tree 010d8eeb421bcd5ae8845ba3d1bad7b85edd2bea
parent 7093a17b0467e77fb41804b2754a44f54a8c9ca2
author Iskander Sharipov <quasilyte@gmail.com> 1576231728 +0300
committer Rebecca Stambler <rstambler@golang.org> 1576262369 +0000

internal/lsp/tests: add error handling for @item()

If there are less than 3 args for @item, it does an early
return without recording it for a given pos.
This leads to a panic when tests runner can't lookup
a non-nil item for a pos.

Since other collect* methods seem to use t.Fatal for error
handling, do the same in collectCompletionItems().

Change-Id: I21960731f532b93029e6e06800e0484dc7d599df
Reviewed-on: https://go-review.googlesource.com/c/tools/+/211257
Run-TryBot: Iskander Sharipov <quasilyte@gmail.com>
Run-TryBot: Rebecca Stambler <rstambler@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Rebecca Stambler <rstambler@golang.org>