Skip to content

Commit

Permalink
chore: remove parentheses in 'sam local start-api' filter (#24513)
Browse files Browse the repository at this point in the history
The dependencies in v1.76.0+ of the SAM CLI no longer contain parenthesis
in the output, making the trigger wait forever
  • Loading branch information
RomainMuller authored Mar 8, 2023
1 parent e672ad9 commit 77c752b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions packages/aws-cdk/test/integ/helpers/sam.ts
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ export class SamIntegrationTestFixture extends TestFixture {
args.push('--port');
args.push(port.toString());

return this.samShell(['sam', 'local', 'start-api', ...args], '(Press CTRL+C to quit)', ()=>{
return this.samShell(['sam', 'local', 'start-api', ...args], 'Press CTRL+C to quit', ()=>{
return new Promise<ActionOutput>((resolve, reject) => {
axios.get(`http://127.0.0.1:${port}${apiPath}`).then( resp => {
resolve(resp.data);
Expand Down Expand Up @@ -242,4 +242,4 @@ function killSubProcess(child: child_process.ChildProcess, command: string) {
child.kill('SIGINT');
}

}
}

0 comments on commit 77c752b

Please sign in to comment.