Fix TS 2.0 one-line rule bug (originally palantir#1429) (palantir#1746)

kc17 · Nov 19, 2016 · fbe5fd4 · fbe5fd4
1 parent be864a1
commit fbe5fd4
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/src/rules/oneLineRule.ts b/src/rules/oneLineRule.ts
@@ -196,7 +196,7 @@ class OneLineWalker extends Lint.RuleWalker {
     public visitModuleDeclaration(node: ts.ModuleDeclaration) {
         const nameNode = node.name;
         const body = node.body;
-        if (body.kind === ts.SyntaxKind.ModuleBlock) {
+        if (body != null && body.kind === ts.SyntaxKind.ModuleBlock) {
             const openBraceToken = body.getChildAt(0);
             this.handleOpeningBrace(nameNode, openBraceToken);
         }

diff --git a/test/rules/one-line/all/test.ts.lint b/test/rules/one-line/all/test.ts.lint
@@ -140,3 +140,6 @@ let geoConfig: {
     timeout: 5000,
     enableHighAccuracy: false
 };
+
+declare module "*";
+declare module "someLibrary/*";
diff --git a/test/rules/one-line/none/test.ts.lint b/test/rules/one-line/none/test.ts.lint
@@ -119,3 +119,6 @@ let geoConfig: {
     timeout: 5000,
     enableHighAccuracy: false
 };
+
+declare module "*";
+declare module "someLibrary/*";