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