1
0
mirror of https://github.com/golang/go synced 2024-09-28 17:14:29 -06:00

net/http: improve error reporting in TestCancelRequestWhenSharingConnection

Ensure that errors are reported if an HTTP request unexpectedly fails.

For #56587

Change-Id: I1adfb4fedc24d4177caf54e34c5033267e32caa6
Reviewed-on: https://go-review.googlesource.com/c/go/+/486075
Reviewed-by: Bryan Mills <bcmills@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Damien Neil <dneil@google.com>
Reviewed-by: Emmanuel Odeke <emmanuel@orijtech.com>
This commit is contained in:
Damien Neil 2023-04-18 14:53:02 -07:00 committed by Gopher Robot
parent a245aced26
commit 7d4d71e52f

View File

@ -6844,23 +6844,25 @@ func testCancelRequestWhenSharingConnection(t *testing.T, mode testMode) {
})
req, _ := NewRequestWithContext(ctx, "GET", ts.URL, nil)
res, err := client.Do(req)
reqerrc <- err
if err == nil {
if err != nil {
reqerrc <- err
} else {
res.Body.Close()
}
}()
// Wait for the first request to receive a response and return the
// connection to the idle pool.
r1c := <-reqc
close(r1c)
select {
case err := <-reqerrc:
t.Fatalf("request 1: got err %v, want nil", err)
case r1c := <-reqc:
close(r1c)
}
var idlec chan struct{}
select {
case err := <-reqerrc:
if err != nil {
t.Fatalf("request 1: got err %v, want nil", err)
}
idlec = <-putidlec
t.Fatalf("request 1: got err %v, want nil", err)
case idlec = <-putidlec:
}