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

Unified Diff: tests/language/const_map2_test.dart

Issue 179293005: Check that const-map keys don't override equals. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Update comment and add other test. Created 6 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: tests/language/const_map2_test.dart
diff --git a/tests/language/const_map2_test.dart b/tests/language/const_map2_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..69d7a198736a9e1ce767bc6002338949ce23a55b
--- /dev/null
+++ b/tests/language/const_map2_test.dart
@@ -0,0 +1,29 @@
+// Copyright (c) 2014, 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.
+
+import "package:expect/expect.dart";
+
+class A {
+ const factory A() = B;
+}
+
+class B implements A {
+ const B();
+
+ operator ==(o) => true; /// 00: compile-time error
+}
+
+confuse(x) {
+ if (new DateTime.now() == 42) return confuse(2);
+ return x;
+}
+
+main() {
+ // It is a compile-time error if the key overrides operator ==.
karlklose 2014/03/03 08:58:45 'key' -> 'key type'.
floitsch 2014/03/03 09:52:32 Done.
+ var m = const { const A(): 42 };
+ Expect.equals(42, m[confuse(const B())]);
+
+ m = const { "foo": 99, const A(): 499 };
+ Expect.equals(499, m[confuse(const B())]);
+}

Powered by Google App Engine
This is Rietveld 408576698