On Fri, 22 Jun 2001, Stas Bekman wrote:
 
> see the attached patch. This one just rotates through all the tests N
> times, where N is --run-tests=N (optional). 

looks good, two issues:
- t/TEST -run-tests without =x will start/stop the server even it is
already running (via t/TEST -start or -debug)
- doesn't work for all tests (when none are specified), i think
Apache::TestHarness is where the x run-tests should be done to fix this.

btw, did you know GetOpt::Long adds abbreviated aliases?  so you can
actually use '-r' as shorthand for '-run-tests'

> Another option that might be interesting to add is replace:
> 
> my $tests =
> [ ( @{ $self->{tests} } ) x $self->{opts}->{'run-tests'} ];
> 
> with:
> 
> my $tests =
> [ map { $_ x $self->{opts}->{'run-tests'} } @{ $self->{tests} } ];
> 
> so in first case (as in the patch) tests: a, b, c are multiplied as a, b,
> c, a, b, c, a, b, c... whereas the second example does: a, a, a, b, b, b,
> c, c, c, ...

i like the current way, but if there was an option to choose between both,
that would be way cool.



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to