-
Notifications
You must be signed in to change notification settings - Fork 76
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
test(api): e2e for iam provider on rds models #1957
test(api): e2e for iam provider on rds models #1957
Conversation
blogIAMUnauthClient = constructBlogHelper(unauthAppSyncClient); | ||
postIAMUnauthClient = constructPostHelper(unauthAppSyncClient); | ||
userIAMUnauthClient = constructUserHelper(unauthAppSyncClient); | ||
profileIAMUnauthClient = constructProfileHelper(unauthAppSyncClient); | ||
blogIAMAuthClient = constructBlogHelper(authAppSyncClient); | ||
postIAMAuthClient = constructPostHelper(authAppSyncClient); | ||
userIAMAuthClient = constructUserHelper(authAppSyncClient); | ||
profileIAMAuthClient = constructProfileHelper(authAppSyncClient); | ||
userApiKeyClient = constructUserHelper(apiKeyClient); | ||
userLambdaClient = constructUserHelper(lambdaClient); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some of these clients are unused today but left them here for future work to add more test cases.
cognitoIdentityPoolId: hasIdentityPoolId ? ref('ctx.stash.identityPoolId') : nul(), | ||
...(role.strategy === 'private' && { cognitoIdentityPoolId: identityPoolId ? str(identityPoolId) : nul() }), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to reference the identityPoolId in stash, right? I don't think that value identityPoolId
you're referencing exists. This should fail to build if I'm not mistaken.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, just double-checked, and it looks like the build is failing on this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, looks rebase didn't go well. Looking into it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR has been updated now with the correct changes.
The build job is still failing |
All the tests are passing now. |
4845915
into
aws-amplify:feature/rds-support-preview2
Adding E2E tests for Auth Directive IAM provider support.