fix: stabilize E2E tests — exclude broken regions, fix Node 22 compatibility#3468
Merged
Conversation
This reverts commit d83d153.
Contributor
Author
|
A successful e2e run can be found here, where only |
svidgen
approved these changes
Apr 13, 2026
adrianjoshua-strutt
approved these changes
Apr 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Stabilizes E2E test suite by excluding regions with infrastructure issues and fixing Node 22 compatibility problems. Batch 10 result: 190✅ / 1❌ (only
cleanup_e2e_resourcesutility failed).Changes
Region Exclusions
Exclude
me-south-1(BAH) andme-central-1(DXB)scripts/e2e-test-regions.jsonandpackages/amplify-e2e-core/src/configure/index.ts.Exclude
sa-east-1(GRU)amplify pushcommand is erroring in this region, causing consistent test failures.scripts/e2e-test-regions.jsonandpackages/amplify-e2e-core/src/configure/index.ts.Node 22 Compatibility Fix
Fix
sql_generate_unauthfailure (packages/amplify-e2e-core/src/categories/api.ts).expect()to.wait()for thegenerateUnauthSQLerror message assertion.[DEP0040]) as the first line of output before the actual error message..expect()usesshift: truewhich only matches the next line — so it matched the deprecation warning instead of the expected error..wait()usesshift: falseand scans all lines until a match is found.Region-Pinned Test
Pin
searchable-previous-deployment-had-node-to-nodetous-west-1(scripts/split-e2e-tests.ts)FORCE_REGION_MAPso it always runs inus-west-1.ap-northeast-3(Osaka) via round-robin, wheret2.small.elasticsearchinstance type is deprecated/unavailable.codebuild_specs/e2e_workflow.ymlto includeCLI_REGION: us-west-1.Other
amplify --versionlogging inpackages/amplify-e2e-core/src/categories/function.tsfor debugging.Test Results
Batch 10 (
fix/exclude-bah-dxb-regions): 190✅ / 1❌cleanup_e2e_resources(utility task, not a real test)