net: lengthen arbitrary SetDeadline timeout by a few orders of magnitude

The "someTimeout" constant in the net test is “just to test that
net.Conn implementations don't explode when their SetFooDeadline
methods are called”. It was set to 10 seconds, which is short enough
that it could actually matter on some platforms.

Since the point of the constant is just to make sure methods don't
explode, we should set it to be at least a couple of orders of
magnitude longer than the test: then it is guaranteed not to have any
unintended side-effects.

Fixes #50227

Change-Id: If97ae7bef5e7f16b336d09ccc37f5ea2ea7e70b3
Reviewed-on: https://go-review.googlesource.com/c/go/+/372796
Trust: Bryan Mills <bcmills@google.com>
Run-TryBot: Bryan Mills <bcmills@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>

diff --git a/src/net/conn_test.go b/src/net/conn_test.go
index 3403eddfd3ff7c8d41ecda191e73c95a711f73f6..d168dda08e19e4bb1df1073b8e15f5f768ae7d71 100644 (file)
--- a/src/net/conn_test.go
+++ b/src/net/conn_test.go
@@ -17,7 +17,7 @@ import (
 // someTimeout is used just to test that net.Conn implementations
 // don't explode when their SetFooDeadline methods are called.
 // It isn't actually used for testing timeouts.
-const someTimeout = 10 * time.Second
+const someTimeout = 1 * time.Hour
 
 func TestConnAndListener(t *testing.T) {
        for i, network := range []string{"tcp", "unix", "unixpacket"} {