fix(rest-spread): Do not require Symbol.iterator for strings

packages/babel-helpers/src/helpers.js

@@ -934,8 +934,9 @@ helpers.arrayWithHoles = helper("7.0.0-beta.0")`
 helpers.iterableToArray = helper("7.0.0-beta.0")`
   export default function _iterableToArray(iter) {
     if (
-      Symbol.iterator in Object(iter) ||
-      Object.prototype.toString.call(iter) === "[object Arguments]"
+      typeof iter === 'string'
+      || Object.prototype.toString.call(iter) === "[object Arguments]"
+      || Symbol.iterator in Object(iter)
     ) return Array.from(iter);
   }
 `;

packages/babel-plugin-proposal-object-rest-spread/test/fixtures/object-rest/parameters-exec/exec.js

@@ -0,0 +1,9 @@
+function sum(x, y, z) {
+  return x + y + z;
+}
+function test() {
+  var args = arguments;
+  return sum(...args);
+}
+
+expect(test(1, 2, 3)).toBe(6);

packages/babel-plugin-proposal-object-rest-spread/test/fixtures/object-rest/parameters-exec/options.json

@@ -0,0 +1,3 @@
+{
+  "plugins": ["proposal-object-rest-spread"]
+}

packages/babel-plugin-proposal-object-rest-spread/test/fixtures/object-spread/string-exec/exec.js

@@ -0,0 +1,3 @@
+expect([...'']).toHaveLength(0);
+expect([...'abc']).toHaveLength(3);
+expect([...'def']).toMatchObject(['d','e','f']);

packages/babel-runtime-corejs2/helpers/esm/iterableToArray.js

@@ -1,5 +1,5 @@
 import _Array$from from "../../core-js/array/from";
 import _isIterable from "../../core-js/is-iterable";
 export default function _iterableToArray(iter) {
-  if (_isIterable(Object(iter)) || Object.prototype.toString.call(iter) === "[object Arguments]") return _Array$from(iter);
+  if (typeof iter === 'string' || Object.prototype.toString.call(iter) === "[object Arguments]" || _isIterable(Object(iter))) return _Array$from(iter);
 }

packages/babel-runtime-corejs2/helpers/iterableToArray.js

@@ -3,7 +3,7 @@ var _Array$from = require("../core-js/array/from");
 var _isIterable = require("../core-js/is-iterable");
 
 function _iterableToArray(iter) {
-  if (_isIterable(Object(iter)) || Object.prototype.toString.call(iter) === "[object Arguments]") return _Array$from(iter);
+  if (typeof iter === 'string' || Object.prototype.toString.call(iter) === "[object Arguments]" || _isIterable(Object(iter))) return _Array$from(iter);
 }
 
 module.exports = _iterableToArray;

packages/babel-runtime/helpers/esm/iterableToArray.js

@@ -1,3 +1,3 @@
 export default function _iterableToArray(iter) {
-  if (Symbol.iterator in Object(iter) || Object.prototype.toString.call(iter) === "[object Arguments]") return Array.from(iter);
+  if (typeof iter === 'string' || Object.prototype.toString.call(iter) === "[object Arguments]" || Symbol.iterator in Object(iter)) return Array.from(iter);
 }

packages/babel-runtime/helpers/iterableToArray.js

@@ -1,5 +1,5 @@
 function _iterableToArray(iter) {
-  if (Symbol.iterator in Object(iter) || Object.prototype.toString.call(iter) === "[object Arguments]") return Array.from(iter);
+  if (typeof iter === 'string' || Object.prototype.toString.call(iter) === "[object Arguments]" || Symbol.iterator in Object(iter)) return Array.from(iter);
 }
 
 module.exports = _iterableToArray;