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

Side by Side Diff: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/models/test_results.py

Issue 1783073002: Run auto-formatter on files in webkitpy/layout_tests/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Ran yapf -i --style '{based_on_style: pep8, column_limit: 132}' then did manual fix-up 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 unified diff | Download patch
OLDNEW
1 # Copyright (C) 2010 Google Inc. All rights reserved. 1 # Copyright (C) 2010 Google Inc. All rights reserved.
2 # 2 #
3 # Redistribution and use in source and binary forms, with or without 3 # Redistribution and use in source and binary forms, with or without
4 # modification, are permitted provided that the following conditions are 4 # modification, are permitted provided that the following conditions are
5 # met: 5 # met:
6 # 6 #
7 # * Redistributions of source code must retain the above copyright 7 # * Redistributions of source code must retain the above copyright
8 # notice, this list of conditions and the following disclaimer. 8 # notice, this list of conditions and the following disclaimer.
9 # * Redistributions in binary form must reproduce the above 9 # * Redistributions in binary form must reproduce the above
10 # copyright notice, this list of conditions and the following disclaimer 10 # copyright notice, this list of conditions and the following disclaimer
(...skipping 20 matching lines...) Expand all
31 from webkitpy.layout_tests.models import test_failures 31 from webkitpy.layout_tests.models import test_failures
32 32
33 33
34 class TestResult(object): 34 class TestResult(object):
35 """Data object containing the results of a single test.""" 35 """Data object containing the results of a single test."""
36 36
37 @staticmethod 37 @staticmethod
38 def loads(string): 38 def loads(string):
39 return cPickle.loads(string) 39 return cPickle.loads(string)
40 40
41 def __init__(self, test_name, failures=None, test_run_time=None, has_stderr= False, reftest_type=None, pid=None, references=None, device_failed=False, has_re paint_overlay=False): 41 def __init__(self,
42 test_name,
43 failures=None,
44 test_run_time=None,
45 has_stderr=False,
46 reftest_type=None,
47 pid=None,
48 references=None,
49 device_failed=False,
50 has_repaint_overlay=False):
42 self.test_name = test_name 51 self.test_name = test_name
43 self.failures = failures or [] 52 self.failures = failures or []
44 self.test_run_time = test_run_time or 0 # The time taken to execute the test itself. 53 self.test_run_time = test_run_time or 0 # The time taken to execute the test itself.
45 self.has_stderr = has_stderr 54 self.has_stderr = has_stderr
46 self.reftest_type = reftest_type or [] 55 self.reftest_type = reftest_type or []
47 self.pid = pid 56 self.pid = pid
48 self.references = references or [] 57 self.references = references or []
49 self.device_failed = device_failed 58 self.device_failed = device_failed
50 self.has_repaint_overlay = has_repaint_overlay 59 self.has_repaint_overlay = has_repaint_overlay
51 60
52 # FIXME: Setting this in the constructor makes this class hard to mutate . 61 # FIXME: Setting this in the constructor makes this class hard to mutate .
53 self.type = test_failures.determine_result_type(failures) 62 self.type = test_failures.determine_result_type(failures)
54 63
55 # These are set by the worker, not by the driver, so they are not passed to the constructor. 64 # These are set by the worker, not by the driver, so they are not passed to the constructor.
56 self.worker_name = '' 65 self.worker_name = ''
57 self.shard_name = '' 66 self.shard_name = ''
58 self.total_run_time = 0 # The time taken to run the test plus any refer ences, compute diffs, etc. 67 self.total_run_time = 0 # The time taken to run the test plus any refer ences, compute diffs, etc.
59 self.test_number = None 68 self.test_number = None
60 69
61 def __eq__(self, other): 70 def __eq__(self, other):
62 return (self.test_name == other.test_name and 71 return (self.test_name == other.test_name and self.failures == other.fai lures and self.test_run_time == other.test_run_time)
63 self.failures == other.failures and
64 self.test_run_time == other.test_run_time)
65 72
66 def __ne__(self, other): 73 def __ne__(self, other):
67 return not (self == other) 74 return not (self == other)
68 75
69 def has_failure_matching_types(self, *failure_classes): 76 def has_failure_matching_types(self, *failure_classes):
70 for failure in self.failures: 77 for failure in self.failures:
71 if type(failure) in failure_classes: 78 if type(failure) in failure_classes:
72 return True 79 return True
73 return False 80 return False
74 81
75 def dumps(self): 82 def dumps(self):
76 return cPickle.dumps(self) 83 return cPickle.dumps(self)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698