Fix: Ensure exports applied to TSModuleDeclaration (#375)

Author: JamesHenry

lib/ast-node-types.js

@@ -113,6 +113,7 @@ module.exports = {
     TSEnumDeclaration: "TSEnumDeclaration",
     TSEnumMember: "TSEnumMember",
     TSExportAssignment: "TSExportAssignment",
+    TSExportKeyword: "TSExportKeyword",
     TSIndexSignature: "TSIndexSignature",
     TSInterfaceBody: "TSInterfaceBody",
     TSInterfaceDeclaration: "TSInterfaceDeclaration",

lib/convert.js

@@ -2096,10 +2096,13 @@ module.exports = function convert(config) {
                 type: AST_NODE_TYPES.TSModuleDeclaration,
                 id: convertChild(node.name)
             });
-            applyModifiersToResult(node.modifiers);
             if (node.body) {
                 result.body = convertChild(node.body);
             }
+            // apply modifiers first...
+            applyModifiersToResult(node.modifiers);
+            // ...then check for exports
+            result = nodeUtils.fixExports(node, result, ast);
             break;
         }
 

tests/ast-alignment/spec.js

@@ -461,6 +461,7 @@ const fixturePatternsToTest = [
     // "typescript/basics/export-type-function-declaration.src.ts", // babylon parse errors
     // "typescript/basics/interface-with-all-property-types.src.ts", // babylon parse errors
     // "typescript/basics/interface-with-construct-signature-with-parameter-accessibility.src.ts", // babylon parse errors
+    // "typescript/namespaces-and-modules/nested-internal-module.src.ts", // babylon parse errors
 
     /**
      * typescript-eslint-parser erroring, but babylon not.

tests/fixtures/typescript/namespaces-and-modules/nested-internal-module.src.ts

@@ -0,0 +1,12 @@
+module A {
+
+    export var x = 'hello world'
+    export class Point {
+        constructor(public x: number, public y: number) { }
+    }
+    export module B {
+        export interface Id {
+            name: string;
+        }
+    }
+}