diff options
author | Jelmer Vernooij <jelmer@samba.org> | 2008-04-16 00:03:45 +0200 |
---|---|---|
committer | Jelmer Vernooij <jelmer@samba.org> | 2008-04-16 00:03:45 +0200 |
commit | 8eb6d59a466cf7d9c79b1a271737873c8744ed16 (patch) | |
tree | 5e7506a4e806add411bbf92486e7de645eaadb04 | |
parent | d3729a8efa1c7e848d263ef320d3114144477be3 (diff) | |
download | samba-8eb6d59a466cf7d9c79b1a271737873c8744ed16.tar.gz samba-8eb6d59a466cf7d9c79b1a271737873c8744ed16.tar.xz samba-8eb6d59a466cf7d9c79b1a271737873c8744ed16.zip |
Properly warn about exit codes that indicate success while tests claim
failure.
(This used to be commit 92873264d707bed88beac54a73a29d66c011f80c)
-rw-r--r-- | source4/selftest/output/plain.pm | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/source4/selftest/output/plain.pm b/source4/selftest/output/plain.pm index 25ff74792e5..f14e26b38d4 100644 --- a/source4/selftest/output/plain.pm +++ b/source4/selftest/output/plain.pm @@ -81,6 +81,9 @@ sub end_testsuite($$$$$) my $out = ""; if ($unexpected) { + if ($result eq "success" and not defined($reason)) { + $reason = "Expected negative exit code, got positive exit code"; + } $self->output_msg("ERROR: $reason\n"); push (@{$self->{suitesfailed}}, $name); } else { |