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

Unified Diff: pkg/analyzer/test/source/analysis_options_provider_test.dart

Issue 1975963002: Support the name analysis_options.yaml for analysis options files (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Created 4 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: pkg/analyzer/test/source/analysis_options_provider_test.dart
diff --git a/pkg/analyzer/test/source/analysis_options_provider_test.dart b/pkg/analyzer/test/source/analysis_options_provider_test.dart
index ee9f0e0278ebf9913788811a6fe4837d245198ca..07a7ff690f135a05e0e97dbcc8728ba4a92751d0 100644
--- a/pkg/analyzer/test/source/analysis_options_provider_test.dart
+++ b/pkg/analyzer/test/source/analysis_options_provider_test.dart
@@ -9,6 +9,7 @@ import 'dart:core' hide Resource;
import 'package:analyzer/file_system/file_system.dart';
import 'package:analyzer/file_system/memory_file_system.dart';
import 'package:analyzer/source/analysis_options_provider.dart';
+import 'package:analyzer/src/generated/engine.dart';
import 'package:unittest/unittest.dart';
import 'package:yaml/yaml.dart';
@@ -18,7 +19,8 @@ import '../utils.dart';
main() {
initializeTestEnvironment();
- runReflectiveTests(AnalysisOptionsProviderTest);
+ runReflectiveTests(AnalysisOptionsProviderOldTest);
+ runReflectiveTests(AnalysisOptionsProviderNewTest);
group('AnalysisOptionsProvider', () {
void expectMergesTo(String defaults, String overrides, String expected) {
var optionsProvider = new AnalysisOptionsProvider();
@@ -100,12 +102,23 @@ analyzer:
}
@reflectiveTest
-class AnalysisOptionsProviderTest {
+class AnalysisOptionsProviderNewTest extends AnalysisOptionsProviderTest {
+ String get optionsFileName => AnalysisEngine.ANALYSIS_OPTIONS_YAML_FILE;
+}
+
+@reflectiveTest
+class AnalysisOptionsProviderOldTest extends AnalysisOptionsProviderTest {
+ String get optionsFileName => AnalysisEngine.ANALYSIS_OPTIONS_FILE;
+}
+
+abstract class AnalysisOptionsProviderTest {
TestPathTranslator pathTranslator;
ResourceProvider resourceProvider;
AnalysisOptionsProvider provider = new AnalysisOptionsProvider();
+ String get optionsFileName;
+
void setUp() {
var rawProvider = new MemoryResourceProvider(isWindows: isWindows);
resourceProvider = new TestResourceProvider(rawProvider);
@@ -115,14 +128,14 @@ class AnalysisOptionsProviderTest {
void test_getOptions_crawlUp_hasInFolder() {
pathTranslator.newFolder('/foo/bar');
pathTranslator.newFile(
- '/foo/.analysis_options',
+ '/foo/$optionsFileName',
r'''
analyzer:
ignore:
- foo
''');
pathTranslator.newFile(
- '/foo/bar/.analysis_options',
+ '/foo/bar/$optionsFileName',
r'''
analyzer:
ignore:
@@ -140,14 +153,14 @@ analyzer:
void test_getOptions_crawlUp_hasInParent() {
pathTranslator.newFolder('/foo/bar/baz');
pathTranslator.newFile(
- '/foo/.analysis_options',
+ '/foo/$optionsFileName',
r'''
analyzer:
ignore:
- foo
''');
pathTranslator.newFile(
- '/foo/bar/.analysis_options',
+ '/foo/bar/$optionsFileName',
r'''
analyzer:
ignore:
@@ -169,14 +182,14 @@ analyzer:
}
void test_getOptions_empty() {
- pathTranslator.newFile('/.analysis_options', r'''#empty''');
+ pathTranslator.newFile('/$optionsFileName', r'''#empty''');
Map<String, YamlNode> options = _getOptions('/');
expect(options, isNotNull);
expect(options, isEmpty);
}
void test_getOptions_invalid() {
- pathTranslator.newFile('/.analysis_options', r''':''');
+ pathTranslator.newFile('/$optionsFileName', r''':''');
expect(() {
_getOptions('/');
}, throws);
@@ -184,7 +197,7 @@ analyzer:
void test_getOptions_simple() {
pathTranslator.newFile(
- '/.analysis_options',
+ '/$optionsFileName',
r'''
analyzer:
ignore:

Powered by Google App Engine
This is Rietveld 408576698