github653224
diff --git a/‎python-frontend/src/main/java/org/sonar/python/metrics/FileLinesVisitor.java‎
Lines changed: 3 additions & 3 deletions b/‎python-frontend/src/main/java/org/sonar/python/metrics/FileLinesVisitor.java‎
Lines changed: 3 additions & 3 deletions
diff --git a/‎python-frontend/src/test/java/org/sonar/python/FileLinesVisitorTest.java‎
Lines changed: 6 additions & 0 deletions b/‎python-frontend/src/test/java/org/sonar/python/FileLinesVisitorTest.java‎
Lines changed: 6 additions & 0 deletions
diff --git a/‎python-frontend/src/test/resources/metrics/empty.py‎ b/‎python-frontend/src/test/resources/metrics/empty.py‎
@@ -86,8 +86,10 @@ private void visitFile() {
   private void visitNode(SubscriptionContext ctx) {
     Tree tree = ctx.syntaxNode();
     if (tree.is(Tree.Kind.FILE_INPUT)) {
-      statements--;
       handleDocString(((FileInput) tree).docstring());
+    } else {
+      statements++;
+      executableLines.add(tree.firstToken().line());
     }
     if (tree.is(Tree.Kind.CLASSDEF)) {
       classDefs++;
@@ -96,8 +98,6 @@ private void visitNode(SubscriptionContext ctx) {
     if (tree.is(Tree.Kind.FUNCDEF)) {
       handleDocString(((FunctionDef) tree).docstring());
     }
-    statements++;
-    executableLines.add(tree.firstToken().line());
   }
 
   private void handleDocString(@Nullable StringLiteral docstring) {
 
@@ -60,4 +60,10 @@ public void executable_lines() {
     assertThat(visitor.getExecutableLines()).containsOnly(1, 2, 4, 7, 11, 13, 14, 15, 16, 18, 20, 21, 22, 23, 25, 27, 28, 29);
   }
 
+  @Test
+  public void empty_file() {
+    FileLinesVisitor visitor = new FileLinesVisitor();
+    TestPythonVisitorRunner.scanFile(new File(BASE_DIR, "empty.py"), visitor);
+    assertThat(visitor.getExecutableLines()).isEmpty();
+  }
 }
Original file line number	Diff line number	Diff line change
`@@ -86,8 +86,10 @@ private void visitFile() {`
`86`	`86`	`private void visitNode(SubscriptionContext ctx) {`
`87`	`87`	`Tree tree = ctx.syntaxNode();`
`88`	`88`	`if (tree.is(Tree.Kind.FILE_INPUT)) {`
`89`		`- statements--;`
`90`	`89`	`handleDocString(((FileInput) tree).docstring());`
	`90`	`+ } else {`
	`91`	`+ statements++;`
	`92`	`+ executableLines.add(tree.firstToken().line());`
`91`	`93`	`}`
`92`	`94`	`if (tree.is(Tree.Kind.CLASSDEF)) {`
`93`	`95`	`classDefs++;`
`@@ -96,8 +98,6 @@ private void visitNode(SubscriptionContext ctx) {`
`96`	`98`	`if (tree.is(Tree.Kind.FUNCDEF)) {`
`97`	`99`	`handleDocString(((FunctionDef) tree).docstring());`
`98`	`100`	`}`
`99`		`- statements++;`
`100`		`- executableLines.add(tree.firstToken().line());`
`101`	`101`	`}`
`102`	`102`
`103`	`103`	`private void handleDocString(@Nullable StringLiteral docstring) {`
Original file line number	Diff line number	Diff line change
`@@ -60,4 +60,10 @@ public void executable_lines() {`
`60`	`60`	`assertThat(visitor.getExecutableLines()).containsOnly(1, 2, 4, 7, 11, 13, 14, 15, 16, 18, 20, 21, 22, 23, 25, 27, 28, 29);`
`61`	`61`	`}`
`62`	`62`
	`63`	`+ @Test`
	`64`	`+ public void empty_file() {`
	`65`	`+ FileLinesVisitor visitor = new FileLinesVisitor();`
	`66`	`+ TestPythonVisitorRunner.scanFile(new File(BASE_DIR, "empty.py"), visitor);`
	`67`	`+ assertThat(visitor.getExecutableLines()).isEmpty();`
	`68`	`+ }`
`63`	`69`	`}`