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

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

Issue 2801143003: Remove the "pkgbuild" step on the bots. (Closed)
Patch Set: Rebase. Created 3 years, 8 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/package_testing_support.dart ('k') | tools/testing/dart/test_runner.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 library test_configurations; 5 library test_configurations;
6 6
7 import "dart:async"; 7 import "dart:async";
8 import 'dart:convert'; 8 import 'dart:convert';
9 import 'dart:io'; 9 import 'dart:io';
10 import "dart:math" as math; 10 import "dart:math" as math;
(...skipping 191 matching lines...) Expand 10 before | Expand all | Expand 10 after
202 } else if (conf['compiler'] == 'none' && 202 } else if (conf['compiler'] == 'none' &&
203 conf['runtime'] == 'vm' && 203 conf['runtime'] == 'vm' &&
204 key == 'vm') { 204 key == 'vm') {
205 // vm tests contain both cc tests (added here) and dart tests (added 205 // vm tests contain both cc tests (added here) and dart tests (added
206 // in [TEST_SUITE_DIRECTORIES]). 206 // in [TEST_SUITE_DIRECTORIES]).
207 testSuites.add(new VMTestSuite(conf)); 207 testSuites.add(new VMTestSuite(conf));
208 } else if (conf['analyzer']) { 208 } else if (conf['analyzer']) {
209 if (key == 'analyze_library') { 209 if (key == 'analyze_library') {
210 testSuites.add(new AnalyzeLibraryTestSuite(conf)); 210 testSuites.add(new AnalyzeLibraryTestSuite(conf));
211 } 211 }
212 } else if (conf['compiler'] == 'none' &&
213 conf['runtime'] == 'vm' &&
214 key == 'pkgbuild') {
215 if (!conf['use_sdk']) {
216 print("Running the 'pkgbuild' test suite requires "
217 "passing the '--use-sdk' to test.py");
218 exit(1);
219 }
220 testSuites.add(
221 new PkgBuildTestSuite(conf, 'pkgbuild', 'pkg/pkgbuild.status'));
222 } 212 }
223 } 213 }
224 } 214 }
225 } 215 }
226 216
227 void allTestsFinished() { 217 void allTestsFinished() {
228 for (var conf in configurations) { 218 for (var conf in configurations) {
229 if (conf.containsKey('_servers_')) { 219 if (conf.containsKey('_servers_')) {
230 conf['_servers_'].stopServers(); 220 conf['_servers_'].stopServers();
231 } 221 }
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
318 maxBrowserProcesses, 308 maxBrowserProcesses,
319 startTime, 309 startTime,
320 testSuites, 310 testSuites,
321 eventListener, 311 eventListener,
322 allTestsFinished, 312 allTestsFinished,
323 verbose, 313 verbose,
324 recordingPath, 314 recordingPath,
325 recordingOutputPath, 315 recordingOutputPath,
326 adbDevicePool); 316 adbDevicePool);
327 } 317 }
OLDNEW
« no previous file with comments | « tools/testing/dart/package_testing_support.dart ('k') | tools/testing/dart/test_runner.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698