mirror of
https://github.com/moparisthebest/curl
synced 2024-11-16 22:45:03 -05:00
runtests: keep logfiles around by default
Make '-k' a no-op. The singletest function now clears the log directory BEFORE each individual test and not after, which makes it possible to always keep the logfiles around after a test has been run. No need to specify -k anymore. Keeping the option parsing around to work with users of old habits. Some tests also didn't work properly when -k was used (since the old logs would be kep when a new test starts) which this change also fixes. Closes #4035
This commit is contained in:
parent
6c2b7d44e3
commit
6617db6a7e
@ -3202,6 +3202,9 @@ sub singletest {
|
|||||||
my $cmd;
|
my $cmd;
|
||||||
my $disablevalgrind;
|
my $disablevalgrind;
|
||||||
|
|
||||||
|
# fist, remove all lingering log files
|
||||||
|
cleardir($LOGDIR);
|
||||||
|
|
||||||
# copy test number to a global scope var, this allows
|
# copy test number to a global scope var, this allows
|
||||||
# testnum checking when starting test harness servers.
|
# testnum checking when starting test harness servers.
|
||||||
$testnumcheck = $testnum;
|
$testnumcheck = $testnum;
|
||||||
@ -3862,9 +3865,6 @@ sub singletest {
|
|||||||
|
|
||||||
# Skip all the verification on torture tests
|
# Skip all the verification on torture tests
|
||||||
if ($torture) {
|
if ($torture) {
|
||||||
if(!$cmdres && !$keepoutfiles) {
|
|
||||||
cleardir($LOGDIR);
|
|
||||||
}
|
|
||||||
# timestamp test result verification end
|
# timestamp test result verification end
|
||||||
$timevrfyend{$testnum} = Time::HiRes::time();
|
$timevrfyend{$testnum} = Time::HiRes::time();
|
||||||
return $cmdres;
|
return $cmdres;
|
||||||
@ -4295,10 +4295,6 @@ sub singletest {
|
|||||||
logmsg "PASS: $testnum - $testname\n";
|
logmsg "PASS: $testnum - $testname\n";
|
||||||
}
|
}
|
||||||
|
|
||||||
# the test succeeded, remove all log files
|
|
||||||
if(!$keepoutfiles) {
|
|
||||||
cleardir($LOGDIR);
|
|
||||||
}
|
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user