third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/views/buildbot_results.py - Issue 1783073002: Run auto-formatter on files in webkitpy/layout_tests/.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/views/buildbot_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: Rebased 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

« no previous file with comments | « third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/servers/wptserve.py ('k') | third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/views/buildbot_results_unittest.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/views/buildbot_results.py

diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/views/buildbot_results.py b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/views/buildbot_results.py

index d7d4445f226ee3e8812c78b0d64081532556ba34..77924c95b8916fb77db1880d3462deca601a872c 100644

--- a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/views/buildbot_results.py

+++ b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/views/buildbot_results.py

@@ -68,14 +68,14 @@ class BuildBotPrinter(object):

self._print("")

# FIXME: We should be skipping anything marked WONTFIX, so we shouldn't bother logging these stats.

self._print_run_results_entry(run_results, test_expectations.WONTFIX,

- "Tests that will only be fixed if they crash (WONTFIX)")

+ "Tests that will only be fixed if they crash (WONTFIX)")

self._print("")

def _print_run_results_entry(self, run_results, timeline, heading):

total = len(run_results.tests_by_timeline[timeline])

not_passing = (total -

- len(run_results.tests_by_expectation[test_expectations.PASS] &

- run_results.tests_by_timeline[timeline]))

+ len(run_results.tests_by_expectation[test_expectations.PASS] &

+ run_results.tests_by_timeline[timeline]))

self._print("=> %s (%d):" % (heading, not_passing))

for result in TestExpectations.EXPECTATION_DESCRIPTIONS.keys():

@@ -143,7 +143,8 @@ class BuildBotPrinter(object):

actual = result['actual'].split(" ")

expected = result['expected'].split(" ")

# FIXME: clean this up once the old syntax is gone

- new_expectations_list = [TestExpectationParser._inverted_expectation_tokens[exp] for exp in list(set(actual) | set(expected))]

+ new_expectations_list = [TestExpectationParser._inverted_expectation_tokens[exp]

+ for exp in list(set(actual) | set(expected))]

self._print(" %s [ %s ]" % (test, " ".join(new_expectations_list)))

self._print("")