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

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

Issue 2633213003: Do some housekeeping in tools/testing/dart (Closed)
Patch Set: Add comment Created 3 years, 11 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
« no previous file with comments | « tools/testing/dart/test_progress.dart ('k') | tools/testing/dart/test_suite.dart » ('j') | 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) 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 3121 matching lines...) Expand 10 before | Expand all | Expand 10 after
3132 bool replaying = recordedInputFile != null; 3132 bool replaying = recordedInputFile != null;
3133 3133
3134 // When the graph building is finished, notify event listeners. 3134 // When the graph building is finished, notify event listeners.
3135 _graph.events 3135 _graph.events
3136 .where((event) => event is dgraph.GraphSealedEvent) 3136 .where((event) => event is dgraph.GraphSealedEvent)
3137 .listen((event) { 3137 .listen((event) {
3138 eventAllTestsKnown(); 3138 eventAllTestsKnown();
3139 }); 3139 });
3140 3140
3141 // Queue commands as they become "runnable" 3141 // Queue commands as they become "runnable"
3142 var commandEnqueuer = new CommandEnqueuer(_graph); 3142 new CommandEnqueuer(_graph);
3143 3143
3144 // CommandExecutor will execute commands 3144 // CommandExecutor will execute commands
3145 var executor; 3145 var executor;
3146 if (recording) { 3146 if (recording) {
3147 executor = new RecordingCommandExecutor(new Path(recordingOutputFile)); 3147 executor = new RecordingCommandExecutor(new Path(recordingOutputFile));
3148 } else if (replaying) { 3148 } else if (replaying) {
3149 executor = new ReplayingCommandExecutor(new Path(recordedInputFile)); 3149 executor = new ReplayingCommandExecutor(new Path(recordedInputFile));
3150 } else { 3150 } else {
3151 executor = new CommandExecutorImpl( 3151 executor = new CommandExecutorImpl(
3152 _globalConfiguration, maxProcesses, maxBrowserProcesses, 3152 _globalConfiguration, maxProcesses, maxBrowserProcesses,
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
3219 } 3219 }
3220 } 3220 }
3221 3221
3222 void eventAllTestsDone() { 3222 void eventAllTestsDone() {
3223 for (var listener in _eventListener) { 3223 for (var listener in _eventListener) {
3224 listener.allDone(); 3224 listener.allDone();
3225 } 3225 }
3226 _allDone(); 3226 _allDone();
3227 } 3227 }
3228 } 3228 }
OLDNEW
« no previous file with comments | « tools/testing/dart/test_progress.dart ('k') | tools/testing/dart/test_suite.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698