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

Unified Diff: editor/tools/plugins/com.google.dart.tools.ui_test/src/com/google/dart/tools/ui/refactoring/RenameMethodProcessorTest.java

Issue 11273064: Issue 5887. Parts must start with 'part of'. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 2 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: editor/tools/plugins/com.google.dart.tools.ui_test/src/com/google/dart/tools/ui/refactoring/RenameMethodProcessorTest.java
diff --git a/editor/tools/plugins/com.google.dart.tools.ui_test/src/com/google/dart/tools/ui/refactoring/RenameMethodProcessorTest.java b/editor/tools/plugins/com.google.dart.tools.ui_test/src/com/google/dart/tools/ui/refactoring/RenameMethodProcessorTest.java
index 263abc7dca33d28bc5c981bf5e5bc7f24c229b45..509ddf612b2723fb10214f6a5038bca9bdb1eb91 100644
--- a/editor/tools/plugins/com.google.dart.tools.ui_test/src/com/google/dart/tools/ui/refactoring/RenameMethodProcessorTest.java
+++ b/editor/tools/plugins/com.google.dart.tools.ui_test/src/com/google/dart/tools/ui/refactoring/RenameMethodProcessorTest.java
@@ -235,9 +235,9 @@ public final class RenameMethodProcessorTest extends RefactoringTest {
}
public void test_OK_multipleUnits_onReference() throws Exception {
- setUnitContent(
- "Test1.dart",
+ setUnitContent("Test1.dart", new String[] {
"// filler filler filler filler filler filler filler filler filler filler",
+ "part of test;",
"class A {",
" test(var p) {}",
" int bar = 2;",
@@ -245,28 +245,28 @@ public final class RenameMethodProcessorTest extends RefactoringTest {
" test(3);",
" bar = 4;",
" }",
- "}");
- setUnitContent(
- "Test2.dart",
+ "}"});
+ setUnitContent("Test2.dart", new String[] {
"// filler filler filler filler filler filler filler filler filler filler",
+ "part of test;",
"f2() {",
" A a = new A();",
" a.test(5);",
- "}");
- setUnitContent(
- "Test3.dart",
+ "}"});
+ setUnitContent("Test3.dart", new String[] {
"// filler filler filler filler filler filler filler filler filler filler",
+ "part of test;",
"class B extends A {",
" f3() {",
" test(6);",
" }",
- "}");
+ "}"});
setTestUnitContent(
"// filler filler filler filler filler filler filler filler filler filler",
- "#library('test');",
- "#source('Test1.dart');",
- "#source('Test2.dart');",
- "#source('Test3.dart');");
+ "library test;",
+ "part 'Test1.dart';",
+ "part 'Test2.dart';",
+ "part 'Test3.dart';");
// get units, because they have not library
CompilationUnit unit1 = testProject.getUnit("Test1.dart");
CompilationUnit unit2 = testProject.getUnit("Test2.dart");
@@ -275,9 +275,9 @@ public final class RenameMethodProcessorTest extends RefactoringTest {
Method method = findElement(unit2, "test(5);");
// do rename
renameMethod(method, "newName");
- assertUnitContent(
- unit1,
+ assertUnitContent(unit1, new String[] {
"// filler filler filler filler filler filler filler filler filler filler",
+ "part of test;",
"class A {",
" newName(var p) {}",
" int bar = 2;",
@@ -285,22 +285,22 @@ public final class RenameMethodProcessorTest extends RefactoringTest {
" newName(3);",
" bar = 4;",
" }",
- "}");
- assertUnitContent(
- unit2,
+ "}"});
+ assertUnitContent(unit2, new String[] {
"// filler filler filler filler filler filler filler filler filler filler",
+ "part of test;",
"f2() {",
" A a = new A();",
" a.newName(5);",
- "}");
- assertUnitContent(
- unit3,
+ "}"});
+ assertUnitContent(unit3, new String[] {
"// filler filler filler filler filler filler filler filler filler filler",
+ "part of test;",
"class B extends A {",
" f3() {",
" newName(6);",
" }",
- "}");
+ "}"});
}
public void test_OK_namedConstructor_newName() throws Exception {

Powered by Google App Engine
This is Rietveld 408576698