tree 9dcf8806b65d640e0966cd1f72cf52e05fbc3d78
parent cc38525d79b9b8cdc77890487817a14770c674a9
author Rob Findley <rfindley@google.com> 1584998765 -0400
committer Robert Findley <rfindley@google.com> 1585081104 +0000

internal/lsp/regtest: remove redundant T and ctx params from regtest funcs

In an effort to be idiomatic, I made the regtest func signature
func(context.Context, testing.T, *Env), despite the fact that Env
already has a Context and a T.

This just ended up causing more confusion, as it's not clear which
Context or T to use. Remove this and just use the fields on Env.

Change-Id: I54da1fdfe6ce17a67601b2af8640d4d2ea676e8c
Reviewed-on: https://go-review.googlesource.com/c/tools/+/225157
Run-TryBot: Robert Findley <rfindley@google.com>
Reviewed-by: Rebecca Stambler <rstambler@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>