Remove ability to disable new analysis driver

pkg/analysis_server/test/domain_execution_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.
 
 import 'package:analysis_server/protocol/protocol.dart';
 import 'package:analysis_server/protocol/protocol_generated.dart';
 import 'package:analysis_server/src/analysis_server.dart';
-import 'package:analysis_server/src/constants.dart';
 import 'package:analysis_server/src/domain_execution.dart';
 import 'package:analysis_server/src/plugin/server_plugin.dart';
-import 'package:analyzer/file_system/file_system.dart';
 import 'package:analyzer/file_system/memory_file_system.dart';
 import 'package:analyzer/instrumentation/instrumentation.dart';
-import 'package:analyzer/src/generated/engine.dart';
 import 'package:analyzer/src/generated/sdk.dart';
 import 'package:analyzer/src/generated/source.dart';
 import 'package:analyzer/src/generated/source_io.dart';
 import 'package:plugin/manager.dart';
 import 'package:test/test.dart';
 import 'package:test_reflective_loader/test_reflective_loader.dart';
 
 import 'analysis_abstract.dart';
 import 'mocks.dart';
 
@@ skipping 52 matching lines @@
       test('delete non-existent context', () {
         Request request = new ExecutionDeleteContextParams('13').toRequest('0');
         Response response = handler.handleRequest(request);
         // TODO(brianwilkerson) It isn't currently specified to be an error if a
         // client attempts to delete a context that doesn't exist. Should it be?
 //        expect(response, isResponseFailure('0'));
         expect(response, isResponseSuccess('0'));
       });
     });
 
