kubernetes
/
apiserver
mirror of https://github.com/kubernetes/apiserver.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix: enable contains rule from testifylint in module `k8s.io/apiserver` 

Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>

Kubernetes-commit: 3b92b9f84d62550a62890d8d3f6a57d96fc70ccc
This commit is contained in:
Matthieu MOREL 2024-09-22 11:57:24 +02:00 committed by Kubernetes Publisher
parent 326460d62e
commit c6ba9681f7
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
pkg/util/proxy/streamtunnel_test.go
View File

@ -170,7 +170,7 @@ func TestTunnelingHandler_BadHandshakeError(t *testing.T) {
// Handshake fails.
_, err := httpstream.Handshake(req, w, []string{constants.PortForwardV1Name})
require.Error(t, err, "handshake should have returned an error")
assert.True(t, strings.Contains(err.Error(), "unable to negotiate protocol"))
assert.ErrorContains(t, err, "unable to negotiate protocol")
w.WriteHeader(http.StatusForbidden)
}))
defer spdyServer.Close()
@ -279,12 +279,12 @@ func TestTunnelingResponseWriter_Hijack(t *testing.T) {
trw = &tunnelingResponseWriter{written: true}
_, _, err = trw.Hijack()
assert.Error(t, err, "Hijack after writing to response writer is error")
assert.True(t, strings.Contains(err.Error(), "connection has already been written to"))
assert.ErrorContains(t, err, "connection has already been written to")
// Hijacking after already hijacked is an error.
trw = &tunnelingResponseWriter{hijacked: true}
_, _, err = trw.Hijack()
assert.Error(t, err, "Hijack after writing to response writer is error")
assert.True(t, strings.Contains(err.Error(), "connection has already been hijacked"))
assert.ErrorContains(t, err, "connection has already been hijacked")
}
func TestTunnelingResponseWriter_DelegateResponseWriter(t *testing.T) {