Skip to content

Commit f461601

Browse files
committed
test: increase timeout for workflow execution tests
Increased the timeout to account for the latency of workflow execution and polling.
1 parent 701ef0f commit f461601

2 files changed

Lines changed: 8 additions & 5 deletions

File tree

workflows/quickstart/test/executeWorkflow.test.js

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -92,8 +92,11 @@ describe('Cloud Workflows JavaScript Execution Samples', () => {
9292
`node --require ts-node/register ./executeWithoutArguments.js ${PROJECT_ID} ${LOCATION_ID} ${WORKFLOW_ID}`
9393
);
9494

95-
assert.ok(result.length > 0, 'Quickstart must return non-empty result');
96-
}).timeout(5000);
95+
assert.ok(
96+
result.length > 0,
97+
'executeWithoutArguments must return non-empty result'
98+
);
99+
}).timeout(60000);
97100

98101
it('should execute the workflow using the executeWithArguments sample', async () => {
99102
// Execute workflow, with long test timeout
@@ -105,5 +108,5 @@ describe('Cloud Workflows JavaScript Execution Samples', () => {
105108
result.length > 0,
106109
'executeWithArguments must return non-empty result'
107110
);
108-
}).timeout(5000);
111+
}).timeout(60000);
109112
});

workflows/quickstart/test/executeWorkflow.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ describe('Cloud Workflows TypeScript Execution Samples', () => {
9292
result.length > 0,
9393
'executeWithoutArguments must return non-empty result'
9494
);
95-
}).timeout(5000);
95+
}).timeout(60000);
9696

9797
it('should execute the workflow using the executeWithArguments sample', async () => {
9898
// Execute workflow, with long test timeout
@@ -104,5 +104,5 @@ describe('Cloud Workflows TypeScript Execution Samples', () => {
104104
result.length > 0,
105105
'executeWithArguments must return non-empty result'
106106
);
107-
}).timeout(5000);
107+
}).timeout(60000);
108108
});

0 commit comments

Comments
 (0)