Skip to content

Commit

Permalink
gopls/internal/regtest/marker: require go/packages
Browse files Browse the repository at this point in the history
Add a missing testenv condition to the marker test runner.

Fixes golang/go#58396

Change-Id: I69a7fe6dfab011ba789a066e2c7677c0315eed00
Reviewed-on: https://go-review.googlesource.com/c/tools/+/466715
Run-TryBot: Robert Findley <[email protected]>
TryBot-Result: Gopher Robot <[email protected]>
gopls-CI: kokoro <[email protected]>
Auto-Submit: Robert Findley <[email protected]>
Reviewed-by: Bryan Mills <[email protected]>
  • Loading branch information
findleyr authored and gopherbot committed Feb 8, 2023
1 parent 1ace7db commit 545ca87
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions gopls/internal/lsp/regtest/marker.go
Original file line number Diff line number Diff line change
Expand Up @@ -236,6 +236,9 @@ var update = flag.Bool("update", false, "if set, update test data during marker
// - AddImport
// - SelectionRanges
func RunMarkerTests(t *testing.T, dir string) {
// The marker tests must be able to run go/packages.Load.
testenv.NeedsGoPackages(t)

tests, err := loadMarkerTests(dir)
if err != nil {
t.Fatal(err)
Expand Down

0 comments on commit 545ca87

Please sign in to comment.