Merge pull request #77 from getsource/port-7227

Only report result status when tests were actually run. Props @danielbachhuber.
WordPress · May 15, 2020 · dfb2bd6 · dfb2bd6
2 parents a132352 + 0db3c98
commit dfb2bd6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/parts/result-set.php b/parts/result-set.php
@@ -35,7 +35,7 @@
 					$status       = 'Errored';
 					$status_title = 'No results found for test.';
 					$results      = get_post_meta( $report->ID, 'results', true );
-					if ( isset( $results['failures'] ) ) {
+					if ( isset( $results['failures'] ) && ! empty( $results['tests'] ) ) {
 						$status       = 0 === (int) $results['failures'] && 0 === (int) $results['errors'] ? 'Passed' : 'Failed';
 						$status_title = (int) $results['tests'] . ' tests, ' . (int) $results['failures'] . ' failed, ' . (int) $results['errors'] . ' errors';
 					}

diff --git a/parts/single-result.php b/parts/single-result.php
@@ -4,7 +4,7 @@
 $status       = 'Errored';
 $status_title = 'No results found for test.';
 $results      = get_post_meta( $report->ID, 'results', true );
-if ( isset( $results['failures'] ) ) {
+if ( isset( $results['failures'] ) && ! empty( $results['tests'] ) ) {
 	$status       = 0 === (int) $results['failures'] && 0 === (int) $results['errors'] ? 'Passed' : 'Failed';
 	$status_title = (int) $results['tests'] . ' tests, ' . (int) $results['failures'] . ' failed, ' . (int) $results['errors'] . ' errors';
 }