test: remove redundant test exectution (#1937)

codyoss · web-flow · commit e13be5c54730 · 2025-09-05T10:20:00.000-05:00
If someone runs `go test ./...` locally all tests were being run twice because of this nested test. Also CI already explicitly runs for coverage so this is not needed. Fixes: #734
diff --git a/all_test.go b/all_test.go
@@ -186,10 +186,6 @@ func TestGodocLint(t *testing.T) {
 		"./...")
 }
 
-func TestCoverage(t *testing.T) {
-	rungo(t, "test", "-coverprofile=coverage.out", "./internal/...", "./cmd/...")
-}
-
 func rungo(t *testing.T, args ...string) {
 	t.Helper()
 

Original file line number	Diff line number	Diff line change
`@@ -186,10 +186,6 @@ func TestGodocLint(t *testing.T) {`
`186`	`186`	`"./...")`
`187`	`187`	`}`
`188`	`188`
`189`		`-func TestCoverage(t *testing.T) {`
`190`		`- rungo(t, "test", "-coverprofile=coverage.out", "./internal/...", "./cmd/...")`
`191`		`-}`
`192`		`-`
`193`	`189`	`func rungo(t *testing.T, args ...string) {`
`194`	`190`	`t.Helper()`
`195`	`191`