Geoffrey Young wrote:

Philippe M. Chiasson wrote:

a release candidate for Apache::Test 1.15 is now available for testing.


any way to sneak in this minor patch?  it fixes cases where there are
multiple all.t files present (such as t/foo/all.t and t/bar/all.t) and more
than one fail - currently directories are properly ignored but only the last
directory's all.t file is run and shown in the output because prior failing
all.t files are (improperly) pruned along with all the other files in the
directory.

Just got a message from Geoff (he's not online at the moment), and he told me he changed his mind, and would rather wait for the A-T release to be out before applying this patch. So, let's leave it up to him to integrate once 1.15 is out.

--Geoff


------------------------------------------------------------------------

Index: lib/Apache/TestHarness.pm
===================================================================
RCS file: /home/cvs/httpd-test/perl-framework/Apache-Test/lib/Apache/TestHarness.pm,v
retrieving revision 1.20
diff -u -r1.20 TestHarness.pm
--- lib/Apache/TestHarness.pm   17 Sep 2004 18:24:13 -0000      1.20
+++ lib/Apache/TestHarness.pm   20 Oct 2004 04:42:01 -0000
@@ -116,7 +116,7 @@
         if (m:\Wall\.t$:) {
             unless ($self->run_t($_)) {
                 $skip_dirs{$dir} = 1;
-                @new_tests = grep { not $skip_dirs{dirname $_} } @new_tests;
+                @new_tests = grep { m:\Wall\.t$: || not $skip_dirs{dirname $_} } 
@new_tests;
                 push @new_tests, $_;
             }
         }

-- -------------------------------------------------------------------------------- Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5 http://gozer.ectoplasm.org/ F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

Attachment: signature.asc
Description: OpenPGP digital signature



Reply via email to