[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