Remove extra bundles argo used to try to validate regretion in favor of cover scenarios with mocks and unit tests

Author: camilamacedo86

Makefile

@@ -167,15 +167,9 @@ generate: $(CONTROLLER_GEN) #EXHELP Generate code containing DeepCopy, DeepCopyI
 	$(CONTROLLER_GEN) --load-build-tags=$(GO_BUILD_TAGS) object:headerFile="hack/boilerplate.go.txt" paths="./..."
 
 .PHONY: verify
-verify: k8s-pin kind-verify-versions fmt generate manifests crd-ref-docs generate-test-data #HELP Verify all generated code is up-to-date. Runs k8s-pin instead of just tidy.
+verify: k8s-pin kind-verify-versions fmt generate manifests crd-ref-docs #HELP Verify all generated code is up-to-date. Runs k8s-pin instead of just tidy.
 	git diff --exit-code
 
-# Renders registry+v1 bundles in test/convert
-# Used by CI in verify to catch regressions in the registry+v1 -> plain conversion code
-.PHONY: generate-test-data
-generate-test-data:
-	go run test/convert/generate-manifests.go
-
 .PHONY: fix-lint
 fix-lint: $(GOLANGCI_LINT) #EXHELP Fix lint issues
 	$(GOLANGCI_LINT) run --fix --build-tags $(GO_BUILD_TAGS) $(GOLANGCI_LINT_ARGS)

go.mod

