Skip to content

Commit be595a2

Browse files
k8s-ci-robotshaneutt
authored andcommitted
Merge pull request #1627 from sunjayBhatia/conformance-additional-path-matching-cases
conformance: Add additional path matching edge cases
1 parent c05f281 commit be595a2

File tree

1 file changed

+13
-0
lines changed

1 file changed

+13
-0
lines changed

conformance/tests/httproute-matching.go

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,19 @@ var HTTPRouteMatching = suite.ConformanceTest{
6464
Request: http.Request{Path: "/", Headers: map[string]string{"Version": "two"}},
6565
Backend: "infra-backend-v2",
6666
Namespace: ns,
67+
}, {
68+
Request: http.Request{Path: "/v2/"},
69+
Backend: "infra-backend-v2",
70+
Namespace: ns,
71+
}, {
72+
// Not a path segment prefix so should not match /v2.
73+
Request: http.Request{Path: "/v2example"},
74+
Backend: "infra-backend-v1",
75+
Namespace: ns,
76+
}, {
77+
Request: http.Request{Path: "/foo/v2/example"},
78+
Backend: "infra-backend-v1",
79+
Namespace: ns,
6780
}}
6881

6982
for i := range testCases {

0 commit comments

Comments
 (0)