Partial implementation of the 'setAnalysisRoots' API.

pkg/analysis_server/test/domain_analysis_test.dart

Index: pkg/analysis_server/test/domain_analysis_test.dart
diff --git a/pkg/analysis_server/test/domain_analysis_test.dart b/pkg/analysis_server/test/domain_analysis_test.dart
index 53570502591bb0858382d0ec017e35515b6eb55b..0470195a1cc186e2e41c752920189fef9383937d 100644
--- a/pkg/analysis_server/test/domain_analysis_test.dart
+++ b/pkg/analysis_server/test/domain_analysis_test.dart
@@ -7,17 +7,21 @@ library test.domain.analysis;
 import 'package:analysis_server/src/analysis_server.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:unittest/unittest.dart';
 
 import 'mocks.dart';
 
 main() {
+  groupSep = ' | ';
+
   AnalysisServer server;
   AnalysisDomainHandler handler;
+  MemoryResourceProvider resourceProvider = new MemoryResourceProvider();
 
   setUp(() {
     var serverChannel = new MockServerChannel();
-    server = new AnalysisServer(serverChannel);
+    server = new AnalysisServer(serverChannel, resourceProvider);
     handler = new AnalysisDomainHandler(server);
   });
 
@@ -40,17 +44,40 @@ main() {
       expect(response, isNull);
     });
 
-    test('setAnalysisRoots', () {
-      var request = new Request('0', AnalysisDomainHandler.SET_ANALYSIS_ROOTS_METHOD);
-      request.setParameter(
-          AnalysisDomainHandler.INCLUDED_PARAM,
-          ['projectA', 'projectB']);
-      request.setParameter(
-          AnalysisDomainHandler.EXCLUDED_PARAM,
-          ['projectA/subAA', 'projectA/subAB', 'projectB/subBA']);
-      var response = handler.handleRequest(request);
-      // TODO(scheglov) implement
-      expect(response, isNull);
+    group('setAnalysisRoots', () {
+      Request request;
+
+      setUp(() {
+        request = new Request('0', AnalysisDomainHandler.SET_ANALYSIS_ROOTS_METHOD);
+        request.setParameter(AnalysisDomainHandler.INCLUDED_PARAM, []);
+        request.setParameter(AnalysisDomainHandler.EXCLUDED_PARAM, []);
+      });
+
+      test('excluded', () {
+        request.setParameter(AnalysisDomainHandler.EXCLUDED_PARAM, ['foo']);
+        // TODO(scheglov) implement
+        var response = handler.handleRequest(request);
+        expect(response, isResponseFailure('0'));
+      });
+
+      group('included', () {
+        test('new folder', () {
+          resourceProvider.newFolder('/project');
+          resourceProvider.newFile('/project/pubspec.yaml', 'name: project');
+          resourceProvider.newFile('/project/bin/test.dart', 'main() {}');
+          request.setParameter(
+              AnalysisDomainHandler.INCLUDED_PARAM,
+              ['/project']);
+          var response = handler.handleRequest(request);
+          var serverRef = server;
+          expect(response, isResponseSuccess('0'));
+          // verify that unit is resolved eventually
+          return pumpEventQueue(1000).then((_) {

[Paul Berry, 2014/05/27 16:30:29]

Why do we need to pump the event queue 1000 times?  This makes me suspicious
that either (a) there is a design flaw that's causing us to defer way too many
things to asynchronous processing, or (b) this test is actually dependent on IO,
in which case we should explicitly wait for the IO to complete rather than loop
1000 times.

[scheglov, 2014/05/27 18:24:24]

We need do perform about 400 analysis tasks to analyze SDK.
I guess we need to the extract SDK analysis into a separate task and do it in
setUp().
Maybe even use a limited SDK from the memory FS.

+            var unit = serverRef.test_getResolvedCompilationUnit('/project/bin/test.dart');
+            expect(unit, isNotNull);
+          });
+        });
+      });
     });
 
     test('setPriorityFiles', () {