Fix: Label empty body functions (fixes #92)

There are rules in eslint that expect FunctionExpression
nodes to contain a body. We should not produce an invalid
estree node if we use the estree node types.

Author: soda0289

lib/ast-converter.js

@@ -879,6 +879,10 @@ module.exports = function(ast, extra) {
                     }
                 }
 
+                if (!node.body) {
+                    functionDeclarationType = "TSEmptyBodyFunctionDeclaration";
+                }
+
                 /**
                  * Prefix FunctionDeclarations within TypeScript namespaces with "TS"
                  */
@@ -1074,6 +1078,7 @@ module.exports = function(ast, extra) {
                 // TODO: double-check that these positions are correct
                 var methodLoc = ast.getLineAndCharacterOfPosition(node.name.end + 1),
                     nodeIsMethod = (node.kind === SyntaxKind.MethodDeclaration),
+                    isEmptyBody = !(node.body),
                     method = {
                         type: "FunctionExpression",
                         id: null,
@@ -1137,6 +1142,11 @@ module.exports = function(ast, extra) {
                         }
                     }
 
+                    if (isEmptyBody) {
+                        methodDefinitionType = "TSEmptyBodyMethodDefinition";
+                        method.type = "TSEmptyBodyFunctionExpression";
+                    }
+
                     assign(result, {
                         type: methodDefinitionType,
                         key: convertChild(node.name),
@@ -1167,6 +1177,7 @@ module.exports = function(ast, extra) {
                 var constructorIsStatic = Boolean(node.flags & ts.NodeFlags.Static),
                     firstConstructorToken = constructorIsStatic ? ts.findNextToken(node.getFirstToken(), ast) : node.getFirstToken(),
                     constructorLoc = ast.getLineAndCharacterOfPosition(node.parameters.pos - 1),
+                    constructorIsEmptyBody = !(node.body),
                     constructor = {
                         type: "FunctionExpression",
                         id: null,
@@ -1230,8 +1241,14 @@ module.exports = function(ast, extra) {
                     };
                 }
 
+                var constructorMethodDefinitionType = "MethodDefinition";
+                if (constructorIsEmptyBody) {
+                    constructorMethodDefinitionType = "TSEmptyBodyMethodDefinition";
+                    constructor.type = "TSEmptyBodyFunctionExpression";
+                }
+
                 assign(result, {
-                    type: "MethodDefinition",
+                    type: constructorMethodDefinitionType,
                     key: constructorKey,
                     value: constructor,
                     computed: constructorIsComputed,