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

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

Issue 11314019: Using 'interface' and 'abstract' for methods should produce error (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/RenameTypeProcessorTest.java
diff --git a/editor/tools/plugins/com.google.dart.tools.ui_test/src/com/google/dart/tools/ui/refactoring/RenameTypeProcessorTest.java b/editor/tools/plugins/com.google.dart.tools.ui_test/src/com/google/dart/tools/ui/refactoring/RenameTypeProcessorTest.java
index 2e54251616e13f16841e700d567825fb6122ce57..224860351793a1849a4b2b163d392723d84395f8 100644
--- a/editor/tools/plugins/com.google.dart.tools.ui_test/src/com/google/dart/tools/ui/refactoring/RenameTypeProcessorTest.java
+++ b/editor/tools/plugins/com.google.dart.tools.ui_test/src/com/google/dart/tools/ui/refactoring/RenameTypeProcessorTest.java
@@ -182,150 +182,6 @@ public final class RenameTypeProcessorTest extends RefactoringTest {
"");
}
- public void test_OK_interfaceFactory_hasImpl_renameFactory() throws Exception {
- setTestUnitContent(
- "// filler filler filler filler filler filler filler filler filler filler",
- "interface I default F {",
- " I();",
- " I.named();",
- "}",
- "class F implements I {",
- " factory F() {}",
- " factory F.named() {}",
- "}",
- "f() {",
- " new I();",
- " new I.named();",
- "}",
- "");
- Type type = findElement("F implements I {");
- // do rename
- renameType(type, "NewName");
- assertTestUnitContent(
- "// filler filler filler filler filler filler filler filler filler filler",
- "interface I default NewName {",
- " I();",
- " I.named();",
- "}",
- "class NewName implements I {",
- " factory NewName() {}",
- " factory NewName.named() {}",
- "}",
- "f() {",
- " new I();",
- " new I.named();",
- "}",
- "");
- }
-
- public void test_OK_interfaceFactory_hasImpl_renameInterface() throws Exception {
- setTestUnitContent(
- "// filler filler filler filler filler filler filler filler filler filler",
- "interface I default F {",
- " I();",
- " I.named();",
- "}",
- "class F implements I {",
- " factory F() {}",
- " factory F.named() {}",
- "}",
- "f() {",
- " new I();",
- " new I.named();",
- "}",
- "");
- Type type = findElement("I default F {");
- // do rename
- renameType(type, "NewName");
- assertTestUnitContent(
- "// filler filler filler filler filler filler filler filler filler filler",
- "interface NewName default F {",
- " NewName();",
- " NewName.named();",
- "}",
- "class F implements NewName {",
- " factory F() {}",
- " factory F.named() {}",
- "}",
- "f() {",
- " new NewName();",
- " new NewName.named();",
- "}",
- "");
- }
-
- public void test_OK_interfaceFactory_notImpl_renameFactory() throws Exception {
- setTestUnitContent(
- "// filler filler filler filler filler filler filler filler filler filler",
- "interface I default F {",
- " I();",
- " I.named();",
- "}",
- "class F { // marker",
- " factory I() {}",
- " factory I.named() {}",
- "}",
- "f() {",
- " new I();",
- " new I.named();",
- "}",
- "");
- Type type = findElement("F { // marker");
- // do rename
- renameType(type, "NewName");
- assertTestUnitContent(
- "// filler filler filler filler filler filler filler filler filler filler",
- "interface I default NewName {",
- " I();",
- " I.named();",
- "}",
- "class NewName { // marker",
- " factory I() {}",
- " factory I.named() {}",
- "}",
- "f() {",
- " new I();",
- " new I.named();",
- "}",
- "");
- }
-
- public void test_OK_interfaceFactory_notImpl_renameInterface() throws Exception {
- setTestUnitContent(
- "// filler filler filler filler filler filler filler filler filler filler",
- "interface I default F {",
- " I();",
- " I.named();",
- "}",
- "class F {",
- " factory I() {}",
- " factory I.named() {}",
- "}",
- "f() {",
- " new I();",
- " new I.named();",
- "}",
- "");
- Type type = findElement("I default F {");
- // do rename
- renameType(type, "NewName");
- assertTestUnitContent(
- "// filler filler filler filler filler filler filler filler filler filler",
- "interface NewName default F {",
- " NewName();",
- " NewName.named();",
- "}",
- "class F {",
- " factory NewName() {}",
- " factory NewName.named() {}",
- "}",
- "f() {",
- " new NewName();",
- " new NewName.named();",
- "}",
- "");
- }
-
public void test_OK_multipleUnits_onReference() throws Exception {
setUnitContent(
"Test1.dart",

Powered by Google App Engine
This is Rietveld 408576698