Remove v1 handler testing
This commit is contained in:
parent
e1b62f7d42
commit
a70b677565
|
|
@ -91,22 +91,12 @@ func TestMakeHandler(t *testing.T) {
|
||||||
request: request{path: "/v2/", redirect: false},
|
request: request{path: "/v2/", redirect: false},
|
||||||
expected: expected{url: defaultUpstreamRegistry + "/v2/", statusCode: http.StatusPermanentRedirect},
|
expected: expected{url: defaultUpstreamRegistry + "/v2/", statusCode: http.StatusPermanentRedirect},
|
||||||
},
|
},
|
||||||
{
|
|
||||||
name: "/v1/ returns 308 without following redirect",
|
|
||||||
request: request{path: "/v1/", redirect: false},
|
|
||||||
expected: expected{url: defaultUpstreamRegistry + "/v1/", statusCode: http.StatusPermanentRedirect},
|
|
||||||
},
|
|
||||||
// when redirecting, results from k8s.gcr.io
|
// when redirecting, results from k8s.gcr.io
|
||||||
{
|
{
|
||||||
name: "/v2/ returns 401 from gcr, with following redirect",
|
name: "/v2/ returns 401 from gcr, with following redirect",
|
||||||
request: request{path: "/v2/", redirect: true},
|
request: request{path: "/v2/", redirect: true},
|
||||||
expected: expected{url: defaultUpstreamRegistry + "/v2/", statusCode: http.StatusUnauthorized},
|
expected: expected{url: defaultUpstreamRegistry + "/v2/", statusCode: http.StatusUnauthorized},
|
||||||
},
|
},
|
||||||
{
|
|
||||||
name: "/v1/ returns 404 from gcr, with following redirect",
|
|
||||||
request: request{path: "/v1/", redirect: true},
|
|
||||||
expected: expected{url: defaultUpstreamRegistry + "/v1/", statusCode: http.StatusNotFound},
|
|
||||||
},
|
|
||||||
},
|
},
|
||||||
tests: defaultTestFuncs(t),
|
tests: defaultTestFuncs(t),
|
||||||
}
|
}
|
||||||
|
|
@ -129,20 +119,3 @@ func TestDoV2(t *testing.T) {
|
||||||
}
|
}
|
||||||
suite.runTestSuite(t)
|
suite.runTestSuite(t)
|
||||||
}
|
}
|
||||||
|
|
||||||
func TestDoV1(t *testing.T) {
|
|
||||||
suite := &suite{
|
|
||||||
handler: http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
||||||
doV1(w, r, defaultUpstreamRegistry)
|
|
||||||
}),
|
|
||||||
scenarios: []scenario{
|
|
||||||
{
|
|
||||||
name: "v1 handler returns 308 without following redirect",
|
|
||||||
request: request{path: "/v1/", redirect: false},
|
|
||||||
expected: expected{url: defaultUpstreamRegistry + "/v1/", statusCode: http.StatusPermanentRedirect},
|
|
||||||
},
|
|
||||||
},
|
|
||||||
tests: defaultTestFuncs(t),
|
|
||||||
}
|
|
||||||
suite.runTestSuite(t)
|
|
||||||
}
|
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue