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

Side by Side Diff: pkg/analyzer/test/source/analysis_options_provider_test.dart

Issue 2298913003: Pull in test_reflective_loader 0.0.4 and switch analyzer to it. (Closed)
Patch Set: Fixes for review comments. Created 4 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
OLDNEW
1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2015, 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 library analyzer.test.source.analysis_options_provider_test; 5 library analyzer.test.source.analysis_options_provider_test;
6 6
7 import 'dart:core' hide Resource; 7 import 'dart:core' hide Resource;
8 8
9 import 'package:analyzer/file_system/file_system.dart'; 9 import 'package:analyzer/file_system/file_system.dart';
10 import 'package:analyzer/file_system/memory_file_system.dart'; 10 import 'package:analyzer/file_system/memory_file_system.dart';
11 import 'package:analyzer/source/analysis_options_provider.dart'; 11 import 'package:analyzer/source/analysis_options_provider.dart';
12 import 'package:analyzer/src/generated/engine.dart'; 12 import 'package:analyzer/src/generated/engine.dart';
13 import 'package:test_reflective_loader/test_reflective_loader.dart';
13 import 'package:unittest/unittest.dart'; 14 import 'package:unittest/unittest.dart';
14 import 'package:yaml/yaml.dart'; 15 import 'package:yaml/yaml.dart';
15 16
16 import '../reflective_tests.dart';
17 import '../resource_utils.dart'; 17 import '../resource_utils.dart';
18 import '../utils.dart'; 18 import '../utils.dart';
19 19
20 main() { 20 main() {
21 initializeTestEnvironment(); 21 initializeTestEnvironment();
22 runReflectiveTests(AnalysisOptionsProviderOldTest); 22 defineReflectiveTests(AnalysisOptionsProviderOldTest);
23 runReflectiveTests(AnalysisOptionsProviderNewTest); 23 defineReflectiveTests(AnalysisOptionsProviderNewTest);
24 group('AnalysisOptionsProvider', () { 24 group('AnalysisOptionsProvider', () {
25 void expectMergesTo(String defaults, String overrides, String expected) { 25 void expectMergesTo(String defaults, String overrides, String expected) {
26 var optionsProvider = new AnalysisOptionsProvider(); 26 var optionsProvider = new AnalysisOptionsProvider();
27 var defaultOptions = optionsProvider.getOptionsFromString(defaults); 27 var defaultOptions = optionsProvider.getOptionsFromString(defaults);
28 var overrideOptions = optionsProvider.getOptionsFromString(overrides); 28 var overrideOptions = optionsProvider.getOptionsFromString(overrides);
29 var merged = optionsProvider.merge(defaultOptions, overrideOptions); 29 var merged = optionsProvider.merge(defaultOptions, overrideOptions);
30 expect(merged, optionsProvider.getOptionsFromString(expected)); 30 expect(merged, optionsProvider.getOptionsFromString(expected));
31 } 31 }
32 32
33 group('merging', () { 33 group('merging', () {
(...skipping 182 matching lines...) Expand 10 before | Expand all | Expand 10 after
216 expect(ignore[1], 'sdk_ext/**'); 216 expect(ignore[1], 'sdk_ext/**');
217 } 217 }
218 } 218 }
219 } 219 }
220 220
221 Map<String, YamlNode> _getOptions(String posixPath, {bool crawlUp: false}) { 221 Map<String, YamlNode> _getOptions(String posixPath, {bool crawlUp: false}) {
222 Resource resource = pathTranslator.getResource(posixPath); 222 Resource resource = pathTranslator.getResource(posixPath);
223 return provider.getOptions(resource, crawlUp: crawlUp); 223 return provider.getOptions(resource, crawlUp: crawlUp);
224 } 224 }
225 } 225 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698