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

Unified Diff: tests/lib/mirrors/hierarchy_invariants_test.dart

Issue 27226003: Do not hide membership of implementation classes in dart:core. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 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: tests/lib/mirrors/hierarchy_invariants_test.dart
diff --git a/tests/lib/mirrors/hierarchy_invariants_test.dart b/tests/lib/mirrors/hierarchy_invariants_test.dart
index 0197dd3805b25452a1888e348f658cb1ad7d44b9..19d3152a0ac77d2c6e2ca1f3fe9cf8600e551742 100644
--- a/tests/lib/mirrors/hierarchy_invariants_test.dart
+++ b/tests/lib/mirrors/hierarchy_invariants_test.dart
@@ -8,12 +8,22 @@ import 'dart:mirrors';
import 'package:expect/expect.dart';
+isAnonymousMixinApplication(classMirror) {
ahe 2013/10/15 07:14:02 Should we have an accessor for this in the API?
+ return MirrorSystem.getName(classMirror.simpleName).contains(' with ');
+}
+
check(classMirror) {
if (classMirror is TypedefMirror) return;
Expect.isTrue(classMirror.simpleName is Symbol);
Expect.notEquals(null, classMirror.owner);
Expect.isTrue(classMirror.owner is LibraryMirror);
+ if (!isAnonymousMixinApplication(classMirror)) {
+ Expect.equals(classMirror.originalDeclaration,
+ classMirror.owner.classes[classMirror.simpleName]);
+ } else {
+ Expect.isNull(classMirror.owner.classes[classMirror.simpleName]);
+ }
Expect.isTrue(classMirror.superinterfaces is List);
if (classMirror.superclass == null) {
Expect.equals(reflectClass(Object), classMirror);
« runtime/lib/mirrors.cc ('K') | « tests/lib/mirrors/basic_types_in_dart_core_test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698