tree e599326d9cc72303bb96441ad9f4a9b0b28be68c
parent 2ffc4dc5cc611d5a10f10db088b6eb56341e254d
author Rob Findley <rfindley@google.com> 1690312895 -0400
committer Robert Findley <rfindley@google.com> 1690377590 +0000

gopls: fix tests that depend on log sequencing

As noted recently in gopls/internal/lsp/protocol/context.go, log
messages are delivered asynchronously to other LSP events. This means
that any test relying on sequencing of log messages with respect to
other operations is fundamentally incorrect.

Searching for such tests turned up several known flakes.

Fixes golang/go#61521
Fixes golang/go#51549
Fixes golang/go#51772

Change-Id: I4cb5031140e34e1c816b221a90d5ffa4055ad76c
Reviewed-on: https://go-review.googlesource.com/c/tools/+/513097
Run-TryBot: Robert Findley <rfindley@google.com>
Reviewed-by: Alan Donovan <adonovan@google.com>
gopls-CI: kokoro <noreply+kokoro@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>