Changeset 7227

Timestamp:

05/24/2018 09:16:48 PM (5 years ago)

Author:

danielbachhuber

Message:

phpunit-test-reporter: Only report result status when tests were actually run

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/parts

Files:

• result-set.php (modified) (1 diff)
• single-result.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/parts/result-set.php

@@ -36 +36 @@
                     $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';

sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/parts/single-result.php

@@ -5 +5 @@
 $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';