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

Unified Diff: pkg/analysis_server/test/analysis_abstract.dart

Issue 2842013003: Add support for watch events and error notifications (Closed)
Patch Set: fix comment Created 3 years, 8 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/analysis_server/test/analysis_abstract.dart
diff --git a/pkg/analysis_server/test/analysis_abstract.dart b/pkg/analysis_server/test/analysis_abstract.dart
index c037b95d8fa1edade6c36efd75db71febc8b8467..b1ff59eb29862241b1b876cfbf939bdac87f1b48 100644
--- a/pkg/analysis_server/test/analysis_abstract.dart
+++ b/pkg/analysis_server/test/analysis_abstract.dart
@@ -27,6 +27,7 @@ import 'package:analyzer_plugin/src/protocol/protocol_internal.dart' as plugin;
import 'package:plugin/manager.dart';
import 'package:plugin/plugin.dart';
import 'package:test/test.dart';
+import 'package:watcher/src/watch_event.dart';
scheglov 2017/04/26 15:55:18 Maybe package:watcher/watcher.dart?
Brian Wilkerson 2017/04/26 16:35:55 Done (We really need to fix the quick fixes, etc.
import 'mock_sdk.dart';
import 'mocks.dart';
@@ -308,12 +309,18 @@ class TestPluginManager implements PluginManager {
}
@override
- Map<PluginInfo, Future<plugin.Response>> broadcast(
+ Map<PluginInfo, Future<plugin.Response>> broadcastRequest(
analyzer.ContextRoot contextRoot, plugin.RequestParams params) {
return broadcastResults ?? <PluginInfo, Future<plugin.Response>>{};
}
@override
+ Future<List<Future<plugin.Response>>> broadcastWatchEvent(
+ WatchEvent watchEvent) async {
+ return <Future<plugin.Response>>[];
+ }
+
+ @override
List<PluginInfo> pluginsForContextRoot(analyzer.ContextRoot contextRoot) {
fail('Unexpected invocation of pluginsForContextRoot');
return null;

Powered by Google App Engine
This is Rietveld 408576698