@@ -30,6 +30,7 @@ require (
 	golang.org/x/sync v0.16.0
 	golang.org/x/tools v0.35.0
 	gopkg.in/yaml.v2 v2.4.0
+	gotest.tools/v3 v3.5.2
 	helm.sh/helm/v3 v3.18.4
 	k8s.io/api v0.33.2
 	k8s.io/apiextensions-apiserver v0.33.2

internal/operator-controller/rukpak/render/render_test.go

@@ -7,8 +7,11 @@ import (
 	"testing"
 
 	"github.com/stretchr/testify/require"
+	"gotest.tools/v3/assert"
 	appsv1 "k8s.io/api/apps/v1"
 	corev1 "k8s.io/api/core/v1"
+	"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
+	"k8s.io/apimachinery/pkg/runtime/schema"
 	"sigs.k8s.io/controller-runtime/pkg/client"
 
 	"github.com/operator-framework/api/pkg/operators/v1alpha1"
@@ -267,3 +270,162 @@ func Test_BundleValidatorCallsAllValidationFnsInOrder(t *testing.T) {
 	require.NoError(t, val.Validate(nil))
 	require.Equal(t, "hi", actual)
 }
+func Test_Render_WithNilBundleValidator(t *testing.T) {
+	defer func() {
+		if r := recover(); r == nil {
+			t.Error("expected panic when BundleValidator is nil")
+		}
+	}()
+
+	renderer := render.BundleRenderer{}
+	_, _ = renderer.Render(bundle.RegistryV1{}, "ns")
+}
+
+func Test_Render_WithNoResourceGenerators(t *testing.T) {
+	renderer := render.BundleRenderer{
+		BundleValidator: render.BundleValidator{
+			func(_ *bundle.RegistryV1) []error { return nil },
+		},
+	}
+
+	_, err := renderer.Render(bundle.RegistryV1{
+		CSV: MakeCSV(WithInstallModeSupportFor(v1alpha1.InstallModeTypeAllNamespaces)),
+	}, "test", render.WithTargetNamespaces("test"), render.WithUniqueNameGenerator(render.DefaultUniqueNameGenerator))
+
+	require.NoError(t, err)
+}
+
+func Test_Render_ReturnsExactSameObjectReferences(t *testing.T) {
+	obj1 := fakeUnstructured("Service", "ns", "svc")
+	obj2 := fakeUnstructured("ConfigMap", "ns", "cfg")
+
+	mockGen := render.ResourceGenerator(func(_ *bundle.RegistryV1, _ render.Options) ([]client.Object, error) {
+		return []client.Object{obj1, obj2}, nil
+	})
+
+	renderer := render.BundleRenderer{
+		BundleValidator:    render.BundleValidator{func(_ *bundle.RegistryV1) []error { return nil }},
+		ResourceGenerators: []render.ResourceGenerator{mockGen},
+	}
+
+	result, err := renderer.Render(bundle.RegistryV1{
+		CSV: MakeCSV(WithInstallModeSupportFor(v1alpha1.InstallModeTypeAllNamespaces)),
+	}, "ns", render.WithTargetNamespaces("ns"), render.WithUniqueNameGenerator(render.DefaultUniqueNameGenerator))
+
+	require.NoError(t, err)
+	require.Same(t, obj1, result[0])
+	require.Same(t, obj2, result[1])
+}
+
+// Test_Render_ValidatesOutputForAllInstallModes ensures that the BundleRenderer
+// correctly generates and returns the exact list of client.Objects produced by the
+// ResourceGenerators, across all supported install modes (AllNamespaces, SingleNamespace, OwnNamespace).
+func Test_Render_ValidatesOutputForAllInstallModes(t *testing.T) {
+	testCases := []struct {
+		name             string
+		installNamespace string
+		watchNamespace   string
+		installModes     []v1alpha1.InstallMode
+		expectedNS       string
+	}{
+		{
+			name:             "AllNamespaces",
+			installNamespace: "mock-system",
+			watchNamespace:   "",
+			installModes: []v1alpha1.InstallMode{
+				{Type: v1alpha1.InstallModeTypeAllNamespaces, Supported: true},
+			},
+			expectedNS: "mock-system",
+		},
+		{
+			name:             "SingleNamespace",
+			installNamespace: "mock-system",
+			watchNamespace:   "mock-watch",
+			installModes: []v1alpha1.InstallMode{
+				{Type: v1alpha1.InstallModeTypeSingleNamespace, Supported: true},
+			},
+			expectedNS: "mock-watch",
+		},
+		{
+			name:             "OwnNamespace",
+			installNamespace: "mock-system",
+			watchNamespace:   "mock-system",
+			installModes: []v1alpha1.InstallMode{
+				{Type: v1alpha1.InstallModeTypeOwnNamespace, Supported: true},
+			},
+			expectedNS: "mock-system",
+		},
+	}
+
+	for _, tc := range testCases {
+		t.Run(tc.name, func(t *testing.T) {
+			// Given the mocks
+			expectedObjects := []client.Object{
+				fakeUnstructured("ClusterRole", "", "mock-operator.v0-role"),
+				fakeUnstructured("ClusterRoleBinding", "", "mock-operator.v0-binding"),
+				fakeUnstructured("ConfigMap", tc.expectedNS, "mock-operator-manager-config"),
+				fakeUnstructured("CustomResourceDefinition", "", "mocks.argoproj.io"),
+				fakeUnstructured("Deployment", tc.expectedNS, "mock-operator-controller-manager"),
+				fakeUnstructured("Service", tc.expectedNS, "mock-operator-controller-manager-metrics-service"),
+				fakeUnstructured("ServiceAccount", tc.expectedNS, "mock-operator-controller-manager"),
+			}
+
+			mockGen := render.ResourceGenerator(func(_ *bundle.RegistryV1, _ render.Options) ([]client.Object, error) {
+				return expectedObjects, nil
+			})
+
+			mockBundle := bundle.RegistryV1{
+				CSV: v1alpha1.ClusterServiceVersion{
+					Spec: v1alpha1.ClusterServiceVersionSpec{
+						InstallModes: tc.installModes,
+					},
+				},
+			}
+
+			renderer := render.BundleRenderer{
+				BundleValidator: render.BundleValidator{
+					func(_ *bundle.RegistryV1) []error { return nil },
+				},
+				ResourceGenerators: []render.ResourceGenerator{mockGen},
+			}
+
+			opts := []render.Option{
+				render.WithTargetNamespaces(tc.watchNamespace),
+				render.WithUniqueNameGenerator(render.DefaultUniqueNameGenerator),
+			}
+
+			// When we call:
+			objs, err := renderer.Render(mockBundle, tc.installNamespace, opts...)
+
+			// We expected have no error and ensure that the objects
+			// returned match with the resource generator's output.
+			require.NoError(t, err)
+			require.Len(t, objs, len(expectedObjects))
+
+			for i, obj := range objs {
+				exp := expectedObjects[i]
+				assert.Equal(t, exp.GetObjectKind().GroupVersionKind().Kind, obj.GetObjectKind().GroupVersionKind().Kind)
+				assert.Equal(t, exp.GetName(), obj.GetName(), "unexpected name at index %d", i)
+				assert.Equal(t, exp.GetNamespace(), obj.GetNamespace(), "unexpected namespace at index %d", i)
+			}
+		})
+	}
+}
+
+func fakeUnstructured(kind, namespace, name string) client.Object {
+	obj := &unstructured.Unstructured{}
+	group := ""
+	version := "v1"
+	if kind == "CustomResourceDefinition" {
+		group = "apiextensions.k8s.io"
+		version = "v1"
+	}
+	obj.SetGroupVersionKind(schema.GroupVersionKind{
+		Group:   group,
+		Version: version,
+		Kind:    kind,
+	})
+	obj.SetNamespace(namespace)
+	obj.SetName(name)
+	return obj
+}