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

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

Issue 2999373002: Migrated test block 218 to Dart 2.0. (Closed)
Patch Set: Addressed Bob's comments. Created 3 years, 3 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 | « tests/lib_strong/mirrors/constructor_optional_args_test.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 600 matching lines...) Expand 10 before | Expand all | Expand 10 after
611 * instead of having to create a custom [StandardTestSuite] subclass. In 611 * instead of having to create a custom [StandardTestSuite] subclass. In
612 * particular, if you add 'path/to/mytestsuite' to [TEST_SUITE_DIRECTORIES] 612 * particular, if you add 'path/to/mytestsuite' to [TEST_SUITE_DIRECTORIES]
613 * in test.dart, this will all be set up for you. 613 * in test.dart, this will all be set up for you.
614 */ 614 */
615 factory StandardTestSuite.forDirectory( 615 factory StandardTestSuite.forDirectory(
616 Configuration configuration, Path directory) { 616 Configuration configuration, Path directory) {
617 var name = directory.filename; 617 var name = directory.filename;
618 var status_paths = [ 618 var status_paths = [
619 '$directory/$name.status', 619 '$directory/$name.status',
620 '$directory/.status', 620 '$directory/.status',
621 '$directory/${name}_app_jit.status',
621 '$directory/${name}_analyzer.status', 622 '$directory/${name}_analyzer.status',
622 '$directory/${name}_analyzer2.status', 623 '$directory/${name}_analyzer2.status',
623 '$directory/${name}_dart2js.status', 624 '$directory/${name}_dart2js.status',
624 '$directory/${name}_dartdevc.status', 625 '$directory/${name}_dartdevc.status',
625 '$directory/${name}_kernel.status', 626 '$directory/${name}_kernel.status',
626 '$directory/${name}_precompiled.status', 627 '$directory/${name}_precompiled.status',
627 '$directory/${name}_vm.status', 628 '$directory/${name}_vm.status',
628 ]; 629 ];
629 630
630 return new StandardTestSuite(configuration, name, directory, status_paths, 631 return new StandardTestSuite(configuration, name, directory, status_paths,
(...skipping 1067 matching lines...) Expand 10 before | Expand all | Expand 10 after
1698 1699
1699 bool isTestFile(String filename) { 1700 bool isTestFile(String filename) {
1700 // NOTE: We exclude tests and patch files for now. 1701 // NOTE: We exclude tests and patch files for now.
1701 return filename.endsWith(".dart") && 1702 return filename.endsWith(".dart") &&
1702 !filename.endsWith("_test.dart") && 1703 !filename.endsWith("_test.dart") &&
1703 !filename.contains("_internal/js_runtime/lib"); 1704 !filename.contains("_internal/js_runtime/lib");
1704 } 1705 }
1705 1706
1706 bool get listRecursively => true; 1707 bool get listRecursively => true;
1707 } 1708 }
OLDNEW
« no previous file with comments | « tests/lib_strong/mirrors/constructor_optional_args_test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698