net/http: deflake TestClientConnReserveAndConsume

This test includes an assertion that a client conn's state hook
is called exactly once, but some of the test cases can result in
two events occurring: A request completes and a connection closes.

Change the assertion to just check that the hook is called
at least once.

Fixes #76480

Change-Id: Ie1438581b072b10623eb3d5fe443294a639c9853
Reviewed-on: https://go-review.googlesource.com/c/go/+/725601
Commit-Queue: Damien Neil <[email protected]>
Auto-Submit: Damien Neil <[email protected]>
Reviewed-by: Nicholas Husin <[email protected]>
Reviewed-by: Nicholas Husin <[email protected]>
TryBot-Bypass: Damien Neil <[email protected]>

Author: neild

src/net/http/clientconn_test.go

@@ -286,9 +286,10 @@ func TestClientConnReserveAndConsume(t *testing.T) {
 				synctest.Wait()
 
 				// State hook should be called, either to report the
-				// connection availability increasing or the connection closing.
-				if got, want := stateHookCalls, 1; got != want {
-					t.Errorf("connection state hook calls: %v, want %v", got, want)
+				// connection availability increasing or the connection closing,
+				// or both.
+				if stateHookCalls == 0 {
+					t.Errorf("connection state hook calls: %v, want >1", stateHookCalls)
 				}
 
 				if test.h1Closed && (mode == http1Mode || mode == https1Mode) {