On Apr 27, 2010, at 02:40 PM, exar...@twistedmatrix.com wrote: >On 01:38 pm, rdmur...@bitdance.com wrote:
>> 2) have unit tests that fail before the patch and succeed after
>
>This list would make a good addition to one of the cpython development
>pages. If potential contributors could find this information, then
>they'd be much more likely to participate by doing reviews.
It would be kind of cool if there were some best practices for running said
unittest both with and without the patch enabled. Kind of like using #ifdefs
in C but without all the commenting-out-commenting-in error proneness. I
guess you could do something like
if os.getenv('BUG1234'):
# Patch the frobnicator to not bloviate.
Maybe more trouble than it's worth, and not always feasible of course, but I'm
wondering how (or maybe if) people do things this way.
With Bazaar, I often use a loom with two threads - a bottom one that contains
the test that fails, and a top one that contains the fix for the test. It's a
great way to develop a patch, but you lose that once you flatten the code for
review.
-Barry
signature.asc
Description: PGP signature
_______________________________________________ Python-Dev mailing list Python-Dev@python.org http://mail.python.org/mailman/listinfo/python-dev Unsubscribe: http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com
