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

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

Issue 2713723003: Remove and hide analyzer cli flags. (Closed)
Patch Set: Created 3 years, 9 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 1860 matching lines...) Expand 10 before | Expand all | Expand 10 after
1871 static String libraryPath(Map configuration) => configuration['use_sdk'] 1871 static String libraryPath(Map configuration) => configuration['use_sdk']
1872 ? '${TestUtils.buildDir(configuration)}/dart-sdk' 1872 ? '${TestUtils.buildDir(configuration)}/dart-sdk'
1873 : 'sdk'; 1873 : 'sdk';
1874 1874
1875 AnalyzeLibraryTestSuite(Map configuration) 1875 AnalyzeLibraryTestSuite(Map configuration)
1876 : super(configuration, 'analyze_library', libraryPath(configuration), 1876 : super(configuration, 'analyze_library', libraryPath(configuration),
1877 ['lib'], ['tests/lib/analyzer/analyze_library.status']); 1877 ['lib'], ['tests/lib/analyzer/analyze_library.status']);
1878 1878
1879 List<String> additionalOptions(Path filePath, {bool showSdkWarnings}) { 1879 List<String> additionalOptions(Path filePath, {bool showSdkWarnings}) {
1880 var options = super.additionalOptions(filePath); 1880 var options = super.additionalOptions(filePath);
1881 // NOTE: This flag has been deprecated. 1881 options.add('--sdk-warnings');
1882 options.add('--show-sdk-warnings');
1883 return options; 1882 return options;
1884 } 1883 }
1885 1884
1886 bool isTestFile(String filename) { 1885 bool isTestFile(String filename) {
1887 // NOTE: We exclude tests and patch files for now. 1886 // NOTE: We exclude tests and patch files for now.
1888 return filename.endsWith(".dart") && 1887 return filename.endsWith(".dart") &&
1889 !filename.endsWith("_test.dart") && 1888 !filename.endsWith("_test.dart") &&
1890 !filename.contains("_internal/js_runtime/lib"); 1889 !filename.contains("_internal/js_runtime/lib");
1891 } 1890 }
1892 1891
(...skipping 528 matching lines...) Expand 10 before | Expand all | Expand 10 after
2421 } 2420 }
2422 if (path.length > WINDOWS_SHORTEN_PATH_LIMIT) { 2421 if (path.length > WINDOWS_SHORTEN_PATH_LIMIT) {
2423 ++shortNameCounter; 2422 ++shortNameCounter;
2424 var pathEnd = path.substring(path.length - WINDOWS_PATH_END_LENGTH); 2423 var pathEnd = path.substring(path.length - WINDOWS_PATH_END_LENGTH);
2425 path = "short${shortNameCounter}_$pathEnd"; 2424 path = "short${shortNameCounter}_$pathEnd";
2426 } 2425 }
2427 } 2426 }
2428 return path; 2427 return path;
2429 } 2428 }
2430 } 2429 }
OLDNEW
« pkg/analyzer_cli/lib/src/options.dart ('K') | « pkg/analyzer_cli/test/options_test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698