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

Unified Diff: pkg/analyzer_plugin/test/plugin/navigation_mixin_test.dart

Issue 3008443002: Enhance NavigationMixin to handle notifications (Closed)
Patch Set: Added a test Created 3 years, 4 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_plugin/test/plugin/mocks.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/analyzer_plugin/test/plugin/navigation_mixin_test.dart
diff --git a/pkg/analyzer_plugin/test/plugin/navigation_mixin_test.dart b/pkg/analyzer_plugin/test/plugin/navigation_mixin_test.dart
index f7573fc343dce99af6e0225baa09966d7509971d..eea7c012a16e743539e855b7610be793ec1ade60 100644
--- a/pkg/analyzer_plugin/test/plugin/navigation_mixin_test.dart
+++ b/pkg/analyzer_plugin/test/plugin/navigation_mixin_test.dart
@@ -8,6 +8,7 @@ import 'package:analyzer/file_system/file_system.dart';
import 'package:analyzer/file_system/memory_file_system.dart';
import 'package:analyzer/src/dart/analysis/driver.dart';
import 'package:analyzer_plugin/plugin/navigation_mixin.dart';
+import 'package:analyzer_plugin/protocol/protocol.dart';
import 'package:analyzer_plugin/protocol/protocol_common.dart';
import 'package:analyzer_plugin/protocol/protocol_generated.dart';
import 'package:analyzer_plugin/src/utilities/navigation/navigation.dart';
@@ -50,13 +51,30 @@ class NavigationMixinTest {
await plugin.handleAnalysisSetContextRoots(
new AnalysisSetContextRootsParams([contextRoot1]));
- var result = await plugin.handleAnalysisGetNavigation(
- new AnalysisGetNavigationParams(filePath1, 1, 2));
+ AnalysisGetNavigationResult result =
+ await plugin.handleAnalysisGetNavigation(
+ new AnalysisGetNavigationParams(filePath1, 1, 2));
expect(result, isNotNull);
expect(result.files, hasLength(1));
expect(result.targets, hasLength(1));
expect(result.regions, hasLength(2));
}
+
+ test_sendNavigationNotification() async {
+ await plugin.handleAnalysisSetContextRoots(
+ new AnalysisSetContextRootsParams([contextRoot1]));
+
+ plugin.mockChannel.listen(null,
+ onNotification: (Notification notification) {
+ expect(notification, isNotNull);
+ AnalysisNavigationParams params =
+ new AnalysisNavigationParams.fromNotification(notification);
+ expect(params.files, hasLength(1));
+ expect(params.targets, hasLength(1));
+ expect(params.regions, hasLength(2));
+ });
+ await plugin.sendNavigationNotification(filePath1);
+ }
}
class _TestNavigationContributor implements NavigationContributor {
@@ -89,8 +107,8 @@ class _TestServerPlugin extends MockServerPlugin with NavigationMixin {
@override
Future<NavigationRequest> getNavigationRequest(
AnalysisGetNavigationParams parameters) async {
- AnalysisResult result = new AnalysisResult(
- null, null, null, null, null, null, null, null, null, null, null);
+ AnalysisResult result = new AnalysisResult(null, null, parameters.file,
+ null, null, null, null, null, null, null, null);
return new DartNavigationRequestImpl(
resourceProvider, parameters.offset, parameters.length, result);
}
« no previous file with comments | « pkg/analyzer_plugin/test/plugin/mocks.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698