Skip to content

Commit 486fd23

Browse files
committed
all: remove unit tests for staticcheck, unparam, and spellcheck
Those should not be unit tests and, more importantly, they should not be running on builders. Change-Id: I70504fedce3bad17ac408dc6f208d61fb7bd6071 Reviewed-on: https://go-review.googlesource.com/c/vuln/+/587097 TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Ian Cottrell <iancottrell@google.com> Run-TryBot: Zvonimir Pavlinovic <zpavlinovic@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
1 parent 6b0fd56 commit 486fd23

File tree

3 files changed

+0
-27
lines changed

3 files changed

+0
-27
lines changed

all_test.go

Lines changed: 0 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ import (
2222
"golang.org/x/mod/modfile"
2323
"golang.org/x/vuln/internal/testenv"
2424
"golang.org/x/vuln/scan"
25-
"mvdan.cc/unparam/check"
2625
)
2726

2827
// excluded contains the set of modules that x/vuln should not depend on.
@@ -83,22 +82,6 @@ func TestGovulncheck(t *testing.T) {
8382
}
8483
}
8584

86-
func TestStaticCheck(t *testing.T) {
87-
skipIfShort(t)
88-
rungo(t, "run", "honnef.co/go/tools/cmd/staticcheck@v0.4.3", "./...")
89-
}
90-
91-
func TestUnparam(t *testing.T) {
92-
testenv.NeedsGoBuild(t)
93-
warns, err := check.UnusedParams(false, false, false, "./...")
94-
if err != nil {
95-
t.Fatalf("check.UnusedParams: %v", err)
96-
}
97-
for _, warn := range warns {
98-
t.Errorf(warn)
99-
}
100-
}
101-
10285
func TestVet(t *testing.T) {
10386
rungo(t, "vet", "-all", "./...")
10487
}
@@ -107,11 +90,6 @@ func TestGoModTidy(t *testing.T) {
10790
rungo(t, "mod", "tidy")
10891
}
10992

110-
func TestMisspell(t *testing.T) {
111-
skipIfShort(t)
112-
rungo(t, "run", "github.com/client9/misspell/cmd/misspell@v0.3.4", "-error", ".")
113-
}
114-
11593
func TestHeaders(t *testing.T) {
11694
sfs := os.DirFS(".")
11795
fs.WalkDir(sfs, ".", func(path string, d fs.DirEntry, _ error) error {

go.mod

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ require (
88
golang.org/x/mod v0.17.0
99
golang.org/x/sync v0.7.0
1010
golang.org/x/tools v0.21.1-0.20240514024235-59d9797072e7
11-
mvdan.cc/unparam v0.0.0-20230312165513-e84e2d14e3b8
1211
)
1312

1413
require github.com/google/renameio v0.1.0 // indirect

go.sum

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,5 @@ golang.org/x/mod v0.17.0 h1:zY54UmvipHiNd+pm+m0x9KhZ9hl1/7QNMyxXbc6ICqA=
99
golang.org/x/mod v0.17.0/go.mod h1:hTbmBsO62+eylJbnUtE2MGJUyE7QWk4xUqPFrRgJ+7c=
1010
golang.org/x/sync v0.7.0 h1:YsImfSBoP9QPYL0xyKJPq0gcaJdG3rInoqxTWbfQu9M=
1111
golang.org/x/sync v0.7.0/go.mod h1:Czt+wKu1gCyEFDUtn0jG5QVvpJ6rzVqr5aXyt9drQfk=
12-
golang.org/x/tools v0.21.0 h1:qc0xYgIbsSDt9EyWz05J5wfa7LOVW0YTLOXrqdLAWIw=
13-
golang.org/x/tools v0.21.0/go.mod h1:aiJjzUbINMkxbQROHiO6hDPo2LHcIPhhQsa9DLh0yGk=
1412
golang.org/x/tools v0.21.1-0.20240514024235-59d9797072e7 h1:DnP3aRQn/r68glNGB8/7+3iE77jA+YZZCxpfIXx2MdA=
1513
golang.org/x/tools v0.21.1-0.20240514024235-59d9797072e7/go.mod h1:aiJjzUbINMkxbQROHiO6hDPo2LHcIPhhQsa9DLh0yGk=
16-
mvdan.cc/unparam v0.0.0-20230312165513-e84e2d14e3b8 h1:VuJo4Mt0EVPychre4fNlDWDuE5AjXtPJpRUWqZDQhaI=
17-
mvdan.cc/unparam v0.0.0-20230312165513-e84e2d14e3b8/go.mod h1:Oh/d7dEtzsNHGOq1Cdv8aMm3KdKhVvPbRQcM8WFpBR8=

0 commit comments

Comments
 (0)