Enhance dart:js interop in a backwards compatible manner. List objects can now be passed back and f…

tests/html/js_array_test.dart

Index: tests/html/js_array_test.dart
diff --git a/tests/html/js_array_test.dart b/tests/html/js_array_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..df036fc4182abff92ba4e23d15bc42d822a332df
--- /dev/null
+++ b/tests/html/js_array_test.dart
@@ -0,0 +1,469 @@
+// 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.
+
+library jsArrayTest;
+
+import 'dart:async';
+import 'dart:collection';
+import 'dart:html';
+import 'dart:js';
+
+import 'package:unittest/unittest.dart';
+import 'package:unittest/html_config.dart';
+
+_injectJs() {
+  final script = new ScriptElement();
+  script.type = 'text/javascript';
+  script.innerHtml = r"""
+function callJsMethod(jsObj, jsMethodName, args) {
+  return jsObj[jsMethodName].apply(jsObj, args);
+}
+
+function jsEnumerateIndices(obj) {
+  var ret = [];
+  for(var i in obj) {
+    ret.push(i);
+  }
+  return ret;
+}
+
+function setValue(obj, index, value) {
+  return obj[index] = value;
+}
+
+function getValue(obj, index) {
+  return obj[index];
+}
+
+function checkIsArray(obj) {
+  this.dartArray = obj;
+  return Array.isArray(obj);
+}
+
+function concatValues(obj) {
+  return obj.concat("a", "b", ["c", "d"], 42);
+}
+
+function concatOntoArray(obj) {
+  return [1,2,3].concat(obj, "foo");
+}
+
+function repeatedConcatOntoArray(obj) {
+  return [1,2,3].concat(obj, obj);
+}
+
+function everyGreaterThanZero(obj) {
+  return obj.every(function(currentValue, index, array) {
+    return currentValue > 0;
+  });
+}
+
+function everyGreaterThanZeroCheckThisArg(obj) {
+  var j = 0;
+  return obj.every(function(currentValue, index, array) {
+    if (j != index) {
+      throw "Unxpected index";
+    }
+    j++;
+    if (array !== obj) {
+      throw "Array argument doesn't match obj";
+    }
+    return currentValue > 0;
+  });
+}
+
+function filterGreater42(obj) {
+  return obj.filter(function(currentValue, index, array) {
+    return currentValue > 42;
+  });
+}
+
+function forEachCollectResult(array, callback) {
+  var result = [];
+  array.forEach(function(currentValue) {
+    result.push(currentValue * 2);
+  });
+  return result;
+}
+
+function someEqual42(array, callback) {
+  return array.some(function(currentValue) {
+    return currentValue == 42;
+  });
+}
+
+function sortNumbersBackwards(array) {
+  return array.sort(function(a, b) {
+    return b - a;
+  });
+}
+
+function spliceDummyItems(array) {
+  return array.splice(1, 2, "quick" ,"brown", "fox");
+}
+
+function spliceTestStringArgs(array) {
+  return array.splice("1.2", "2.01", "quick" ,"brown", "fox");
+}
+
+function splicePastEnd(array) {
+  return array.splice(1, 5332, "quick" ,"brown", "fox");
+}
+
+function callJsToString(array) {
+  return array.toString();
+}
+
+function mapAddIndexToEachElement(array) {
+  return array.map(function(currentValue, index) {
+    return currentValue + index;
+  });
+}
+
+function reduceSumDoubledElements(array) {
+  return array.reduce(function(previousValue, currentValue) { return previousValue + currentValue*2; }, 0);
+}
+
+// TODO(jacobr): add a test that distinguishes reduce from reduceRight.
+function reduceRightSumDoubledElements(array) {
+  return array.reduceRight(function(previousValue, currentValue) { return previousValue + currentValue*2; }, 0);
+}
+
+function identical(o1, o2) {
+  return o1 === o2;
+}
+
+function getOwnPropertyDescriptorJson(array, property) {
+  return JSON.stringify(Object.getOwnPropertyDescriptor(array, property));
+}
+
+""";
+  document.body.append(script);
+}
+
+class Foo {
+}
+
+callJsMethod(List array, String methodName, List args) =>
+  context.callMethod("callJsMethod", [array, methodName, args]);
+
+callIndexOf(List array, value) => callJsMethod(array, "indexOf", [value]);
+callLastIndexOf(List array, value) => callJsMethod(array, "lastIndexOf", [value]);
+
+callPop(List array) => callJsMethod(array, "pop", []);
+callPush(List array, element) => callJsMethod(array, "push", [element]);
+callShift(List array) => callJsMethod(array, "shift", []);
+callReverse(List array) => callJsMethod(array, "reverse", []);
+
+main() {
+  _injectJs();
+  useHtmlConfiguration();
+
+  group('indexOf', () {
+    var div = new DivElement();
+    var list = [3, 42, "foo", 42, div];
+    test('found', () {
+      expect(callIndexOf(list, 3), equals(0));
+      expect(callIndexOf(list, 42), equals(1));
+      expect(callIndexOf(list, "foo"), equals(2));
+      expect(callIndexOf(list, div), equals(4));
+    });
+
+    test('missing', () {
+      expect(callIndexOf(list, 31), equals(-1));
+      expect(callIndexOf(list, "42"), equals(-1));
+      expect(callIndexOf(list, null), equals(-1));
+    });
+  });
+
+  group('join', () {
+    var list = [3, 42, "foo"];
+    var listWithDartClasses = [3, new Foo(), 42, "foo", new Object()];
+    test('default', () {
+      expect(callJsMethod(list, "join", []), equals("3,42,foo"));
+      expect(callJsMethod(listWithDartClasses, "join", []), equals("3,Instance of 'Foo',42,foo,Instance of 'Object'"));
+    });
+
+    test('custom separator', () {
+      expect(callJsMethod(list, "join", ["##"]), equals("3##42##foo"));
+    });
+  });
+
+  group('lastIndexOf', () {
+    var list = [3, 42, "foo", 42];
+    test('found', () {
+      expect(callLastIndexOf(list, 3), equals(0));
+      expect(callLastIndexOf(list, 42), equals(3));
+      expect(callLastIndexOf(list, "foo"), equals(2));
+    });
+
+    test('missing', () {
+      expect(callLastIndexOf(list, 31), equals(-1));
+      expect(callLastIndexOf(list, "42"), equals(-1));
+      expect(callLastIndexOf(list, null), equals(-1));
+    });
+  });
+
+  group('pop', () {
+    test('all', () {
+      var foo = new Foo();
+      var div = new DivElement();
+      var list = [3, 42, "foo", foo, div];
+      expect(callPop(list), equals(div));
+      expect(list.length, equals(4));
+      expect(callPop(list), equals(foo));
+      expect(list.length, equals(3));
+      expect(callPop(list), equals("foo"));
+      expect(list.length, equals(2));
+      expect(callPop(list), equals(42));
+      expect(list.length, equals(1));
+      expect(callPop(list), equals(3));
+      expect(list.length, equals(0));
+    });
+  });
+
+  group('push', () {
+    test('strings', () {
+      var list = [];
+      expect(callPush(list, "foo"), equals(1));
+      expect(callPush(list, "bar"), equals(2));
+      expect(callPush(list, "baz"), equals(3));
+      expect(list.toString(), equals("[foo, bar, baz]"));
+    });
+  });
+
+  group('shift', () {
+    test('all', () {
+      var foo = new Foo();
+      var div = new DivElement();
+      var list = [3, 42, "foo", foo, div];
+      expect(callShift(list), equals(3));
+      expect(list.length, equals(4));
+      expect(callShift(list), equals(42));
+      expect(list.length, equals(3));
+      expect(callShift(list), equals("foo"));
+      expect(list.length, equals(2));
+      expect(callShift(list), equals(foo));
+      expect(list.length, equals(1));
+      expect(callShift(list), equals(div));
+      expect(list.length, equals(0));
+    });
+  });
+
+  group('reverse', () {
+    test('simple', () {
+      var foo = new Foo();
+      var div = new DivElement();
+      var list = [3, 42, "foo"];
+      callReverse(list);
+      expect(list.toString(), equals("[foo, 42, 3]"));
+      list = [3, 42];
+      callReverse(list);
+      expect(list.toString(), equals("[42, 3]"));
+    });
+  });
+
+  group('slice', () {
+    test('copy', () {
+      var foo = new Foo();
+      var div = new DivElement();
+      var list = [3, 42, "foo", foo, div];
+      var copy = callJsMethod(list, "slice", []);
+      expect(identical(list, copy), equals(false));
+      expect(copy.length, equals(list.length));
+      for (var i = 0; i < list.length; i++) {
+        expect(list[i], equals(copy[i]));
+      }
+      copy.add("dummy");
+      expect(list.length + 1, equals(copy.length));
+    });
+    test('specify start', () {
+      var list = [3, 42, "foo"];
+      var copy = callJsMethod(list, "slice", [1]);
+      expect(copy.first, equals(42));
+    });
+    test('specify start and end', () {
+      var list = [3, 42, 92, "foo"];
+      var copy = callJsMethod(list, "slice", [1,3]);
+      expect(copy.first, equals(42));
+      expect(copy.last, equals(92));
+    });
+  });
+
+  group("js snippet tests", () {
+    test("enumerate indices", () {
+      var list = ["a", "b", "c", "d"];
+      var indices = context.callMethod('jsEnumerateIndices', [list]);
+      expect(indices.length, equals(4));
+      for (int i = 0; i < 4; i++) {
+        expect(indices[i], equals('$i'));
+      }
+    });
+
+    test("set element", () {
+      var list = ["a", "b", "c", "d"];
+      context.callMethod('setValue', [list, 0, 42]);
+      expect(list[0], equals(42));
+      context.callMethod('setValue', [list, 1, 84]);
+      expect(list[1], equals(84));
+      context.callMethod('setValue', [list, 6, 100]); // Off the end of the list.
+      expect(list.length, equals(7));
+      expect(list[4], equals(null));
+      expect(list[6], equals(100));
+
+      // These tests have to be commented out because we don't persist
+      // JS proxies for Dart objects like we could/should.
+      // context.callMethod('setValue', [list, -1, "foo"]); // Not a valid array index
+      // expect(context.callMethod('getValue', [list, -1]), equals("foo"));
+      // expect(context.callMethod('getValue', [list, "-1"]), equals("foo"));
+    });
+
+    test("get element", () {
+      var list = ["a", "b", "c", "d"];
+      expect(context.callMethod('getValue', [list, 0]), equals("a"));
+      expect(context.callMethod('getValue', [list, 1]), equals("b"));
+      expect(context.callMethod('getValue', [list, 6]), equals(null));
+      expect(context.callMethod('getValue', [list, -1]), equals(null));
+
+      expect(context.callMethod('getValue', [list, "0"]), equals("a"));
+      expect(context.callMethod('getValue', [list, "1"]), equals("b"));
+    });
+
+    test("is array", () {
+      var list  = ["a", "b"];
+      expect(context.callMethod("checkIsArray", [list]), isTrue);
+    });
+
+    test("property descriptors", () {
+      // This test matters to make behavior consistent with JS native arrays
+      // and to make devtools integration work well.
+      var list = ["a", "b"];
+      expect(context.callMethod("getOwnPropertyDescriptorJson", [list, 0]),
+        equals('{"value":"a","writable":true,"enumerable":true,"configurable":true}'));
+
+      expect(context.callMethod("getOwnPropertyDescriptorJson", [list, "length"]),
+        equals('{"value":2,"writable":true,"enumerable":false,"configurable":false}'));
+    });
+
+    test("concat js arrays", () {
+      var list = ["1", "2"];
+      var ret = context.callMethod("concatValues", [list]);
+      expect(list.length, equals(2));
+      expect(ret.length, equals(7));
+      expect(ret[0], equals("1"));
+      expect(ret[3], equals("b")); 
+      expect(ret[5], equals("d")); 
+      expect(ret[6], equals(42)); 
+    });
+
+    test("concat onto arrays", () {
+      // This test only passes if we have monkey patched the core Array object
+      // prototype to handle Dart Lists.
+      var list = ["a", "b"];
+      var ret = context.callMethod("concatOntoArray", [list]);
+      expect(list.length, equals(2));
+      expect(ret.length, equals(6));
+      expect(ret[0], equals(1));
+      expect(ret[3], equals("a")); 
+      expect(ret[4], equals("b")); 
+      expect(ret[5], equals("foo")); 
+    });
+
+    test("every greater than zero", () {
+      expect(context.callMethod("everyGreaterThanZero", [[1, 5]]), equals(true));
+      expect(context.callMethod("everyGreaterThanZeroCheckThisArg", [[1, 5]]), equals(true));
+      expect(context.callMethod("everyGreaterThanZero", [[1, 0]]), equals(false));
+    });
+
+    test("filter greater than 42", () {
+      expect(context.callMethod("filterGreater42", [[1, 5]]), equals([]));
+      expect(context.callMethod("filterGreater42", [[43, 5, 49]]), equals([43, 49]));
+      expect(context.callMethod("filterGreater42", [["43", "5", "49"]]), equals(["43", "49"]));
+    });
+
+    test("for each collect result", () {
+      expect(context.callMethod("forEachCollectResult", [[1, 5, 7]]), equals([2, 10, 14]));
+    });
+
+    test("some", () {
+      expect(context.callMethod("someEqual42", [[1, 5, 9]]), equals(false));
+      expect(context.callMethod("someEqual42", [[1, 42, 9]]), equals(true));
+    });
+
+    test("sort backwards", () {
+      var arr = [1, 5, 9];
+      var ret = context.callMethod("sortNumbersBackwards", [arr]);
+      expect(identical(arr, ret), equals(true));
+      expect(ret, equals([9, 5, 1]));
+    });
+
+
+    test("splice dummy items", () {
+      var list = [1,2,3,4];
+      var removed = context.callMethod("spliceDummyItems", [list]);
+      expect(removed.length, equals(2));
+      expect(removed[0], equals(2));
+      expect(removed[1], equals(3));
+      expect(list.first, equals(1));
+      expect(list[1], equals("quick"));
+      expect(list[2], equals("brown"));
+      expect(list[3], equals("fox"));
+      expect(list.last, equals(4));
+    });
+
+    test("splice string args", () {
+      var list = [1,2,3,4];
+      var removed = context.callMethod("spliceTestStringArgs", [list]);
+      expect(removed.length, equals(2));
+      expect(removed[0], equals(2));
+      expect(removed[1], equals(3));
+      expect(list.first, equals(1));
+      expect(list[1], equals("quick"));
+      expect(list[2], equals("brown"));
+      expect(list[3], equals("fox"));
+      expect(list.last, equals(4));
+    });
+
+    test("splice pastEndOfArray", () {
+      var list = [1,2,3,4];
+      var removed = context.callMethod("splicePastEnd", [list]);
+      expect(removed.length, equals(3));
+      expect(list.first, equals(1));
+      expect(list.length, equals(4));
+      expect(list[1], equals("quick"));
+      expect(list[2], equals("brown"));
+      expect(list[3], equals("fox"));
+    });
+
+    test("splice both bounds past end of array", () {
+      var list = [1];
+      var removed = context.callMethod("splicePastEnd", [list]);
+      expect(removed.length, equals(0));
+      expect(list.first, equals(1));
+      expect(list.length, equals(4));
+      expect(list[1], equals("quick"));
+      expect(list[2], equals("brown"));
+      expect(list[3], equals("fox"));
+    });
+  });
+
+  // This test group is disabled until we figure out an efficient way to
+  // distinguish between "array" Dart List types and non-array Dart list types.
+  /*
+  group('Non-array Lists', () {
+    test('opaque proxy', () {
+      // Dartium could easily support making LinkedList and all other classes
+      // implementing List behave like a JavaScript array but that would
+      // be challenging to implement in dart2js until browsers support ES6.
+      var list = ["a", "b", "c", "d"];
+      var listView = new UnmodifiableListView(list.getRange(1,3));
+      expect(listView is List, equals(true));
+      expect(listView.length, equals(2));
+      expect(context.callMethod("checkIsArray", [listView]), isFalse);
+      expect(context.callMethod("checkIsArray", [listView.toList()]), isTrue);
+      expect(context.callMethod("getOwnPropertyDescriptorJson", [listView, "length"]), equals("null"));
+    });
+  });
+  */
+}