[Distutils] setuptools tracker "needpatch" keyword, tests, etc.

Phillip J. Eby pje at telecommunity.com
Fri Aug 8 18:28:26 CEST 2008

At 10:32 AM 8/8/2008 -0500, Chris Galvan wrote:
>Phillip J. Eby wrote:
>>Hello all.  I had some spare time the other day and went through 
>>the tracker, reclassifying a few things and marking some with a 
>>"needpatch" keyword.  The ones marked "needpatch" vary from things 
>>I have no idea what to do with, to ones where I've practically 
>>spelled out the needed patch in the tracker.
>>Issues with patches that passed my initial review have been marked 
>>"in-progress"; these could use some testing before check-in.  At 
>>this point, I haven't had an opportunity to review the results of 
>>the test sprint that was done; if somebody could throw that up as a 
>>patch on the tracker, or at least repost a link to where I can find 
>>that stuff, that would help.
>The work done on the test sprint is hosted in this bzr branch.  We 
>wanted to get your feedback on what had been done so far to make 
>sure we were heading in the right direction.

My initial reaction is that it's off to a good start, but the tests 
themselves seem rather shallow; more like "smoke tests" (i.e., turn 
it on and see if smoke comes out) than functional tests.

I'm thinking it might help to use the setuptools.sandbox facility to 
log files created, deleted, modified, etc. by the process.  That 
would probably be a better test of what has/hasn't been done than 
using ellipses on the logs, which is order-dependent as well as 
having the ability to skip lines where the wrong thing is being done, 
etc.  The way things are being done now, they probably won't be able 
to test some of the things that are most likely to break (i.e., the 
complexities of easy_install).

(Probably in order to do that I'll need to add a new sandboxing class 
that creates a "mock" filesystem and allows before/after expectations 
to be set.)

More information about the Distutils-SIG mailing list