risky test should not fire when we have a skipped test

zynga · Jul 19, 2019 · b1e57f5 · b1e57f5
1 parent 07c1eb2
commit b1e57f5
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/Zynga/PHPUnit/V2/TestResult.hh b/src/Zynga/PHPUnit/V2/TestResult.hh
@@ -1015,6 +1015,8 @@ class TestResult {
       error_log(
         'JEO WARNING - NON-Exception based error thrown e='.gettype($e),
       );
+    } else if ($error === true && $skipped == true) {
+      // skipped is a noop atm.
     } else if ($error === true && $e instanceof Exception) {
       $this->addError($test, $e, $time);
     } else if ($failure === true && $e instanceof Exception) {