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

refactor: modify the class name of credential model #54

Merged
merged 1 commit into from
Jul 7, 2023

Conversation

yndu13
Copy link
Collaborator

@yndu13 yndu13 commented Jul 7, 2023

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

@codecov-commenter
Copy link

codecov-commenter commented Jul 7, 2023

Codecov Report

Merging #54 (cdf3090) into master (25c239d) will not change coverage.
The diff coverage is 90.00%.

❗ Current head cdf3090 differs from pull request most recent head 45a0fcb. Consider uploading reports for the commit 45a0fcb to get more accurate results

@@            Coverage Diff            @@
##             master      #54   +/-   ##
=========================================
  Coverage     84.69%   84.69%           
  Complexity      508      508           
=========================================
  Files            44       44           
  Lines          1784     1784           
  Branches        113      113           
=========================================
  Hits           1511     1511           
  Misses          236      236           
  Partials         37       37           
Flag Coverage Δ
unittest 84.69% <90.00%> (ø)

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

Impacted Files Coverage Δ
...edentials/provider/DefaultCredentialsProvider.java 81.25% <ø> (ø)
...entials/provider/EcsRamRoleCredentialProvider.java 65.11% <ø> (ø)
...redentials/provider/StaticCredentialsProvider.java 100.00% <ø> (ø)
...un/credentials/provider/URLCredentialProvider.java 40.22% <0.00%> (ø)
src/main/java/com/aliyun/credentials/Client.java 70.23% <66.66%> (ø)
...a/com/aliyun/credentials/EcsRamRoleCredential.java 100.00% <100.00%> (ø)
.../com/aliyun/credentials/OIDCRoleArnCredential.java 95.83% <100.00%> (ø)
...a/com/aliyun/credentials/RamRoleArnCredential.java 100.00% <100.00%> (ø)
...a/com/aliyun/credentials/RsaKeyPairCredential.java 100.00% <100.00%> (ø)
...ain/java/com/aliyun/credentials/URLCredential.java 95.83% <100.00%> (ø)
... and 9 more

@JacksonTian JacksonTian merged commit 0205ca1 into master Jul 7, 2023
11 checks passed
@JacksonTian JacksonTian deleted the update-model branch July 7, 2023 03:28
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.

3 participants