Skip to content
This repository has been archived by the owner on Apr 7, 2024. It is now read-only.

test: enhance test coverage #53

Merged
merged 4 commits into from
Apr 26, 2023
Merged

Conversation

Wwwsylvia
Copy link
Member

Resolves: #51

Signed-off-by: Sylvia Lei <[email protected]>
Signed-off-by: Sylvia Lei <[email protected]>
Signed-off-by: Sylvia Lei <[email protected]>
@Wwwsylvia
Copy link
Member Author

In case the codecov report is not showing, see https://app.codecov.io/gh/oras-project/oras-credentials-go/tree/refactor_tests
image

registry.go Show resolved Hide resolved
Signed-off-by: Sylvia Lei <[email protected]>
@codecov-commenter
Copy link

Codecov Report

Merging #53 (850c63b) into main (ba6b33c) will increase coverage by 22.37%.
The diff coverage is 100.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff             @@
##             main      #53       +/-   ##
===========================================
+ Coverage   58.76%   81.13%   +22.37%     
===========================================
  Files           6        6               
  Lines         371      371               
===========================================
+ Hits          218      301       +83     
+ Misses        129       48       -81     
+ Partials       24       22        -2     
Impacted Files Coverage Δ
registry.go 83.33% <100.00%> (+9.25%) ⬆️

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Contributor

@shizhMSFT shizhMSFT left a comment

Choose a reason for hiding this comment

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

LGTM

@shizhMSFT shizhMSFT merged commit d610c11 into oras-project:main Apr 26, 2023
@Wwwsylvia Wwwsylvia deleted the refactor_tests branch April 26, 2023 07:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Increase test coverage of internal/config
4 participants