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

Unified Diff: tests/compiler/dart2js_native/native_class_fields_test.dart

Issue 2379173002: Add native_testing library to mock @Native classes (Closed)
Patch Set: xxx Created 4 years, 1 month 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/compiler/dart2js_native/native_class_fields_test.dart
diff --git a/tests/compiler/dart2js_native/native_class_fields_test.dart b/tests/compiler/dart2js_native/native_class_fields_test.dart
index 45a45bf3235b8d73c9f1b59daa62c28a779873ea..67380cb1e3fd644f855e591637c3e2489d64f2b7 100644
--- a/tests/compiler/dart2js_native/native_class_fields_test.dart
+++ b/tests/compiler/dart2js_native/native_class_fields_test.dart
@@ -2,8 +2,7 @@
// 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 "dart:_js_helper";
-import "package:expect/expect.dart";
+import "native_testing.dart";
// Verify that native fields on classes are not renamed by the minifier.
@Native("A")
@@ -25,7 +24,7 @@ function setter(x) {
function A(){
var a = Object.create(
- { constructor: { name: 'A'}},
+ { constructor: A},
{ myLongPropertyName: { get: getter,
set: setter,
configurable: false,
@@ -37,17 +36,24 @@ function A(){
}
makeA = function(){return new A;};
+self.nativeConstructor(A);
""";
A makeA() native ;
main() {
+ nativeTesting();
setup();
var a = makeA();
a.myLongPropertyName = 21;
int gotten = a.myLongPropertyName;
Expect.equals(11, gotten);
+ // Force interceptor dispatch.
+ confuse(a).myLongPropertyName = 99;
+ gotten = confuse(a).myLongPropertyName;
+ Expect.equals(22, gotten);
+
var a2 = makeA();
if (a2 is A) {
// Inside this 'if' the compiler knows that a2 is an A, so it is tempted

Powered by Google App Engine
This is Rietveld 408576698