-    group('mapUri', () {
-      String contextId;
-
-      void createExecutionContextIdForFile(String path) {
-        Request request = new ExecutionCreateContextParams(path).toRequest('0');
-        Response response = handler.handleRequest(request);
-        expect(response, isResponseSuccess('0'));
-        ExecutionCreateContextResult result =
-            new ExecutionCreateContextResult.fromResponse(response);
-        contextId = result.id;
-      }
-
-      setUp(() {
-        Folder folder = provider.newFile('/a/b.dart', '').parent;
-        server.folderMap.putIfAbsent(folder, () {
-          SourceFactory factory =
-              new SourceFactory([new ResourceUriResolver(provider)]);
-          AnalysisContext context =
-              AnalysisEngine.instance.createAnalysisContext();
-          context.sourceFactory = factory;
-          return context;
-        });
-        createExecutionContextIdForFile('/a/b.dart');
-      });
-
-      tearDown(() {
-        Request request =
-            new ExecutionDeleteContextParams(contextId).toRequest('1');
-        Response response = handler.handleRequest(request);
-        expect(response, isResponseSuccess('1'));
-      });
-
-      group('file to URI', () {
-        test('does not exist', () {
-          Request request =
-              new ExecutionMapUriParams(contextId, file: '/a/c.dart')
-                  .toRequest('2');
-          Response response = handler.handleRequest(request);
-          expect(response, isResponseFailure('2'));
-        });
-
-        test('directory', () {
-          provider.newFolder('/a/d');
-          Request request =
-              new ExecutionMapUriParams(contextId, file: '/a/d').toRequest('2');
-          Response response = handler.handleRequest(request);
-          expect(response, isResponseFailure('2'));
-        });
-      });
-
-      group('URI to file', () {
-        test('invalid', () {
-          Request request =
-              new ExecutionMapUriParams(contextId, uri: 'foo:///a/b.dart')
-                  .toRequest('2');
-          Response response = handler.handleRequest(request);
-          expect(response, isResponseFailure('2'));
-        });
-      });
-
-      test('invalid context id', () {
-        Request request =
-            new ExecutionMapUriParams('xxx', uri: '').toRequest('4');
-        Response response = handler.handleRequest(request);
-        expect(response, isResponseFailure('4'));
-      });
-
-      test('both file and uri', () {
-        Request request =
-            new ExecutionMapUriParams('xxx', file: '', uri: '').toRequest('5');
-        Response response = handler.handleRequest(request);
-        expect(response, isResponseFailure('5'));
-      });
-
-      test('neither file nor uri', () {
-        Request request = new ExecutionMapUriParams('xxx').toRequest('6');
-        Response response = handler.handleRequest(request);
-        expect(response, isResponseFailure('6'));
-      });
-    });
-
-    group('setSubscriptions', () {
-      test('failure - invalid service name', () {
-        expect(handler.onFileAnalyzed, isNull);
-
-        Request request = new Request('0', EXECUTION_SET_SUBSCRIPTIONS, {
-          SUBSCRIPTIONS: ['noSuchService']
-        });
-        Response response = handler.handleRequest(request);
-        expect(response, isResponseFailure('0'));
-        expect(handler.onFileAnalyzed, isNull);
-      });
-
-      test('success - setting and clearing', () {
-        expect(handler.onFileAnalyzed, isNull);
-
-        Request request =
-            new ExecutionSetSubscriptionsParams([ExecutionService.LAUNCH_DATA])
-                .toRequest('0');
-        Response response = handler.handleRequest(request);
-        expect(response, isResponseSuccess('0'));
-        expect(handler.onFileAnalyzed, isNotNull);
-
-        request = new ExecutionSetSubscriptionsParams([]).toRequest('0');
-        response = handler.handleRequest(request);
-        expect(response, isResponseSuccess('0'));
-        expect(handler.onFileAnalyzed, isNull);
-      });
-    });
+    // TODO(brianwilkerson) Re-enable these tests if we re-enable the
+    // execution.mapUri request.
+//    group('mapUri', () {
+//      String contextId;
+//
+//      void createExecutionContextIdForFile(String path) {
+//        Request request = new ExecutionCreateContextParams(path).toRequest('0');
+//        Response response = handler.handleRequest(request);
+//        expect(response, isResponseSuccess('0'));
+//        ExecutionCreateContextResult result =
+//            new ExecutionCreateContextResult.fromResponse(response);
+//        contextId = result.id;
+//      }
+//
+//      setUp(() {
+//        Folder folder = provider.newFile('/a/b.dart', '').parent;
+//        server.folderMap.putIfAbsent(folder, () {
+//          SourceFactory factory =
+//              new SourceFactory([new ResourceUriResolver(provider)]);
+//          AnalysisContext context =
+//              AnalysisEngine.instance.createAnalysisContext();
+//          context.sourceFactory = factory;
+//          return context;
+//        });
+//        createExecutionContextIdForFile('/a/b.dart');
+//      });
+//
+//      tearDown(() {
+//        Request request =
+//            new ExecutionDeleteContextParams(contextId).toRequest('1');
+//        Response response = handler.handleRequest(request);
+//        expect(response, isResponseSuccess('1'));
+//      });
+//
+//      group('file to URI', () {
+//        test('does not exist', () {
+//          Request request =
+//              new ExecutionMapUriParams(contextId, file: '/a/c.dart')
+//                  .toRequest('2');
+//          Response response = handler.handleRequest(request);
+//          expect(response, isResponseFailure('2'));
+//        });
+//
+//        test('directory', () {
+//          provider.newFolder('/a/d');
+//          Request request =
+//              new ExecutionMapUriParams(contextId, file: '/a/d').toRequest('2');
+//          Response response = handler.handleRequest(request);
+//          expect(response, isResponseFailure('2'));
+//        });
+//      });
+//
+//      group('URI to file', () {
+//        test('invalid', () {
+//          Request request =
+//              new ExecutionMapUriParams(contextId, uri: 'foo:///a/b.dart')
+//                  .toRequest('2');
+//          Response response = handler.handleRequest(request);
+//          expect(response, isResponseFailure('2'));
+//        });
+//      });
+//
+//      test('invalid context id', () {
+//        Request request =
+//            new ExecutionMapUriParams('xxx', uri: '').toRequest('4');
+//        Response response = handler.handleRequest(request);
+//        expect(response, isResponseFailure('4'));
+//      });
+//
+//      test('both file and uri', () {
+//        Request request =
+//            new ExecutionMapUriParams('xxx', file: '', uri: '').toRequest('5');
+//        Response response = handler.handleRequest(request);
+//        expect(response, isResponseFailure('5'));
+//      });
+//
+//      test('neither file nor uri', () {
+//        Request request = new ExecutionMapUriParams('xxx').toRequest('6');
+//        Response response = handler.handleRequest(request);
+//        expect(response, isResponseFailure('6'));
+//      });
+//    });
   });
 }
 
 @reflectiveTest
 class ExecutionDomainTest extends AbstractAnalysisTest {
   String contextId;
 
   @override
   void setUp() {
     super.setUp();
@@ skipping 65 matching lines @@
  * A [Source] that knows it's [fullName].
  */
 class TestSource implements Source {
   String fullName;
 
   TestSource(this.fullName);
 
   @override
   noSuchMethod(Invocation invocation) => super.noSuchMethod(invocation);
 }