tree 1eb6baf0b01c7f0f9599125feaeae085e68b2ab8
parent c97a9db72cfb15897a4889900072c5a57d85d01c
author Daisuke Suzuki <daisuzu@gmail.com> 1583582604 +0900
committer Rebecca Stambler <rstambler@golang.org> 1584651847 +0000

internal/lsp/cmd: fix definition test to run independently

definition.Run requires a nil check of opts before applying to avoid
panic, and test must be run with markdown enabled.
When running 'go test' without -run flag, connection.initialize() was
not called and there was no problem because the default value of
Options.PreferredContentFormat was Markdown.
In addition, currently using the same connection despite different
options, therefore make to use a different connection for different
options.

Change-Id: I6ef773c14cbdcae621da9295d4c618c3aff0beee
Reviewed-on: https://go-review.googlesource.com/c/tools/+/222497
Reviewed-by: Rebecca Stambler <rstambler@golang.org>
Run-TryBot: Rebecca Stambler <rstambler@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>