Add unit tests for HTTPRoute

Signed-off-by: Sheng Lin <shelin@nvidia.com>

Author: shengnuo

api/apps/v1alpha1/nimservice_types_test.go

@@ -21,6 +21,7 @@ import (
 	"testing"
 
 	corev1 "k8s.io/api/core/v1"
+	gatewayv1 "sigs.k8s.io/gateway-api/apis/v1"
 )
 
 // TestGetVolumes tests the GetVolumes function.
@@ -113,5 +114,91 @@ func TestGetVolumes(t *testing.T) {
 			}
 		})
 	}
+}
+
+func TestHTTpRoute(t *testing.T) {
+	enabled := true
+	prefixTypeMatch := gatewayv1.PathMatchPathPrefix
+	root := "/"
+	var port gatewayv1.PortNumber = DefaultAPIPort
 
+	tests := []struct {
+		name          string
+		nimSpec       *HTTPRoute
+		desiredGWSpec gatewayv1.HTTPRouteSpec
+	}{
+		{
+			name: "should return empty gatewayv1.HTTPRouteSpec if HTTPRouteSpec is nil",
+			nimSpec: &HTTPRoute{
+				Enabled: &enabled,
+				Spec:    nil,
+			},
+			desiredGWSpec: gatewayv1.HTTPRouteSpec{},
+		},
+		{
+			name: "should correctly translate to gatewayv1.HTTPRouteSpec",
+			nimSpec: &HTTPRoute{
+				Enabled: &enabled,
+				Spec: &HTTPRouteSpec{
+					CommonRouteSpec: gatewayv1.CommonRouteSpec{
+						ParentRefs: []gatewayv1.ParentReference{
+							{
+								Name: "istio-gateway",
+							},
+						},
+					},
+					Host: "foobar.nim",
+					Paths: []HTTPPathMatch{
+						{
+							Type:  &prefixTypeMatch,
+							Value: &root,
+						},
+					},
+				},
+			},
+			desiredGWSpec: gatewayv1.HTTPRouteSpec{
+				CommonRouteSpec: gatewayv1.CommonRouteSpec{
+					ParentRefs: []gatewayv1.ParentReference{
+						{
+							Name: "istio-gateway",
+						},
+					},
+				},
+				Hostnames: []gatewayv1.Hostname{
+					"foobar.nim",
+				},
+				Rules: []gatewayv1.HTTPRouteRule{
+					{
+						Matches: []gatewayv1.HTTPRouteMatch{
+							{
+								Path: &gatewayv1.HTTPPathMatch{
+									Type:  &prefixTypeMatch,
+									Value: &root,
+								},
+							},
+						},
+						BackendRefs: []gatewayv1.HTTPBackendRef{
+							{
+								BackendRef: gatewayv1.BackendRef{
+									BackendObjectReference: gatewayv1.BackendObjectReference{
+										Name: "test",
+										Port: &port,
+									},
+								},
+							},
+						},
+					},
+				},
+			},
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			gwSpec := tt.nimSpec.GenerateGatewayHTTPRouteSpec("test")
+			if !reflect.DeepEqual(gwSpec, tt.desiredGWSpec) {
+				t.Errorf("GenerateGatewayHTTPRouteSpec() = %+v, want %+v", gwSpec, tt.desiredGWSpec)
+			}
+		})
+	}
 }

internal/render/render_test.go

@@ -33,6 +33,8 @@ import (
 
 	corev1 "k8s.io/api/core/v1"
 
+	gatewayv1 "sigs.k8s.io/gateway-api/apis/v1"
+
 	"github.com/NVIDIA/k8s-nim-operator/internal/render"
 	"github.com/NVIDIA/k8s-nim-operator/internal/render/types"
 )
@@ -500,6 +502,45 @@ var _ = Describe("K8s Resources Rendering", func() {
 			Expect(ingress.Namespace).To(Equal("default"))
 		})
 
+		It("should render HTTPRoute template correctly", func() {
+			params := types.HTTPRouteParams{
+				Enabled:   true,
+				Name:      "test-httproute",
+				Namespace: "default",
+				Spec: gatewayv1.HTTPRouteSpec{
+					CommonRouteSpec: gatewayv1.CommonRouteSpec{
+						ParentRefs: []gatewayv1.ParentReference{},
+					},
+					Hostnames: []gatewayv1.Hostname{
+						"host.name",
+					},
+					Rules: []gatewayv1.HTTPRouteRule{
+						{
+							Matches: []gatewayv1.HTTPRouteMatch{
+								{
+									Path: &gatewayv1.HTTPPathMatch{},
+								},
+							},
+							BackendRefs: []gatewayv1.HTTPBackendRef{
+								{
+									BackendRef: gatewayv1.BackendRef{
+										BackendObjectReference: gatewayv1.BackendObjectReference{
+											Name: "foobar-service",
+										},
+									},
+								},
+							},
+						},
+					},
+				},
+			}
+			r := render.NewRenderer(templatesDir)
+			httpRoute, err := r.HTTPRoute(&params)
+			Expect(err).NotTo(HaveOccurred())
+			Expect(httpRoute.Name).To(Equal("test-httproute"))
+			Expect(httpRoute.Namespace).To(Equal("default"))
+		})
+
 		It("should render HPA template correctly", func() {
 			minRep := int32(1)
 			params := types.HPAParams{