Commit 3cf777e1 authored by Bjorn Munch's avatar Bjorn Munch

Bug #43148 MTR2 + PB2 should display correct number of failures

Retried tests are reported the same as ordinary ones
Prepend "retry-" to fail or pass for retried tests
parent 7a6133c7
...@@ -113,6 +113,7 @@ sub mtr_report_test ($) { ...@@ -113,6 +113,7 @@ sub mtr_report_test ($) {
my $logfile= $tinfo->{'logfile'}; my $logfile= $tinfo->{'logfile'};
my $warnings= $tinfo->{'warnings'}; my $warnings= $tinfo->{'warnings'};
my $result= $tinfo->{'result'}; my $result= $tinfo->{'result'};
my $retry= $tinfo->{'retries'} ? "retry-" : "";
if ($result eq 'MTR_RES_FAILED'){ if ($result eq 'MTR_RES_FAILED'){
...@@ -147,7 +148,7 @@ sub mtr_report_test ($) { ...@@ -147,7 +148,7 @@ sub mtr_report_test ($) {
if ( $warnings ) if ( $warnings )
{ {
mtr_report("[ $fail ] Found warnings/errors in server log file!"); mtr_report("[ $retry$fail ] Found warnings/errors in server log file!");
mtr_report(" Test ended at $timest"); mtr_report(" Test ended at $timest");
mtr_report($warnings); mtr_report($warnings);
return; return;
...@@ -155,14 +156,14 @@ sub mtr_report_test ($) { ...@@ -155,14 +156,14 @@ sub mtr_report_test ($) {
my $timeout= $tinfo->{'timeout'}; my $timeout= $tinfo->{'timeout'};
if ( $timeout ) if ( $timeout )
{ {
mtr_report("[ $fail ] timeout after $timeout seconds"); mtr_report("[ $retry$fail ] timeout after $timeout seconds");
mtr_report(" Test ended at $timest"); mtr_report(" Test ended at $timest");
mtr_report("\n$tinfo->{'comment'}"); mtr_report("\n$tinfo->{'comment'}");
return; return;
} }
else else
{ {
mtr_report("[ $fail ]\n Test ended at $timest"); mtr_report("[ $retry$fail ]\n Test ended at $timest");
} }
if ( $logfile ) if ( $logfile )
...@@ -205,7 +206,7 @@ sub mtr_report_test ($) { ...@@ -205,7 +206,7 @@ sub mtr_report_test ($) {
{ {
my $timer_str= $tinfo->{timer} || ""; my $timer_str= $tinfo->{timer} || "";
$tot_real_time += ($timer_str/1000); $tot_real_time += ($timer_str/1000);
mtr_report("[ pass ] ", sprintf("%5s", $timer_str)); mtr_report("[ ${retry}pass ] ", sprintf("%5s", $timer_str));
# Show any problems check-testcase found # Show any problems check-testcase found
if ( defined $tinfo->{'check'} ) if ( defined $tinfo->{'check'} )
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment