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

feat(experimentalIdentityAndAuth): add @aws.auth#sigv4 integration tests #1012

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

syall
Copy link
Contributor

@syall syall commented Oct 8, 2023

Issue #, if available:

N/A.

Description of changes:

Dependent on: #1018, #1021

Add @aws.auth#sigv4 integration tests.

  • Thread sigv4 cases

Testing:

In staging, not all NPM packages are officially published, so the following replacement is necessary in the smithy-typescript-codegen-test projections:

# Replace
"@smithy/experimental-identity-and-auth"
# with the local `node_modules/` path, e.g.
"/Users/yuasteve/development/ts-ia/smithy-typescript/node_modules/@smithy/experimental-identity-and-auth"
  1. Run ./gradlew :smithy-typescript-codegen-test:clean :smithy-typescript-codegen-test:build
  2. Do the replacement mentioned above
  3. Run node scripts/build-generated-test-packages.js (may need to do more than once...)
  4. Ran yarn test:integration in @smithy/experimental-identity-and-auth

If one or more of the packages in the /packages directory has been modified, be sure yarn changeset add has been run and its output has
been committed and included in this pull request. See CONTRIBUTING.md.


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

@syall syall force-pushed the identity-auth-integration-tests-sigv4 branch 8 times, most recently from 7d0feb1 to 016537d Compare October 10, 2023 21:30
@syall syall force-pushed the identity-auth-integration-tests-sigv4 branch from 016537d to a797db7 Compare October 10, 2023 21:40
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.

1 participant