Chromium Code Reviews| Index: test/checker/checker_test.dart |
| diff --git a/test/checker/checker_test.dart b/test/checker/checker_test.dart |
| index f435275541b24515f4d6087f2a2118b16d07525c..ac340bfc4a6b2643dbdcbe05cf933c31d0b264d3 100644 |
| --- a/test/checker/checker_test.dart |
| +++ b/test/checker/checker_test.dart |
| @@ -1372,31 +1372,6 @@ void main() { |
| inferFromOverrides: true); |
| testChecker( |
| - 'field/field override 2', |
| - { |
| - '/main.dart': ''' |
| - class A {} |
| - class B extends A {} |
| - class C extends B {} |
| - |
| - class Base { |
| - B f1; |
| - B f2; |
| - B f3; |
| - B f4; |
| - } |
| - |
| - class Child extends Base { |
| - /*severe:InvalidMethodOverride*/A f1; // invalid for getter |
| - /*severe:InvalidMethodOverride*/C f2; // invalid for setter |
| - /*severe:InvalidMethodOverride,severe:InvalidMethodOverride*/var f3; |
| - /*severe:InvalidMethodOverride,severe:InvalidMethodOverride*/dynamic f4; |
| - } |
| - ''' |
| - }, |
| - inferFromOverrides: false); |
| - |
| - testChecker( |
| 'getter/getter override', |
| { |
| '/main.dart': ''' |
| @@ -1422,31 +1397,6 @@ void main() { |
| inferFromOverrides: true); |
| testChecker( |
| - 'getter/getter override 2', |
| - { |
| - '/main.dart': ''' |
| - class A {} |
| - class B extends A {} |
| - class C extends B {} |
| - |
| - abstract class Base { |
| - B get f1; |
| - B get f2; |
| - B get f3; |
| - B get f4; |
| - } |
| - |
| - class Child extends Base { |
| - /*severe:InvalidMethodOverride*/A get f1 => null; |
| - C get f2 => null; |
| - /*severe:InvalidMethodOverride*/get f3 => null; |
| - /*severe:InvalidMethodOverride*/dynamic get f4 => null; |
| - } |
| - ''' |
| - }, |
| - inferFromOverrides: false); |
| - |
| - testChecker( |
| 'field/getter override', |
| { |
| '/main.dart': ''' |
| @@ -2025,14 +1975,14 @@ void main() { |
| implements I1 {} |
| class T2 extends Base implements I1 { |
| - /*severe:InvalidMethodOverride*/m(a) {} |
| + /*severe:InvalidMethodOverride,severe:InvalidMethodOverride*/m(a) {} |
|
Jennifer Messerly
2015/10/07 21:50:26
is this an example of a duplicate error message? I
vsm
2015/10/07 22:02:21
It's actually now 2 different override errors - on
|
| } |
| class T3 extends Object with /*severe:InvalidMethodOverride*/Base |
| implements I1 {} |
| class T4 extends Object with Base implements I1 { |
| - /*severe:InvalidMethodOverride*/m(a) {} |
| + /*severe:InvalidMethodOverride,severe:InvalidMethodOverride*/m(a) {} |
| } |
| ''' |
| }); |
| @@ -2479,22 +2429,6 @@ void main() { |
| }); |
| }); |
| - // This is a regression test: we used to report it twice because it was |
| - // the top super class and top super interface. |
| - // TODO(sigmund): maybe we generalize this and don't report again errors |
| - // when an interface is also a superclass. |
| - testChecker( |
| - 'no reporting of overrides with Object twice.', |
| - { |
| - '/main.dart': ''' |
| - class A {} |
| - class T1 implements A { |
| - /*severe:InvalidMethodOverride*/toString() {} |
| - } |
| - ''' |
| - }, |
| - inferFromOverrides: false); |
| - |
| testChecker('invalid runtime checks', { |
| '/main.dart': ''' |
| typedef int I2I(int x); |