fixup! Use Patch instead of Update for finalizer operations

tmshort · tmshort · commit 0ac87e3e4df2 · 2025-11-19T15:41:00.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
@@ -507,7 +507,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
`@@ -507,7 +507,7 @@ func Test_ClusterExtensionRevisionReconciler_Reconcile_Deletion(t *testing.T) {`
`507`	`507`	`// Kubernetes deletes the object immediately. The fake client simulates`
`508`	`508`	`// this behavior, so we expect a NotFound error.`
`509`	`509`	`require.Error(t, err)`
`510`		`- require.True(t, client.IgnoreNotFound(err) == nil, "expected NotFound error, got: %v", err)`
	`510`	`+ require.NoError(t, client.IgnoreNotFound(err), "expected NotFound error, got: %v", err)`
`511`	`511`	`},`
`512`	`512`	`},`
`513`	`513`	`{`