Skip to content

chore: migration test suite#51

Merged
aripalo merged 3 commits into
mainfrom
migration-test
May 19, 2026
Merged

chore: migration test suite#51
aripalo merged 3 commits into
mainfrom
migration-test

Conversation

@aripalo

@aripalo aripalo commented May 13, 2026

Copy link
Copy Markdown
Owner

This pull request introduces a comprehensive migration test harness for aws-cdk-github-oidc, enabling automated validation of migration flows across multiple major versions. The harness is implemented as an isolated CDK app with supporting scripts and configuration, and it programmatically exercises upgrade, resource retention, lookup, and import scenarios. The most important changes are as follows:

Migration Test Harness Implementation

  • Added a new migration-test directory containing a standalone CDK app, which can switch between different aws-cdk-github-oidc versions and migration phases using the MIGRATION_PHASE environment variable. The app includes logic to create, retain, look up, and import the OIDC provider resource, validating real-world migration flows. (migration-test/lib/migration-test-stack.ts, migration-test/lib/migration-test-stack.tsR1-R77)
  • Added a shell script (scripts/run-migration.sh) that automates the full migration sequence, including dependency upgrades, stack deployments, resource lookups, and post-import verification, ensuring end-to-end coverage. (migration-test/scripts/run-migration.sh, migration-test/scripts/run-migration.shR1-R109)

Configuration and Documentation

  • Added a detailed README.md describing the migration flow, stack behavior in each phase, setup instructions, and resource mapping process. (migration-test/README.md, migration-test/README.mdR1-R66)
  • Added cdk.json, package.json, and pnpm-workspace.yaml to configure the CDK app, dependency management, and workspace policies, ensuring isolated and reproducible test runs. (migration-test/cdk.json, [1]; migration-test/package.json, [2]; migration-test/pnpm-workspace.yaml, [3]

Resource Mapping and Ignore Files

@aripalo aripalo marked this pull request as draft May 13, 2026 17:08
@codecov

codecov Bot commented May 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (5d8f580) to head (6fd23ed).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #51   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          262       262           
  Branches         9         9           
=========================================
  Hits           262       262           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aripalo aripalo changed the title test: migration test suite chore: migration test suite May 13, 2026
@aripalo aripalo marked this pull request as ready for review May 13, 2026 17:19
@sonarqubecloud

Copy link
Copy Markdown

@aripalo aripalo merged commit 3b5e77a into main May 19, 2026
13 checks passed
@aripalo aripalo deleted the migration-test branch May 19, 2026 06:01
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