Should call return()

packages/babel-helpers/src/helpers.ts

@@ -787,12 +787,13 @@ helpers.iterableToArrayLimit = helper("7.0.0-beta.0")`
     var _n = true;
     var _d = false;
     var _s, _e;
-    _i = _i.call(arr);
-    if (i === 0) {
-      if (typeof _i !== "object" && typeof _i !== "function") return;
-      return _arr;
-    }
     try {
+      _i = _i.call(arr);
+      if (i === 0) {
+        if (typeof _i !== "object" && typeof _i !== "function") return;
+        _n = false;
+        return _arr;
+      }
       for (; !(_n = (_s = _i.next()).done); _n = true) {
         _arr.push(_s.value);
         if (i && _arr.length === i) break;

packages/babel-plugin-transform-destructuring/test/fixtures/destructuring/empty-array-pattern/exec.js

@@ -1,17 +1,17 @@
 expect(function () {
-    var [] = null;
+  var [] = null;
 }).toThrow("Invalid attempt to destructure non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method.");
 
 expect(function () {
-    var [] = 42;
+  var [] = 42;
 }).toThrow("Invalid attempt to destructure non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method.");
 
 expect(function () {
-    var [] = {};
+  var [] = {};
 }).toThrow("Invalid attempt to destructure non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method.");
 
 expect(function () {
-    var [] = { [Symbol.iterator]: function() {} };
+  var [] = { [Symbol.iterator]: function() {} };
 }).toThrow("Invalid attempt to destructure non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method.");
 
 var [] = [];
@@ -21,3 +21,16 @@ var [] = (function*() { throw new Error("Should not throw"); })();
 var [] = { [Symbol.iterator]: function() { return {}; } }
 var [] = { [Symbol.iterator]: function() { return function() {}; } }
 var [] = { [Symbol.iterator]: async function*() {} }
+
+var returnCalled = false;
+var [] = {
+  [Symbol.iterator]: function() {
+    return {
+      return: function() {
+        returnCalled = true;
+        return {};
+      }
+    };
+  }
+};
+expect(returnCalled).toStrictEqual(true);

packages/babel-plugin-transform-destructuring/test/fixtures/destructuring/empty-array-pattern/input.js

@@ -10,3 +10,15 @@ var [] = (function*() { throw new Error("Should not throw"); })();
 var [] = { [Symbol.iterator]: function() { return {}; } }
 var [] = { [Symbol.iterator]: function() { return function() {}; } }
 var [] = { [Symbol.iterator]: async function*() {} }
+
+var returnCalled = false;
+var [] = {
+  [Symbol.iterator]: function() {
+    return {
+      return: function() {
+        returnCalled = true;
+        return {};
+      }
+    };
+  }
+};

packages/babel-plugin-transform-destructuring/test/fixtures/destructuring/empty-array-pattern/output.js

@@ -51,3 +51,15 @@ var _Symbol$iterator7 = {
     })
   },
   _Symbol$iterator8 = babelHelpers.slicedToArray(_Symbol$iterator7, 0);
+var returnCalled = false;
+var _Symbol$iterator9 = {
+    [Symbol.iterator]: function () {
+      return {
+        return: function () {
+          returnCalled = true;
+          return {};
+        }
+      };
+    }
+  },
+  _Symbol$iterator10 = babelHelpers.slicedToArray(_Symbol$iterator9, 0);