mirror of
https://github.com/golang/go
synced 2024-11-18 12:54:44 -07:00
d2abdd5413
+ test Fixes issue golang/go#14684 Change-Id: I56023bf36c307d02c71b4ddf08aee9c229fc66c1 Reviewed-on: https://go-review.googlesource.com/20247 Reviewed-by: Daniel Morsing <daniel.morsing@gmail.com>
292 lines
7.5 KiB
Go
292 lines
7.5 KiB
Go
// Copyright 2013 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 main_test
|
|
|
|
// This file defines a test framework for guru queries.
|
|
//
|
|
// The files beneath testdata/src/main contain Go programs containing
|
|
// query annotations of the form:
|
|
//
|
|
// @verb id "select"
|
|
//
|
|
// where verb is the query mode (e.g. "callers"), id is a unique name
|
|
// for this query, and "select" is a regular expression matching the
|
|
// substring of the current line that is the query's input selection.
|
|
//
|
|
// The expected output for each query is provided in the accompanying
|
|
// .golden file.
|
|
//
|
|
// (Location information is not included because it's too fragile to
|
|
// display as text. TODO(adonovan): think about how we can test its
|
|
// correctness, since it is critical information.)
|
|
//
|
|
// Run this test with:
|
|
// % go test golang.org/x/tools/cmd/guru -update
|
|
// to update the golden files.
|
|
|
|
import (
|
|
"bytes"
|
|
"encoding/json"
|
|
"flag"
|
|
"fmt"
|
|
"go/build"
|
|
"go/parser"
|
|
"go/token"
|
|
"io"
|
|
"io/ioutil"
|
|
"os"
|
|
"os/exec"
|
|
"path/filepath"
|
|
"regexp"
|
|
"runtime"
|
|
"strconv"
|
|
"strings"
|
|
"testing"
|
|
|
|
guru "golang.org/x/tools/cmd/guru"
|
|
)
|
|
|
|
var updateFlag = flag.Bool("update", false, "Update the golden files.")
|
|
|
|
type query struct {
|
|
id string // unique id
|
|
verb string // query mode, e.g. "callees"
|
|
posn token.Position // query position
|
|
filename string
|
|
queryPos string // query position in command-line syntax
|
|
}
|
|
|
|
func parseRegexp(text string) (*regexp.Regexp, error) {
|
|
pattern, err := strconv.Unquote(text)
|
|
if err != nil {
|
|
return nil, fmt.Errorf("can't unquote %s", text)
|
|
}
|
|
return regexp.Compile(pattern)
|
|
}
|
|
|
|
// parseQueries parses and returns the queries in the named file.
|
|
func parseQueries(t *testing.T, filename string) []*query {
|
|
filedata, err := ioutil.ReadFile(filename)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
// Parse the file once to discover the test queries.
|
|
fset := token.NewFileSet()
|
|
f, err := parser.ParseFile(fset, filename, filedata, parser.ParseComments)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
lines := bytes.Split(filedata, []byte("\n"))
|
|
|
|
var queries []*query
|
|
queriesById := make(map[string]*query)
|
|
|
|
// Find all annotations of these forms:
|
|
expectRe := regexp.MustCompile(`@([a-z]+)\s+(\S+)\s+(\".*)$`) // @verb id "regexp"
|
|
for _, c := range f.Comments {
|
|
text := strings.TrimSpace(c.Text())
|
|
if text == "" || text[0] != '@' {
|
|
continue
|
|
}
|
|
posn := fset.Position(c.Pos())
|
|
|
|
// @verb id "regexp"
|
|
match := expectRe.FindStringSubmatch(text)
|
|
if match == nil {
|
|
t.Errorf("%s: ill-formed query: %s", posn, text)
|
|
continue
|
|
}
|
|
|
|
id := match[2]
|
|
if prev, ok := queriesById[id]; ok {
|
|
t.Errorf("%s: duplicate id %s", posn, id)
|
|
t.Errorf("%s: previously used here", prev.posn)
|
|
continue
|
|
}
|
|
|
|
q := &query{
|
|
id: id,
|
|
verb: match[1],
|
|
filename: filename,
|
|
posn: posn,
|
|
}
|
|
|
|
if match[3] != `"nopos"` {
|
|
selectRe, err := parseRegexp(match[3])
|
|
if err != nil {
|
|
t.Errorf("%s: %s", posn, err)
|
|
continue
|
|
}
|
|
|
|
// Find text of the current line, sans query.
|
|
// (Queries must be // not /**/ comments.)
|
|
line := lines[posn.Line-1][:posn.Column-1]
|
|
|
|
// Apply regexp to current line to find input selection.
|
|
loc := selectRe.FindIndex(line)
|
|
if loc == nil {
|
|
t.Errorf("%s: selection pattern %s doesn't match line %q",
|
|
posn, match[3], string(line))
|
|
continue
|
|
}
|
|
|
|
// Assumes ASCII. TODO(adonovan): test on UTF-8.
|
|
linestart := posn.Offset - (posn.Column - 1)
|
|
|
|
// Compute the file offsets.
|
|
q.queryPos = fmt.Sprintf("%s:#%d,#%d",
|
|
filename, linestart+loc[0], linestart+loc[1])
|
|
}
|
|
|
|
queries = append(queries, q)
|
|
queriesById[id] = q
|
|
}
|
|
|
|
// Return the slice, not map, for deterministic iteration.
|
|
return queries
|
|
}
|
|
|
|
// WriteResult writes res (-format=plain) to w, stripping file locations.
|
|
func WriteResult(w io.Writer, q *guru.Query) {
|
|
capture := new(bytes.Buffer) // capture standard output
|
|
q.WriteTo(capture)
|
|
for _, line := range strings.Split(capture.String(), "\n") {
|
|
// Remove a "file:line: " prefix.
|
|
if i := strings.Index(line, ": "); i >= 0 {
|
|
line = line[i+2:]
|
|
}
|
|
fmt.Fprintf(w, "%s\n", line)
|
|
}
|
|
}
|
|
|
|
// doQuery poses query q to the guru and writes its response and
|
|
// error (if any) to out.
|
|
func doQuery(out io.Writer, q *query, useJson bool) {
|
|
fmt.Fprintf(out, "-------- @%s %s --------\n", q.verb, q.id)
|
|
|
|
var buildContext = build.Default
|
|
buildContext.GOPATH = "testdata"
|
|
pkg := filepath.Dir(strings.TrimPrefix(q.filename, "testdata/src/"))
|
|
query := guru.Query{
|
|
Mode: q.verb,
|
|
Pos: q.queryPos,
|
|
Build: &buildContext,
|
|
Scope: []string{pkg},
|
|
Reflection: true,
|
|
}
|
|
if err := guru.Run(&query); err != nil {
|
|
fmt.Fprintf(out, "\nError: %s\n", err)
|
|
return
|
|
}
|
|
|
|
if useJson {
|
|
// JSON output
|
|
b, err := json.MarshalIndent(query.Serial(), "", "\t")
|
|
if err != nil {
|
|
fmt.Fprintf(out, "JSON error: %s\n", err.Error())
|
|
return
|
|
}
|
|
out.Write(b)
|
|
fmt.Fprintln(out)
|
|
} else {
|
|
// "plain" (compiler diagnostic format) output
|
|
WriteResult(out, &query)
|
|
}
|
|
}
|
|
|
|
func TestGuru(t *testing.T) {
|
|
switch runtime.GOOS {
|
|
case "android":
|
|
t.Skipf("skipping test on %q (no testdata dir)", runtime.GOOS)
|
|
case "windows":
|
|
t.Skipf("skipping test on %q (no /usr/bin/diff)", runtime.GOOS)
|
|
}
|
|
|
|
for _, filename := range []string{
|
|
"testdata/src/calls/main.go",
|
|
"testdata/src/describe/main.go",
|
|
"testdata/src/freevars/main.go",
|
|
"testdata/src/implements/main.go",
|
|
"testdata/src/implements-methods/main.go",
|
|
"testdata/src/imports/main.go",
|
|
"testdata/src/peers/main.go",
|
|
"testdata/src/pointsto/main.go",
|
|
"testdata/src/referrers/main.go",
|
|
"testdata/src/reflection/main.go",
|
|
"testdata/src/what/main.go",
|
|
"testdata/src/whicherrs/main.go",
|
|
// JSON:
|
|
// TODO(adonovan): most of these are very similar; combine them.
|
|
"testdata/src/calls-json/main.go",
|
|
"testdata/src/peers-json/main.go",
|
|
"testdata/src/describe-json/main.go",
|
|
"testdata/src/implements-json/main.go",
|
|
"testdata/src/implements-methods-json/main.go",
|
|
"testdata/src/pointsto-json/main.go",
|
|
"testdata/src/referrers-json/main.go",
|
|
"testdata/src/what-json/main.go",
|
|
} {
|
|
useJson := strings.Contains(filename, "-json/")
|
|
queries := parseQueries(t, filename)
|
|
golden := filename + "lden"
|
|
got := filename + "t"
|
|
gotfh, err := os.Create(got)
|
|
if err != nil {
|
|
t.Errorf("Create(%s) failed: %s", got, err)
|
|
continue
|
|
}
|
|
defer gotfh.Close()
|
|
defer os.Remove(got)
|
|
|
|
// Run the guru on each query, redirecting its output
|
|
// and error (if any) to the foo.got file.
|
|
for _, q := range queries {
|
|
doQuery(gotfh, q, useJson)
|
|
}
|
|
|
|
// Compare foo.got with foo.golden.
|
|
var cmd *exec.Cmd
|
|
switch runtime.GOOS {
|
|
case "plan9":
|
|
cmd = exec.Command("/bin/diff", "-c", golden, got)
|
|
default:
|
|
cmd = exec.Command("/usr/bin/diff", "-u", golden, got)
|
|
}
|
|
buf := new(bytes.Buffer)
|
|
cmd.Stdout = buf
|
|
cmd.Stderr = os.Stderr
|
|
if err := cmd.Run(); err != nil {
|
|
t.Errorf("Guru tests for %s failed: %s.\n%s\n",
|
|
filename, err, buf)
|
|
|
|
if *updateFlag {
|
|
t.Logf("Updating %s...", golden)
|
|
if err := exec.Command("/bin/cp", got, golden).Run(); err != nil {
|
|
t.Errorf("Update failed: %s", err)
|
|
}
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestIssue14684(t *testing.T) {
|
|
var buildContext = build.Default
|
|
buildContext.GOPATH = "testdata"
|
|
query := guru.Query{
|
|
Mode: "freevars",
|
|
Pos: "testdata/src/README.txt:#1",
|
|
Build: &buildContext,
|
|
}
|
|
err := guru.Run(&query)
|
|
if err == nil {
|
|
t.Fatal("guru query succeeded unexpectedly")
|
|
}
|
|
if got, want := err.Error(), "testdata/src/README.txt is not a Go source file"; got != want {
|
|
t.Errorf("query error was %q, want %q", got, want)
|
|
}
|
|
}
|