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

Unified Diff: third_party/WebKit/LayoutTests/external/wpt/check_stability.py

Issue 2695813009: Import wpt@503f5b5f78ec4e87d144f78609f363f0ed0ea8db (Closed)
Patch Set: Skip some tests Created 3 years, 10 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/LayoutTests/external/wpt/check_stability.py
diff --git a/third_party/WebKit/LayoutTests/external/wpt/check_stability.py b/third_party/WebKit/LayoutTests/external/wpt/check_stability.py
index 4e6d3232358c6bbc2a721aa055481b73c8d51d6e..a411f338b04b51b732ff298b466e0f0295ad9971 100644
--- a/third_party/WebKit/LayoutTests/external/wpt/check_stability.py
+++ b/third_party/WebKit/LayoutTests/external/wpt/check_stability.py
@@ -575,17 +575,23 @@ def write_inconsistent(inconsistent, iterations):
def write_results(results, iterations, comment_pr):
"""Output all test results to logger.info."""
+ pr_number = None
+ if comment_pr:
+ try:
+ pr_number = int(comment_pr)
+ except ValueError:
+ pass
logger.info("## All results ##\n")
+ if pr_number:
+ logger.info("<details>\n")
+ logger.info("<summary>%i %s ran</summary>\n\n" % (len(results),
+ "tests" if len(results) > 1
+ else "test"))
+
for test, test_results in results.iteritems():
baseurl = "http://w3c-test.org/submissions"
if "https" in os.path.splitext(test)[0].split(".")[1:]:
baseurl = "https://w3c-test.org/submissions"
- pr_number = None
- if comment_pr:
- try:
- pr_number = int(comment_pr)
- except ValueError:
- pass
if pr_number:
logger.info("<details>\n")
logger.info('<summary><a href="%s/%s%s">%s</a></summary>\n\n' %
@@ -601,6 +607,9 @@ def write_results(results, iterations, comment_pr):
if pr_number:
logger.info("</details>\n")
+ if pr_number:
+ logger.info("</details>\n")
+
def get_parser():
"""Create and return script-specific argument parser."""

Powered by Google App Engine
This is Rietveld 408576698