fix: HOME with different permissions

[gauron99]

Make sure standard functionality works even when HOME has different permissions

• added new tests
• test for permissions that is skipped for windows and prow
• print warning for not sufficient permissions when trying to access func config file (usually func/.config.yaml in $HOME)

/kind enhancement

[knative-prow]

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

[codecov]

Codecov Report

Attention: Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 67.10%. Comparing base (f4bd1d8) to head (1fbb2a8).
Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
cmd/root.go	33.33%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2361      +/-   ##
==========================================
+ Coverage   61.54%   67.10%   +5.55%     
==========================================
  Files         129      129              
  Lines       15334    11931    -3403     
==========================================
- Hits         9437     8006    -1431     
+ Misses       4971     2962    -2009     
- Partials      926      963      +37     

Flag	Coverage Δ
e2e-test	36.65% <66.66%> (-0.01%)	⬇️
e2e-test-oncluster	33.44% <66.66%> (+0.01%)	⬆️
e2e-test-oncluster-runtime	29.57% <66.66%> (?)
e2e-test-runtime-go	26.62% <66.66%> (?)
e2e-test-runtime-node	26.00% <66.66%> (?)
e2e-test-runtime-python	26.00% <66.66%> (?)
e2e-test-runtime-quarkus	26.10% <66.66%> (?)
e2e-test-runtime-rust	25.15% <66.66%> (?)
e2e-test-runtime-springboot	25.18% <66.66%> (?)
e2e-test-runtime-typescript	26.09% <66.66%> (?)
integration-tests	53.14% <50.00%> (+2.41%)	⬆️
unit-tests	51.28% <50.00%> (?)
unit-tests-macos-latest	?
unit-tests-ubuntu-latest	?
unit-tests-windows-latest	?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[gauron99]

/rerun all

[gauron99]

/test all

[gauron99]

/retest-required

[gauron99]

/retest-required

[lkingland]

/lgtm

/hold for potential squelching of the permission error

[gauron99]

The main unit test is failing because "knative" user (GH actions/ something similar) has greater permissions, I suspect. Will have to update the new test.

[gauron99]

@lkingland PTAL

I have moved the printing to the root.go file which significantly lowers the number of prints to just one per command.
I have also added a TODO for myself for the opportunity to simplify the whole process.

[lkingland]

Looking good to me!

[knative-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gauron99, lkingland

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [gauron99,lkingland]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[gauron99]

/unhold