support destructing on assignment

fixes #4870

lib/Parser.js

@@ -993,8 +993,9 @@ class Parser extends Tapable {
 			}
 		} else {
 			this.walkExpression(expression.right);
-			this.scope.renames["$" + expression.left.name] = undefined;
-			this.walkExpression(expression.left);
+			this.enterPattern(expression.left, (name, decl) => {
+				this.scope.renames["$" + name] = undefined;
+			});
 		}
 	}
 

test/cases/parsing/issue-4870/file.js

@@ -0,0 +1 @@
+export var test = "test";

test/cases/parsing/issue-4870/index.js

@@ -0,0 +1,13 @@
+import { test } from "./file";
+
+it("should allow import in array destructing", function() {
+	var other;
+	[other = test] = [];
+	other.should.be.eql("test");
+});
+
+it("should allow import in object destructing", function() {
+	var other;
+	({other = test} = {});
+	other.should.be.eql("test");
+});

test/cases/parsing/issue-4870/test.filter.js

@@ -0,0 +1,6 @@
+var supportsIteratorDestructuring = require("../../../helpers/supportsIteratorDestructuring");
+var supportsObjectDestructuring = require("../../../helpers/supportsObjectDestructuring");
+
+module.exports = function(config) {
+	return !config.minimize && supportsObjectDestructuring() && supportsIteratorDestructuring();
+};