Skip to content

Commit ed22a5d

Browse files
committed
fix: update test comment to explain why 2s wait is unnecessary
1 parent 4b30d19 commit ed22a5d

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

test/e2e-azuremanaged/orchestration.spec.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,8 @@ describe("Durable Task Scheduler (DTS) E2E Tests", () => {
231231
expect(state?.serializedOutput).toEqual(JSON.stringify("handled-failure"));
232232
expect(failActivityCounter).toEqual(1);
233233

234-
// Wait a bit then verify orchestration stays COMPLETED (not corrupted by late activity completions)
234+
// Verify orchestration stays COMPLETED. The sidecar won't deliver activity
235+
// completion events to an already-completed orchestration, so no delay is needed.
235236
const finalState = await taskHubClient.getOrchestrationState(id);
236237
expect(finalState).toBeDefined();
237238
expect(finalState?.runtimeStatus).toEqual(OrchestrationStatus.ORCHESTRATION_STATUS_COMPLETED);

0 commit comments

Comments
 (0)