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

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

Issue 2671913002: Update status for analyze_libraries suite, and make it work with --use-sdk (Closed)
Patch Set: Created 3 years, 10 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
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 1944 matching lines...) Expand 10 before | Expand all | Expand 10 after
1955 if (dir.existsSync()) { 1955 if (dir.existsSync()) {
1956 enqueueDirectory(dir, group); 1956 enqueueDirectory(dir, group);
1957 } 1957 }
1958 } 1958 }
1959 1959
1960 return group.future; 1960 return group.future;
1961 } 1961 }
1962 } 1962 }
1963 1963
1964 class AnalyzeLibraryTestSuite extends DartcCompilationTestSuite { 1964 class AnalyzeLibraryTestSuite extends DartcCompilationTestSuite {
1965 static String libraryPath(Map configuration) =>
1966 configuration['use_sdk'] ? '${TestUtils.buildDir(configuration)}/dart-sdk' : 'sdk';
1967
1965 AnalyzeLibraryTestSuite(Map configuration) 1968 AnalyzeLibraryTestSuite(Map configuration)
1966 : super(configuration, 'analyze_library', 'sdk', ['lib'], 1969 : super(configuration, 'analyze_library', libraryPath(configuration), ['li b'],
1967 ['tests/lib/analyzer/analyze_library.status']); 1970 ['tests/lib/analyzer/analyze_library.status']);
1968 1971
1969 List<String> additionalOptions(Path filePath, {bool showSdkWarnings}) { 1972 List<String> additionalOptions(Path filePath, {bool showSdkWarnings}) {
1970 var options = super.additionalOptions(filePath); 1973 var options = super.additionalOptions(filePath);
1971 // NOTE: This flag has been deprecated. 1974 // NOTE: This flag has been deprecated.
1972 options.add('--show-sdk-warnings'); 1975 options.add('--show-sdk-warnings');
1973 return options; 1976 return options;
1974 } 1977 }
1975 1978
1976 bool isTestFile(String filename) { 1979 bool isTestFile(String filename) {
(...skipping 557 matching lines...) Expand 10 before | Expand all | Expand 10 after
2534 } 2537 }
2535 if (path.length > WINDOWS_SHORTEN_PATH_LIMIT) { 2538 if (path.length > WINDOWS_SHORTEN_PATH_LIMIT) {
2536 ++shortNameCounter; 2539 ++shortNameCounter;
2537 var pathEnd = path.substring(path.length - WINDOWS_PATH_END_LENGTH); 2540 var pathEnd = path.substring(path.length - WINDOWS_PATH_END_LENGTH);
2538 path = "short${shortNameCounter}_$pathEnd"; 2541 path = "short${shortNameCounter}_$pathEnd";
2539 } 2542 }
2540 } 2543 }
2541 return path; 2544 return path;
2542 } 2545 }
2543 } 2546 }
OLDNEW
« tests/lib/analyzer/analyze_library.status ('K') | « tests/lib/analyzer/analyze_library.status ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698