Page MenuHomePhabricator

test: simplify test-amend.t to avoid race condition
ClosedPublic

Authored by marmoute on Mon, Feb 10, 5:38 AM.

Details

Summary

Insted on relying on sleep, we could simply have the editor do the file change.
This remove the reliance on "sleep" and avoid test failing on heavy load
machine.

To test this, I reverted the code change in 5558e3437872 and the test started
failing again.

This is a graft on stable of 141ceec06b55 which should have targeted for stable.

Diff Detail

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

Event Timeline

marmoute created this revision.Mon, Feb 10, 5:38 AM
pulkit accepted this revision.Mon, Feb 10, 12:44 PM
This revision is now accepted and ready to land.Mon, Feb 10, 12:44 PM
This revision was automatically updated to reflect the committed changes.