D5966: test: stabilize test-run-tests.t output
durin42 (Augie Fackler)
phabricator at mercurial-scm.org
Tue Feb 19 10:10:21 EST 2019
durin42 added inline comments.
INLINE COMMENTS
> lothiraldan wrote in test-run-tests.t:1177
> Yes we could.
>
> I think the regex was previously `{4,5}` for two reasons:
>
> - We expected the duration to be less than 10 seconds.
>
> - The precision of the number was 3 digits or 4 digits after the dot depending if we used json or simplejson for dumping the data.
>
> Right now, it seems that `run-tests.py` itself is limitation the precision of the digits.
>
> So we can either drop the {3,4} as we should have a stable number of digits after the dot, and anyone changing the precision of times in `run-tests.py` will have a diff in this test.
>
> Or we make it more permissive so we don't need to bother about updating it later.
>
> I don't have a strong opinion about either, what do you think we should do?
I don't feel strongly, let's just make it permissive so the test passes on both fast and slow systems.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D5966
To: lothiraldan, #hg-reviewers
Cc: durin42, mercurial-devel
More information about the Mercurial-devel
mailing list