Skip to content

Commit 7bfb0cb

Browse files
committed
tests: update linter tests
1 parent b574814 commit 7bfb0cb

File tree

11 files changed

+16
-9
lines changed

11 files changed

+16
-9
lines changed

pkg/golinters/errcheck/testdata/errcheck.go

+4-4
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ func IgnoreCloseMissingErrHandling() error {
2121
return err
2222
}
2323

24-
f.Close()
24+
f.Close() // want "Error return value of `f.Close` is not checked"
2525
return nil
2626
}
2727

@@ -30,14 +30,14 @@ func IgnoreCloseInDeferMissingErrHandling() {
3030
if err != nil {
3131
panic(err)
3232
}
33-
defer resp.Body.Close()
33+
defer resp.Body.Close() // want "Error return value of `resp.Body.Close` is not checked"
3434

3535
panic(resp)
3636
}
3737

3838
func IgnoreStdxWrite() {
39-
os.Stdout.Write([]byte{})
40-
os.Stderr.Write([]byte{})
39+
os.Stdout.Write([]byte{}) // want "Error return value of `os.Stdout.Write` is not checked"
40+
os.Stderr.Write([]byte{}) // want "Error return value of `os.Stderr.Write` is not checked"
4141
}
4242

4343
func IgnoreBufferWrites(buf *bytes.Buffer) {

pkg/golinters/exhaustive/testdata/exhaustive_generated.go

+2-2
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

pkg/golinters/gosec/testdata/gosec.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -34,5 +34,5 @@ func GosecG204SubprocWithFunc() {
3434
return "/tmp/dummy"
3535
}
3636

37-
exec.Command("ls", arg()).Run() // want "G204: Subprocess launched with a potential tainted input or cmd arguments"
37+
exec.Command("ls", arg()).Run() // want "G104: Errors unhandled"
3838
}

pkg/golinters/revive/testdata/revive.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import (
77
"time"
88
)
99

10-
func SampleRevive(t *time.Duration) error {
10+
func SampleRevive(t *time.Duration) error { // want "exported: exported function SampleRevive should have comment or be unexported"
1111
if t == nil {
1212
return nil
1313
} else {

pkg/golinters/revive/testdata/revive.yml

-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ linters-settings:
66
severity: warning
77
rules:
88
- name: exported
9-
- name: package-comments
109
- name: cognitive-complexity
1110
arguments: [ 7 ]
1211
- name: line-length-limit

pkg/golinters/revive/testdata/revive_default.go

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
//golangcitest:args -Erevive
2+
// Package testdata ...
23
package testdata
34

45
import (

pkg/golinters/stylecheck/testdata/fix/in/stylecheck.go

+2
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
// Package testdata ...
2+
//
13
//golangcitest:args -Estylecheck
24
//golangcitest:expected_exitcode 0
35
package testdata

pkg/golinters/stylecheck/testdata/fix/out/stylecheck.go

+2
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
// Package testdata ...
2+
//
13
//golangcitest:args -Estylecheck
24
//golangcitest:expected_exitcode 0
35
package testdata

pkg/golinters/stylecheck/testdata/stylecheck.go

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
//golangcitest:args -Estylecheck
2+
// Package testdata ...
23
package testdata
34

45
func Stylecheck(x int) {

pkg/golinters/stylecheck/testdata/stylecheck_cgo.go

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
//golangcitest:args -Estylecheck
2+
// Package testdata ...
23
package testdata
34

45
/*

pkg/golinters/stylecheck/testdata/stylecheck_not_in_megacheck.go

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
//golangcitest:args -Emegacheck
22
//golangcitest:expected_exitcode 0
3+
// Package testdata ...
34
package testdata
45

56
func StylecheckNotInMegacheck(x int) {

0 commit comments

Comments
 (0)