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

Side by Side Diff: Tools/Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py

Issue 294973008: Add --sanitizer-mode to run-webkit-tests (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: rename command line flag, add tests; ready for review Created 6 years, 6 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 | « Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py ('k') | 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 # Copyright (C) 2010 Google Inc. All rights reserved. 1 # Copyright (C) 2010 Google Inc. All rights reserved.
2 # Copyright (C) 2010 Gabor Rapcsanyi (rgabor@inf.u-szeged.hu), University of Sze ged 2 # Copyright (C) 2010 Gabor Rapcsanyi (rgabor@inf.u-szeged.hu), University of Sze ged
3 # Copyright (C) 2011 Apple Inc. All rights reserved. 3 # Copyright (C) 2011 Apple Inc. All rights reserved.
4 # 4 #
5 # Redistribution and use in source and binary forms, with or without 5 # Redistribution and use in source and binary forms, with or without
6 # modification, are permitted provided that the following conditions are 6 # modification, are permitted provided that the following conditions are
7 # met: 7 # met:
8 # 8 #
9 # * Redistributions of source code must retain the above copyright 9 # * Redistributions of source code must retain the above copyright
10 # notice, this list of conditions and the following disclaimer. 10 # notice, this list of conditions and the following disclaimer.
(...skipping 248 matching lines...) Expand 10 before | Expand all | Expand 10 after
259 tests_included=True, shared_port=False) 259 tests_included=True, shared_port=False)
260 self.assertTrue(any(['Running 1 ' in line for line in regular_output .buflist])) 260 self.assertTrue(any(['Running 1 ' in line for line in regular_output .buflist]))
261 261
262 def test_dryrun(self): 262 def test_dryrun(self):
263 tests_run = get_tests_run(['--dry-run']) 263 tests_run = get_tests_run(['--dry-run'])
264 self.assertEqual(tests_run, []) 264 self.assertEqual(tests_run, [])
265 265
266 tests_run = get_tests_run(['-n']) 266 tests_run = get_tests_run(['-n'])
267 self.assertEqual(tests_run, []) 267 self.assertEqual(tests_run, [])
268 268
269 def test_enable_sanitizer(self):
270 self.assertTrue(passing_run(['--enable-sanitizer', 'failures/expected/te xt.html']))
271
269 def test_exception_raised(self): 272 def test_exception_raised(self):
270 # Exceptions raised by a worker are treated differently depending on 273 # Exceptions raised by a worker are treated differently depending on
271 # whether they are in-process or out. inline exceptions work as normal, 274 # whether they are in-process or out. inline exceptions work as normal,
272 # which allows us to get the full stack trace and traceback from the 275 # which allows us to get the full stack trace and traceback from the
273 # worker. The downside to this is that it could be any error, but this 276 # worker. The downside to this is that it could be any error, but this
274 # is actually useful in testing. 277 # is actually useful in testing.
275 # 278 #
276 # Exceptions raised in a separate process are re-packaged into 279 # Exceptions raised in a separate process are re-packaged into
277 # WorkerExceptions (a subclass of BaseException), which have a string ca pture of the stack which can 280 # WorkerExceptions (a subclass of BaseException), which have a string ca pture of the stack which can
278 # be printed, but don't display properly in the unit test exception hand lers. 281 # be printed, but don't display properly in the unit test exception hand lers.
(...skipping 764 matching lines...) Expand 10 before | Expand all | Expand 10 after
1043 stderr = StringIO.StringIO() 1046 stderr = StringIO.StringIO()
1044 res = run_webkit_tests.main(['--platform', 'test', '--exit-after-n-failu res', '1', 1047 res = run_webkit_tests.main(['--platform', 'test', '--exit-after-n-failu res', '1',
1045 'failures/unexpected/missing_text.html', 1048 'failures/unexpected/missing_text.html',
1046 'failures/unexpected/missing_image.html'], 1049 'failures/unexpected/missing_image.html'],
1047 stdout, stderr) 1050 stdout, stderr)
1048 self.assertEqual(res, test_run_results.EARLY_EXIT_STATUS) 1051 self.assertEqual(res, test_run_results.EARLY_EXIT_STATUS)
1049 self.assertEqual(stdout.getvalue(), 1052 self.assertEqual(stdout.getvalue(),
1050 ('\n' 1053 ('\n'
1051 'Regressions: Unexpected missing results (1)\n' 1054 'Regressions: Unexpected missing results (1)\n'
1052 ' failures/unexpected/missing_image.html [ Missing ]\n\n')) 1055 ' failures/unexpected/missing_image.html [ Missing ]\n\n'))
OLDNEW
« no previous file with comments | « Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698