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

Side by Side Diff: tools/test.dart

Issue 11029024: When only listing and not running tests, don't start the http server. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 2 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/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 /** 6 /**
7 * This file is the entrypoint of the dart test suite. This suite is used 7 * This file is the entrypoint of the dart test suite. This suite is used
8 * to test: 8 * to test:
9 * 9 *
10 * 1. the dart vm 10 * 1. the dart vm
(...skipping 109 matching lines...) Expand 10 before | Expand all | Expand 10 after
120 if (selectors.containsKey(name)) { 120 if (selectors.containsKey(name)) {
121 queue.addTestSuite( 121 queue.addTestSuite(
122 new StandardTestSuite.forDirectory(conf, testSuiteDir)); 122 new StandardTestSuite.forDirectory(conf, testSuiteDir));
123 } 123 }
124 } 124 }
125 } 125 }
126 126
127 // Start global http server that serves the entire dart repo. 127 // Start global http server that serves the entire dart repo.
128 // The http server is available on localhost:9876 for any 128 // The http server is available on localhost:9876 for any
129 // test that needs to load resources from the repo over http. 129 // test that needs to load resources from the repo over http.
130 startHttpServer('127.0.0.1', 9876); 130 if (!listTests) startHttpServer('127.0.0.1', 9876);
131 131
132 // Start process queue. 132 // Start process queue.
133 new ProcessQueue( 133 new ProcessQueue(
134 maxProcesses, 134 maxProcesses,
135 progressIndicator, 135 progressIndicator,
136 startTime, 136 startTime,
137 printTiming, 137 printTiming,
138 enqueueConfiguration, 138 enqueueConfiguration,
139 () => terminateHttpServer(), 139 () {
140 if (!listTests) terminateHttpServer();
141 },
140 verbose, 142 verbose,
141 listTests); 143 listTests);
142 } 144 }
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