Checks for shadowing in 'Rename Class Member' refactoring.

pkg/analysis_services/test/refactoring/rename_class_member_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_class_member;
 
 import 'package:analysis_services/correction/status.dart';
 import 'package:analysis_testing/reflective_tests.dart';
 import 'package:unittest/unittest.dart';
 
 import 'abstract_rename.dart';
-import 'package:analysis_services/correction/change.dart';
 
 
 main() {
   groupSep = ' | ';
   runReflectiveTests(RenameClassMemberTest);
 }
 
 
 @ReflectiveTestCase()
 class RenameClassMemberTest extends RenameRefactoringTest {
+  test_checkFinalConditions_OK_noShadow() {
+    indexTestUnit('''
+class A {
+  int newName;
+}
+class B {
+  test() {}
+}
+class C extends A {
+  main() {
+    print(newName);
+  }
+}
+''');
+    createRenameRefactoringAtString('test() {}');
+    // check status
+    refactoring.newName = 'newName';
+    return refactoring.checkFinalConditions().then((status) {
+      assertRefactoringStatusOK(status);
+    });
+  }
+
+  test_checkFinalConditions_hasMember_MethodElement() {
+    indexTestUnit('''
+class A {
+  test() {}
+  newName() {} // existing
+}
+''');
+    createRenameRefactoringAtString('test() {}');
+    // check status
+    refactoring.newName = 'newName';
+    return refactoring.checkFinalConditions().then((status) {
+      assertRefactoringStatus(
+          status,
+          RefactoringStatusSeverity.ERROR,
+          expectedMessage: "Class 'A' already declares method with name 'newName'.",
+          expectedContextSearch: 'newName() {} // existing');
+    });
+  }
+
+  test_checkFinalConditions_shadowed_byLocal_OK_qualifiedReference() {
+    indexTestUnit('''
+class A {
+  test() {}
+  main() {
+    var newName;
+    this.test(); // marker
+  }
+}
+''');
+    createRenameRefactoringAtString('test() {}');
+    // check status
+    refactoring.newName = 'newName';
+    return refactoring.checkFinalConditions().then((status) {
+      assertRefactoringStatusOK(status);
+    });
+  }
+
+  test_checkFinalConditions_shadowed_byLocal_OK_renamedNotUsed() {
+    indexTestUnit('''
+class A {
+  test() {}
+  main() {
+    var newName;
+  }
+}
+''');
+    createRenameRefactoringAtString('test() {}');
+    // check status
+    refactoring.newName = 'newName';
+    return refactoring.checkFinalConditions().then((status) {
+      assertRefactoringStatusOK(status);
+    });
+  }
+
+  test_checkFinalConditions_shadowed_byLocal_inSameClass() {
+    indexTestUnit('''
+class A {
+  test() {}
+  main() {
+    var newName;
+    test(); // marker
+  }
+}
+''');
+    createRenameRefactoringAtString('test() {}');
+    // check status
+    refactoring.newName = 'newName';
+    return refactoring.checkFinalConditions().then((status) {
+      assertRefactoringStatus(
+          status,
+          RefactoringStatusSeverity.ERROR,
+          expectedMessage:
+              "Usage of renamed method will be shadowed by local variable 'newName'.",
+          expectedContextSearch: 'test(); // marker');
+    });
+  }
+
+  test_checkFinalConditions_shadowed_byLocal_inSubClass() {
+    indexTestUnit('''
+class A {
+  test() {}
+}
+class B extends A {
+  main() {
+    var newName;
+    test(); // marker
+  }
+}
+''');
+    createRenameRefactoringAtString('test() {}');
+    // check status
+    refactoring.newName = 'newName';
+    return refactoring.checkFinalConditions().then((status) {
+      assertRefactoringStatus(
+          status,
+          RefactoringStatusSeverity.ERROR,
+          expectedMessage:
+              "Usage of renamed method will be shadowed by local variable 'newName'.",
+          expectedContextSearch: 'test(); // marker');
+    });
+  }
+
+  test_checkFinalConditions_shadowed_byParameter_inSameClass() {
+    indexTestUnit('''
+class A {
+  test() {}
+  main(newName) {
+    test(); // marker
+  }
+}
+''');
+    createRenameRefactoringAtString('test() {}');
+    // check status
+    refactoring.newName = 'newName';
+    return refactoring.checkFinalConditions().then((status) {
+      assertRefactoringStatus(
+          status,
+          RefactoringStatusSeverity.ERROR,
+          expectedMessage:
+              "Usage of renamed method will be shadowed by parameter 'newName'.",
+          expectedContextSearch: 'test(); // marker');
+    });
+  }
+
+  test_checkFinalConditions_shadowed_inSubClass() {
+    indexTestUnit('''
+class A {
+  newName() {} // marker
+}
+class B extends A {
+  test() {}
+  main() {
+    newName();
+  }
+}
+''');
+    createRenameRefactoringAtString('test() {}');
+    // check status
+    refactoring.newName = 'newName';
+    return refactoring.checkFinalConditions().then((status) {
+      assertRefactoringStatus(
+          status,
+          RefactoringStatusSeverity.ERROR,
+          expectedMessage: "Renamed method will shadow method 'A.newName'.",
+          expectedContextSearch: 'newName() {} // marker');
+    });
+  }
+
+  test_checkFinalConditions_shadowsSuper_MethodElement() {
+    indexTestUnit('''
+class A {
+  test() {}
+}
+class B extends A {
+  newName() {} // marker
+  main() {
+    test();
+  }
+}
+''');
+    createRenameRefactoringAtString('test() {}');
+    // check status
+    refactoring.newName = 'newName';
+    return refactoring.checkFinalConditions().then((status) {
+      assertRefactoringStatus(
+          status,
+          RefactoringStatusSeverity.ERROR,
+          expectedMessage: "Renamed method will be shadowed by method 'B.newName'.",
+          expectedContextSearch: 'newName() {} // marker');
+    });
+  }
+
+  test_checkFinalConditions_shadowsSuper_inSubClass_FieldElement() {
+    indexTestUnit('''
+class A {
+  int newName; // marker
+}
+class B extends A {
+  test() {}
+}
+class C extends B {
+  main() {
+    print(newName);
+  }
+}
+''');
+    createRenameRefactoringAtString('test() {}');
+    // check status
+    refactoring.newName = 'newName';
+    return refactoring.checkFinalConditions().then((status) {
+      assertRefactoringStatus(
+          status,
+          RefactoringStatusSeverity.ERROR,
+          expectedMessage: "Renamed method will shadow field 'A.newName'.",
+          expectedContextSearch: 'newName; // marker');
+    });
+  }
+
   test_checkInitialConditions_operator() {
     indexTestUnit('''
 class A {
   operator -(other) => this;
 }
 ''');
     createRenameRefactoringAtString('-(other)');
     // check status
     refactoring.newName = 'newName';
     return refactoring.checkInitialConditions().then((status) {
@@ skipping 340 matching lines @@
 }
 main(var a) {
   a.newName();
   new A().newName();
 }
 ''').then((_) {
       assertNoFileChange('/lib.dart');
     });
   }
 
-  test_createChange_TypeParameterElement() {
-    indexTestUnit('''
-class A<Test> {
-  Test field;
-  List<Test> items;
-  Test method(Test p) => null;
-}
-''');
-    // configure refactoring
-    createRenameRefactoringAtString('Test> {');
-    expect(refactoring.refactoringName, 'Rename Type Parameter');
-    expect(refactoring.oldName, 'Test');
-    refactoring.newName = 'NewName';
-    // validate change
-    return assertSuccessfulRename('''
-class A<NewName> {
-  NewName field;
-  List<NewName> items;
-  NewName method(NewName p) => null;
-}
-''');
-  }
-
   test_createChange_PropertyAccessorElement_getter() {
     indexTestUnit('''
 class A {
   get test {} // marker
   set test(x) {}
   main() {
     print(test);
     test = 1;
   }
 }
@@ skipping 89 matching lines @@
   A a = new A();
   print(a.newName);
   a.newName = 2;
 
   B b = new B();
   print(b.newName);
   b.newName = 2;
 }
 ''');
   }
+
+  test_createChange_TypeParameterElement() {
+    indexTestUnit('''
+class A<Test> {
+  Test field;
+  List<Test> items;
+  Test method(Test p) => null;
+}
+''');
+    // configure refactoring
+    createRenameRefactoringAtString('Test> {');
+    expect(refactoring.refactoringName, 'Rename Type Parameter');
+    expect(refactoring.oldName, 'Test');
+    refactoring.newName = 'NewName';
+    // validate change
+    return assertSuccessfulRename('''
+class A<NewName> {
+  NewName field;
+  List<NewName> items;
+  NewName method(NewName p) => null;
+}
+''');
+  }
 }