Chromium Code Reviews| Index: tests/lib/mirrors/abstract_class_test.dart |
| diff --git a/tests/lib/mirrors/abstract_class_test.dart b/tests/lib/mirrors/abstract_class_test.dart |
| index 230097e771f26e1de2cc70868d839bf03144eb90..00e108172dfd62cee2c395411510acc96021a2a8 100644 |
| --- a/tests/lib/mirrors/abstract_class_test.dart |
| +++ b/tests/lib/mirrors/abstract_class_test.dart |
| @@ -130,7 +130,7 @@ abstract class NamedMA = S with M; |
| class SubNamedMA extends NamedMA { |
| mixinFoo() {} |
| } |
| -class ConcreteNamedMA = S with M; |
| +class ConcreteNamedMA = S with M; /// 00: static type warning |
|
jwren
2014/04/07 22:34:41
Do you want this to be marked as abstract too?
rmacnak
2014/04/07 22:39:51
No, here we want to test the mirror system says th
|
| abstract class NamedMA2 = S with M2; |
| class SubNamedMA2 extends NamedMA2 { |
| @@ -159,8 +159,8 @@ testNamedMixinApplication() { |
| // Application is concrete. |
| { |
| // Mixin is abstract. |
| - Expect.isFalse(reflectClass(ConcreteNamedMA).isAbstract); |
| - Expect.isFalse(reflectClass(ConcreteNamedMA).superclass.isAbstract); |
| + Expect.isFalse(reflectClass(ConcreteNamedMA).isAbstract); /// 00: ok |
| + Expect.isFalse(reflectClass(ConcreteNamedMA).superclass.isAbstract); /// 00: ok |
| // Mixin is concrete. |
| Expect.isFalse(reflectClass(ConcreteNamedMA2).isAbstract); |