Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(220)

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_failures.py

Issue 1806393002: Run yapf on files in webkit/layout_tests/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_failures.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_failures.py b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_failures.py
index 5c16b948bdfb019ee6fe5644262d1b13b9889704..59506648c3be80ee31c0fe5e1bd1e98e8eeebc6b 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_failures.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_failures.py
@@ -33,7 +33,9 @@ from webkitpy.layout_tests.models import test_expectations
def is_reftest_failure(failure_list):
failure_types = [type(f) for f in failure_list]
- return set((FailureReftestMismatch, FailureReftestMismatchDidNotOccur, FailureReftestNoImagesGenerated)).intersection(failure_types)
+ return set((FailureReftestMismatch, FailureReftestMismatchDidNotOccur, FailureReftestNoImagesGenerated)).intersection(
+ failure_types)
+
# FIXME: This is backwards. Each TestFailure subclass should know what
# test_expectation type it corresponds too. Then this method just
@@ -57,16 +59,12 @@ def determine_result_type(failure_list):
return test_expectations.TIMEOUT
elif FailureEarlyExit in failure_types:
return test_expectations.SKIP
- elif (FailureMissingResult in failure_types or
- FailureMissingImage in failure_types or
- FailureMissingImageHash in failure_types or
- FailureMissingAudio in failure_types):
+ elif (FailureMissingResult in failure_types or FailureMissingImage in failure_types or
+ FailureMissingImageHash in failure_types or FailureMissingAudio in failure_types):
return test_expectations.MISSING
else:
- is_text_failure = (FailureTextMismatch in failure_types or
- FailureTestHarnessAssertion in failure_types)
- is_image_failure = (FailureImageHashIncorrect in failure_types or
- FailureImageHashMismatch in failure_types)
+ is_text_failure = (FailureTextMismatch in failure_types or FailureTestHarnessAssertion in failure_types)
+ is_image_failure = (FailureImageHashIncorrect in failure_types or FailureImageHashMismatch in failure_types)
is_audio_failure = (FailureAudioMismatch in failure_types)
if is_text_failure and is_image_failure:
return test_expectations.IMAGE_PLUS_TEXT
@@ -77,8 +75,7 @@ def determine_result_type(failure_list):
elif is_audio_failure:
return test_expectations.AUDIO
else:
- raise ValueError("unclassifiable set of failures: "
- + str(failure_types))
+ raise ValueError("unclassifiable set of failures: " + str(failure_types))
class TestFailure(object):
@@ -226,14 +223,9 @@ class FailureEarlyExit(TestFailure):
def message(self):
return "skipped due to early exit"
-
# Convenient collection of all failure classes for anything that might
# need to enumerate over them all.
-ALL_FAILURE_CLASSES = (FailureTimeout, FailureCrash, FailureMissingResult,
- FailureTestHarnessAssertion,
- FailureTextMismatch, FailureMissingImageHash,
- FailureMissingImage, FailureImageHashMismatch,
- FailureImageHashIncorrect, FailureReftestMismatch,
- FailureReftestMismatchDidNotOccur, FailureReftestNoImagesGenerated,
- FailureMissingAudio, FailureAudioMismatch,
- FailureEarlyExit)
+ALL_FAILURE_CLASSES = (FailureTimeout, FailureCrash, FailureMissingResult, FailureTestHarnessAssertion, FailureTextMismatch,
+ FailureMissingImageHash, FailureMissingImage, FailureImageHashMismatch, FailureImageHashIncorrect,
+ FailureReftestMismatch, FailureReftestMismatchDidNotOccur, FailureReftestNoImagesGenerated,
+ FailureMissingAudio, FailureAudioMismatch, FailureEarlyExit)

Powered by Google App Engine
This is Rietveld 408576698