You cannot select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
tailscale/ipn/ipnlocal
Josh Bleecher Snyder b681edc572 ipn/ipnlocal: add failing test
Concurrent calls to LocalBackend.setWgengineStatus
could result in some of the status updates being dropped.
This was exacerbated by 92077ae78c,
which increases the probability of concurrent status updates,
causing test failures (tailscale/corp#2579).

It's going to take a bit of work to fix this test.
The ipnlocal state machine is difficult to reason about,
particularly in the face of concurrency.
We could fix the test trivially by throwing a new mutex around
setWgengineStatus to serialize calls to it,
but I'd like to at least try to do better than cosmetics.

In the meantime, commit the test.

Signed-off-by: Josh Bleecher Snyder <josh@tailscale.com>
3 years ago
..
dnsconfig_test.go ipn/ipnlocal: flesh out the dnsConfigForNetmap tests 3 years ago
local.go ipn/ipnlocal: flesh out the dnsConfigForNetmap tests 3 years ago
local_test.go all: close fake userspace engines when tests complete 3 years ago
loglines_test.go all: close fake userspace engines when tests complete 3 years ago
peerapi.go all: adapt to opaque netaddr types 3 years ago
peerapi_macios_ext.go all: simplify ts_macext build tags 3 years ago
peerapi_test.go all: update tests to use tstest.MemLogger 3 years ago
state_test.go ipn/ipnlocal: add failing test 3 years ago