Page MenuHomePhabricator

Increase timeout on avalanche test.
ClosedPublic

Authored by deadalnix on Wed, Aug 28, 15:22.

Details

Summary

This shouldn't be an issue, because the loop break as soon as the event is reached.

Test Plan

Run the test and check there was no failure, and that it did not take forever to run.

As the issue cannot be reproduced reliably, this cannot ensure that this fixes it, but it should be enough.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

deadalnix created this revision.Wed, Aug 28, 15:22
Herald added a reviewer: Restricted Project. · View Herald TranscriptWed, Aug 28, 15:22
Fabien requested changes to this revision.Wed, Aug 28, 15:28
Fabien added a subscriber: Fabien.
Fabien added inline comments.
src/test/avalanche_tests.cpp
801 ↗(On Diff #10996)

5 minutes

This revision now requires changes to proceed.Wed, Aug 28, 15:28
deadalnix updated this revision to Diff 10997.Wed, Aug 28, 15:29

Actually do wait for 1 minute

Fabien accepted this revision.Wed, Aug 28, 15:46
This revision is now accepted and ready to land.Wed, Aug 28, 15:46
This revision was automatically updated to reflect the committed changes.