tree 7c82ea0e079cc65aaf86b9ac3eb601f3bde257ce
parent b29caf79d417d3744f726c1210133fa3e8e1bd24
author Rob Findley <rfindley@google.com> 1602772896 -0400
committer Robert Findley <rfindley@google.com> 1602786029 +0000

internal/lsp/source: be consistent about command identifiers

The primary identifier for gopls commands was changed from Command.Name
to Command.ID(), but this change was not made in the commands passed
back to the client via ExecuteCommandOptions.

Fix this, and ensure that our regtests actually check commands against
the list of supported commands that has been sent.

For golang/go#41985

Change-Id: If566584f157e8a86d26eac6353dbfd772b298cfc
Reviewed-on: https://go-review.googlesource.com/c/tools/+/262597
Run-TryBot: Robert Findley <rfindley@google.com>
Trust: Robert Findley <rfindley@google.com>
gopls-CI: kokoro <noreply+kokoro@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>