From 7726d5d138bfbcee3c62e97a152bca6ca19e4da2 Mon Sep 17 00:00:00 2001 From: Xunzhuo Date: Wed, 8 Feb 2023 14:11:49 +0800 Subject: [PATCH] e2e: add rewrite hostname testcase (#182) Signed-off-by: bitliu --- .../tests/httproute-rewrite-host.go | 91 +++++++++++++++++++ .../tests/httproute-rewrite-host.yaml | 44 +++++++++ test/ingress/e2e_test.go | 1 + 3 files changed, 136 insertions(+) create mode 100644 test/ingress/conformance/tests/httproute-rewrite-host.go create mode 100644 test/ingress/conformance/tests/httproute-rewrite-host.yaml diff --git a/test/ingress/conformance/tests/httproute-rewrite-host.go b/test/ingress/conformance/tests/httproute-rewrite-host.go new file mode 100644 index 000000000..d7efca702 --- /dev/null +++ b/test/ingress/conformance/tests/httproute-rewrite-host.go @@ -0,0 +1,91 @@ +// Copyright (c) 2022 Alibaba Group Holding Ltd. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +package tests + +import ( + "testing" + + "github.com/alibaba/higress/test/ingress/conformance/utils/http" + "github.com/alibaba/higress/test/ingress/conformance/utils/suite" +) + +func init() { + HigressConformanceTests = append(HigressConformanceTests, HTTPRouteRewritePath) +} + +var HTTPRouteRewriteHost = suite.ConformanceTest{ + ShortName: "HTTPRouteRewriteHost", + Description: "A single Ingress in the higress-conformance-infra namespace uses the rewrite host", + Manifests: []string{"tests/httproute-rewrite-host.yaml"}, + Test: func(t *testing.T, suite *suite.ConformanceTestSuite) { + testcases := []http.Assertion{ + { + Request: http.AssertionRequest{ + ActualRequest: http.Request{ + Path: "/foo", + Host: "foo.com", + }, + ExpectedRequest: &http.ExpectedRequest{ + Request: http.Request{ + Host: "higress.io", + Path: "/foo", + }, + }, + }, + + Response: http.AssertionResponse{ + ExpectedResponse: http.Response{ + StatusCode: 200, + }, + }, + + Meta: http.AssertionMeta{ + TargetBackend: "infra-backend-v1", + TargetNamespace: "higress-conformance-infra", + }, + }, { + Request: http.AssertionRequest{ + ActualRequest: http.Request{ + Path: "/foo", + Host: "bar.com", + }, + ExpectedRequest: &http.ExpectedRequest{ + Request: http.Request{ + Host: "higress.io", + Path: "/foo", + }, + }, + }, + + Response: http.AssertionResponse{ + ExpectedResponse: http.Response{ + StatusCode: 200, + }, + }, + + Meta: http.AssertionMeta{ + TargetBackend: "infra-backend-v2", + TargetNamespace: "higress-conformance-infra", + }, + }, + } + + t.Run("Rewrite HTTPRoute Host", func(t *testing.T) { + for _, testcase := range testcases { + http.MakeRequestAndExpectEventuallyConsistentResponse(t, suite.RoundTripper, suite.TimeoutConfig, suite.GatewayAddress, testcase) + } + }) + }, +} diff --git a/test/ingress/conformance/tests/httproute-rewrite-host.yaml b/test/ingress/conformance/tests/httproute-rewrite-host.yaml new file mode 100644 index 000000000..3b99d621d --- /dev/null +++ b/test/ingress/conformance/tests/httproute-rewrite-host.yaml @@ -0,0 +1,44 @@ +# Copyright (c) 2022 Alibaba Group Holding Ltd. +# +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. + +apiVersion: networking.k8s.io/v1 +kind: Ingress +metadata: + annotations: + nginx.ingress.kubernetes.io/upstream-vhost: "higress.io" + name: httproute-rewrite-host + namespace: higress-conformance-infra +spec: + ingressClassName: higress + rules: + - host: "foo.com" + http: + paths: + - pathType: Prefix + path: "/foo" + backend: + service: + name: infra-backend-v1 + port: + number: 8080 + - host: "bar.com" + http: + paths: + - pathType: Prefix + path: "/foo" + backend: + service: + name: infra-backend-v2 + port: + number: 8080 diff --git a/test/ingress/e2e_test.go b/test/ingress/e2e_test.go index 9234cdf83..2ad7ac86e 100644 --- a/test/ingress/e2e_test.go +++ b/test/ingress/e2e_test.go @@ -53,6 +53,7 @@ func TestHigressConformanceTests(t *testing.T) { tests.HTTPRouteSimpleSameNamespace, tests.HTTPRouteHostNameSameNamespace, tests.HTTPRouteRewritePath, + tests.HTTPRouteRewriteHost, tests.HTTPRouteCanaryHeader, }