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

Unified Diff: generate_test_report.py

Issue 4156003: Exclude fake crash reporter directories from report. (Closed) Base URL: http://git.chromium.org/git/crosutils.git
Patch Set: Remove comment Created 10 years, 2 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: generate_test_report.py
diff --git a/generate_test_report.py b/generate_test_report.py
index 97c3ec6f0da48e921caaa4eb2c11774ed3d84573..f152a61a7468d34e09012bb58f44faa4aed3d8bf 100755
--- a/generate_test_report.py
+++ b/generate_test_report.py
@@ -98,6 +98,10 @@ class ReportGenerator(object):
if not os.path.isfile(status_file):
return
+ # Remove false positives that are missing a debug dir.
+ if not os.path.exists(os.path.join(testdir, 'debug')):
+ return
+
status_raw = open(status_file, 'r').read()
status = 'FAIL'
if (re.search(r'GOOD.+completed successfully', status_raw) and
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698