Skip to content

Commit 2fd7b16

Browse files
committed
TESTrun: Use more 'newdir' and 'diffdir' variables
1 parent 1abe1f8 commit 2fd7b16

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

tests/TESTrun

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -97,8 +97,8 @@ sub runtest {
9797
my $coredump = false;
9898
my $status = 0;
9999
my $linecount = 0;
100-
my $rawstderrlog = "tests/NEW/${outputbase}.raw.stderr";
101-
my $stderrlog = "tests/NEW/${outputbase}.stderr";
100+
my $rawstderrlog = "${newdir}/${outputbase}.raw.stderr";
101+
my $stderrlog = "${newdir}/${outputbase}.stderr";
102102
my $diffstat = 0;
103103
my $errdiffstat = 0;
104104

@@ -107,7 +107,7 @@ sub runtest {
107107
#
108108
# Furthermore, on Windows, fc can't read the standard input, so we
109109
# can't do it as a pipeline in any case.
110-
$r = system "$TCPDUMP -# -n -r $input $options >tests/NEW/${outputbase} 2>${rawstderrlog}";
110+
$r = system "$TCPDUMP -# -n -r $input $options >${newdir}/${outputbase} 2>${rawstderrlog}";
111111
if($r != 0) {
112112
#
113113
# Something other than "tcpdump opened the file, read it, and
@@ -117,7 +117,7 @@ sub runtest {
117117
# wrote out, so it shows up when we diff the expected output
118118
# with it.
119119
#
120-
open(OUTPUT, ">>"."tests/NEW/$outputbase") || die "fail to open $outputbase\n";
120+
open(OUTPUT, ">>"."${newdir}/$outputbase") || die "fail to open $outputbase\n";
121121
if($r == -1) {
122122
# failed to start due to error.
123123
$status = $!;
@@ -228,7 +228,7 @@ sub runtest {
228228
$r = system "fc /lb1000 /t /1 $winoutput tests\\NEW\\$outputbase >tests\\DIFF\\$outputbase.diff";
229229
$diffstat = $r >> 8;
230230
} else {
231-
$r = system "diff $diff_flags $output tests/NEW/$outputbase >tests/DIFF/$outputbase.diff";
231+
$r = system "diff $diff_flags $output ${newdir}/$outputbase >${diffdir}/$outputbase.diff";
232232
$diffstat = WEXITSTATUS($r);
233233
}
234234
}
@@ -261,7 +261,7 @@ sub runtest {
261261
$nr = system "fc /lb1000 /t /1 $winoutput.stderr $canonstderrlog >tests\DIFF\$outputbase.stderr.diff";
262262
$errdiffstat = $nr >> 8;
263263
} else {
264-
$nr = system "diff $output.stderr $stderrlog >tests/DIFF/$outputbase.stderr.diff";
264+
$nr = system "diff $output.stderr $stderrlog >${diffdir}/$outputbase.stderr.diff";
265265
$errdiffstat = WEXITSTATUS($nr);
266266
}
267267
if($r == 0) {
@@ -286,22 +286,22 @@ sub runtest {
286286
printf " %-40s: passed with error messages:\n", $name;
287287
showfile($stderrlog);
288288
}
289-
unlink "tests/DIFF/$outputbase.diff";
289+
unlink "${diffdir}/$outputbase.diff";
290290
return 0;
291291
}
292292
# must have failed!
293293
printf " %-40s: TEST FAILED(exit core=%d/diffstat=%d,%d/r=%d)", $name, $coredump, $diffstat, $errdiffstat, $r;
294294
open FOUT, '>>tests/failure-outputs.txt';
295295
printf FOUT "\nFailed test: $name\n\n";
296296
close FOUT;
297-
if(-f "tests/DIFF/$outputbase.diff") {
297+
if(-f "${diffdir}/$outputbase.diff") {
298298
#
299299
# XXX - just do this directly in Perl?
300300
#
301301
if ($^O eq 'MSWin32') {
302302
system "type tests\\DIFF\\$outputbase.diff >> tests\\failure-outputs.txt";
303303
} else {
304-
system "cat tests/DIFF/$outputbase.diff >> tests/failure-outputs.txt";
304+
system "cat ${diffdir}/$outputbase.diff >> tests/failure-outputs.txt";
305305
}
306306
}
307307

0 commit comments

Comments
 (0)