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

Unified Diff: compiler/javatests/com/google/dart/compiler/resolver/ResolverTest.java

Issue 9403026: Issue 1575. Allow declare new named parameters in method override. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Style tweaks. Created 8 years, 10 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: compiler/javatests/com/google/dart/compiler/resolver/ResolverTest.java
diff --git a/compiler/javatests/com/google/dart/compiler/resolver/ResolverTest.java b/compiler/javatests/com/google/dart/compiler/resolver/ResolverTest.java
index 2fb3263c8cb7b570471b97b4ea3ececaa8ab33d6..c48b79ba89fe4272af890f6885f6a68d5ae15a93 100644
--- a/compiler/javatests/com/google/dart/compiler/resolver/ResolverTest.java
+++ b/compiler/javatests/com/google/dart/compiler/resolver/ResolverTest.java
@@ -15,9 +15,7 @@ import com.google.dart.compiler.type.Types;
import junit.framework.Assert;
-import java.util.LinkedHashSet;
import java.util.List;
-import java.util.Set;
/**
* Basic tests of the resolver.
@@ -1043,66 +1041,6 @@ public class ResolverTest extends ResolverTestCase {
"}"));
}
- public void testImplementsAndOverrides1() {
- resolveAndTest(Joiner.on("\n").join(
- "class Object {}",
- "interface Interface {",
- " foo(x);",
- "}",
- "class Class implements Interface {",
- " foo() {}", // error
- "}"),
- ResolverErrorCode.CANNOT_OVERRIDE_METHOD_WRONG_NUM_PARAMS);
- }
-
- public void testImplementsAndOverrides2() {
- resolveAndTest(Joiner.on("\n").join(
- "class Object {}",
- "interface Interface {",
- " foo([x]);",
- "}",
- "class Class implements Interface {",
- " foo([x,y]) {}", // error
- "}"),
- ResolverErrorCode.CANNOT_OVERRIDE_METHOD_WRONG_NUM_PARAMS);
- }
-
- public void testImplementsAndOverrides3() {
- resolveAndTest(Joiner.on("\n").join(
- "class Object {}",
- "interface Interface {",
- " foo(x, [y]);",
- "}",
- "class Class implements Interface {",
- " foo([x,y]) {}", // error
- "}"),
- ResolverErrorCode.CANNOT_OVERRIDE_METHOD_NUM_NAMED_PARAMS);
- }
-
- public void testImplementsAndOverrides4() {
- resolveAndTest(Joiner.on("\n").join(
- "class Object {}",
- "interface Interface {",
- " foo([x,y]);",
- "}",
- "class Class implements Interface {",
- " foo([x]) {}", // error
- "}"),
- ResolverErrorCode.CANNOT_OVERRIDE_METHOD_WRONG_NUM_PARAMS);
- }
-
- public void testImplementsAndOverrides5() {
- resolveAndTest(Joiner.on("\n").join(
- "class Object {}",
- "interface Interface {",
- " foo([y,x]);",
- "}",
- "class Class implements Interface {",
- " foo([x,y]) {}", // error
- "}"),
- ResolverErrorCode.CANNOT_OVERRIDE_METHOD_ORDER_NAMED_PARAMS);
- }
-
public void testTypeVariableInStatic() {
resolveAndTest(Joiner.on("\n").join(
"class Object {}",

Powered by Google App Engine
This is Rietveld 408576698