review: strip trailing blank lines, modernize a test loop to slices.ContainsFunc

Author: CMGS

k8s/admission/admission_test.go

@@ -97,4 +97,3 @@ func TestServeRejectsMissingRequest(t *testing.T) {
 		t.Errorf("handler should not be invoked when Request is nil")
 	}
 }
-

k8s/tls_test.go

@@ -11,6 +11,7 @@ import (
 	"net"
 	"os"
 	"path/filepath"
+	"slices"
 	"testing"
 	"time"
 )
@@ -30,13 +31,7 @@ func TestGenerateSelfSignedCertIsParseable(t *testing.T) {
 	if parsed.Subject.CommonName != "testhost" {
 		t.Errorf("CN = %q", parsed.Subject.CommonName)
 	}
-	found := false
-	for _, ip := range parsed.IPAddresses {
-		if ip.String() == "10.0.0.1" {
-			found = true
-		}
-	}
-	if !found {
+	if !slices.ContainsFunc(parsed.IPAddresses, func(ip net.IP) bool { return ip.String() == "10.0.0.1" }) {
 		t.Errorf("SAN IP 10.0.0.1 missing from cert: %v", parsed.IPAddresses)
 	}
 }

k8s/unstructured_test.go

@@ -30,4 +30,3 @@ func TestDecodeUnstructured(t *testing.T) {
 		t.Fatalf("decoded name mismatch: got %q", got.Spec.Name)
 	}
 }
-