Remove extra newlines when no-summary=output

gotestyourself · Oct 11, 2020 · 1530e18 · 1530e18
1 parent e3cd29c
commit 1530e18
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/testjson/summary.go b/testjson/summary.go
@@ -183,7 +183,9 @@ func writeTestCaseSummary(out io.Writer, execution executionSummary, conf testCa
 			}
 			fmt.Fprint(out, line)
 		}
-		fmt.Fprintln(out)
+		if _, isNoOutput := execution.(*noOutputSummary); !isNoOutput {
+			fmt.Fprintln(out)
+		}
 	}
 }
 

diff --git a/testjson/summary_test.go b/testjson/summary_test.go
@@ -186,17 +186,12 @@ DONE 13 tests, 1 skipped, 4 failures, 1 error in 34.123s
 === Skipped
 === SKIP: project/pkg/more TestOnlySometimes (0.00s)
 
-
 === Failed
 === FAIL: project/badmain  (0.00s)
-
 === FAIL: project/fs TestFileDo (1.41s)
-
 === FAIL: project/fs TestFileDoError (0.01s)
-
 === FAIL: project/pkg/more TestAlbatross (0.04s)
 
-
 === Errors
 pkg/file.go:99:12: missing ',' before newline