Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(25)

Unified Diff: pkg/analysis_services/test/refactoring/rename_class_member_test.dart

Issue 484733003: Import analysis_services.dart into analysis_server.dart. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: pkg/analysis_services/test/refactoring/rename_class_member_test.dart
diff --git a/pkg/analysis_services/test/refactoring/rename_class_member_test.dart b/pkg/analysis_services/test/refactoring/rename_class_member_test.dart
deleted file mode 100644
index f1b716304ea550223da333c9b0a0b4bec49f8629..0000000000000000000000000000000000000000
--- a/pkg/analysis_services/test/refactoring/rename_class_member_test.dart
+++ /dev/null
@@ -1,734 +0,0 @@
-// 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';
-
-
-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) {
- assertRefactoringStatus(status, RefactoringStatusSeverity.FATAL);
- });
- }
-
- test_checkNewName_FieldElement() {
- indexTestUnit('''
-class A {
- int test;
-}
-''');
- createRenameRefactoringAtString('test;');
- // null
- refactoring.newName = null;
- assertRefactoringStatus(
- refactoring.checkNewName(),
- RefactoringStatusSeverity.ERROR,
- expectedMessage: "Field name must not be null.");
- // OK
- refactoring.newName = 'newName';
- assertRefactoringStatusOK(refactoring.checkNewName());
- }
-
- test_checkNewName_FieldElement_const() {
- indexTestUnit('''
-class A {
- static const int TEST = 0;
-}
-''');
- createRenameRefactoringAtString('TEST =');
- // null
- refactoring.newName = null;
- assertRefactoringStatus(
- refactoring.checkNewName(),
- RefactoringStatusSeverity.ERROR,
- expectedMessage: "Constant name must not be null.");
- // not upper case
- refactoring.newName = 'newName';
- assertRefactoringStatus(
- refactoring.checkNewName(),
- RefactoringStatusSeverity.WARNING);
- // OK
- refactoring.newName = 'NEW_NAME';
- assertRefactoringStatusOK(refactoring.checkNewName());
- }
-
- test_checkNewName_MethodElement() {
- indexTestUnit('''
-class A {
- test() {}
-}
-''');
- createRenameRefactoringAtString('test() {}');
- // null
- refactoring.newName = null;
- assertRefactoringStatus(
- refactoring.checkNewName(),
- RefactoringStatusSeverity.ERROR,
- expectedMessage: "Method name must not be null.");
- // empty
- refactoring.newName = '';
- assertRefactoringStatus(
- refactoring.checkNewName(),
- RefactoringStatusSeverity.ERROR,
- expectedMessage: "Method name must not be empty.");
- // same
- refactoring.newName = 'test';
- assertRefactoringStatus(
- refactoring.checkNewName(),
- RefactoringStatusSeverity.FATAL,
- expectedMessage: "The new name must be different than the current name.");
- // OK
- refactoring.newName = 'newName';
- assertRefactoringStatusOK(refactoring.checkNewName());
- }
-
- test_createChange_FieldElement() {
- indexTestUnit('''
-class A {
- int test; // marker
- main() {
- print(test);
- test = 1;
- test += 2;
- }
-}
-class B extends A {
-}
-class C extends B {
- get test => 1;
- set test(x) {}
-}
-main() {
- A a = new A();
- B b = new B();
- C c = new C();
- print(a.test);
- a.test = 1;
- a.test += 2;
- print(b.test);
- b.test = 1;
- print(c.test);
- c.test = 1;
-}
-''');
- // configure refactoring
- createRenameRefactoringAtString('test; // marker');
- expect(refactoring.refactoringName, 'Rename Field');
- expect(refactoring.oldName, 'test');
- refactoring.newName = 'newName';
- // validate change
- return assertSuccessfulRename('''
-class A {
- int newName; // marker
- main() {
- print(newName);
- newName = 1;
- newName += 2;
- }
-}
-class B extends A {
-}
-class C extends B {
- get newName => 1;
- set newName(x) {}
-}
-main() {
- A a = new A();
- B b = new B();
- C c = new C();
- print(a.newName);
- a.newName = 1;
- a.newName += 2;
- print(b.newName);
- b.newName = 1;
- print(c.newName);
- c.newName = 1;
-}
-''');
- }
-
- test_createChange_FieldElement_constructorFieldInitializer() {
- indexTestUnit('''
-class A {
- final test;
- A() : test = 5;
-}
-''');
- // configure refactoring
- createRenameRefactoringAtString('test;');
- expect(refactoring.refactoringName, 'Rename Field');
- expect(refactoring.oldName, 'test');
- refactoring.newName = 'newName';
- // validate change
- return assertSuccessfulRename('''
-class A {
- final newName;
- A() : newName = 5;
-}
-''');
- }
-
- test_createChange_FieldElement_fieldFormalParameter() {
- indexTestUnit('''
-class A {
- final test;
- A(this.test);
-}
-''');
- // configure refactoring
- createRenameRefactoringAtString('test;');
- expect(refactoring.refactoringName, 'Rename Field');
- expect(refactoring.oldName, 'test');
- refactoring.newName = 'newName';
- // validate change
- return assertSuccessfulRename('''
-class A {
- final newName;
- A(this.newName);
-}
-''');
- }
-
- test_createChange_FieldElement_invocation() {
- indexTestUnit('''
-typedef F(a);
-class A {
- F test;
- main() {
- test(1);
- }
-}
-main() {
- A a = new A();
- a.test(2);
-}
-''');
- // configure refactoring
- createRenameRefactoringAtString('test(2);');
- expect(refactoring.refactoringName, 'Rename Field');
- expect(refactoring.oldName, 'test');
- refactoring.newName = 'newName';
- // validate change
- return assertSuccessfulRename('''
-typedef F(a);
-class A {
- F newName;
- main() {
- newName(1);
- }
-}
-main() {
- A a = new A();
- a.newName(2);
-}
-''');
- }
-
- test_createChange_MethodElement() {
- indexTestUnit('''
-class A {
- test() {}
-}
-class B extends A {
- test() {} // marker
-}
-class C extends B {
- test() {}
-}
-class D implements A {
- test() {}
-}
-class E {
- test() {}
-}
-main() {
- A a = new A();
- B b = new B();
- C c = new C();
- D d = new D();
- E e = new E();
- a.test();
- b.test();
- c.test();
- d.test();
- e.test();
-}
-''');
- // configure refactoring
- createRenameRefactoringAtString('test() {} // marker');
- expect(refactoring.refactoringName, 'Rename Method');
- expect(refactoring.oldName, 'test');
- refactoring.newName = 'newName';
- // validate change
- return assertSuccessfulRename('''
-class A {
- newName() {}
-}
-class B extends A {
- newName() {} // marker
-}
-class C extends B {
- newName() {}
-}
-class D implements A {
- newName() {}
-}
-class E {
- test() {}
-}
-main() {
- A a = new A();
- B b = new B();
- C c = new C();
- D d = new D();
- E e = new E();
- a.newName();
- b.newName();
- c.newName();
- d.newName();
- e.test();
-}
-''');
- }
-
- test_createChange_MethodElement_potential() {
- indexTestUnit('''
-class A {
- test() {}
-}
-main(var a) {
- a.test(); // 1
- new A().test();
- a.test(); // 2
-}
-''');
- // configure refactoring
- createRenameRefactoringAtString('test() {}');
- expect(refactoring.refactoringName, 'Rename Method');
- expect(refactoring.oldName, 'test');
- refactoring.newName = 'newName';
- // validate change
- return assertSuccessfulRename('''
-class A {
- newName() {}
-}
-main(var a) {
- a.newName(); // 1
- new A().newName();
- a.newName(); // 2
-}
-''').then((_) {
- assertPotentialEdits(['test(); // 1', 'test(); // 2']);
- });
- }
-
- test_createChange_MethodElement_potential_private_otherLibrary() {
- indexUnit('/lib.dart', '''
-library lib;
-main(p) {
- p._test();
-}
-''');
- indexTestUnit('''
-class A {
- _test() {}
-}
-main(var a) {
- a._test();
- new A()._test();
-}
-''');
- // configure refactoring
- createRenameRefactoringAtString('_test() {}');
- expect(refactoring.refactoringName, 'Rename Method');
- expect(refactoring.oldName, '_test');
- refactoring.newName = 'newName';
- // validate change
- return assertSuccessfulRename('''
-class A {
- newName() {}
-}
-main(var a) {
- a.newName();
- new A().newName();
-}
-''').then((_) {
- assertNoFileChange('/lib.dart');
- });
- }
-
- test_createChange_PropertyAccessorElement_getter() {
- indexTestUnit('''
-class A {
- get test {} // marker
- set test(x) {}
- main() {
- print(test);
- test = 1;
- }
-}
-class B extends A {
- get test {}
- set test(x) {}
-}
-main() {
- A a = new A();
- print(a.test);
- a.test = 2;
-
- B b = new B();
- print(b.test);
- b.test = 2;
-}
-''');
- // configure refactoring
- createRenameRefactoringAtString('test {} // marker');
- expect(refactoring.refactoringName, 'Rename Field');
- expect(refactoring.oldName, 'test');
- refactoring.newName = 'newName';
- // validate change
- return assertSuccessfulRename('''
-class A {
- get newName {} // marker
- set newName(x) {}
- main() {
- print(newName);
- newName = 1;
- }
-}
-class B extends A {
- get newName {}
- set newName(x) {}
-}
-main() {
- A a = new A();
- print(a.newName);
- a.newName = 2;
-
- B b = new B();
- print(b.newName);
- b.newName = 2;
-}
-''');
- }
-
- test_createChange_PropertyAccessorElement_setter() {
- indexTestUnit('''
-class A {
- get test {}
- set test(x) {} // marker
- main() {
- print(test);
- test = 1;
- }
-}
-class B extends A {
- get test {}
- set test(x) {}
-}
-main() {
- A a = new A();
- print(a.test);
- a.test = 2;
-
- B b = new B();
- print(b.test);
- b.test = 2;
-}
-''');
- // configure refactoring
- createRenameRefactoringAtString('test(x) {} // marker');
- expect(refactoring.refactoringName, 'Rename Field');
- expect(refactoring.oldName, 'test');
- refactoring.newName = 'newName';
- // validate change
- return assertSuccessfulRename('''
-class A {
- get newName {}
- set newName(x) {} // marker
- main() {
- print(newName);
- newName = 1;
- }
-}
-class B extends A {
- get newName {}
- set newName(x) {}
-}
-main() {
- 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;
-}
-''');
- }
-}

Powered by Google App Engine
This is Rietveld 408576698