deps: upgrade to gopls and x/tools a3f652f1

* tools/gopls: add cmd support for folding_ranges a3f652f1
* internal/lsp/cache: avoid returning errors when building source.Errors 52adfe5c
* cmd/go-contrib-init: add unit test for the cmdErr function 9d59ce8a
* internal/testenv: reject the resolved 'go' command if it does not match runtime.GOROOT 7dd52f09
* internal/lsp/cache: have NewView create view even if load all packages fails 50fa39b7
* go/analysis/unitchecker: add erroras analysis to align with go vet 8cb0d021
* internal/lsp: support implements for methods on an interface a99edfee
* internal/lsp/protocol: define types in alphabetical order in tsprotocol.go 9f4e7946

cmd/govim/internal/golang_org_x_tools/lsp/cache/check.go

@@ -310,7 +310,8 @@ func typeCheck(ctx context.Context, fset *token.FileSet, m *metadata, mode sourc
 		for _, e := range rawErrors {
 			srcErr, err := sourceError(ctx, fset, pkg, e)
 			if err != nil {
-				return nil, err
+				log.Error(ctx, "unable to compute error positions", err, telemetry.Package.Of(pkg.ID()))
+				continue
 			}
 			pkg.errors = append(pkg.errors, srcErr)
 		}

cmd/govim/internal/golang_org_x_tools/lsp/cache/errors.go

@@ -12,7 +12,9 @@ import (
 	"golang.org/x/tools/go/packages"
 	"github.com/govim/govim/cmd/govim/internal/golang_org_x_tools/lsp/protocol"
 	"github.com/govim/govim/cmd/govim/internal/golang_org_x_tools/lsp/source"
+	"github.com/govim/govim/cmd/govim/internal/golang_org_x_tools/lsp/telemetry"
 	"github.com/govim/govim/cmd/govim/internal/golang_org_x_tools/span"
+	"github.com/govim/govim/cmd/govim/internal/golang_org_x_tools/telemetry/log"
 	errors "golang.org/x/xerrors"
 )
 
@@ -40,7 +42,8 @@ func sourceError(ctx context.Context, fset *token.FileSet, pkg *pkg, e interface
 		kind = source.ParseError
 		spn, err = scannerErrorRange(ctx, fset, pkg, e.Pos)
 		if err != nil {
-			return nil, err
+			log.Error(ctx, "no span for scanner.Error pos", err, telemetry.Package.Of(pkg.ID()))
+			spn = span.Parse(e.Pos.String())
 		}
 
 	case scanner.ErrorList:
@@ -52,9 +55,9 @@ func sourceError(ctx context.Context, fset *token.FileSet, pkg *pkg, e interface
 		kind = source.ParseError
 		spn, err = scannerErrorRange(ctx, fset, pkg, e[0].Pos)
 		if err != nil {
-			return nil, err
+			log.Error(ctx, "no span for scanner.Error pos", err, telemetry.Package.Of(pkg.ID()))
+			spn = span.Parse(e[0].Pos.String())
 		}
-
 	case types.Error:
 		msg = e.Msg
 		kind = source.TypeError

cmd/govim/internal/golang_org_x_tools/lsp/cache/load.go

@@ -64,10 +64,12 @@ func (s *snapshot) load(ctx context.Context, scope source.Scope) ([]*metadata, e
 		return nil, errors.Errorf("no metadata for %s: %v", uri, err)
 	}
 	log.Print(ctx, "go/packages.Load", tag.Of("packages", len(pkgs)))
-	if _, ok := scope.(source.FileURI); len(pkgs) == 0 && ok {
+	if len(pkgs) == 0 {
 		if err == nil {
 			err = errNoPackagesFound
 		}
+	}
+	if err != nil {
 		return nil, err
 	}
 	m, prevMissingImports, err := s.updateMetadata(ctx, scope, pkgs, cfg)

cmd/govim/internal/golang_org_x_tools/lsp/cache/session.go

@@ -6,6 +6,7 @@ package cache
 
 import (
 	"context"
+	"fmt"
 	"path/filepath"
 	"sort"
 	"strconv"
@@ -126,8 +127,9 @@ func (s *session) NewView(ctx context.Context, name string, folder span.URI, opt
 	v.snapshotMu.Lock()
 	defer v.snapshotMu.Unlock() // The code after the snapshot is used isn't expensive.
 	m, err := v.snapshot.load(ctx, source.DirectoryURI(folder))
+	var loadErr error
 	if err != nil && err != errNoPackagesFound {
-		return nil, err
+		loadErr = fmt.Errorf("Error loading packages: %v", err)
 	}
 
 	// Prepare CheckPackageHandles for every package that's been loaded.
@@ -144,7 +146,7 @@ func (s *session) NewView(ctx context.Context, name string, folder span.URI, opt
 	// we always need to drop the view map
 	s.viewMap = make(map[span.URI]source.View)
 	debug.AddView(debugView{v})
-	return v, nil
+	return v, loadErr
 }
 
 // View returns the view by name.

cmd/govim/internal/golang_org_x_tools/lsp/cmd/cmd.go

@@ -142,6 +142,7 @@ func (app *Application) commands() []tool.Application {
 		&app.Serve,
 		&bug{},
 		&check{app: app},
+		&foldingRanges{app: app},
 		&format{app: app},
 		&links{app: app},
 		&imports{app: app},

cmd/govim/internal/golang_org_x_tools/lsp/cmd/folding_range.go

@@ -0,0 +1,72 @@
+// Copyright 2019 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+package cmd
+
+import (
+	"context"
+	"flag"
+	"fmt"
+
+	"github.com/govim/govim/cmd/govim/internal/golang_org_x_tools/lsp/protocol"
+	"github.com/govim/govim/cmd/govim/internal/golang_org_x_tools/span"
+	"github.com/govim/govim/cmd/govim/internal/golang_org_x_tools/tool"
+)
+
+// foldingRanges implements the folding_ranges verb for gopls
+type foldingRanges struct {
+	app *Application
+}
+
+func (r *foldingRanges) Name() string      { return "folding_ranges" }
+func (r *foldingRanges) Usage() string     { return "<file>" }
+func (r *foldingRanges) ShortHelp() string { return "display selected file's folding ranges" }
+func (r *foldingRanges) DetailedHelp(f *flag.FlagSet) {
+	fmt.Fprint(f.Output(), `
+Example:
+
+  $ gopls folding_ranges helper/helper.go
+`)
+	f.PrintDefaults()
+}
+
+func (r *foldingRanges) Run(ctx context.Context, args ...string) error {
+	if len(args) != 1 {
+		return tool.CommandLineErrorf("folding_ranges expects 1 argument (file)")
+	}
+
+	conn, err := r.app.connect(ctx)
+	if err != nil {
+		return err
+	}
+	defer conn.terminate(ctx)
+
+	from := span.Parse(args[0])
+	file := conn.AddFile(ctx, from.URI())
+	if file.err != nil {
+		return file.err
+	}
+
+	p := protocol.FoldingRangeParams{
+		TextDocument: protocol.TextDocumentIdentifier{
+			URI: protocol.NewURI(from.URI()),
+		},
+	}
+
+	ranges, err := conn.FoldingRange(ctx, &p)
+	if err != nil {
+		return err
+	}
+
+	for _, r := range ranges {
+		fmt.Printf("%v:%v-%v:%v\n",
+			r.StartLine+1,
+			r.StartCharacter+1,
+			r.EndLine+1,
+			r.EndCharacter,
+		)
+	}
+
+	return nil
+}

cmd/govim/internal/golang_org_x_tools/lsp/cmd/test/cmdtest.go

@@ -65,10 +65,6 @@ func (r *runner) RankCompletion(t *testing.T, src span.Span, test tests.Completi
 	//TODO: add command line completions tests when it works
 }
 
-func (r *runner) FoldingRange(t *testing.T, spn span.Span) {
-	//TODO: add command line folding range tests when it works
-}
-
 func (r *runner) Highlight(t *testing.T, name string, locations []span.Span) {
 	//TODO: add command line highlight tests when it works
 }

cmd/govim/internal/golang_org_x_tools/lsp/cmd/test/folding_range.go

@@ -0,0 +1,32 @@
+package cmdtest
+
+import (
+	"fmt"
+	"testing"
+
+	"github.com/govim/govim/cmd/govim/internal/golang_org_x_tools/lsp/cmd"
+	"github.com/govim/govim/cmd/govim/internal/golang_org_x_tools/span"
+	"github.com/govim/govim/cmd/govim/internal/golang_org_x_tools/tool"
+)
+
+func (r *runner) FoldingRanges(t *testing.T, spn span.Span) {
+	goldenTag := "foldingRange-cmd"
+	uri := spn.URI()
+	filename := uri.Filename()
+
+	app := cmd.New("gopls-test", r.data.Config.Dir, r.data.Config.Env, r.options)
+	got := CaptureStdOut(t, func() {
+		err := tool.Run(r.ctx, app, append([]string{"-remote=internal", "folding_ranges"}, filename))
+		if err != nil {
+			fmt.Println(err)
+		}
+	})
+
+	expect := string(r.data.Golden(goldenTag, filename, func() ([]byte, error) {
+		return []byte(got), nil
+	}))
+
+	if expect != got {
+		t.Errorf("folding_ranges failed failed for %s expected:\n%s\ngot:\n%s", filename, expect, got)
+	}
+}