Make RefactoringStatus a collection of generated RefactoringProblems.

pkg/analysis_server/test/services/refactoring/rename_constructor_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.services.refactoring.rename_constructor;
 
-import 'package:analysis_server/src/services/correction/status.dart';
+import 'package:analysis_server/src/protocol2.dart';
 import 'package:analysis_testing/reflective_tests.dart';
 import 'package:analyzer/src/generated/ast.dart';
 import 'package:analyzer/src/generated/element.dart';
 import 'package:unittest/unittest.dart';
 
 import 'abstract_rename.dart';
 
 
 main() {
   groupSep = ' | ';
   runReflectiveTests(RenameConstructorTest);
 }
 
 
 @ReflectiveTestCase()
 class RenameConstructorTest extends RenameRefactoringTest {
   test_checkFinalConditions_hasMember_constructor() {
     indexTestUnit('''
 class A {
   A.test() {}
   A.newName() {} // existing
 }
 ''');
     _createConstructorDeclarationRefactoring('test() {}');
     // check status
     refactoring.newName = 'newName';
     return refactoring.checkFinalConditions().then((status) {
       assertRefactoringStatus(
           status,
-          RefactoringStatusSeverity.ERROR,
+          RefactoringProblemSeverity.ERROR,
           expectedMessage: "Class 'A' already declares constructor with name 'newName'.",
           expectedContextSearch: 'newName() {} // existing');
     });
   }
 
   test_checkFinalConditions_hasMember_method() {
     indexTestUnit('''
 class A {
   A.test() {}
   newName() {} // existing
 }
 ''');
     _createConstructorDeclarationRefactoring('test() {}');
     // check status
     refactoring.newName = 'newName';
     return refactoring.checkFinalConditions().then((status) {
       assertRefactoringStatus(
           status,
-          RefactoringStatusSeverity.ERROR,
+          RefactoringProblemSeverity.ERROR,
           expectedMessage: "Class 'A' already declares method with name 'newName'.",
           expectedContextSearch: 'newName() {} // existing');
     });
   }
 
   test_checkNewName() {
     indexTestUnit('''
 class A {
   A.test() {}
 }
 ''');
     createRenameRefactoringAtString('test() {}');
     expect(refactoring.oldName, 'test');
     // null
     refactoring.newName = null;
     assertRefactoringStatus(
         refactoring.checkNewName(),
-        RefactoringStatusSeverity.ERROR,
+        RefactoringProblemSeverity.ERROR,
         expectedMessage: "Constructor name must not be null.");
     // same
     refactoring.newName = 'test';
     assertRefactoringStatus(
         refactoring.checkNewName(),
-        RefactoringStatusSeverity.FATAL,
+        RefactoringProblemSeverity.FATAL,
         expectedMessage: "The new name must be different than the current name.");
     // empty
     refactoring.newName = '';
     assertRefactoringStatusOK(refactoring.checkNewName());
     // OK
     refactoring.newName = 'newName';
     assertRefactoringStatusOK(refactoring.checkNewName());
   }
 
   test_createChange_add() {
@@ skipping 94 matching lines @@
 }
 ''');
   }
 
   void _createConstructorDeclarationRefactoring(String search) {
     ConstructorElement element =
         findNodeElementAtString(search, (node) => node is ConstructorDeclaration);
     createRenameRefactoringForElement(element);
   }
 }