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: |