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

Unified Diff: pkg/analyzer_plugin/test/integration/support/protocol_matchers.dart

Issue 2947743002: Add support to pass the location of the analysis options file to plugins (Closed)
Patch Set: Created 3 years, 6 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_plugin/test/integration/support/protocol_matchers.dart
diff --git a/pkg/analyzer_plugin/test/integration/support/protocol_matchers.dart b/pkg/analyzer_plugin/test/integration/support/protocol_matchers.dart
index 63751e53c965e924c10feadc0f7e3f07a1f68343..ed457e4b8d5914d3db44347dee4e03d5248d54fb 100644
--- a/pkg/analyzer_plugin/test/integration/support/protocol_matchers.dart
+++ b/pkg/analyzer_plugin/test/integration/support/protocol_matchers.dart
@@ -209,32 +209,34 @@ final Matcher isCompletionSuggestionKind =
* ContextBuilderOptions
*
* {
- * "dartSdkSummaryPath": optional String
- * "defaultAnalysisOptionsFilePath": optional List<String>
+ * "dartSdkSummaryPath": optional FilePath
+ * "defaultAnalysisOptionsFilePath": optional List<FilePath>
* "declaredVariables": optional Map<String, String>
- * "defaultPackageFilePath": optional List<String>
- * "defaultPackagesDirectoryPath": optional List<String>
+ * "defaultPackageFilePath": optional List<FilePath>
+ * "defaultPackagesDirectoryPath": optional List<FilePath>
* }
*/
final Matcher isContextBuilderOptions = new LazyMatcher(
() => new MatchesJsonObject("ContextBuilderOptions", null, optionalFields: {
- "dartSdkSummaryPath": isString,
- "defaultAnalysisOptionsFilePath": isListOf(isString),
+ "dartSdkSummaryPath": isFilePath,
+ "defaultAnalysisOptionsFilePath": isListOf(isFilePath),
"declaredVariables": isMapOf(isString, isString),
- "defaultPackageFilePath": isListOf(isString),
- "defaultPackagesDirectoryPath": isListOf(isString)
+ "defaultPackageFilePath": isListOf(isFilePath),
+ "defaultPackagesDirectoryPath": isListOf(isFilePath)
}));
/**
* ContextRoot
*
* {
- * "root": String
- * "exclude": List<String>
+ * "root": FilePath
+ * "exclude": List<FilePath>
+ * "optionsFile": optional FilePath
* }
*/
final Matcher isContextRoot = new LazyMatcher(() => new MatchesJsonObject(
- "ContextRoot", {"root": isString, "exclude": isListOf(isString)}));
+ "ContextRoot", {"root": isFilePath, "exclude": isListOf(isFilePath)},
+ optionalFields: {"optionsFile": isFilePath}));
/**
* Element
@@ -865,11 +867,11 @@ final Matcher isSourceFileEdit = new LazyMatcher(() => new MatchesJsonObject(
*
* {
* "type": WatchEventType
- * "path": String
+ * "path": FilePath
* }
*/
final Matcher isWatchEvent = new LazyMatcher(() => new MatchesJsonObject(
- "WatchEvent", {"type": isWatchEventType, "path": isString}));
+ "WatchEvent", {"type": isWatchEventType, "path": isFilePath}));
/**
* WatchEventType
@@ -1438,14 +1440,17 @@ final Matcher isPluginShutdownResult = isNull;
* plugin.versionCheck params
*
* {
- * "byteStorePath": String
- * "sdkPath": String
+ * "byteStorePath": FilePath
+ * "sdkPath": FilePath
* "version": String
* }
*/
final Matcher isPluginVersionCheckParams = new LazyMatcher(() =>
- new MatchesJsonObject("plugin.versionCheck params",
- {"byteStorePath": isString, "sdkPath": isString, "version": isString}));
+ new MatchesJsonObject("plugin.versionCheck params", {
+ "byteStorePath": isFilePath,
+ "sdkPath": isFilePath,
+ "version": isString
+ }));
/**
* plugin.versionCheck result

Powered by Google App Engine
This is Rietveld 408576698