[PATCH 1 of 8] tests: add a randomized test for pathencode

Nikolaj Sjujskij sterkrig at myopera.com
Fri Nov 16 15:25:14 CST 2012


Den 2012-11-15 02:49:23 skrev Martin Schröder
<martin.schroeder at nerdluecht.de>:

> Am 13.11.2012 23:00, schrieb Bryan O'Sullivan:
>> This is a probabilistic test - it generates different test cases on  
>> every
>> run, unless invoked from the command line with a specific seed.
> Is that really a good thing? ... A test might fail in one case and  
> succeed
> in another.
Yes, that's the main problem, especially for a first-time contributor,
whose one-line patch suddenly "breaks" pathencode test. What about putting
it into /contrib/ and running it periodically? And every found cornercase
adds new regression test in regular test suite.


More information about the Mercurial-devel mailing list