This is an archive of the discontinued Mercurial Phabricator instance.

run-tests: allow some slack about 'waiting on lock' message

Authored by marmoute on Dec 2 2020, 5:19 PM.



It is common to run the tests on very loaded machine when concurrent run might
take a bit longer. Such message are usually harmless, but anoying as they break
the tests.

Test that explicitly depends on this value have been adjusted. This make them
more robust anyway.

A fun case was test-clone-pull-corruption.t which, without the previous
changeset introducing extra flushing, ended use having a line 31 (`pulling from
../source`) changing order because the warning message was no longer flushing
stdin before using stderr (stderr being invisible in the test).

Diff Detail

rHG Mercurial
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

marmoute created this revision.Dec 2 2020, 5:19 PM
pulkit accepted this revision.Dec 3 2020, 3:14 AM
This revision is now accepted and ready to land.Dec 3 2020, 3:14 AM