controller/test: adjust controller-test for new webhook

Theoretically, we just setup the webhook and can use the webhook in our
controller test. Somehow related to WebhookInstallOptions it should
work. Except, I don't get it to work.

Instead, there webhook is not running in this unit test. We however have
them in the "Webhook Suite", so there are unit tests about the webhook
there.

Instead, adjust the controller test for the (lack of) webhook changes.
Note that this doesn't actually check the webhook, since it's not
running in this setup. It rather tests, that the webhook is not running.
If we ever make the webhook working for dpuoperatorconfig_controller_test
test, we will need to adjust this. That is intentional.

internal/controller/dpuoperatorconfig_controller_test.go

@@ -39,7 +39,7 @@ import (
 
 var (
 	testNamespace              = "openshift-dpu-operator"
-	testDpuOperatorConfigName  = "default"
+	testDpuOperatorConfigName  = "dpu-operator-config"
 	testDpuOperatorConfigKind  = "DpuOperatorConfig"
 	testDpuDaemonName          = "dpu-daemon"
 	testNetworkFunctionNADDpu  = "dpunfcni-conf"
@@ -218,7 +218,7 @@ var _ = Describe("Main Controller", Ordered, func() {
 		Context("When DpuOperatorConfig CR exists with host mode", func() {
 			BeforeAll(func() {
 				ns := dpuOperatorNameSpace()
-				cr = dpuOperatorCR("operator-config", "host", ns)
+				cr = dpuOperatorCR(testDpuOperatorConfigName, "host", ns)
 				createNameSpace(mgr.GetClient(), ns)
 				createDpuOperatorCR(mgr.GetClient(), cr)
 			})
@@ -233,9 +233,21 @@ var _ = Describe("Main Controller", Ordered, func() {
 					return mgr.GetClient().Get(context.Background(), types.NamespacedName{Namespace: "default", Name: testNetworkFunctionNADHost}, nad)
 				}, testutils.TestAPITimeout*3, testutils.TestRetryInterval).ShouldNot(HaveOccurred())
 			})
+			It("only DpuOperatorConfig \"openshift-dpu-operator/dpu-operator-config\" is allowed", func() {
+				ns := dpuOperatorNameSpace()
+				client := mgr.GetClient()
+
+				cr2 := dpuOperatorCR("foo", "host", ns)
+				err2 := client.Create(context.Background(), cr2)
+				/* The validating webhook does not run in this setup. If it were, adding
+				 * this CR would be rejected. Instead, it passes. This indicates that the
+				 * webhook is not running. */
+				Expect(err2).NotTo(HaveOccurred())
+				deleteDpuOperatorCR(mgr.GetClient(), cr2)
+			})
 			AfterAll(func() {
 				ns := dpuOperatorNameSpace()
-				cr = dpuOperatorCR("operator-config", "host", ns)
+				cr = dpuOperatorCR(testDpuOperatorConfigName, "host", ns)
 				deleteDpuOperatorCR(mgr.GetClient(), cr)
 			})
 		})

internal/testutils/kindcluster.go

@@ -26,7 +26,7 @@ import (
 
 var (
 	testNamespace             = "openshift-dpu-operator"
-	testDpuOperatorConfigName = "default"
+	testDpuOperatorConfigName = "dpu-operator-config"
 	testDpuOperatorConfigKind = "DpuOperatorConfig"
 	testDpuDaemonName         = "dpu-daemon"
 	testClusterName           = "dpu-operator-test-cluster"