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

Side by Side 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/python 1 #!/usr/bin/python
2 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved. 2 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 6
7 """Parses and displays the contents of one or more autoserv result directories. 7 """Parses and displays the contents of one or more autoserv result directories.
8 8
9 This script parses the contents of one or more autoserv results folders and 9 This script parses the contents of one or more autoserv results folders and
10 generates test reports. 10 generates test reports.
(...skipping 259 matching lines...) Expand 10 before | Expand all | Expand 10 after
270 help='Don\'t strip a prefix from test directory names') 270 help='Don\'t strip a prefix from test directory names')
271 (options, args) = parser.parse_args() 271 (options, args) = parser.parse_args()
272 272
273 if not args: 273 if not args:
274 parser.print_help() 274 parser.print_help()
275 Die('no result directories provided') 275 Die('no result directories provided')
276 276
277 generator = ReportGenerator(options, args) 277 generator = ReportGenerator(options, args)
278 generator.Run() 278 generator.Run()
279 279
280 for v in generator._results.itervalues():
petkov 2010/08/25 21:21:15 theoretically, _ signifies a private member. care
281 if v["status"] != 'PASS':
petkov 2010/08/25 21:21:15 change to 'status' for consistency.
282 sys.exit(1)
280 283
281 if __name__ == '__main__': 284 if __name__ == '__main__':
282 main() 285 main()
OLDNEW
« 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