Skip to content

Conversation

@yndu13
Copy link
Collaborator

@yndu13 yndu13 commented Dec 3, 2024

…error scenarios

You need to complete
  • unit tests and/or feature tests
  • documentation is changed or added

@codecov-commenter
Copy link

codecov-commenter commented Dec 3, 2024

Codecov Report

Attention: Patch coverage is 84.16988% with 41 lines in your changes missing coverage. Please review.

Project coverage is 91.70%. Comparing base (2b0ee12) to head (0ab9ddf).

Files with missing lines Patch % Lines
...entials/provider/EcsRamRoleCredentialProvider.java 74.28% 6 Missing and 3 partials ⚠️
...un/credentials/provider/RefreshCachedSupplier.java 89.41% 4 Missing and 5 partials ⚠️
...ava/com/aliyun/credentials/policy/NonBlocking.java 72.41% 7 Missing and 1 partial ⚠️
...provider/ECSMetadataServiceCredentialsFetcher.java 66.66% 1 Missing and 3 partials ⚠️
...edentials/provider/SessionCredentialsProvider.java 60.00% 4 Missing ⚠️
...un/credentials/provider/URLCredentialProvider.java 76.47% 2 Missing and 2 partials ⚠️
.../com/aliyun/credentials/utils/ParameterHelper.java 93.10% 2 Missing ⚠️
...ntials/provider/OIDCRoleArnCredentialProvider.java 91.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #96      +/-   ##
============================================
+ Coverage     91.39%   91.70%   +0.31%     
- Complexity      714      754      +40     
============================================
  Files            46       49       +3     
  Lines          2359     2532     +173     
  Branches        213      223      +10     
============================================
+ Hits           2156     2322     +166     
- Misses          129      133       +4     
- Partials         74       77       +3     

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

@yndu13 yndu13 force-pushed the refreshers branch 3 times, most recently from cec5d11 to 017236c Compare December 27, 2024 06:38
@yndu13 yndu13 force-pushed the refreshers branch 2 times, most recently from e80aaef to 7a2a8e1 Compare January 23, 2025 06:46
@yndu13 yndu13 force-pushed the refreshers branch 4 times, most recently from a7fe5f5 to eac4533 Compare February 12, 2025 08:50
@yndu13 yndu13 merged commit 9662fc8 into master Feb 12, 2025
4 checks passed
@yndu13 yndu13 deleted the refreshers branch February 12, 2025 09:37
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.

4 participants