Revert "cmd/go/internal/test: add 'tests' vet check to 'go test' suite"

This reverts commit f1d60500bcc48f06c8d322dac1e48dd4c22f5863.

Reason for revert: Broke the longtest builders.

Change-Id: I5f3510c8ffc24fae5e71fac0a2dbda01ecfe5d5c
Cq-Include-Trybots: luci.golang.try:gotip-linux-amd64-longtest
Reviewed-on: https://go-review.googlesource.com/c/go/+/571695
Reviewed-by: Alan Donovan <adonovan@google.com>
Auto-Submit: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Russ Cox <rsc@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>

diff --git a/doc/next/3-tools.md b/doc/next/3-tools.md
index 56d82e4926ece02ea493215211fff55d6db09fe9..bdbe6c077173d05ac85cc18410702060400f51aa 100644 (file)
--- a/doc/next/3-tools.md
+++ b/doc/next/3-tools.md
@@ -10,12 +10,3 @@ or copying the `go` binary.
 
 ### Cgo {#cgo}
 
-### Vet
-
-The new `tests` analyzer reports common mistakes in declarations of
-tests, fuzzers, benchmarks, and examples in test packages, such as
-malformed names, wrong signatures, or examples that document
-non-existent identifiers. Some of these mistakes may cause tests not
-to run.
-
-This analyzer is among the subset of analyzers that are run by `go test`.

diff --git a/src/cmd/go/alldocs.go b/src/cmd/go/alldocs.go
index cf3485faac27bd46231ec2bb669e751a3e68b925..dde47ac1b86bb3c66bbd1accd250577e44f4195f 100644 (file)
--- a/src/cmd/go/alldocs.go
+++ b/src/cmd/go/alldocs.go
@@ -1756,7 +1756,7 @@
 // finds any problems, go test reports those and does not run the test
 // binary. Only a high-confidence subset of the default go vet checks are
 // used. That subset is: atomic, bool, buildtags, directive, errorsas,
-// ifaceassert, nilfunc, printf, stringintconv, and tests. You can see
+// ifaceassert, nilfunc, printf, and stringintconv. You can see
 // the documentation for these and other vet tests via "go doc cmd/vet".
 // To disable the running of go vet, use the -vet=off flag. To run all
 // checks, use the -vet=all flag.

diff --git a/src/cmd/go/internal/test/test.go b/src/cmd/go/internal/test/test.go
index 17dc3a0fdbe059ea132479aa166ec9459eb2edfd..f1555fd5d42b7c2b3f270e911d67c614422c5f4e 100644 (file)
--- a/src/cmd/go/internal/test/test.go
+++ b/src/cmd/go/internal/test/test.go
@@ -79,7 +79,7 @@ and its test source files to identify significant problems. If go vet
 finds any problems, go test reports those and does not run the test
 binary. Only a high-confidence subset of the default go vet checks are
 used. That subset is: atomic, bool, buildtags, directive, errorsas,
-ifaceassert, nilfunc, printf, stringintconv, and tests. You can see
+ifaceassert, nilfunc, printf, and stringintconv. You can see
 the documentation for these and other vet tests via "go doc cmd/vet".
 To disable the running of go vet, use the -vet=off flag. To run all
 checks, use the -vet=all flag.
@@ -667,7 +667,7 @@ var defaultVetFlags = []string{
        "-slog",
        "-stringintconv",
        // "-structtags",
-       "-tests",
+       // "-tests",
        // "-unreachable",
        // "-unsafeptr",
        // "-unusedresult",

diff --git a/src/cmd/go/testdata/script/test_bad_example.txt b/src/cmd/go/testdata/script/test_bad_example.txt
index 46bc264779e3f869684f84a187de651d39221716..1d147b663fe62af62ab320b7baa9516a6a14f833 100644 (file)
--- a/src/cmd/go/testdata/script/test_bad_example.txt
+++ b/src/cmd/go/testdata/script/test_bad_example.txt
@@ -1,7 +1,6 @@
 # Tests that invalid examples are ignored.
 # Verifies golang.org/issue/35284
-# Disable vet, as 'tests' analyzer objects to surplus parameter.
-go test -vet=off x_test.go
+go test x_test.go
 
 -- x_test.go --
 package  x
@@ -11,4 +10,4 @@ import "fmt"
 func ExampleThisShouldNotHaveAParameter(thisShouldntExist int) {
        fmt.Println("X")
        // Output:
-}
+}
\ No newline at end of file