Skip to content

Commit

Permalink
chore: ensure aws test failure blocks publish (#4307)
Browse files Browse the repository at this point in the history
  • Loading branch information
MarkMcCulloh authored Sep 27, 2023
1 parent 5297fe2 commit 6d29256
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 12 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -331,7 +331,7 @@ jobs:
- aws-sdk-spec-test
steps:
- name: Check failure
if: needs.build.result == 'failure' || needs.e2e-test.result == 'failure' || needs.benchmark.result == 'failure' || needs.test.result == 'failure'
if: needs.build.result == 'failure' || needs.e2e-test.result == 'failure' || needs.benchmark.result == 'failure' || needs.test.result == 'failure' || needs.aws-sdk-spec-test.result == 'failure'
run: exit 1
- name: Download patches
uses: actions/download-artifact@v3
Expand Down
7 changes: 0 additions & 7 deletions libs/wingsdk/test/target-sim/__snapshots__/queue.test.ts.snap
Original file line number Diff line number Diff line change
Expand Up @@ -80,13 +80,6 @@ exports[`messages are not requeued if the function fails after retention timeout
"Invoke (payload=\\"{\\\\\\"messages\\\\\\":[\\\\\\"BAD MESSAGE\\\\\\"]}\\").",
"Subscriber error - returning 1 messages to queue: ERROR",
"1 messages pushed back to queue after visibility timeout.",
"Sending messages (messages=[\\"BAD MESSAGE\\"], subscriber=sim-1).",
"Invoke (payload=\\"{\\\\\\"messages\\\\\\":[\\\\\\"BAD MESSAGE\\\\\\"]}\\").",
"Subscriber error - returning 1 messages to queue: ERROR",
"wingsdk.sim.EventMapping deleted.",
"wingsdk.cloud.Queue deleted.",
"wingsdk.cloud.Function deleted.",
"wingsdk.cloud.TestRunner deleted.",
]
`;

Expand Down
6 changes: 2 additions & 4 deletions libs/wingsdk/test/target-sim/queue.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -265,24 +265,22 @@ test("messages are not requeued if the function fails after retention timeout",
// THEN
await s.stop();

expect(listMessages(s)).toMatchSnapshot();
expect(listMessages(s).slice(0, 9)).toMatchSnapshot();
expect(app.snapshot()).toMatchSnapshot();

expect(
s
.listTraces()
.filter((v) => v.sourceType == QUEUE_TYPE)
.map((trace) => trace.data.message)
.slice(0, 5)
).toMatchInlineSnapshot(`
[
"wingsdk.cloud.Queue created.",
"Push (messages=BAD MESSAGE).",
"Sending messages (messages=[\\"BAD MESSAGE\\"], subscriber=sim-1).",
"Subscriber error - returning 1 messages to queue: ERROR",
"1 messages pushed back to queue after visibility timeout.",
"Sending messages (messages=[\\"BAD MESSAGE\\"], subscriber=sim-1).",
"Subscriber error - returning 1 messages to queue: ERROR",
"wingsdk.cloud.Queue deleted.",
]
`);
});
Expand Down

0 comments on commit 6d29256

Please sign in to comment.