fixup! Use Patch instead of Update for finalizer operations

tmshort · tmshort · commit 68b3412b9c7f · 2025-11-19T10:43:31.000-05:00
Signed-off-by: Todd Short &lt;tshort@redhat.com&gt;
diff --git a/internal/operator-controller/controllers/clusterextensionrevision_controller_test.go b/internal/operator-controller/controllers/clusterextensionrevision_controller_test.go
@@ -506,7 +506,7 @@ func Test_ClusterExtensionRevisionReconciler_Reconcile_Deletion(t *testing.T) {
 				// Kubernetes deletes the object immediately. The fake client simulates
 				// this behavior, so we expect a NotFound error.
 				require.Error(t, err)
-				require.True(t, client.IgnoreNotFound(err) == nil, "expected NotFound error, got: %v", err)
+				require.NoError(t, client.IgnoreNotFound(err), "expected NotFound error, got: %v", err)
 			},
 		},
 		{

Original file line number	Diff line number	Diff line change
`@@ -506,7 +506,7 @@ func Test_ClusterExtensionRevisionReconciler_Reconcile_Deletion(t *testing.T) {`
`506`	`506`	`// Kubernetes deletes the object immediately. The fake client simulates`
`507`	`507`	`// this behavior, so we expect a NotFound error.`
`508`	`508`	`require.Error(t, err)`
`509`		`- require.True(t, client.IgnoreNotFound(err) == nil, "expected NotFound error, got: %v", err)`
	`509`	`+ require.NoError(t, client.IgnoreNotFound(err), "expected NotFound error, got: %v", err)`
`510`	`510`	`},`
`511`	`511`	`},`
`512`	`512`	`{`