Code review comment for lp:~james-w/subunit/discovery

Revision history for this message
James Westby (james-w) wrote :

On Tue, 22 Jun 2010 19:10:49 -0000, Robert Collins <email address hidden> wrote:
> I think I'd like to see something like this:
>
> python -m subunit.run --asdf
> option --asdf not recognized
>
> Execute a python test suite with subunit reporting.
>
> Usage: python -m subunit.run [options] [tests]
>
> .. the stock stuff here.
>
> And in testtools it should say
>
> python -m testtools.run --asdf
> option --asdf not recognized
>
> Execute a python test suite.
>
> Usage: testtools.run [options] [tests]
>
> ...
>
> ----
>
> I note that the thing to run is being misdetected for you; I thought I
> fixed that, perhaps you didn't grab testtools trunk - or my fix was
> incomplete.

I wasn't using trunk.

Doing what you ask is possible, but means we move further from 2.7's
TestProgram, and don't pick up extra options in the usage output
that easily. That at least would go away once 2.7 was the minimum.

I'm happy to do that if you like.

Thanks,

James

« Back to merge proposal