Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 /** | 5 /** |
| 6 * Classes and methods for enumerating and preparing tests. | 6 * Classes and methods for enumerating and preparing tests. |
| 7 * | 7 * |
| 8 * This library includes: | 8 * This library includes: |
| 9 * | 9 * |
| 10 * - Creating tests by listing all the Dart files in certain directories, | 10 * - Creating tests by listing all the Dart files in certain directories, |
| (...skipping 430 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 441 args.add('--out=$tempDir/out.js'); | 441 args.add('--out=$tempDir/out.js'); |
| 442 List<Command> commands = <Command>[new Command(shellPath(), args)]; | 442 List<Command> commands = <Command>[new Command(shellPath(), args)]; |
| 443 if (configuration['runtime'] == 'd8') { | 443 if (configuration['runtime'] == 'd8') { |
| 444 var d8 = TestUtils.d8FileName(configuration); | 444 var d8 = TestUtils.d8FileName(configuration); |
| 445 commands.add(new Command(d8, ['$tempDir/out.js'])); | 445 commands.add(new Command(d8, ['$tempDir/out.js'])); |
| 446 } | 446 } |
| 447 return commands; | 447 return commands; |
| 448 | 448 |
| 449 case 'dart2dart': | 449 case 'dart2dart': |
| 450 var compilerArguments = new List.from(args); | 450 var compilerArguments = new List.from(args); |
| 451 var additionalFlags = configuration['additional-compiler-flags'] | |
| 452 .split(const RegExp(' ')); | |
|
ricow1
2012/08/27 09:43:14
can't we just split on ' ', instead of using a Reg
Anton Muhin
2012/08/27 09:45:37
Done.
| |
| 453 compilerArguments.addAll(additionalFlags); | |
| 451 compilerArguments.add('--output-type=dart'); | 454 compilerArguments.add('--output-type=dart'); |
| 452 String tempDir = createOutputDirectory(info.filePath, ''); | 455 String tempDir = createOutputDirectory(info.filePath, ''); |
| 453 compilerArguments.add('--out=$tempDir/out.dart'); | 456 compilerArguments.add('--out=$tempDir/out.dart'); |
| 454 List<Command> commands = | 457 List<Command> commands = |
| 455 <Command>[new Command(shellPath(), compilerArguments)]; | 458 <Command>[new Command(shellPath(), compilerArguments)]; |
| 456 if (configuration['runtime'] == 'vm') { | 459 if (configuration['runtime'] == 'vm') { |
| 457 // TODO(antonm): support checked. | 460 // TODO(antonm): support checked. |
| 458 var vmArguments = new List.from(vmOptions); | 461 var vmArguments = new List.from(vmOptions); |
| 459 vmArguments.addAll([ | 462 vmArguments.addAll([ |
| 460 '--ignore-unrecognized-flags', '$tempDir/out.dart']); | 463 '--ignore-unrecognized-flags', '$tempDir/out.dart']); |
| (...skipping 946 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 1407 * $noCrash tests are expected to be flaky but not crash | 1410 * $noCrash tests are expected to be flaky but not crash |
| 1408 * $pass tests are expected to pass | 1411 * $pass tests are expected to pass |
| 1409 * $failOk tests are expected to fail that we won't fix | 1412 * $failOk tests are expected to fail that we won't fix |
| 1410 * $fail tests are expected to fail that we should fix | 1413 * $fail tests are expected to fail that we should fix |
| 1411 * $crash tests are expected to crash that we should fix | 1414 * $crash tests are expected to crash that we should fix |
| 1412 * $timeout tests are allowed to timeout | 1415 * $timeout tests are allowed to timeout |
| 1413 """; | 1416 """; |
| 1414 print(report); | 1417 print(report); |
| 1415 } | 1418 } |
| 1416 } | 1419 } |
| OLD | NEW |