Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unit-test: increase coverage, path: pkg/ingress/kube/secret #105

Merged
merged 2 commits into from
Jan 29, 2023

Conversation

iutx
Copy link
Contributor

@iutx iutx commented Dec 8, 2022

Signed-off-by: iutx root@viper.run

Ⅰ. Describe what this PR did

increase coverage, path: pkg/ingress/kube/secret

0% -> 100%

Ⅱ. Does this pull request fix one issue?

fixes #87

Ⅲ. Why don't you add test cases (unit test/integration test)?

None

Ⅳ. Describe how to verify it

None

Ⅴ. Special notes for reviews

None

@iutx
Copy link
Contributor Author

iutx commented Dec 14, 2022

get, i will rework for this pr later.

Copy link
Collaborator

@johnlanni johnlanni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you update UT to be compatible with the new code?

@iutx iutx force-pushed the unit-test/ingress-kube-secret branch from f8ee4af to 55a34d2 Compare January 29, 2023 06:24
@codecov-commenter
Copy link

Codecov Report

Merging #105 (3b705c0) into main (4cd7975) will increase coverage by 0.62%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
+ Coverage   26.09%   26.71%   +0.62%     
==========================================
  Files          27       28       +1     
  Lines        4759     4772      +13     
==========================================
+ Hits         1242     1275      +33     
+ Misses       3438     3418      -20     
  Partials       79       79              
Impacted Files Coverage Δ
pkg/ingress/kube/secret/controller.go 100.00% <0.00%> (ø)
pkg/bootstrap/server.go 62.97% <0.00%> (+7.63%) ⬆️

@iutx iutx requested a review from johnlanni January 29, 2023 09:27
Copy link
Collaborator

@johnlanni johnlanni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

pkg/ingress/kube/secret 单测覆盖率提高(Increase unit test coverage)
3 participants