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

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: 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
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 print(runningBrowserTests);
ricow1 2012/11/01 09:48:55 debug print?
Mads Ager (google) 2012/11/01 10:12:57 Done.
102 }
96 103
97 // Start process queue. 104 // Start process queue.
98 new ProcessQueue( 105 new ProcessQueue(
99 maxProcesses, 106 maxProcesses,
100 progressIndicator, 107 progressIndicator,
101 startTime, 108 startTime,
102 printTiming, 109 printTiming,
103 enqueueConfiguration, 110 enqueueConfiguration,
104 () => terminateHttpServer(), 111 () => terminateHttpServer(),
105 verbose, 112 verbose,
106 listTests); 113 listTests);
107 } 114 }
OLDNEW
« tools/test.dart ('K') | « 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