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

Side by Side Diff: tools/test-runtime.dart

Issue 12035053: Support for running a limited amount of browser tests in parallel (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 10 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 #!/usr/bin/env dart 1 #!/usr/bin/env dart
2 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 2 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
3 // for details. All rights reserved. Use of this source code is governed by a 3 // for details. All rights reserved. Use of this source code is governed by a
4 // BSD-style license that can be found in the LICENSE file. 4 // BSD-style license that can be found in the LICENSE file.
5 5
6 // TODO(ager): Get rid of this version of test.dart when we don't have 6 // TODO(ager): Get rid of this version of test.dart when we don't have
7 // to worry about the special runtime checkout anymore. 7 // to worry about the special runtime checkout anymore.
8 // This file is identical to test.dart with test suites in the 8 // This file is identical to test.dart with test suites in the
9 // directories samples, client, compiler, and utils removed. 9 // directories samples, client, compiler, and utils removed.
10 10
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
95 // The http server is available on localhost:9876 for any 95 // The http server is available on localhost:9876 for any
96 // test that needs to load resources from the repo over http. 96 // test that needs to load resources from the repo over http.
97 if (!listTests) { 97 if (!listTests) {
98 // Only start the server if we are running browser tests. 98 // Only start the server if we are running browser tests.
99 var runningBrowserTests = configurations.any((config) { 99 var runningBrowserTests = configurations.any((config) {
100 return TestUtils.isBrowserRuntime(config['runtime']); 100 return TestUtils.isBrowserRuntime(config['runtime']);
101 }); 101 });
102 if (runningBrowserTests) startHttpServer('127.0.0.1', 9876); 102 if (runningBrowserTests) startHttpServer('127.0.0.1', 9876);
103 } 103 }
104 104
105 var maxBrowserProcesses = maxProcesses;
106
105 // Start process queue. 107 // Start process queue.
106 new ProcessQueue( 108 new ProcessQueue(
107 maxProcesses, 109 maxProcesses,
110 maxBrowserProcesses,
108 progressIndicator, 111 progressIndicator,
109 startTime, 112 startTime,
110 printTiming, 113 printTiming,
111 testSuites, 114 testSuites,
112 () => TestingServerRunner.terminateHttpServers(), 115 () => TestingServerRunner.terminateHttpServers(),
113 verbose, 116 verbose,
114 listTests); 117 listTests);
115 } 118 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698