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

Unified Diff: pkg/analysis_server/test/services/completion/completion_computer_test.dart

Issue 1115843003: restructure internal request to implement CompletionRequest API, (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: merge Created 5 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/services/completion/completion_computer_test.dart
diff --git a/pkg/analysis_server/test/services/completion/completion_computer_test.dart b/pkg/analysis_server/test/services/completion/completion_computer_test.dart
index d9485dde74c67ec7b4c6fd006191d736aaa1843f..5b23689c94c4febcdbba405733d55ebb99b7ae80 100644
--- a/pkg/analysis_server/test/services/completion/completion_computer_test.dart
+++ b/pkg/analysis_server/test/services/completion/completion_computer_test.dart
@@ -6,6 +6,9 @@ library test.services.completion.suggestion;
import 'dart:async';
+import 'package:analysis_server/completion/completion_core.dart'
+ show CompletionRequest;
+import 'package:analysis_server/src/analysis_server.dart';
import 'package:analysis_server/src/protocol.dart';
import 'package:analysis_server/src/services/completion/completion_manager.dart';
import 'package:analysis_server/src/services/completion/dart_completion_manager.dart';
@@ -19,6 +22,7 @@ import 'package:test_reflective_loader/test_reflective_loader.dart';
import 'package:unittest/unittest.dart';
import '../../abstract_single_unit.dart';
+import '../../operation/operation_queue_test.dart';
main() {
groupSep = ' | ';
@@ -44,7 +48,6 @@ class DartCompletionManagerTest extends AbstractSingleUnitTest {
Index index;
SearchEngineImpl searchEngine;
Source source;
- CompletionPerformance perf;
DartCompletionManager manager;
MockCompletionContributor contributor1;
MockCompletionContributor contributor2;
@@ -63,7 +66,6 @@ class DartCompletionManagerTest extends AbstractSingleUnitTest {
index = createLocalMemoryIndex();
searchEngine = new SearchEngineImpl(index);
source = addSource('/does/not/exist.dart', '');
- perf = new CompletionPerformance();
manager = new DartCompletionManager.create(context, searchEngine, source);
suggestion1 = new CompletionSuggestion(CompletionSuggestionKind.INVOCATION,
DART_RELEVANCE_DEFAULT, "suggestion1", 1, 1, false, false);
@@ -83,7 +85,9 @@ class DartCompletionManagerTest extends AbstractSingleUnitTest {
manager.contributors = [contributor1, contributor2];
int count = 0;
bool done = false;
- CompletionRequest completionRequest = new CompletionRequest(0, perf);
+ AnalysisServer server = new AnalysisServerMock(searchEngine: searchEngine);
+ CompletionRequest completionRequest =
+ new CompletionRequestImpl(server, context, source, 0);
manager.results(completionRequest).listen((CompletionResult r) {
switch (++count) {
case 1:
@@ -120,7 +124,9 @@ class DartCompletionManagerTest extends AbstractSingleUnitTest {
manager.contributors = [contributor1, contributor2];
int count = 0;
bool done = false;
- CompletionRequest completionRequest = new CompletionRequest(0, perf);
+ AnalysisServer server = new AnalysisServerMock(searchEngine: searchEngine);
+ CompletionRequest completionRequest =
+ new CompletionRequestImpl(server, context, source, 0);
manager.results(completionRequest).listen((CompletionResult r) {
switch (++count) {
case 1:

Powered by Google App Engine
This is Rietveld 408576698