diff --git a/mozilla/webtools/bugzilla/runtests.sh b/mozilla/webtools/bugzilla/runtests.sh index ab18e2e0044..7b67b176166 100755 --- a/mozilla/webtools/bugzilla/runtests.sh +++ b/mozilla/webtools/bugzilla/runtests.sh @@ -1,11 +1,11 @@ #!/bin/sh -export TEST_VERBOSE=0 +TEST_VERBOSE=0 PART1='use Test::Harness qw(&runtests $verbose); $verbose=' PART2='; runtests @ARGV;' for f in $*; do if [ $f == "--verbose" ] ; then - export TEST_VERBOSE=1 + TEST_VERBOSE=1 fi done diff --git a/mozilla/webtools/bugzilla/t/001compile.t b/mozilla/webtools/bugzilla/t/001compile.t index c297241dfb1..4fc5686625d 100644 --- a/mozilla/webtools/bugzilla/t/001compile.t +++ b/mozilla/webtools/bugzilla/t/001compile.t @@ -45,8 +45,9 @@ use strict; # First now we test the scripts my @testitems = @Support::Files::testitems; -#my %warnings; -my $verbose = $::ENV{TEST_VERBOSE}; +# Capture the TESTERR from Test::More for printing errors. +# This will handle verbosity for us automatically +*TESTOUT = \*Test::More::TESTOUT; my $perlapp = $^X; foreach my $file (@testitems) { @@ -63,15 +64,14 @@ foreach my $file (@testitems) { my $loginfo=`$command`; #print '@@'.$loginfo.'##'; if ($loginfo =~ /syntax ok$/im) { -# $warnings{$_} = 1 foreach ($loginfo =~ /\((W.*?)\)/mg); if ($loginfo ne "$file syntax OK\n") { - if ($verbose) { print STDERR $loginfo; } + print TESTOUT $loginfo; ok(0,$file."--WARNING"); } else { ok(1,$file); } } else { - if ($verbose) { print STDERR $loginfo; } + print TESTOUT $loginfo; ok(0,$file."--ERROR"); } } diff --git a/mozilla/webtools/bugzilla/t/002goodperl.t b/mozilla/webtools/bugzilla/t/002goodperl.t index 3615d489a6e..9047e09ff59 100644 --- a/mozilla/webtools/bugzilla/t/002goodperl.t +++ b/mozilla/webtools/bugzilla/t/002goodperl.t @@ -44,7 +44,6 @@ BEGIN { use Test::More tests => $tests; } use strict; my @testitems = @Support::Files::testitems; # get the files to test. -my $verbose = $::ENV{TEST_VERBOSE}; foreach my $file (@testitems) { $file =~ s/\s.*$//; # nuke everything after the first space (#comment) diff --git a/mozilla/webtools/bugzilla/t/003safesys.t b/mozilla/webtools/bugzilla/t/003safesys.t index 94d137aa97e..98e9d2d64be 100644 --- a/mozilla/webtools/bugzilla/t/003safesys.t +++ b/mozilla/webtools/bugzilla/t/003safesys.t @@ -44,7 +44,9 @@ BEGIN { use Test::More tests => $tests; } use strict; my @testitems = @Support::Files::testitems; -my $verbose = $::ENV{TEST_VERBOSE}; +# Capture the TESTERR from Test::More for printing errors. +# This will handle verbosity for us automatically +*TESTOUT = \*Test::More::TESTOUT; my $perlapp = $^X; foreach my $file (@testitems) { @@ -54,7 +56,7 @@ foreach my $file (@testitems) { my $loginfo=`$command`; if ($loginfo =~ /arguments for Support::Systemexec::(system|exec)/im) { ok(0,"$file DOES NOT use proper system or exec calls"); - if ($verbose) { print STDERR $loginfo; } + print TESTOUT $loginfo; } else { ok(1,"$file uses proper system and exec calls"); } diff --git a/mozilla/webtools/bugzilla/t/004template.t b/mozilla/webtools/bugzilla/t/004template.t index f66176a7217..92d9c511c97 100644 --- a/mozilla/webtools/bugzilla/t/004template.t +++ b/mozilla/webtools/bugzilla/t/004template.t @@ -34,7 +34,6 @@ use Template; my @testitems = @Support::Templates::testitems; my $include_path = $Support::Templates::include_path; -my $verbose = $::ENV{TEST_VERBOSE}; # Check to make sure all templates that are referenced in # Bugzilla exist in the proper place. diff --git a/mozilla/webtools/bugzilla/t/005no_tabs.t b/mozilla/webtools/bugzilla/t/005no_tabs.t index 66b2af5d613..e420baaaaab 100644 --- a/mozilla/webtools/bugzilla/t/005no_tabs.t +++ b/mozilla/webtools/bugzilla/t/005no_tabs.t @@ -32,7 +32,6 @@ BEGIN { use Test::More tests => $tests; } use strict; my @testitems = @Support::Files::testitems; -my $verbose = $::ENV{TEST_VERBOSE}; foreach my $file (@testitems) { open (FILE, "$file");