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

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

Issue 2383273002: Revert "Add native_testing library to mock @Native classes" (Closed)
Patch Set: Created 4 years, 3 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/compiler/dart2js_native/native_call_arity1_frog_test.dart
diff --git a/tests/compiler/dart2js_native/native_call_arity1_frog_test.dart b/tests/compiler/dart2js_native/native_call_arity1_frog_test.dart
index d435bdd11ea2fc49bdd4e4d1286471a3617c54d7..ed3d13153ac0513c5f2be3b65610a60035f9b308 100644
--- a/tests/compiler/dart2js_native/native_call_arity1_frog_test.dart
+++ b/tests/compiler/dart2js_native/native_call_arity1_frog_test.dart
@@ -2,7 +2,8 @@
// 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 "native_testing.dart";
+import "dart:_js_helper";
+import "package:expect/expect.dart";
// Test that native methods with unnamed* optional arguments are called with the
// number of arguments in the call site. This is necessary because native
@@ -37,14 +38,12 @@ B.prototype.foo = function () { return arguments.length; };
makeA = function(){return new A;};
makeB = function(){return new B;};
-
-self.nativeConstructor(A);
-self.nativeConstructor(B);
""";
testDynamicContext() {
- var a = confuse(makeA());
- var b = confuse(makeB());
+ var things = [makeA(), makeB()];
+ var a = things[0];
+ var b = things[1];
Expect.throws(() => a.foo());
Expect.equals(1, a.foo(10));
@@ -77,7 +76,6 @@ testStaticContext() {
}
main() {
- nativeTesting();
setup();
testDynamicContext();
testStaticContext();

Powered by Google App Engine
This is Rietveld 408576698