From: Bryan C. Mills <bcmills@google.com>
Date: Wed, 14 Sep 2022 18:21:23 +0000 (-0400)
Subject: net/http: deflake TestServerCancelsReadHeaderTimeoutWhenIdle
X-Git-Tag: go1.20rc1~1012
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=a9ca741d311772efb828c4b4f45e737783a89ca3;p=gostls13.git

net/http: deflake TestServerCancelsReadHeaderTimeoutWhenIdle

Return errors instead of calling t.Fatalf for timing-sensitive failure
modes. Scale the Sleep call to the timeout (so that it isn't too short
for the longer durations).

Fixes #54891.

Change-Id: I574e85e121becdda9ab8ee6bfd37c18a549d366d
Reviewed-on: https://go-review.googlesource.com/c/go/+/430955
Reviewed-by: Joel Sing <joel@sing.id.au>
Run-TryBot: Bryan Mills <bcmills@google.com>
Reviewed-by: Damien Neil <dneil@google.com>
Auto-Submit: Bryan Mills <bcmills@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
---

diff --git a/src/net/http/serve_test.go b/src/net/http/serve_test.go
index 21f23c652f..dd1ae55b8a 100644
--- a/src/net/http/serve_test.go
+++ b/src/net/http/serve_test.go
@@ -5872,23 +5872,23 @@ func TestServerCancelsReadHeaderTimeoutWhenIdle(t *testing.T) {
 		defer conn.Close()
 
 		if _, err := conn.Write([]byte("GET / HTTP/1.1\r\nHost: e.com\r\n\r\n")); err != nil {
-			t.Fatalf("writing first request failed: %v", err)
+			return fmt.Errorf("writing first request failed: %v", err)
 		}
 
 		if _, err := ReadResponse(br, nil); err != nil {
-			t.Fatalf("first response (before timeout) failed: %v", err)
+			return fmt.Errorf("first response (before timeout) failed: %v", err)
 		}
 
 		// wait for longer than the server's ReadHeaderTimeout, and then send
 		// another request
-		time.Sleep(timeout + 10*time.Millisecond)
+		time.Sleep(timeout * 3 / 2)
 
 		if _, err := conn.Write([]byte("GET / HTTP/1.1\r\nHost: e.com\r\n\r\n")); err != nil {
-			t.Fatalf("writing second request failed: %v", err)
+			return fmt.Errorf("writing second request failed: %v", err)
 		}
 
 		if _, err := ReadResponse(br, nil); err != nil {
-			t.Fatalf("second response (after timeout) failed: %v", err)
+			return fmt.Errorf("second response (after timeout) failed: %v", err)
 		}
 
 		return nil