fix test

Author: AlinsRan

internal/controller/consumer_controller.go

@@ -19,7 +19,6 @@ package controller
 
 import (
 	"context"
-	"fmt"
 
 	"github.com/go-logr/logr"
 	corev1 "k8s.io/api/core/v1"
@@ -276,13 +275,9 @@ func (r *ConsumerReconciler) updateStatus(consumer *v1alpha1.Consumer, err error
 		NamespacedName: utils.NamespacedName(consumer),
 		Resource:       consumer.DeepCopy(),
 		Mutator: status.MutatorFunc(func(obj client.Object) client.Object {
-			t, ok := obj.(*v1alpha1.Consumer)
-			if !ok {
-				err := fmt.Errorf("unsupported object type %T", obj)
-				panic(err)
-			}
-			t.Status = consumer.Status
-			return t
+			cp := obj.(*v1alpha1.Consumer).DeepCopy()
+			cp.Status = consumer.Status
+			return cp
 		}),
 	})
 }

internal/controller/gatewayclass_controller.go

@@ -122,7 +122,6 @@ func (r *GatewayClassReconciler) Reconcile(ctx context.Context, req ctrl.Request
 			Resource:       gc.DeepCopy(),
 			Mutator: status.MutatorFunc(func(obj client.Object) client.Object {
 				cp := obj.(*gatewayv1.GatewayClass).DeepCopy()
-
 				cp.Status = gc.Status
 				return cp
 			}),

internal/controller/httproutepolicy.go

@@ -20,7 +20,6 @@ package controller
 import (
 	"cmp"
 	"context"
-	"fmt"
 	"slices"
 
 	networkingv1 "k8s.io/api/networking/v1"
@@ -92,13 +91,9 @@ func (r *HTTPRouteReconciler) processHTTPRoutePolicies(tctx *provider.TranslateC
 				NamespacedName: utils.NamespacedName(&policy),
 				Resource:       policy.DeepCopy(),
 				Mutator: status.MutatorFunc(func(obj client.Object) client.Object {
-					t, ok := obj.(*v1alpha1.HTTPRoutePolicy)
-					if !ok {
-						err := fmt.Errorf("unsupported object type %T", obj)
-						panic(err)
-					}
-					t.Status = policy.Status
-					return t
+					cp := obj.(*v1alpha1.HTTPRoutePolicy).DeepCopy()
+					cp.Status = policy.Status
+					return cp
 				}),
 			})
 		}
@@ -170,13 +165,9 @@ func (r *IngressReconciler) processHTTPRoutePolicies(tctx *provider.TranslateCon
 				NamespacedName: utils.NamespacedName(&policy),
 				Resource:       policy.DeepCopy(),
 				Mutator: status.MutatorFunc(func(obj client.Object) client.Object {
-					t, ok := obj.(*v1alpha1.HTTPRoutePolicy)
-					if !ok {
-						err := fmt.Errorf("unsupported object type %T", obj)
-						panic(err)
-					}
-					t.Status = policy.Status
-					return t
+					cp := obj.(*v1alpha1.HTTPRoutePolicy).DeepCopy()
+					cp.Status = policy.Status
+					return cp
 				}),
 			})
 		}
@@ -282,13 +273,9 @@ func updateDeleteAncestors(updater status.Updater, policy v1alpha1.HTTPRoutePoli
 			NamespacedName: utils.NamespacedName(&policy),
 			Resource:       policy.DeepCopy(),
 			Mutator: status.MutatorFunc(func(obj client.Object) client.Object {
-				t, ok := obj.(*v1alpha1.HTTPRoutePolicy)
-				if !ok {
-					err := fmt.Errorf("unsupported object type %T", obj)
-					panic(err)
-				}
-				t.Status = policy.Status
-				return t
+				cp := obj.(*v1alpha1.HTTPRoutePolicy).DeepCopy()
+				cp.Status = policy.Status
+				return cp
 			}),
 		})
 	}

internal/controller/ingress_controller.go

@@ -639,13 +639,9 @@ func (r *IngressReconciler) updateStatus(ctx context.Context, tctx *provider.Tra
 			NamespacedName: utils.NamespacedName(ingress),
 			Resource:       ingress.DeepCopy(),
 			Mutator: status.MutatorFunc(func(obj client.Object) client.Object {
-				t, ok := obj.(*networkingv1.Ingress)
-				if !ok {
-					err := fmt.Errorf("unsupported object type %T", obj)
-					panic(err)
-				}
-				t.Status = ingress.Status
-				return t
+				cp := obj.(*networkingv1.Ingress).DeepCopy()
+				cp.Status = ingress.Status
+				return cp
 			}),
 		})
 		return nil

internal/controller/policies.go

@@ -158,13 +158,9 @@ func ProcessBackendTrafficPolicy(
 				NamespacedName: utils.NamespacedName(policy),
 				Resource:       policy.DeepCopy(),
 				Mutator: status.MutatorFunc(func(obj client.Object) client.Object {
-					t, ok := obj.(*v1alpha1.BackendTrafficPolicy)
-					if !ok {
-						err := fmt.Errorf("unsupported object type %T", obj)
-						panic(err)
-					}
-					t.Status = policy.Status
-					return t
+					cp := obj.(*v1alpha1.BackendTrafficPolicy).DeepCopy()
+					cp.Status = policy.Status
+					return cp
 				}),
 			})
 		}