Allow empty export statements (#338)

Fixes #335
alangpierce · Nov 18, 2018 · de4c909 · de4c909
1 parent 9fdabab
commit de4c909
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 0 deletions.
diff --git a/src/CJSImportProcessor.ts b/src/CJSImportProcessor.ts
@@ -387,6 +387,11 @@ get: () => ${primaryImportName}[key]}); });`;
   private getNamedImports(index: number): {newIndex: number; namedImports: Array<NamedImport>} {
     const namedImports = [];
     while (true) {
+      if (this.tokens.matchesAtIndex(index, [tt.braceR])) {
+        index++;
+        break;
+      }
+
       // Flow type imports should just be ignored.
       let isTypeImport = false;
       if (

diff --git a/src/transformers/CJSImportTransformer.ts b/src/transformers/CJSImportTransformer.ts
@@ -540,6 +540,11 @@ export default class CJSImportTransformer extends Transformer {
 
     const exportStatements = [];
     while (true) {
+      if (this.tokens.matches1(tt.braceR)) {
+        this.tokens.removeToken();
+        break;
+      }
+
       const localName = this.tokens.identifierName();
       let exportedName;
       this.tokens.removeToken();

diff --git a/test/imports-test.ts b/test/imports-test.ts
@@ -204,6 +204,17 @@ describe("transform imports", () => {
     );
   });
 
+  it("allows an empty export statement", () => {
+    assertResult(
+      `
+      export {};
+    `,
+      `"use strict";${ESMODULE_PREFIX}
+      
+    `,
+    );
+  });
+
   it("allows trailing commas in exported names", () => {
     assertResult(
       `
@@ -314,6 +325,17 @@ return obj && obj.__esModule ? obj : { default: obj }; }
     );
   });
 
+  it("allows an import statement with no import bindings", () => {
+    assertResult(
+      `
+      import {} from 'moduleName';
+    `,
+      `"use strict";
+      
+    `,
+    );
+  });
+
   it("handles trailing commas in named imports", () => {
     assertResult(
       `