fix test

zhaohuabing · zhaohuabing · commit d5ea0a472db4 · 2026-02-23T09:39:42.000+08:00
Signed-off-by: Huabing (Robin) Zhao &lt;zhaohuabing@gmail.com&gt;
diff --git a/internal/gatewayapi/backendtrafficpolicy.go b/internal/gatewayapi/backendtrafficpolicy.go
@@ -295,12 +295,7 @@ func (t *Translator) processBackendTrafficPolicyForRoute(
 			// Do need a section name since the policy is targeting to a route.
 			ancestorRef := getAncestorRefForPolicy(mapKey.NamespacedName, p.SectionName)
 			ancestorRefs = append(ancestorRefs, &ancestorRef)
-
-			// Only process parentRefs that were handled by this translator
-			// (skip those referencing Gateways with different GatewayClasses)
-			if parentRefCtx := targetedRoute.GetRouteParentContext(p); parentRefCtx != nil {
-				parentRefCtxs = append(parentRefCtxs, parentRefCtx)
-			}
+			parentRefCtxs = append(parentRefCtxs, targetedRoute.GetRouteParentContext(p))
 		}
 	}
 
diff --git a/internal/gatewayapi/securitypolicy_test.go b/internal/gatewayapi/securitypolicy_test.go
@@ -884,6 +884,23 @@ func hasParentFalseCondition(p *egv1a1.SecurityPolicy) bool {
 	return false
 }
 
+func SetRouteParentContext(route RouteContext, parentRef gwapiv1.ParentReference) {
+	ctx := &RouteParentContext{ParentReference: &parentRef}
+	switch r := route.(type) {
+	case *HTTPRouteContext:
+		ctx.HTTPRoute = r.HTTPRoute
+	case *GRPCRouteContext:
+		ctx.GRPCRoute = r.GRPCRoute
+	case *TLSRouteContext:
+		ctx.TLSRoute = r.TLSRoute
+	case *TCPRouteContext:
+		ctx.TCPRoute = r.TCPRoute
+	case *UDPRouteContext:
+		ctx.UDPRoute = r.UDPRoute
+	}
+	route.SetRouteParentContext(parentRef, ctx)
+}
+
 // --- TCP branch: validateSecurityPolicyForTCP(...) returns err -> SetTranslationErrorForPolicyAncestors(...) + return
 func Test_SecurityPolicy_TCP_Invalid_setsStatus_and_returns(t *testing.T) {
 	tr := &Translator{GatewayControllerName: "gateway.envoyproxy.io/gatewayclass-controller"}
@@ -923,6 +940,7 @@ func Test_SecurityPolicy_TCP_Invalid_setsStatus_and_returns(t *testing.T) {
 			},
 		},
 	}
+	SetRouteParentContext(tcpRoute, tcpRoute.Spec.ParentRefs[0])
 
 	// Create the target reference
 	target := gwapiv1.LocalPolicyTargetReferenceWithSectionName{
@@ -1000,6 +1018,7 @@ func Test_SecurityPolicy_HTTP_Invalid_setsStatus_and_returns(t *testing.T) {
 			},
 		},
 	}
+	SetRouteParentContext(httpRoute, httpRoute.Spec.ParentRefs[0])
 
 	// Create the target reference
 	target := gwapiv1.LocalPolicyTargetReferenceWithSectionName{

Original file line number	Diff line number	Diff line change
`@@ -295,12 +295,7 @@ func (t *Translator) processBackendTrafficPolicyForRoute(`
`295`	`295`	`// Do need a section name since the policy is targeting to a route.`
`296`	`296`	`ancestorRef := getAncestorRefForPolicy(mapKey.NamespacedName, p.SectionName)`
`297`	`297`	`ancestorRefs = append(ancestorRefs, &ancestorRef)`
`298`		`-`
`299`		`- // Only process parentRefs that were handled by this translator`
`300`		`- // (skip those referencing Gateways with different GatewayClasses)`
`301`		`- if parentRefCtx := targetedRoute.GetRouteParentContext(p); parentRefCtx != nil {`
`302`		`- parentRefCtxs = append(parentRefCtxs, parentRefCtx)`
`303`		`- }`
	`298`	`+ parentRefCtxs = append(parentRefCtxs, targetedRoute.GetRouteParentContext(p))`
`304`	`299`	`}`
`305`	`300`	`}`
`306`	`301`