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

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

Issue 365193004: Move Index and IndexStore implementations into Engine. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 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/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 323197666498a32aa7b56175d7ec814bd3d13bf8..bdc15804c0c12517c042989859c293987d233841 100644
--- a/pkg/analysis_server/test/analysis_abstract.dart
+++ b/pkg/analysis_server/test/analysis_abstract.dart
@@ -7,15 +7,42 @@ library test.domain.analysis.abstract;
import 'dart:async';
import 'package:analysis_server/src/analysis_server.dart';
-import 'package:analysis_server/src/domain_analysis.dart';
import 'package:analysis_server/src/constants.dart';
+import 'package:analysis_server/src/domain_analysis.dart';
import 'package:analysis_server/src/protocol.dart';
import 'package:analysis_server/src/resource.dart';
+import 'package:analyzer/index/index.dart';
import 'package:analyzer/src/generated/source.dart';
import 'package:unittest/unittest.dart';
import 'mocks.dart';
-import 'package:analyzer/src/generated/index.dart';
+
+
+int findIdentifierLength(String search) {
+ int length = 0;
+ while (length < search.length) {
+ int c = search.codeUnitAt(length);
+ if (!(c >= 'a'.codeUnitAt(0) && c <= 'z'.codeUnitAt(0) ||
+ c >= 'A'.codeUnitAt(0) && c <= 'Z'.codeUnitAt(0) ||
+ c >= '0'.codeUnitAt(0) && c <= '9'.codeUnitAt(0))) {
+ break;
+ }
+ length++;
+ }
+ return length;
+}
+
+
+
+AnalysisError _jsonToAnalysisError(Map<String, Object> json) {
+ return new AnalysisError(
+ json['file'],
+ json['errorCode'],
+ json['offset'],
+ json['length'],
+ json['message'],
+ json['correction']);
+}
/**
@@ -42,26 +69,6 @@ class AbstractAnalysisTest {
AbstractAnalysisTest() {
}
- void setUp() {
- serverChannel = new MockServerChannel();
- resourceProvider = new MemoryResourceProvider();
- packageMapProvider = new MockPackageMapProvider();
- Index index = createIndex();
- server = new AnalysisServer(
- serverChannel, resourceProvider, packageMapProvider, index);
- server.defaultSdk = new MockSdk();
- handler = new AnalysisDomainHandler(server);
- // listen for notifications
- Stream<Notification> notificationStream = serverChannel.notificationController.stream;
- notificationStream.listen((Notification notification) {
- processNotification(notification);
- });
- }
-
- Index createIndex() {
- return null;
- }
-
void addAnalysisSubscription(AnalysisService service, String file) {
// add file to subscription
var files = analysisSubscriptions[service.name];
@@ -76,36 +83,30 @@ class AbstractAnalysisTest {
handleSuccessfulRequest(request);
}
- void tearDown() {
- server.done();
- handler = null;
- server = null;
- resourceProvider = null;
- serverChannel = null;
+ String addFile(String path, String content) {
+ resourceProvider.newFile(path, content);
+ return path;
}
- void processNotification(Notification notification) {
-// if (notification.event == NOTIFICATION_ERRORS) {
-// String file = notification.getParameter(FILE);
-// List<Map<String, Object>> errorMaps = notification.getParameter(ERRORS);
-// filesErrors[file] = errorMaps.map(jsonToAnalysisError).toList();
-// }
-// if (notification.event == NOTIFICATION_HIGHLIGHTS) {
-// String file = notification.getParameter(FILE);
-// filesHighlights[file] = notification.getParameter(REGIONS);
-// }
-// if (notification.event == NOTIFICATION_NAVIGATION) {
-// String file = notification.getParameter(FILE);
-// filesNavigation[file] = notification.getParameter(REGIONS);
-// }
+ String addTestFile(String content) {
+ addFile(testFile, content);
+ this.testCode = content;
+ return testFile;
+ }
+
+ Index createIndex() {
+ return null;
}
/**
- * Returns a [Future] that completes when the [AnalysisServer] finishes
- * all its scheduled tasks.
+ * Creates a project `/project`.
*/
- Future waitForTasksFinished() {
- return waitForServerOperationsPerformed(server);
+ void createProject() {
+ resourceProvider.newFolder(projectPath);
+ Request request = new Request('0', ANALYSIS_SET_ANALYSIS_ROOTS);
+ request.setParameter(INCLUDED, [projectPath]);
+ request.setParameter(EXCLUDED, []);
+ handleSuccessfulRequest(request);
}
/**
@@ -130,6 +131,15 @@ class AbstractAnalysisTest {
return offset;
}
+ /**
+ * Validates that the given [request] is handled successfully.
+ */
+ Response handleSuccessfulRequest(Request request) {
+ Response response = handler.handleRequest(request);
+ expect(response, isResponseSuccess('0'));
+ return response;
+ }
+
// /**
// * Returns [AnalysisError]s recorded for the given [file].
// * May be empty, but not `null`.
@@ -190,15 +200,20 @@ class AbstractAnalysisTest {
// return getNavigation(testFile);
// }
- /**
- * Creates a project `/project`.
- */
- void createProject() {
- resourceProvider.newFolder(projectPath);
- Request request = new Request('0', ANALYSIS_SET_ANALYSIS_ROOTS);
- request.setParameter(INCLUDED, [projectPath]);
- request.setParameter(EXCLUDED, []);
- handleSuccessfulRequest(request);
+ void processNotification(Notification notification) {
+// if (notification.event == NOTIFICATION_ERRORS) {
+// String file = notification.getParameter(FILE);
+// List<Map<String, Object>> errorMaps = notification.getParameter(ERRORS);
+// filesErrors[file] = errorMaps.map(jsonToAnalysisError).toList();
+// }
+// if (notification.event == NOTIFICATION_HIGHLIGHTS) {
+// String file = notification.getParameter(FILE);
+// filesHighlights[file] = notification.getParameter(REGIONS);
+// }
+// if (notification.event == NOTIFICATION_NAVIGATION) {
+// String file = notification.getParameter(FILE);
+// filesNavigation[file] = notification.getParameter(REGIONS);
+// }
}
// /**
@@ -215,24 +230,36 @@ class AbstractAnalysisTest {
// handleSuccessfulRequest(request);
// }
- String addFile(String path, String content) {
- resourceProvider.newFile(path, content);
- return path;
+ void setUp() {
+ serverChannel = new MockServerChannel();
+ resourceProvider = new MemoryResourceProvider();
+ packageMapProvider = new MockPackageMapProvider();
+ Index index = createIndex();
+ server = new AnalysisServer(
+ serverChannel, resourceProvider, packageMapProvider, index);
+ server.defaultSdk = new MockSdk();
+ handler = new AnalysisDomainHandler(server);
+ // listen for notifications
+ Stream<Notification> notificationStream = serverChannel.notificationController.stream;
+ notificationStream.listen((Notification notification) {
+ processNotification(notification);
+ });
}
- String addTestFile(String content) {
- addFile(testFile, content);
- this.testCode = content;
- return testFile;
+ void tearDown() {
+ server.done();
+ handler = null;
+ server = null;
+ resourceProvider = null;
+ serverChannel = null;
}
/**
- * Validates that the given [request] is handled successfully.
+ * Returns a [Future] that completes when the [AnalysisServer] finishes
+ * all its scheduled tasks.
*/
- Response handleSuccessfulRequest(Request request) {
- Response response = handler.handleRequest(request);
- expect(response, isResponseSuccess('0'));
- return response;
+ Future waitForTasksFinished() {
+ return waitForServerOperationsPerformed(server);
}
static String _getCodeString(code) {
@@ -244,7 +271,6 @@ class AbstractAnalysisTest {
}
-
class AnalysisError {
final String file;
final String errorCode;
@@ -261,29 +287,3 @@ class AnalysisError {
'offset=$offset; length=$length; message=$message)';
}
}
-
-
-AnalysisError _jsonToAnalysisError(Map<String, Object> json) {
- return new AnalysisError(
- json['file'],
- json['errorCode'],
- json['offset'],
- json['length'],
- json['message'],
- json['correction']);
-}
-
-
-int findIdentifierLength(String search) {
- int length = 0;
- while (length < search.length) {
- int c = search.codeUnitAt(length);
- if (!(c >= 'a'.codeUnitAt(0) && c <= 'z'.codeUnitAt(0) ||
- c >= 'A'.codeUnitAt(0) && c <= 'Z'.codeUnitAt(0) ||
- c >= '0'.codeUnitAt(0) && c <= '9'.codeUnitAt(0))) {
- break;
- }
- length++;
- }
- return length;
-}

Powered by Google App Engine
This is Rietveld 408576698