internal/scan: fix double line spacing

Change-Id: I67d096196a4e54a12b4213c8011406a9ff65759e Reviewed-on: https://go-review.googlesource.com/c/vuln/+/508921 TryBot-Result: Gopher Robot <[email protected]> Reviewed-by: Julie Qiu <[email protected]> Reviewed-by: Julie Qiu <[email protected]> Run-TryBot: Ian Cottrell <[email protected]> Auto-Submit: Ian Cottrell <[email protected]>
golang · Jul 13, 2023 · f69de67 · f69de67
1 parent 05958cb
commit f69de67
Show file tree

Hide file tree

Showing 8 changed files with 1 addition and 15 deletions.
diff --git a/cmd/govulncheck/testdata/source_informational_text.ct b/cmd/govulncheck/testdata/source_informational_text.ct
@@ -5,7 +5,6 @@ Using go1.18 and [email protected] with vulnerabilit
 
 Scanning your code and P packages across M dependent modules for known vulnerabilities...
 
-
 === Informational ===
 
 Found 1 vulnerability in packages that you import, but there are no call

diff --git a/internal/scan/testdata/no-vulns.txt b/internal/scan/testdata/no-vulns.txt
@@ -1,6 +1,5 @@
 Using govulncheck with vulnerability data from .
 
-
 === Informational ===
 
 Found 1 vulnerability in packages that you import, but there are no call

diff --git a/internal/scan/testdata/platform-all.txt b/internal/scan/testdata/platform-all.txt
@@ -1,6 +1,5 @@
 Using govulncheck with vulnerability data from .
 
-
 === Informational ===
 
 Found 1 vulnerability in packages that you import, but there are no call

diff --git a/internal/scan/testdata/platform-one-arch-only.txt b/internal/scan/testdata/platform-one-arch-only.txt
@@ -1,6 +1,5 @@
 Using govulncheck with vulnerability data from .
 
-
 === Informational ===
 
 Found 1 vulnerability in packages that you import, but there are no call

diff --git a/internal/scan/testdata/platform-one-import.txt b/internal/scan/testdata/platform-one-import.txt
@@ -1,6 +1,5 @@
 Using govulncheck with vulnerability data from .
 
-
 === Informational ===
 
 Found 1 vulnerability in packages that you import, but there are no call

diff --git a/internal/scan/testdata/platform-two-imports.txt b/internal/scan/testdata/platform-two-imports.txt
@@ -1,6 +1,5 @@
 Using govulncheck with vulnerability data from .
 
-
 === Informational ===
 
 Found 1 vulnerability in packages that you import, but there are no call

diff --git a/internal/scan/testdata/platform-two-os-only.txt b/internal/scan/testdata/platform-two-os-only.txt
@@ -1,6 +1,5 @@
 Using govulncheck with vulnerability data from .
 
-
 === Informational ===
 
 Found 1 vulnerability in packages that you import, but there are no call

diff --git a/internal/scan/text.go b/internal/scan/text.go
@@ -130,9 +130,6 @@ func (h *TextHandler) byVulnerability(findings []*findingSummary) {
 	called := 0
 	for _, findings := range byVuln {
 		if isCalled(findings) {
-			if called > 0 {
-				h.print("\n")
-			}
 			h.vulnerability(called, findings)
 			called++
 		}
@@ -141,7 +138,6 @@ func (h *TextHandler) byVulnerability(findings []*findingSummary) {
 	if unCalled == 0 {
 		return
 	}
-	h.print("\n")
 	h.style(sectionStyle, "=== Informational ===\n")
 	h.print("\nFound ", unCalled)
 	h.print(choose(unCalled == 1, ` vulnerability`, ` vulnerabilities`))
@@ -151,9 +147,6 @@ func (h *TextHandler) byVulnerability(findings []*findingSummary) {
 	index := 0
 	for _, findings := range byVuln {
 		if !isCalled(findings) {
-			if index > 0 {
-				h.print("\n")
-			}
 			h.vulnerability(index, findings)
 			index++
 		}
@@ -226,6 +219,7 @@ func (h *TextHandler) vulnerability(index int, findings []*findingSummary) {
 		}
 		h.traces(module)
 	}
+	h.print("\n")
 }
 
 func (h *TextHandler) traces(traces []*findingSummary) {
@@ -258,7 +252,6 @@ func (h *TextHandler) traces(traces []*findingSummary) {
 
 func (h *TextHandler) summary(findings []*findingSummary) {
 	counters := counters(findings)
-	h.print("\n")
 	if counters.VulnerabilitiesCalled == 0 {
 		h.print("No vulnerabilities found.\n")
 		return