Check for duplicate named exports in exported rest elements/properties (#164)

src/parser/statement.js

@@ -947,6 +947,8 @@ pp.checkDeclaration = function(node) {
     }
   } else if (node.type === "ObjectProperty") {
     this.checkDeclaration(node.value);
+  } else if (node.type === "RestElement" || node.type === "RestProperty") {
+    this.checkDeclaration(node.argument);
   } else if (node.type === "Identifier") {
     this.checkDuplicateExports(node, node.name);
   }

test/fixtures/es2015/modules/duplicate-named-export-destructuring16/actual.js

@@ -0,0 +1,3 @@
+export const foo = 1;
+export const [bar, ...foo] = baz;
+

test/fixtures/es2015/modules/duplicate-named-export-destructuring16/options.json

@@ -0,0 +1,3 @@
+{
+  "throws": "`foo` has already been exported. Exported identifiers must be unique. (2:22)"
+}

test/fixtures/es2015/modules/duplicate-named-export-destructuring17/actual.js

@@ -0,0 +1,3 @@
+export const [foo, ...bar] = baz;
+export const bar = 1;
+

test/fixtures/es2015/modules/duplicate-named-export-destructuring17/options.json

@@ -0,0 +1,3 @@
+{
+  "throws": "`bar` has already been exported. Exported identifiers must be unique. (2:13)"
+}

test/fixtures/es2015/modules/duplicate-named-export-destructuring18/actual.js

@@ -0,0 +1,3 @@
+export const foo = 1;
+export const [bar, [baz, ...foo]] = qux;
+

test/fixtures/es2015/modules/duplicate-named-export-destructuring18/options.json

@@ -0,0 +1,3 @@
+{
+  "throws": "`foo` has already been exported. Exported identifiers must be unique. (2:28)"
+}

test/fixtures/es2015/modules/duplicate-named-export-destructuring19/actual.js

@@ -0,0 +1,3 @@
+export const foo = 1;
+export const { bar: [baz, ...foo] } = qux;
+

test/fixtures/es2015/modules/duplicate-named-export-destructuring19/options.json

@@ -0,0 +1,3 @@
+{
+  "throws": "`foo` has already been exported. Exported identifiers must be unique. (2:29)"
+}

test/fixtures/experimental/object-rest-spread/11/actual.js

@@ -0,0 +1,2 @@
+export const foo = 1;
+export const { bar, ...foo } = baz;

test/fixtures/experimental/object-rest-spread/11/options.json

@@ -0,0 +1,5 @@
+{
+  "sourceType": "module",
+  "throws": "`foo` has already been exported. Exported identifiers must be unique. (2:23)"
+}
+

test/fixtures/experimental/object-rest-spread/12/actual.js

@@ -0,0 +1,2 @@
+export const { foo, ...bar } = baz;
+export const bar = 1;

test/fixtures/experimental/object-rest-spread/12/options.json

@@ -0,0 +1,5 @@
+{
+  "sourceType": "module",
+  "throws": "`bar` has already been exported. Exported identifiers must be unique. (2:13)"
+}
+

test/fixtures/experimental/object-rest-spread/13/actual.js

@@ -0,0 +1,2 @@
+export const foo = 1;
+export const { bar: { baz, ...foo } } = qux;

test/fixtures/experimental/object-rest-spread/13/options.json

@@ -0,0 +1,5 @@
+{
+  "sourceType": "module",
+  "throws": "`foo` has already been exported. Exported identifiers must be unique. (2:30)"
+}
+

test/fixtures/experimental/object-rest-spread/14/actual.js

@@ -0,0 +1,2 @@
+export const foo = 1;
+export const [bar, { baz, ...foo }] = qux;

test/fixtures/experimental/object-rest-spread/14/options.json

@@ -0,0 +1,5 @@
+{
+  "sourceType": "module",
+  "throws": "`foo` has already been exported. Exported identifiers must be unique. (2:29)"
+}
+

test/fixtures/experimental/object-rest-spread/15/actual.js

@@ -0,0 +1,2 @@
+export const foo = 1;
+export const [bar, [{ baz, ...foo }]] = qux;

test/fixtures/experimental/object-rest-spread/15/options.json

@@ -0,0 +1,5 @@
+{
+  "sourceType": "module",
+  "throws": "`foo` has already been exported. Exported identifiers must be unique. (2:30)"
+}
+