Skip to content

Commit

Permalink
test: remove bootstrap in test code (#2916)
Browse files Browse the repository at this point in the history
  • Loading branch information
palpatim authored Sep 30, 2024
1 parent 5e376d5 commit 78614a5
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 12 deletions.
3 changes: 0 additions & 3 deletions packages/amplify-graphql-api-construct-tests/src/commands.ts
Original file line number Diff line number Diff line change
Expand Up @@ -122,9 +122,6 @@ export const cdkDeploy = async (cwd: string, option: string, props?: CdkDeployPr
env: { npm_config_registry: 'https://registry.npmjs.org/' },
noOutputTimeout,
};
// This prevents us from maintaining a separate CDK account bootstrap process as we add support for new accounts, regions.
// Checks and succeeds early (a no-op) if the account-region combination is already bootstrapped.
await spawn(getNpxPath(), ['cdk', 'bootstrap'], commandOptions).runAsync();

await spawn(
getNpxPath(),
Expand Down
9 changes: 0 additions & 9 deletions packages/graphql-transformers-e2e-tests/src/cdkUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -26,15 +26,6 @@ export const deployJsonServer = () => {
throw new Error(`'yarn' failed with exit code: ${yarnServerResult.exitCode}`);
}

const cdkBootstrapResult = execa.sync('npx', ['cdk', 'bootstrap', '--require-approval', 'never'], {
cwd: jsonServerRootDirectory,
stdio: 'inherit',
});

if (cdkBootstrapResult.exitCode !== 0) {
throw new Error(`CDK bootstrap failed with exit code: ${cdkBootstrapResult.exitCode}`);
}

const cdkDeployResult = execa.sync('npx', ['cdk', 'deploy', '--outputsFile', outputValuesFile, '--require-approval', 'never'], {
cwd: jsonServerRootDirectory,
stdio: 'inherit',
Expand Down

0 comments on commit 78614a5

Please sign in to comment.