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

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

Issue 17644011: Bugfix in test_suite.dart (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 5 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 // 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 732 matching lines...) Expand 10 before | Expand all | Expand 10 after
743 completeHandler, 743 completeHandler,
744 expectations, 744 expectations,
745 isNegative: isNegative, 745 isNegative: isNegative,
746 info: info)); 746 info: info));
747 } 747 }
748 } 748 }
749 749
750 List<Command> makeCommands(TestInformation info, var vmOptions, var args) { 750 List<Command> makeCommands(TestInformation info, var vmOptions, var args) {
751 switch (configuration['compiler']) { 751 switch (configuration['compiler']) {
752 case 'dart2js': 752 case 'dart2js':
753 var compiledFile = '$tempDir/out-${runtime}.js';
754 var compiledShadowFile = '$tempDir/out_shadow.js';
755 args = new List.from(args); 753 args = new List.from(args);
756 String tempDir = createOutputDirectory(info.filePath, ''); 754 String tempDir = createOutputDirectory(info.filePath, '');
755 var compiledFile = '$tempDir/out-${configuration['runtime']}.js';
756 var compiledShadowFile = '$tempDir/out_shadow.js';
757 args.add('--out=$compiledFile'); 757 args.add('--out=$compiledFile');
758 758
759 List<Command> commands = 759 List<Command> commands =
760 <Command>[new CompilationCommand(compiledFile, 760 <Command>[new CompilationCommand(compiledFile,
761 compiledShadowFile, 761 compiledShadowFile,
762 !useSdk, 762 !useSdk,
763 dart2JsBootstrapDependencies, 763 dart2JsBootstrapDependencies,
764 compilerPath, 764 compilerPath,
765 args)]; 765 args)];
766 if (info.hasCompileError) { 766 if (info.hasCompileError) {
767 // Do not attempt to run the compiled result. A compilation 767 // Do not attempt to run the compiled result. A compilation
768 // error should be reported by the compilation command. 768 // error should be reported by the compilation command.
769 } else if (configuration['runtime'] == 'd8') { 769 } else if (configuration['runtime'] == 'd8') {
770 commands.add(new Command(d8FileName, ['$tempDir/out.js'])); 770 commands.add(new Command(d8FileName, [compiledFile]));
771 } else if (configuration['runtime'] == 'jsshell') { 771 } else if (configuration['runtime'] == 'jsshell') {
772 commands.add(new Command(jsShellFileName, ['$tempDir/out.js'])); 772 commands.add(new Command(jsShellFileName, [compiledFile]));
773 } 773 }
774 return commands; 774 return commands;
775 775
776 case 'dart2dart': 776 case 'dart2dart':
777 var compiledFile = '$tempDir/out-${runtime}.dart';
778 var compiledShadowFile = '$tempDir/out_shadow.dart';
779 args = new List.from(args); 777 args = new List.from(args);
780 args.add('--output-type=dart'); 778 args.add('--output-type=dart');
781 String tempDir = createOutputDirectory(info.filePath, ''); 779 String tempDir = createOutputDirectory(info.filePath, '');
780 var compiledFile = '$tempDir/out-${configuration['runtime']}.dart';
781 var compiledShadowFile = '$tempDir/out_shadow.dart';
782 args.add('--out=$tempDir/out.dart'); 782 args.add('--out=$tempDir/out.dart');
783 783
784 List<Command> commands = 784 List<Command> commands =
785 <Command>[new CompilationCommand(compiledFile, 785 <Command>[new CompilationCommand(compiledFile,
786 compiledShadowFile, 786 compiledShadowFile,
787 !useSdk, 787 !useSdk,
788 dart2JsBootstrapDependencies, 788 dart2JsBootstrapDependencies,
789 compilerPath, 789 compilerPath,
790 args)]; 790 args)];
791 if (info.hasCompileError) { 791 if (info.hasCompileError) {
(...skipping 1187 matching lines...) Expand 10 before | Expand all | Expand 10 after
1979 * $pass tests are expected to pass 1979 * $pass tests are expected to pass
1980 * $failOk tests are expected to fail that we won't fix 1980 * $failOk tests are expected to fail that we won't fix
1981 * $fail tests are expected to fail that we should fix 1981 * $fail tests are expected to fail that we should fix
1982 * $crash tests are expected to crash that we should fix 1982 * $crash tests are expected to crash that we should fix
1983 * $timeout tests are allowed to timeout 1983 * $timeout tests are allowed to timeout
1984 * $compileErrorSkip tests are skipped on browsers due to compile-time error 1984 * $compileErrorSkip tests are skipped on browsers due to compile-time error
1985 """; 1985 """;
1986 print(report); 1986 print(report);
1987 } 1987 }
1988 } 1988 }
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