Merge pull request #96920 from bl-ue/fix-typo-112720-01

Fix typo in kubectl/.../attach_test.go
This commit is contained in:
Kubernetes Prow Robot 2021-01-25 14:15:00 -08:00 committed by GitHub
commit 927042c2cf
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -200,7 +200,7 @@ func TestAttach(t *testing.T) {
name, version, podPath, fetchPodPath, attachPath, container string name, version, podPath, fetchPodPath, attachPath, container string
pod *corev1.Pod pod *corev1.Pod
remoteAttachErr bool remoteAttachErr bool
exepctedErr string expectedErr string
}{ }{
{ {
name: "pod attach", name: "pod attach",
@ -220,7 +220,7 @@ func TestAttach(t *testing.T) {
pod: attachPod(), pod: attachPod(),
remoteAttachErr: true, remoteAttachErr: true,
container: "bar", container: "bar",
exepctedErr: "attach error", expectedErr: "attach error",
}, },
{ {
name: "container not found error", name: "container not found error",
@ -230,7 +230,7 @@ func TestAttach(t *testing.T) {
attachPath: "/api/" + version + "/namespaces/test/pods/foo/attach", attachPath: "/api/" + version + "/namespaces/test/pods/foo/attach",
pod: attachPod(), pod: attachPod(),
container: "foo", container: "foo",
exepctedErr: "cannot attach to the container: container not found (foo)", expectedErr: "cannot attach to the container: container not found (foo)",
}, },
} }
for _, test := range tests { for _, test := range tests {
@ -290,15 +290,15 @@ func TestAttach(t *testing.T) {
} }
err := options.Run() err := options.Run()
if test.exepctedErr != "" && err.Error() != test.exepctedErr { if test.expectedErr != "" && err.Error() != test.expectedErr {
t.Errorf("%s: Unexpected exec error: %v", test.name, err) t.Errorf("%s: Unexpected exec error: %v", test.name, err)
return return
} }
if test.exepctedErr == "" && err != nil { if test.expectedErr == "" && err != nil {
t.Errorf("%s: Unexpected error: %v", test.name, err) t.Errorf("%s: Unexpected error: %v", test.name, err)
return return
} }
if test.exepctedErr != "" { if test.expectedErr != "" {
return return
} }
if remoteAttach.url.Path != test.attachPath { if remoteAttach.url.Path != test.attachPath {