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

Side by Side Diff: tools/testing/dart/test_runner.dart

Issue 9960050: Add runtime=dartium webdriver support to test.dart (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Fix for batching Created 8 years, 8 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
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 /** 5 /**
6 * Classes and methods for executing tests. 6 * Classes and methods for executing tests.
7 * 7 *
8 * This module includes: 8 * This module includes:
9 * - Managing parallel execution of tests, including timeout checks. 9 * - Managing parallel execution of tests, including timeout checks.
10 * - Evaluating the output of each test as pass/fail/crash/timeout. 10 * - Evaluating the output of each test as pass/fail/crash/timeout.
(...skipping 118 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 final mode = configuration['mode']; 129 final mode = configuration['mode'];
130 final arch = configuration['arch']; 130 final arch = configuration['arch'];
131 return "$compiler-$runtime ${mode}_$arch"; 131 return "$compiler-$runtime ${mode}_$arch";
132 } 132 }
133 133
134 List<String> get batchRunnerArguments() => ['-batch']; 134 List<String> get batchRunnerArguments() => ['-batch'];
135 List<String> get batchTestArguments() => commands.last().arguments; 135 List<String> get batchTestArguments() => commands.last().arguments;
136 136
137 void completed() { completedHandler(this); } 137 void completed() { completedHandler(this); }
138 138
139 bool get usesWebDriver() => (const ['chrome', 'ff', 'safari', 'ie', 'opera']) 139 bool get usesWebDriver() => (const ['chrome', 'dartium', 'ff', 'safari',
140 'ie', 'opera'])
140 .indexOf(configuration['runtime']) >= 0; 141 .indexOf(configuration['runtime']) >= 0;
141 } 142 }
142 143
143 144
144 /** 145 /**
145 * BrowserTestCase has an extra compilation command that is run in a separate 146 * BrowserTestCase has an extra compilation command that is run in a separate
146 * process, before the regular test is run as in the base class [TestCase]. 147 * process, before the regular test is run as in the base class [TestCase].
147 * If the compilation command fails, then the rest of the test is not run. 148 * If the compilation command fails, then the rest of the test is not run.
148 */ 149 */
149 class BrowserTestCase extends TestCase { 150 class BrowserTestCase extends TestCase {
(...skipping 991 matching lines...) Expand 10 before | Expand all | Expand 10 after
1141 // the developer doesn't waste his or her time trying to fix a bunch of 1142 // the developer doesn't waste his or her time trying to fix a bunch of
1142 // tests that appear to be broken but were actually just flakes that 1143 // tests that appear to be broken but were actually just flakes that
1143 // didn't get retried because there had already been one failure. 1144 // didn't get retried because there had already been one failure.
1144 bool allowRetry = _MAX_FAILED_NO_RETRY > _progress.numFailedTests; 1145 bool allowRetry = _MAX_FAILED_NO_RETRY > _progress.numFailedTests;
1145 new RunningProcess(test, allowRetry, this).start(); 1146 new RunningProcess(test, allowRetry, this).start();
1146 } 1147 }
1147 _numProcesses++; 1148 _numProcesses++;
1148 } 1149 }
1149 } 1150 }
1150 } 1151 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698