Changeset 9752

Timestamp:

04/20/2020 10:28:27 PM (5 years ago)

Author:

iandunn

Message:

Events: Show test failure diff for easier comparison.

The full results can still be seen with the new --verbose flag.

File:

• sites/trunk/api.wordpress.org/public_html/events/1.0/tests/test-index.php (modified) (2 diffs)

sites/trunk/api.wordpress.org/public_html/events/1.0/tests/test-index.php

@@ -64 +64 @@
         $actual_output   = is_scalar( $actual_result   ) ? var_export( $actual_result,   true ) : print_r( $actual_result,   true );
 
-        printf(
-            "\n\nExpected result: %s\nActual result: %s",
-            $expected_output,
-            $actual_output
-        );
+        if ( VERBOSE_OUTPUT ) {
+            printf(
+                "\n\nExpected result: %s\nActual result: %s",
+                $expected_output,
+                $actual_output
+            );
+
+        } else {
+            $folder = sys_get_temp_dir();
+            file_put_contents( $folder . '/events-test-expected.txt', $expected_output);
+            file_put_contents( $folder . '/events-test-actual.txt',   $actual_output );
+
+            $diff_results = shell_exec( sprintf(
+                '/usr/bin/diff --unified %s --label "Expected" %s --label "Actual"',
+                $folder . '/events-test-expected.txt',
+                $folder . '/events-test-actual.txt'
+            ) );
+
+            if ( empty( $diff_results ) ) {
+                $diff_results = "Error: diff appears to be empty even though test failed.\n";
+            }
+
+            $short_output = sprintf( "\n\n%s
+                Rerun with `--verbose` for detailed failure results.\n",
+                $diff_results
+            );
+
+            printf( str_replace( "\t", '', $short_output ) );
+        }
     }
 }
@@ -1526 +1550 @@
 }
 
+
+define( 'VERBOSE_OUTPUT', in_array( '--verbose', $argv, true ) );
+
 run_tests();