Skip to content

tests: add oidc integration test#45

Merged
JacksonTian merged 1 commit intomasterfrom
test/intergration
Jul 9, 2024
Merged

tests: add oidc integration test#45
JacksonTian merged 1 commit intomasterfrom
test/intergration

Conversation

@yndu13
Copy link
Contributor

@yndu13 yndu13 commented Jul 8, 2024

No description provided.

@yndu13 yndu13 force-pushed the test/intergration branch from 7b02423 to 815e51f Compare July 8, 2024 10:27
self.assertIsNotNone(credential.access_key_id)
self.assertIsNotNone(credential.access_key_secret)
self.assertIsNotNone(credential.security_token)
self.assertEqual('oidc_role_arn', credential.type)
Copy link
Contributor

Choose a reason for hiding this comment

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

默认凭证链也可以测一个

@codecov-commenter
Copy link

codecov-commenter commented Jul 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.61%. Comparing base (fed05c6) to head (728c24a).

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #45      +/-   ##
==========================================
+ Coverage   80.28%   80.61%   +0.33%     
==========================================
  Files          17       17              
  Lines        1785     1795      +10     
==========================================
+ Hits         1433     1447      +14     
+ Misses        352      348       -4     

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

@yndu13 yndu13 force-pushed the test/intergration branch 2 times, most recently from c9b1be2 to de85347 Compare July 9, 2024 02:12
@yndu13 yndu13 force-pushed the test/intergration branch from de85347 to 728c24a Compare July 9, 2024 02:13
@JacksonTian JacksonTian merged commit bfc59c8 into master Jul 9, 2024
@yndu13 yndu13 deleted the test/intergration branch October 9, 2025 09:23
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