Fix #358: handle array destructuring assignment

src/ast-utils.js

@@ -60,6 +60,10 @@ const extractors = {
 
 	AssignmentPattern(names, node) {
 		extractors[node.left.type](names, node.left);
+	},
+
+	MemberExpression(names, node) {
+		extractors[node.property.type](names, node.property)
 	}
 };
 

test/samples/array-destructuring-assignment/main.js

@@ -0,0 +1,9 @@
+
+function shuffleArray(array) {
+  for (let i = array.length - 1; i > 0; i--) {
+    const j = Math.floor(Math.random() * (i + 1));
+    [array[i], array[j]] = [array[j], array[i]];
+  }
+}
+
+exports.shuffleArray = shuffleArray;

test/test.js

@@ -727,6 +727,38 @@ var esm = /*#__PURE__*/Object.freeze({
 var main = esm;
 
 module.exports = main;
+`
+			);
+		});
+
+		it('handles array destructuring assignment', async () => {
+			const bundle = await rollup({
+				input: 'samples/array-destructuring-assignment/main.js',
+				plugins: [commonjs({ sourceMap: true })]
+			});
+
+			const code = await getCodeFromBundle(bundle);
+			assert.equal(
+				code,
+				`'use strict';
+
+Object.defineProperty(exports, '__esModule', { value: true });
+
+function shuffleArray(array) {
+  for (let i = array.length - 1; i > 0; i--) {
+    const j = Math.floor(Math.random() * (i + 1));
+    [array[i], array[j]] = [array[j], array[i]];
+  }
+}
+
+var shuffleArray_1 = shuffleArray;
+
+var main = {
+	shuffleArray: shuffleArray_1
+};
+
+exports.default = main;
+exports.shuffleArray = shuffleArray_1;
 `
 			);
 		});