Reasonable and intuitive line counting (proposed fix for #468) (#469)

* Proposed fix for issue #468 * Revision of a test script responding to a fix of #468 * Avoid using tsutils v2.29.0, which breaks tests * Revert "Avoid using tsutils v2.29.0, which breaks tests" This reverts commit f1e963f. See also: #469 (comment)
microsoft · Jul 25, 2018 · f4c6aae · f4c6aae
1 parent ba235ad
commit f4c6aae
Show file tree

Hide file tree

Showing 2 changed files with 149 additions and 162 deletions.
diff --git a/src/maxFuncBodyLengthRule.ts b/src/maxFuncBodyLengthRule.ts
@@ -121,7 +121,7 @@ class MaxFunctionBodyLengthRuleWalker extends Lint.RuleWalker {
         const sourceFile: ts.SourceFile = this.getSourceFile();
         const startLine: number = sourceFile.getLineAndCharacterOfPosition(node.body.pos).line;
         const endLine: number = sourceFile.getLineAndCharacterOfPosition(node.body.end).line;
-        return endLine - startLine;
+        return endLine - startLine + 1;
     }
 
     private calcBodyCommentLength(node: ts.FunctionLikeDeclaration) {