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

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

Issue 38543002: Align behavior of VM's ObjectMirror.setField with language semantics. (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/set_field_with_final_test.dart
diff --git a/tests/lib/mirrors/set_field_with_final_test.dart b/tests/lib/mirrors/set_field_with_final_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..63b07a57ace79190cd747ca6d57ac40c662e2741
--- /dev/null
+++ b/tests/lib/mirrors/set_field_with_final_test.dart
@@ -0,0 +1,38 @@
+// Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+library test.set_field_with_final;
+
+import 'dart:mirrors';
+import 'package:expect/expect.dart';
+
+class C {
+ final instanceField = 1;
+ get instanceGetter => 2;
+ static final staticFinal = 3;
+ static get staticGetter => 4;
+}
+
+final toplevelFinal = 5;
+get toplevelGetter => 6;
+
+main() {
+ InstanceMirror im = reflect(new C());
+ Expect.throws(() => im.setField(#instanceField, 7),
+ (e) => e is NoSuchMethodError);
+ Expect.throws(() => im.setField(#instanceGetter, 8),
+ (e) => e is NoSuchMethodError);
+
+ ClassMirror cm = im.type;
+ Expect.throws(() => cm.setField(#staticFinal, 9),
+ (e) => e is NoSuchMethodError);
+ Expect.throws(() => cm.setField(#staticGetter, 10),
+ (e) => e is NoSuchMethodError);
+
+ LibraryMirror lm = cm.owner;
+ Expect.throws(() => lm.setField(#toplevelFinal, 11),
+ (e) => e is NoSuchMethodError);
+ Expect.throws(() => lm.setField(#toplevelGetter, 12),
+ (e) => e is NoSuchMethodError);
+}
« runtime/lib/mirrors.cc ('K') | « tests/lib/mirrors/set_field_with_final_inheritance_test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698