Skip to content

chore: pin auth construct in tests#3282

Merged
sobolk merged 4 commits into
mainfrom
pin-auth-construct
May 15, 2025
Merged

chore: pin auth construct in tests#3282
sobolk merged 4 commits into
mainfrom
pin-auth-construct

Conversation

@sobolk
Copy link
Copy Markdown
Contributor

@sobolk sobolk commented May 14, 2025

Fixes the error that appears in e2e tests. When testing baseline CDK version defined here

.

Backend repo released new version of auth construct that raised baseline.
Data repo doesn't seem to require newer CDK baseline.
Therefore, pinning auth construct in data (which is used only in e2e tests to provision auth) to a version that can use older CDK.

{"version":2,"width":120,"height":30,"timestamp":15.632,"title":"Recording","env":{}}
[0.042,"o","user@host⮀/tmp/amplify-e2e-tests/sqlmodelsiamaccess_267419a209_21db5b2f⮀ npm install /codebuild/output/src2495/src/github.com/aws-amplify/amplify-category-api/packages/amplify-graphql-api-construct/dist/js/graphql-api-construct@1.20.1.jsii.tgz aws-cdk-lib@2.187.0 @aws-amplify/auth-construct@^1.0.0\r\n"]
[15.61,"o","\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m \u001b[0m"]
[15.61,"o","\u001b[35mcode\u001b[0m ERESOLVE\r\n\u001b[0m"]
[15.612,"o","\u001b[37;40mnpm\u001b[0m"]
[15.613,"o"," \u001b[0m"]
[15.613,"o","\u001b[31;40mERR!\u001b[0m \u001b[0m\u001b[35mERESOLVE\u001b[0m"]
[15.613,"o"," unable to resolve dependency tree\r\n\u001b[0m"]
[15.613,"o","\u001b[37;40mnpm\u001b[0m \u001b[0m"]
[15.613,"o","\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m"]
[15.613,"o"," \r\n\u001b[0m"]
[15.613,"o","\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m"]
[15.613,"o"," While resolving: sqlmodelsiamaccess_267419a209_21db5b2f@0.1.0\r\n\u001b[0m\u001b[37;40mnpm\u001b[0m"]
[15.613,"o"," \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m Found: aws-cdk-lib@2.187.0\r\n\u001b[0m"]
[15.613,"o","\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m node_modules/aws-cdk-lib\r\n\u001b[0m\u001b[37;40mnpm\u001b[0m \u001b[0m"]
[15.613,"o","\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m   aws-cdk-lib@\"2.187.0\" from the root project\r\n\u001b[0m\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m"]
[15.613,"o"," \r\n\u001b[0m\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m Could not resolve dependency:\r\n\u001b[0m\u001b[37;40mnpm\u001b[0m"]
[15.613,"o"," \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m peer aws-cdk-lib@\"^2.189.1\" from @aws-amplify/auth-construct@1.8.1\r\n\u001b[0m\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m"]
[15.613,"o","\u001b[35m\u001b[0m node_modules/@aws-amplify/auth-construct\r\n\u001b[0m\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m   @aws-amplify/auth-construct@\"^1.0.0\" from the root project\r\n\u001b[0m"]
[15.613,"o","\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m \r\n\u001b[0m\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m"]
[15.614,"o","\u001b[35m\u001b[0m Fix the upstream dependency conflict, or retry\r\n\u001b[0m\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m this command with --force or --legacy-peer-deps\r\n\u001b[0m"]
[15.614,"o","\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m to accept an incorrect (and potentially broken) dependency resolution.\r\n\u001b[0m"]
[15.614,"o","\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m \r\n\u001b[0m"]
[15.614,"o","\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m \r\n\u001b[0m\u001b[37;40mnpm\u001b[0m \u001b[0m"]
[15.614,"o","\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m For a full report see:\r\n\u001b[0m\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m"]
[15.614,"o"," /root/.npm/_logs/2025-05-14T20_20_26_593Z-eresolve-report.txt\r\n\u001b[0m"]
[15.615,"o","\r\n"]
[15.615,"o","\u001b[37;40mnpm\u001b[0m"]
[15.615,"o"," \u001b[0m\u001b[31;40mERR!\u001b[0m\u001b[35m\u001b[0m A complete log of this run can be found in:\r\n\u001b[0m\u001b[37;40mnpm\u001b[0m \u001b[0m\u001b[31;40mERR!\u001b[0m"]
[15.615,"o","\u001b[35m\u001b[0m     /root/.npm/_logs/2025-05-14T20_20_26_593Z-debug-0.log\r\n\u001b[0m"]
[15.633,"o","user@host⮀/tmp/amplify-e2e-tests/sqlmodelsiamaccess_267419a209_21db5b2f⮀ \r\n"]

svidgen
svidgen previously approved these changes May 15, 2025
@sobolk sobolk marked this pull request as ready for review May 15, 2025 02:00
@sobolk sobolk requested review from a team as code owners May 15, 2025 02:00
@sobolk sobolk dismissed svidgen’s stale review May 15, 2025 02:01

draft pr.

@sobolk sobolk enabled auto-merge (squash) May 15, 2025 15:21
@sobolk sobolk merged commit 677fc00 into main May 15, 2025
8 checks passed
@sobolk sobolk deleted the pin-auth-construct branch May 15, 2025 15:24
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