increase code cov for sources

[itsmurugappan]

Description

Increase code coverage for sources

[codecov]

Codecov Report

Merging #1343 (a2ee858) into main (217df5c) will increase coverage by 0.83%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1343      +/-   ##
==========================================
+ Coverage   72.97%   73.80%   +0.83%     
==========================================
  Files         160      160              
  Lines        8151     8075      -76     
==========================================
+ Hits         5948     5960      +12     
+ Misses       1476     1416      -60     
+ Partials      727      699      -28     

Impacted Files	Coverage Δ
pkg/kn/commands/source/apiserver/create.go	82.60% <ø> (+10.60%)	⬆️
pkg/kn/commands/source/container/create.go	90.00% <100.00%> (+22.55%)	⬆️
pkg/kn/commands/source/container/describe.go	76.27% <100.00%> (+26.27%)	⬆️
pkg/kn/commands/source/container/update.go	86.95% <100.00%> (+25.73%)	⬆️
cmd/kn/main.go	40.00% <0.00%> (-25.72%)	⬇️
pkg/kn/commands/channel/list_types.go	64.70% <0.00%> (-5.89%)	⬇️
pkg/kn/commands/source/list_types.go	56.86% <0.00%> (-5.89%)	⬇️
pkg/kn/commands/testing_helper.go	75.00% <0.00%> (ø)
pkg/kn/commands/source/apiserver/describe.go	72.88% <0.00%> (+3.38%)	⬆️
pkg/kn/commands/source/ping/describe.go	74.07% <0.00%> (+3.70%)	⬆️
... and 9 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 217df5c...a2ee858. Read the comment docs.

[itsmurugappan]

/hold

[dsimansk]

Can you rerun Error: Please run ./hack/update-codegen.sh. pls? Probably a conflict from main merge.

[maximilien]

/ok-to-test
Left a comment. Thanks for doing this. Seems like a third_party dependency is causing some of the issue in CI.

[maximilien]

Whenever I see a function with so many parameters (in code or test) I wonder whether a struct should be created to hold the related params and the struct passed to the func or be a method?

[dsimansk]

@itsmurugappan wrt the verify error, In my case I've update go-license tool to get the same result as CI.

go install github.com/google/go-licenses@latest

[navidshaikh]

/retitle increase code cov for sources

[navidshaikh]

/lgtm

@itsmurugappan: thanks! Are you going to add more more to the PR? else please unhold

[knative-prow-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: itsmurugappan, navidshaikh

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 [navidshaikh]

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

[itsmurugappan]

@navidshaikh i have pushed some more changes. This PR is good to go. I am also working on a follow up or with some unit tests to increase the coverage.
/unhold

[knative-metrics-robot]

The following is the coverage report on the affected files.
Say /test pull-knative-client-go-coverage to re-run this coverage report

File	Old Coverage	New Coverage	Delta
pkg/kn/commands/source/apiserver/create.go	83.3%	88.2%	4.9
pkg/kn/commands/source/apiserver/delete.go	82.4%	88.2%	5.9
pkg/kn/commands/source/apiserver/describe.go	84.7%	86.4%	1.7
pkg/kn/commands/source/apiserver/update.go	75.5%	87.8%	12.2
pkg/kn/commands/source/container/create.go	84.4%	93.5%	9.2
pkg/kn/commands/source/container/delete.go	82.4%	88.2%	5.9
pkg/kn/commands/source/container/describe.go	58.6%	87.9%	29.3
pkg/kn/commands/source/container/update.go	78.9%	91.9%	12.9
pkg/kn/commands/source/ping/delete.go	85.7%	92.9%	7.1
pkg/kn/commands/source/ping/describe.go	84.9%	86.8%	1.9
pkg/kn/commands/source/ping/update.go	77.3%	90.9%	13.6

[dsimansk]

/lgtm