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

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

Issue 10001006: Fix some type warnings in the test scripts, that show up when using the editor. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 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 | Annotate | Revision Log
« no previous file with comments | « tools/testing/dart/test_runner.dart ('k') | 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 738 matching lines...) Expand 10 before | Expand all | Expand 10 after
749 case 'dart2js': 749 case 'dart2js':
750 case 'dartc': 750 case 'dartc':
751 return 'text/javascript'; 751 return 'text/javascript';
752 default: 752 default:
753 Expect.fail('Non-web runtime, so no scriptType for: ' + 753 Expect.fail('Non-web runtime, so no scriptType for: ' +
754 '${configuration["compiler"]}'); 754 '${configuration["compiler"]}');
755 return null; 755 return null;
756 } 756 }
757 } 757 }
758 758
759 String get hasRuntime() { 759 bool get hasRuntime() {
760 switch(configuration['runtime']) { 760 switch(configuration['runtime']) {
761 case null: 761 case null:
762 Expect.fail("configuration['runtime'] is not set"); 762 Expect.fail("configuration['runtime'] is not set");
763 case 'none': 763 case 'none':
764 return false; 764 return false;
765 default: 765 default:
766 return true; 766 return true;
767 } 767 }
768 } 768 }
769 769
(...skipping 610 matching lines...) Expand 10 before | Expand all | Expand 10 after
1380 * $noCrash tests are expected to be flaky but not crash 1380 * $noCrash tests are expected to be flaky but not crash
1381 * $pass tests are expected to pass 1381 * $pass tests are expected to pass
1382 * $failOk tests are expected to fail that we won't fix 1382 * $failOk tests are expected to fail that we won't fix
1383 * $fail tests are expected to fail that we should fix 1383 * $fail tests are expected to fail that we should fix
1384 * $crash tests are expected to crash that we should fix 1384 * $crash tests are expected to crash that we should fix
1385 * $timeout tests are allowed to timeout 1385 * $timeout tests are allowed to timeout
1386 """; 1386 """;
1387 print(report); 1387 print(report);
1388 } 1388 }
1389 } 1389 }
OLDNEW
« no previous file with comments | « tools/testing/dart/test_runner.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698