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

Unified Diff: pkg/analyzer/test/src/context/builder_test.dart

Issue 2976473003: add analysis server --flutter-repo startup flag (Closed)
Patch Set: address comments Created 3 years, 5 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
« no previous file with comments | « pkg/analyzer/lib/src/context/builder.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/analyzer/test/src/context/builder_test.dart
diff --git a/pkg/analyzer/test/src/context/builder_test.dart b/pkg/analyzer/test/src/context/builder_test.dart
index cdae0dfa1e3733d5df4da5de87a3158703b04d10..c6278839327200a8309824e90d390b6f123238bb 100644
--- a/pkg/analyzer/test/src/context/builder_test.dart
+++ b/pkg/analyzer/test/src/context/builder_test.dart
@@ -77,6 +77,7 @@ class ContextBuilderTest extends EngineTestCase {
_MockLintRule _mockLintRule;
_MockLintRule _mockLintRule2;
_MockLintRule _mockLintRule3;
+ _MockLintRule _mockPublicMemberApiDocs;
Uri convertedDirectoryUri(String directoryPath) {
return new Uri.directory(resourceProvider.convertPath(directoryPath),
@@ -771,6 +772,36 @@ linter:
_expectEqualOptions(options, expected);
}
+ void test_getAnalysisOptions_default_flutter_repo() {
+ _defineMockLintRules();
+ AnalysisOptionsImpl defaultOptions = new AnalysisOptionsImpl();
+ builderOptions.defaultOptions = defaultOptions;
+ AnalysisOptionsImpl expected = new AnalysisOptionsImpl();
+ expected.lint = true;
+ expected.lintRules = <Linter>[_mockLintRule, _mockPublicMemberApiDocs];
+ String packagesFilePath =
+ resourceProvider.convertPath('/some/directory/path/.packages');
+ createFile(packagesFilePath, 'flutter:/pkg/flutter/lib/');
+ String optionsFilePath = resourceProvider
+ .convertPath('/pkg/flutter/lib/analysis_options_user.yaml');
+ createFile(
+ optionsFilePath,
+ '''
+linter:
+ rules:
+ - mock_lint_rule
+''');
+ String projPath = resourceProvider.convertPath('/some/directory/path');
+ AnalysisOptions options;
+ try {
+ ContextBuilderOptions.flutterRepo = true;
+ options = builder.getAnalysisOptions(projPath);
+ } finally {
+ ContextBuilderOptions.flutterRepo = false;
+ }
+ _expectEqualOptions(options, expected);
+ }
+
void test_getAnalysisOptions_default_noOverrides() {
AnalysisOptionsImpl defaultOptions = new AnalysisOptionsImpl();
defaultOptions.enableLazyAssignmentOperators = true;
@@ -1012,6 +1043,8 @@ linter:
Registry.ruleRegistry.registerDefault(_mockLintRule2);
_mockLintRule3 = new _MockLintRule('mock_lint_rule3');
Registry.ruleRegistry.register(_mockLintRule3);
+ _mockPublicMemberApiDocs = new _MockLintRule('public_member_api_docs');
+ Registry.ruleRegistry.register(_mockPublicMemberApiDocs);
}
void _expectEqualOptions(
« no previous file with comments | « pkg/analyzer/lib/src/context/builder.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698