From bdce6c8361a0ee54a06d49c948be14225f9dd2aa Mon Sep 17 00:00:00 2001 From: Marc-Antoine Ruel Date: Tue, 7 May 2019 15:32:53 +0000 Subject: [PATCH] Fix auto_stub.py for skipped tests. This is blocking a luci roll in infra. R=nodir@chromium.org Change-Id: I996a74ba519cf05f72d430ca4a2e63e76436b509 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/1598469 Reviewed-by: Nodir Turakulov Commit-Queue: Marc-Antoine Ruel --- testing_support/auto_stub.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/testing_support/auto_stub.py b/testing_support/auto_stub.py index c5a3495c6..b266692e9 100644 --- a/testing_support/auto_stub.py +++ b/testing_support/auto_stub.py @@ -70,4 +70,7 @@ class TestCase(unittest.TestCase, AutoStubMixIn): def has_failed(self): """Returns True if the test has failed.""" + if not self._resultForDoCleanups: + # Maybe skipped. + return False return not self._resultForDoCleanups.wasSuccessful()