Create a public API for contributing fixes and make fixes pluggable

pkg/analysis_server/test/edit/refactoring_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.edit.refactoring;
 
 import 'dart:async';
 
 import 'package:analysis_server/src/edit/edit_domain.dart';
+import 'package:analysis_server/src/plugin/server_plugin.dart';
 import 'package:analysis_server/src/protocol.dart';
 import 'package:analysis_server/src/services/index/index.dart';
 import 'package:analysis_server/src/services/index/local_memory_index.dart';
+import 'package:analyzer/src/plugin/plugin_impl.dart';
 import 'package:unittest/unittest.dart' hide ERROR;
 
 import '../analysis_abstract.dart';
 import '../mocks.dart';
 import '../reflective_tests.dart';
 
 main() {
   groupSep = ' | ';
   runReflectiveTests(ConvertGetterMethodToMethodTest);
   runReflectiveTests(ConvertMethodToGetterTest);
@@ skipping 579 matching lines @@
     offset = findOffset(search);
     length = search.length;
   }
 }
 
 @reflectiveTest
 class GetAvailableRefactoringsTest extends AbstractAnalysisTest {
   List<RefactoringKind> kinds;
 
   /**
-   * Tests that there is a RENAME refactoring available at the [search] offset.
-   */
-  Future assertHasRenameRefactoring(String code, String search) async {
-    return assertHasKind(code, search, RefactoringKind.RENAME, true);
-  }
-
-  /**
    * Tests that there is refactoring of the given [kind] is available at the
    * [search] offset.
    */
   Future assertHasKind(
       String code, String search, RefactoringKind kind, bool expected) async {
     addTestFile(code);
     await waitForTasksFinished();
     await getRefactoringsAtString(search);
     // verify
     Matcher matcher = contains(kind);
     if (!expected) {
       matcher = isNot(matcher);
     }
     expect(kinds, matcher);
   }
 
+  /**
+   * Tests that there is a RENAME refactoring available at the [search] offset.
+   */
+  Future assertHasRenameRefactoring(String code, String search) async {
+    return assertHasKind(code, search, RefactoringKind.RENAME, true);
+  }
+
   @override
   Index createIndex() {
     return createLocalMemoryIndex();
   }
 
   /**
    * Returns the list of available refactorings for the given [offset] and
    * [length].
    */
   Future getRefactorings(int offset, int length) async {
@@ skipping 15 matching lines @@
 
   Future getRefactoringsForString(String search) {
     int offset = findOffset(search);
     return getRefactorings(offset, search.length);
   }
 
   @override
   void setUp() {
     super.setUp();
     createProject();
-    handler = new EditDomainHandler(server);
+    ExtensionManager manager = new ExtensionManager();
+    ServerPlugin plugin = new ServerPlugin();
+    manager.processPlugins([plugin]);
+    handler = new EditDomainHandler(server, plugin);
     server.handlers = [handler];
   }
 
+  Future test_convertMethodToGetter_hasElement() {
+    return assertHasKind('''
+int getValue() => 42;
+''', 'getValue', RefactoringKind.CONVERT_METHOD_TO_GETTER, true);
+  }
+
   Future test_extractLocal() async {
     addTestFile('''
 main() {
   var a = 1 + 2;
 }
 ''');
     await waitForTasksFinished();
     await getRefactoringsForString('1 + 2');
     expect(kinds, contains(RefactoringKind.EXTRACT_LOCAL_VARIABLE));
     expect(kinds, contains(RefactoringKind.EXTRACT_METHOD));
   }
 
-  Future test_convertMethodToGetter_hasElement() {
-    return assertHasKind('''
-int getValue() => 42;
-''', 'getValue', RefactoringKind.CONVERT_METHOD_TO_GETTER, true);
-  }
-
   Future test_rename_hasElement_class() {
     return assertHasRenameRefactoring('''
 class Test {}
 main() {
   Test v;
 }
 ''', 'Test v');
   }
 
   Future test_rename_hasElement_constructor() {
@@ skipping 993 matching lines @@
       [bool validateOnly = false]) {
     Request request = new EditGetRefactoringParams(
             kind, testFile, offset, length, validateOnly, options: options)
         .toRequest('0');
     return serverChannel.sendRequest(request);
   }
 
   @override
   void setUp() {
     super.setUp();
-    server.handlers = [new EditDomainHandler(server),];
     createProject();
-    handler = new EditDomainHandler(server);
+    ExtensionManager manager = new ExtensionManager();
+    ServerPlugin plugin = new ServerPlugin();
+    manager.processPlugins([plugin]);
+    handler = new EditDomainHandler(server, plugin);
+    server.handlers = [handler];
   }
 }
 
 @reflectiveTest
 class _NoSearchEngine extends _AbstractGetRefactoring_Test {
   @override
   Index createIndex() {
     return null;
   }
 
@@ skipping 21 matching lines @@
         .toRequest('0');
     return _assertErrorResposeNoIndex(request);
   }
 
   _assertErrorResposeNoIndex(Request request) async {
     Response response = await serverChannel.sendRequest(request);
     expect(response.error, isNotNull);
     expect(response.error.code, RequestErrorCode.NO_INDEX_GENERATED);
   }
 }