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

Unified Diff: generate_test_report.py

Issue 3150036: adds a flag to exit if a test fails (Closed) Base URL: ssh://git@gitrw.chromium.org:9222/crosutils.git
Patch Set: exit on fail is now the default Created 10 years, 4 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 9235482cb9818f2493afbb97d30430933ee884a3..e148c599df306395a2bf2a39b937d040acfd757e 100755
--- a/generate_test_report.py
+++ b/generate_test_report.py
@@ -277,6 +277,9 @@ def main():
generator = ReportGenerator(options, args)
generator.Run()
+ for v in generator._results.itervalues():
petkov 2010/08/25 21:21:15 theoretically, _ signifies a private member. care
+ if v["status"] != 'PASS':
petkov 2010/08/25 21:21:15 change to 'status' for consistency.
+ sys.exit(1)
if __name__ == '__main__':
main()
« 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