fix: address CI test and lint failures

- Set spec.cluster.clusterRef.name on test CRs so they pass CRD
  validation (Topic/User require cluster or kafkaApiSpec)
- Remove trailing blank line in resource_controller_test.go that
  caused git diff --exit-code lint failure

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: david-yu

operator/cmd/finalizerremoval/finalizer_removal_test.go

@@ -69,8 +69,8 @@ func TestRemoveFinalizersForGVK(t *testing.T) {
 		obj.SetGroupVersionKind(gvk)
 		obj.SetName(name)
 		obj.SetNamespace(ns)
-		// Set required spec field so the API server accepts the object.
-		_ = unstructured.SetNestedMap(obj.Object, map[string]interface{}{}, "spec")
+		// CRDs require spec.cluster.clusterRef to pass validation.
+		_ = unstructured.SetNestedField(obj.Object, "dummy", "spec", "cluster", "clusterRef", "name")
 		require.NoError(t, k8sClient.Create(ctx, obj))
 		// Add finalizer via patch after creation.
 		patch := client.MergeFrom(obj.DeepCopy())
@@ -116,7 +116,7 @@ func TestRemoveFinalizersForGVK(t *testing.T) {
 		obj.SetGroupVersionKind(gvk)
 		obj.SetName("no-finalizer-topic")
 		obj.SetNamespace("default")
-		_ = unstructured.SetNestedMap(obj.Object, map[string]interface{}{}, "spec")
+		_ = unstructured.SetNestedField(obj.Object, "dummy", "spec", "cluster", "clusterRef", "name")
 		require.NoError(t, k8sClient.Create(ctx, obj))
 
 		require.NoError(t, removeFinalizersForGVK(ctx, k8sClient, gvk))

operator/internal/controller/redpanda/resource_controller_test.go

@@ -607,7 +607,6 @@ func TestResourceController(t *testing.T) { // nolint:funlen // These tests have
 
 	require.Equal(t, int32(size/2), reconciler.deletes.Load())
 	require.Equal(t, int32(size), reconciler.syncs.Load())
-
 }
 
 func TestIsNetworkDialError(t *testing.T) {