test: restore len check for goimports (#1710)

codyoss · web-flow · commit 0c92cde1b55e · 2025-08-14T09:48:59.000-04:00
I must have forgot to revert removing this in #1662. Currently this check is not being enforced. Found reviewing: #1686
diff --git a/all_test.go b/all_test.go
@@ -165,6 +165,9 @@ func TestGoImports(t *testing.T) {
 	if err := cmd.Run(); err != nil {
 		t.Fatalf("goimports failed to run: %v\nOutput:\n%s", err, out.String())
 	}
+	if out.Len() > 0 {
+		t.Errorf("goimports found unformatted files:\n%s", out.String())
+	}
 }
 
 func TestGoModTidy(t *testing.T) {

Original file line number	Diff line number	Diff line change
`@@ -165,6 +165,9 @@ func TestGoImports(t *testing.T) {`
`165`	`165`	`if err := cmd.Run(); err != nil {`
`166`	`166`	`t.Fatalf("goimports failed to run: %v\nOutput:\n%s", err, out.String())`
`167`	`167`	`}`
	`168`	`+ if out.Len() > 0 {`
	`169`	`+ t.Errorf("goimports found unformatted files:\n%s", out.String())`
	`170`	`+ }`
`168`	`171`	`}`
`169`	`172`
`170`	`173`	`func TestGoModTidy(t *testing.T) {`