tree bd7f737f7e7b71ea26ecbc61cdf75e20031f99cc
parent 4ddde0e984e9c8d85913f8a8be88ce66b1b0fbae
author Bryan C. Mills <bcmills@google.com> 1639595032 -0500
committer Bryan Mills <bcmills@google.com> 1639624154 +0000

netutil: streamline and simplify LimitListener tests

TestLimitListener had made a lot of assumptions about the kernel's
willingness to queue unaccepted connections, and relied on arbitrary
timeouts to shed load if the queue saturates.

This change eliminates the arbitrary timeouts, replacing them with
synchronization and cancellation and leaving only a couple of
arbitrary sleeps (that can be exceeded by arbitrary amounts without
causing the test to fail).

Fixes golang/go#22926

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