[tools] gopls/test/completion: construct index explicitly

[Peter Weinberger (Gerrit)]

Peter Weinberger submitted the change

Change information

Commit message:

gopls/test/completion: construct index explicitly

TestIssue61208 is flaky because it relies on a background go routine
finishing before the index is needed.

Instead of hoping, explicity ensure the index exists.

Fixes: golang/go#80551

Change-Id: I73512131277fd8f69e6fe9d80947288f9c2c3262

Reviewed-on: https://go-review.googlesource.com/c/tools/+/805040

LUCI-TryBot-Result: golang...@luci-project-accounts.iam.gserviceaccount.com <golang...@luci-project-accounts.iam.gserviceaccount.com>

Reviewed-by: Madeline Kalil <mka...@google.com>

Files:

• M gopls/internal/test/integration/completion/completion_test.go

Change size: XS

Delta: 1 file changed, 1 insertion(+), 0 deletions(-)

Branch: refs/heads/master

Submit Requirements:

• Code-Review: +2 by Madeline Kalil
• TryBots-Pass: LUCI-TryBot-Result+1 by golang...@luci-project-accounts.iam.gserviceaccount.com

Open in Gerrit

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

Gerrit-MessageType: merged

Gerrit-Project: tools

Gerrit-Branch: master

Gerrit-Change-Id: I73512131277fd8f69e6fe9d80947288f9c2c3262

Gerrit-Change-Number: 805040

Gerrit-PatchSet: 2

Gerrit-Owner: Peter Weinberger <p...@google.com>

Gerrit-Reviewer: Madeline Kalil <mka...@google.com>

Gerrit-Reviewer: Peter Weinberger <p...@google.com>

Gerrit-Reviewer: golang...@luci-project-accounts.iam.gserviceaccount.com <golang...@luci-project-accounts.iam.gserviceaccount.com>