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

Unified Diff: test/codegen/lib/html/js_typed_interop_default_arg_test.dart

Issue 1930043002: Add all dart:html tests from the sdk to test/codegen. (Closed) Base URL: git@github.com:dart-lang/dev_compiler.git@master
Patch Set: ptal Created 4 years, 8 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: test/codegen/lib/html/js_typed_interop_default_arg_test.dart
diff --git a/test/codegen/lib/html/js_typed_interop_default_arg_test.dart b/test/codegen/lib/html/js_typed_interop_default_arg_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..69a07569e29c01fa2c176a2823abbd9e56d57a88
--- /dev/null
+++ b/test/codegen/lib/html/js_typed_interop_default_arg_test.dart
@@ -0,0 +1,58 @@
+// Copyright (c) 2015, 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.
+
+@JS()
+library js_typed_interop_test;
+
+import 'dart:html';
+
+import 'package:expect/expect.dart' show NoInline;
+import 'package:js/js.dart';
+import 'package:unittest/unittest.dart';
+import 'package:unittest/html_config.dart';
+
+_injectJs() {
+ document.body.append(new ScriptElement()
+ ..type = 'text/javascript'
+ ..innerHtml = r"""
+ var Foo = {
+ get42: function(b) { return arguments.length >= 1 ? b : 42; },
+ get43: function(b) { return arguments.length >= 1 ? b : 43; }
+ };
+""");
+}
+
+@JS()
+class Foo {
+ // Note: it's invalid to provide a default value.
+ external static num get42([num b
+ = 3 /// default_value: compile-time error
+ ]);
+ external static num get43([num b]);
+}
+
+main() {
+ _injectJs();
+ useHtmlConfiguration();
+
+ test('call directly from dart', () {
+ expect(Foo.get42(2), 2);
+ expect(Foo.get42(), 42);
+ });
+
+ test('call tearoff from dart with arg', () {
+ var f = Foo.get42;
+ expect(f(2), 2); /// explicit_argument: ok
+ });
+
+ test('call tearoff from dart with default', () {
+ var f = Foo.get42;
+ // Note: today both SSA and CPS remove the extra argument on static calls,
+ // but they fail to do so on tearoffs.
+ expect(f(), 3); /// default_value: continued
+
+ f = Foo.get43;
+ expect(f(), 43);
+ });
+}

Powered by Google App Engine
This is Rietveld 408576698