Merge pull request #26025 from liggitt/auth-test-flake

Automatic merge from submit-queue Fix panic in auth test failure I got a spurious failure in the webhook integration test, but couldn't see the error returned because a panic was hit that assumed a body was always returned with the response

Merge pull request #26025 from liggitt/auth-test-flake
adf44ea1 · k8s-merge-robot · 4215fe57 · 22649a19 · adf44ea1
Commit adf44ea1 authored May 23, 2016 by k8s-merge-robot
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

auth_test.go test/integration/auth_test.go +1 -1

No files found.
--- a/test/integration/auth_test.go
+++ b/test/integration/auth_test.go
@@ -1312,11 +1312,11 @@ func TestWebhookTokenAuthenticator(t *testing.T) {

 		func() {
 			resp, err := transport.RoundTrip(req)
-			defer resp.Body.Close()
 			if err != nil {
 				t.Logf("case %v", r)
 				t.Fatalf("unexpected error: %v", err)
 			}
+			defer resp.Body.Close()
 			// Expect all of Alice's actions to at least get past authn/authz.
 			if resp.StatusCode == http.StatusUnauthorized || resp.StatusCode == http.StatusForbidden {
 				t.Logf("case %v", r)