Changeset 33:d466f464c871 in pyenvjasmine for pyenvjasmine/runner.py

Timestamp:

Jan 19, 2018, 11:43:48 AM (6 years ago)

Author:

Borja Lopez <borja@…>

Branch:

default

Phase:

public

Message:

Fix for Runner.did_test_pass(), ensuring we catch failed tests
even if they do not appear in the "Failed:" report at the end of the
tests run

File:

• pyenvjasmine/runner.py (modified) (1 diff)

pyenvjasmine/runner.py

@@ -186 +186 @@
 
     def did_test_pass(self, stdout):
+        if 'FAILED' in stdout:
+            # it can happen that a test fails because of some timing issues
+            # (timer error). In such case it may happen that the test does
+            # not appear in the "Failed" report at the end, even if it
+            # failed, because the execution is interrupted there (no more
+            # tests are even run afterwards)
+            #
+            # in such case, we consider tests failed
+            return False
+        # Otherwise, look for a "Failed: 0" status, which we consider as
+        # tests passing ok
         for line in stdout.splitlines():
             if 'Failed' in line: