Skip to content

Commit

Permalink
fix(agoric-cli): increase integration-test timeout
Browse files Browse the repository at this point in the history
Now that spawner uses createVat() (#1343), deploys to the solo machine take
longer. The integration test was allowing 60s for `agoric deploy
./contract/deploy.js ./api/deploy.js` to work, and that used to take 47s on
my local machine, but now it takes 76s, and would exceed the timeout.

I increased the timeout to 120s to fix this.

When we switch vats to use XS, this should be dramatically faster (about
10x), so we'll have have plenty of margin.
  • Loading branch information
warner committed Apr 23, 2021
1 parent 86c0a58 commit 942c2a2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion packages/agoric-cli/integration-tests/test-workflow.js
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,7 @@ test('workflow', async t => {
]);
finalizers.push(() => pkill(deployP.childProcess, 'SIGINT'));

timeout = setTimeout(deployResult.resolve, 60000, 'timeout');
timeout = setTimeout(deployResult.resolve, 120 * 1000, 'timeout');
const done = await Promise.race([deployResult.promise, deployP]);
t.is(done, 0, `deploy successful before timeout`);
clearTimeout(timeout);
Expand Down

0 comments on commit 942c2a2

Please sign in to comment.