On Sat, 7 Sep 2002, Jeff wrote:
> In attempting to figure out why tests in languages/perl6 aren't running,
> I came across something of an oddity in t/harness.
>
> > my @tests = @ARGV ? @ARGV : map { glob( "t/$_/*.t" ) } ( qw(compiler rx) );
>
> Apparently designed to loop through just compiler/ and rx/ directories,
> yet there are also builtins/ and parser/ directories. It occurs to me
> that code like this is prone to someone forgetting to add a new test/
> directory when updating a test suite. Though it could easily be argued
> that it's better than some equivalent opendir/readdir/closedir code
> because it lets you explicitly declare which test directories you want
> to run. Thoughts on this?
What this code is trying to say is "don't run the parser tests, because
they're unmaintained". So probably something like
grep !m{t/parser/} glob "t/*/*.t"
would suck less. Maybe with a note about why it's done that way.
/s
