Implement analysis.updateOptions

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 87d0c31a75c39c6cb55ab2ae50dc920b6fb776ca..0cc7da40a0062e56ddbfcb218e7f4f9322d506f6 100644
--- a/pkg/analysis_server/test/domain_analysis_test.dart
+++ b/pkg/analysis_server/test/domain_analysis_test.dart
@@ -113,18 +113,28 @@ main() {
       });
     });
 
-    test('updateOptions', () {
-      var request = new Request('0', ANALYSIS_UPDATE_OPTIONS);
-      request.setParameter(
-          OPTIONS,
-          {
-            'analyzeAngular' : true,
-            'enableDeferredLoading': true,
-            'enableEnums': false
-          });
-      var response = handler.handleRequest(request);
-      // TODO(scheglov) implement
-      expect(response, isNull);
+    group('updateOptions', () {
+      test('invalid', () {
+        var request = new Request('0', ANALYSIS_UPDATE_OPTIONS);
+        request.setParameter(OPTIONS, {'not-an-option' : true});
+        var response = handler.handleRequest(request);
+        expect(response, isResponseFailure('0'));
+      });
+
+      test('valid', () {
+        var request = new Request('0', ANALYSIS_UPDATE_OPTIONS);
+        request.setParameter(
+            OPTIONS,
+            {
+              'analyzeAngular' : true,
+              'enableDeferredLoading': true,
+              'enableEnums': false
+            });
+        var response = handler.handleRequest(request);
+        expect(response, isResponseSuccess('0'));
+        // TODO(brianwilkerson) Figure out how to test that the options were

[Paul Berry, 2014/06/17 20:40:54]

Personally I'd prefer to see this addressed before the CL gets committed.

[Brian Wilkerson, 2014/06/17 21:30:12]

And yet you gave it an LGTM...

I added tests.

[Paul Berry, 2014/06/17 22:24:56]

I assumed that LGTM + comments meant "there's some stuff that I would have done
differently, but I'm not going to be a stickler about it--it's your call whether
to take my advice".  Or maybe I was just being unintentionally vague :)
They look good, thanks.

You might want to consider adding a test to confirm that if options are supplied
before a context is created, they are correctly applied to the newly created
context.  Your call :)

[Brian Wilkerson, 2014/06/18 17:01:30]

That would be good, but I don't know how to find the analysis context that owns
the options that need to be tested. I suppose we can expose that information for
testing purposes, but that goes against my training, so I'm hesitant.

+        // actually changed.
+      });
     });
 
     test('updateSdks', () {