Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd/go: running go test ./... elides most test results due to vet #23047

Closed
stuartcarnie opened this issue Dec 8, 2017 · 4 comments
Closed
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. release-blocker
Milestone

Comments

@stuartcarnie
Copy link

What version of Go are you using (go version)?

go version go1.10beta1 darwin/amd64

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/stuartcarnie/Library/Caches/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/stuartcarnie/projects/go/influxdata/influxdb"
GORACE=""
GOROOT="/Users/stuartcarnie/.gvm/versions/go1.10beta1.darwin.amd64"
GOTMPDIR=""
GOTOOLDIR="/Users/stuartcarnie/.gvm/versions/go1.10beta1.darwin.amd64/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/98/6s2bgx_x3vg8c9yrf1v4vp700000gn/T/go-build941673718=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

What did you expect to see?

Test results for all packages

What did you see instead?

Test results stop at the github.com/influxdata/influxdb/cmd package

Workaround

go test -vet=off

@bradfitz bradfitz changed the title running go test ./... elides most test results cmd/go: running go test ./... elides most test results due to vet Dec 8, 2017
@bradfitz bradfitz added NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. release-blocker labels Dec 8, 2017
@bradfitz bradfitz added this to the Go1.10 milestone Dec 8, 2017
@ianlancetaylor
Copy link
Contributor

Can you show us the output of go test -x ./...?

@stuartcarnie
Copy link
Author

stuartcarnie commented Dec 8, 2017

relevant gist of go test -x ./... output https://gist.github.com/stuartcarnie/a9a703f73d368aa4b56b296258ce8e91

@rsc
Copy link
Contributor

rsc commented Dec 13, 2017

Reproduced. Thanks for the good report.

@gopherbot
Copy link

Change https://golang.org/cl/83955 mentions this issue: cmd/go: do not let test vet failures stop reporting of later test results

@golang golang locked and limited conversation to collaborators Dec 14, 2018
@rsc rsc removed their assignment Jun 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. release-blocker
Projects
None yet
Development

No branches or pull requests

5 participants