Implementation for 'analysis.updateContent' 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 'dart:async';
 
 import 'package:analysis_server/src/analysis_server.dart';
 import 'package:analysis_server/src/domain_analysis.dart';
@@ skipping 12 matching lines @@
   AnalysisDomainHandler handler;
   MemoryResourceProvider resourceProvider = new MemoryResourceProvider();
 
   setUp(() {
     serverChannel = new MockServerChannel();
     server = new AnalysisServer(serverChannel, resourceProvider);
     handler = new AnalysisDomainHandler(server);
   });
 
   group('notification.errors', testNotificationErrors);
+  group('notification.errors', testUpdateContent);

[Brian Wilkerson, 2014/05/28 20:38:15]

"errors" --> "updateContent"?

I still don't understand why these tests are in separate methods.

[scheglov, 2014/05/28 20:44:05]

Done.

 
   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);
     });
 
@@ skipping 60 matching lines @@
           {
             AnalysisService.HIGHLIGHTS : ['project/a.dart', 'project/b.dart'],
             AnalysisService.NAVIGATION : ['project/c.dart'],
             AnalysisService.OUTLINE : ['project/d.dart', 'project/e.dart']
           });
       var response = handler.handleRequest(request);
       // TODO(scheglov) implement
       expect(response, isNull);
     });
 
-    test('updateContent', () {
-      var request = new Request('0', AnalysisDomainHandler.UPDATE_CONTENT_METHOD);
-//      request.setParameter(
-//          AnalysisDomainHandler.FILES_PARAM,
-//          {'project/test.dart' : null});
-      var response = handler.handleRequest(request);
-      // TODO(scheglov) implement
-      expect(response, isNull);
-    });
-
     test('updateOptions', () {
       var request = new Request('0', AnalysisDomainHandler.UPDATE_OPTIONS_METHOD);
       request.setParameter(
           AnalysisDomainHandler.OPTIONS_PARAM,
           {
             'analyzeAngular' : true,
             'enableDeferredLoading': true,
             'enableEnums': false
           });
       var response = handler.handleRequest(request);
@@ skipping 73 matching lines @@
    * May be empty, but not `null`.
    */
   List<AnalysisError> getTestErrors() {
     return getErrors(testFile);
   }
 
   /**
    * Creates a project with a single Dart file `/project/bin/test.dart` with
    * the given [code].
    */
-  createSingleFileProject(code) {
-    if (code is List<String>) {
-      code = code.join('\n');
-    }
-    this.testCode = code;
+  void createSingleFileProject(code) {
+    this.testCode = _getCodeString(code);
     resourceProvider.newFolder('/project');
     resourceProvider.newFile('/project/pubspec.yaml', 'name: project');
     resourceProvider.newFile(testFile, testCode);
     Request request = new Request('0', AnalysisDomainHandler.SET_ANALYSIS_ROOTS_METHOD);
     request.setParameter(AnalysisDomainHandler.INCLUDED_PARAM, ['/project']);
     request.setParameter(AnalysisDomainHandler.EXCLUDED_PARAM, []);
+    handleSuccessfulRequest(request);
+  }
+
+  /**
+   * Validates that the given [request] is handled successfully.
+   */
+  void handleSuccessfulRequest(Request request) {
     Response response = handler.handleRequest(request);
     expect(response, isResponseSuccess('0'));
   }
+
+  /**
+   * Stops the associated server.
+   */
+  void stopServer() {
+    server.done();
+  }
+
+  static String _getCodeString(code) {
+    if (code is List<String>) {
+      code = code.join('\n');
+    }
+    return code as String;
+  }
 }
 
 
 testNotificationErrors() {
   AnalysisTestHelper helper;
 
   setUp(() {
     helper = new AnalysisTestHelper();
   });
 
@@ skipping 19 matching lines @@
     return helper.waitForTasksFinished().then((_) {
       List<AnalysisError> errors = helper.getTestErrors();
       expect(errors, hasLength(1));
       AnalysisError error = errors[0];
       expect(error.errorCode, 'StaticWarningCode.UNDEFINED_IDENTIFIER');
     });
   });
 }
 
 
+testUpdateContent() {
+  test('full content', () {
+    AnalysisTestHelper helper = new AnalysisTestHelper();
+    helper.createSingleFileProject('// empty');
+    return helper.waitForTasksFinished().then((_) {
+      // no errors initially
+      List<AnalysisError> errors = helper.getTestErrors();
+      expect(errors, isEmpty);
+      // update code
+      {
+        Request request = new Request('0', AnalysisDomainHandler.UPDATE_CONTENT_METHOD);
+        request.setParameter('files',
+            {
+              helper.testFile : {
+                AnalysisDomainHandler.CONTENT_PARAM : 'library lib'
+              }
+            });
+        helper.handleSuccessfulRequest(request);
+      }
+      // wait, there is an error
+      helper.waitForTasksFinished().then((_) {
+        List<AnalysisError> errors = helper.getTestErrors();
+        expect(errors, hasLength(1));
+      });
+    });
+  });
+
+  test('incremental', () {
+    AnalysisTestHelper helper = new AnalysisTestHelper();
+    helper.createSingleFileProject('library A;');
+    return helper.waitForTasksFinished().then((_) {
+      // no errors initially
+      List<AnalysisError> errors = helper.getTestErrors();
+      expect(errors, isEmpty);
+      // update code
+      {
+        Request request = new Request('0', AnalysisDomainHandler.UPDATE_CONTENT_METHOD);
+        request.setParameter('files',
+            {
+              helper.testFile : {
+                AnalysisDomainHandler.CONTENT_PARAM : 'library lib',
+                AnalysisDomainHandler.OFFSET_PARAM : 'library '.length,
+                AnalysisDomainHandler.OLD_LENGTH_PARAM : 'A;'.length,
+                AnalysisDomainHandler.NEW_LENGTH_PARAM : 'lib'.length,
+              }
+            });
+        helper.handleSuccessfulRequest(request);
+      }
+      // wait, there is an error
+      helper.waitForTasksFinished().then((_) {
+        List<AnalysisError> errors = helper.getTestErrors();
+        expect(errors, hasLength(1));
+      });
+    });
+  });
+}
+
+
 class AnalysisError {
   final String file;
   final String errorCode;
   final int offset;
   final int length;
   final String message;
   final String correction;
   AnalysisError(this.file, this.errorCode, this.offset, this.length,
       this.message, this.correction);
 
   @override
   String toString() {
     return 'NotificationError(file=$file; errorCode=$errorCode; '
         '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']);
 }