Robert Findley 090b14e850 internal/lsp/regtest: make TestResolveImportCycle robust to error order ... In JSON output, the Go command was only setting the Error field for one package involved in an import cycle. As a result, TestResolveImportCycle was dependent on the chosen package, causing flakes when the chosen package is not deterministic. Arguably this behavior should be fixed, both in the go command and in gopls, but for now make the test resilient to choice by asserting on any of the possible errors. Add a new AnyOf expectation to support this type of assertion and tweak the test output formatting. Also update the test to eagerly fail once the didOpen notification has been fully processed, so that we don't have to wait for the assertion timeout. Updates golang/go#52904 Change-Id: Ic209d8fdcb7308c041b287a8f122c47e96d29a96 Reviewed-on: https://go-review.googlesource.com/c/tools/+/406274 Reviewed-by: Michael Matloob <matloob@golang.org> Run-TryBot: Robert Findley <rfindley@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> gopls-CI: kokoro <noreply+kokoro@google.com>		2022-05-13 22:16:40 +00:00
..
builtin_test.go	internal/lsp: don't call PackagesForFile on builtin.go	2021-04-29 13:06:21 +00:00
diagnostics_test.go	internal/lsp/regtest: make TestResolveImportCycle robust to error order	2022-05-13 22:16:40 +00:00
undeclared_test.go	internal/lsp/cache: fix incorrect detection of useless code	2021-10-14 14:04:14 +00:00