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

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

Issue 11360028: Only start http server for browser tests. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Address comment. Created 8 years, 1 month 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/test.dart ('k') | tools/testing/dart/browser_test.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 #!/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 74 matching lines...) Expand 10 before | Expand all | Expand 10 after
85 if (selectors.containsKey(name)) { 85 if (selectors.containsKey(name)) {
86 queue.addTestSuite( 86 queue.addTestSuite(
87 new StandardTestSuite.forDirectory(conf, testSuiteDir)); 87 new StandardTestSuite.forDirectory(conf, testSuiteDir));
88 } 88 }
89 } 89 }
90 } 90 }
91 91
92 // Start global http server that serves the entire dart repo. 92 // Start global http server that serves the entire dart repo.
93 // The http server is available on localhost:9876 for any 93 // The http server is available on localhost:9876 for any
94 // test that needs to load resources from the repo over http. 94 // test that needs to load resources from the repo over http.
95 startHttpServer('127.0.0.1', 9876); 95 if (!listTests) {
96 // Only start the server if we are running browser tests.
97 var runningBrowserTests = configurations.some((config) {
98 return TestUtils.isBrowserRuntime(config['runtime']);
99 });
100 if (runningBrowserTests) startHttpServer('127.0.0.1', 9876);
101 }
96 102
97 // Start process queue. 103 // Start process queue.
98 new ProcessQueue( 104 new ProcessQueue(
99 maxProcesses, 105 maxProcesses,
100 progressIndicator, 106 progressIndicator,
101 startTime, 107 startTime,
102 printTiming, 108 printTiming,
103 enqueueConfiguration, 109 enqueueConfiguration,
104 () => terminateHttpServer(), 110 () => terminateHttpServer(),
105 verbose, 111 verbose,
106 listTests); 112 listTests);
107 } 113 }
OLDNEW
« no previous file with comments | « tools/test.dart ('k') | tools/testing/dart/browser_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698