Skip to content
New issue

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

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

Already on GitHub? # to your account

test: use dummy credential when executing tests #268

Merged
merged 1 commit into from
Nov 29, 2024

Conversation

StoneDot
Copy link
Contributor

Issue #, if available:
N/A

Description of changes:
Currently, the cargo test command uses a default credential chain to determine an AWS credential. However, it should use a dummy credential instead because an actual credential may change the production environment. Switching the default credential to a dummy credential also mitigates the pain when executing a cargo test that may require setting AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@StoneDot StoneDot requested review from a team as code owners November 28, 2024 17:53
Signed-off-by: Hiroaki Goto <hiroag@amazon.com>
@StoneDot StoneDot force-pushed the cargo-test-with-default-cred branch from 1b413e1 to a4f9b7c Compare November 28, 2024 18:04
@ryota-sakamoto ryota-sakamoto merged commit 6e39f09 into awslabs:main Nov 29, 2024
5 checks passed
@StoneDot StoneDot deleted the cargo-test-with-default-cred branch November 29, 2024 06:51
kzym-w pushed a commit to kzym-w/dynein that referenced this pull request Dec 11, 2024
Signed-off-by: Hiroaki Goto <hiroag@amazon.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants