Partial implementation of the 'setAnalysisRoots' API.

pkg/analysis_server/test/domain_analysis_test.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 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);
   });
 
   group('AnalysisDomainHandler', () {
     test('getFixes', () {
       var request = new Request('0', AnalysisDomainHandler.GET_FIXES_METHOD);
       request.setParameter(AnalysisDomainHandler.ERRORS_PARAM, []);
       var response = handler.handleRequest(request);
       // TODO(scheglov) implement
       expect(response, isNull);
     });
 
     test('getMinorRefactorings', () {
       var request = new Request('0', AnalysisDomainHandler.GET_MINOR_REFACTORINGS_METHOD);
       request.setParameter(AnalysisDomainHandler.FILE_PARAM, 'test.dart');
       request.setParameter(AnalysisDomainHandler.OFFSET_PARAM, 10);
       request.setParameter(AnalysisDomainHandler.LENGTH_PARAM, 20);
       var response = handler.handleRequest(request);
       // TODO(scheglov) implement
       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', () {
       var request = new Request('0', AnalysisDomainHandler.SET_PRIORITY_FILES_METHOD);
       request.setParameter(
           AnalysisDomainHandler.FILES_PARAM,
           ['projectA/aa.dart', 'projectB/ba.dart']);
       var response = handler.handleRequest(request);
       // TODO(scheglov) implement
       expect(response, isNull);
@@ skipping 45 matching lines @@
       request.setParameter(
           AnalysisDomainHandler.REMOVED_PARAM,
           ['/dart/sdk-1.2']);
       request.setParameter(AnalysisDomainHandler.DEFAULT_PARAM, '/dart/sdk-1.4');
       var response = handler.handleRequest(request);
       // TODO(scheglov) implement
       expect(response, isNull);
     });
   });
 }