dev: group linter implementation and integration tests (#4603)

Author: ldez

.gitignore

@@ -3,6 +3,7 @@
 /*.pdf
 /*.pprof
 /*.txt
+/test.lock
 /.idea/
 /.vscode/
 /dist/

.golangci.yml

@@ -163,27 +163,27 @@ issues:
       text: "SA1019: c.cfg.Run.ShowStats is deprecated: use Output.ShowStats instead."
 
     # Deprecated linter options.
-    - path: pkg/golinters/errcheck.go
+    - path: pkg/golinters/errcheck/errcheck.go
       linters: [staticcheck]
       text: "SA1019: errCfg.Exclude is deprecated: use ExcludeFunctions instead"
-    - path: pkg/golinters/govet.go
+    - path: pkg/golinters/govet/govet.go
       linters: [staticcheck]
       text: "SA1019: cfg.CheckShadowing is deprecated: the linter should be enabled inside Enable."
-    - path: pkg/golinters/godot.go
+    - path: pkg/golinters/godot/godot.go
       linters: [staticcheck]
       text: "SA1019: settings.CheckAll is deprecated: use Scope instead"
-    - path: pkg/golinters/gci.go
+    - path: pkg/golinters/gci/gci.go
       linters: [staticcheck]
       text: "SA1019: settings.LocalPrefixes is deprecated: use Sections instead."
-    - path: pkg/golinters/mnd.go
+    - path: pkg/golinters/mnd/mnd.go
       linters: [staticcheck]
       text: "SA1019: settings.Settings is deprecated: use root level settings instead."
-    - path: pkg/golinters/mnd.go
+    - path: pkg/golinters/mnd/mnd.go
       linters: [staticcheck]
       text: "SA1019: config.GoMndSettings is deprecated: use MndSettings."
 
     # Related to `run.go`, it cannot be removed.
-    - path: pkg/golinters/gofumpt.go
+    - path: pkg/golinters/gofumpt/gofumpt.go
       linters: [staticcheck]
       text: "SA1019: settings.LangVersion is deprecated: use the global `run.go` instead."
     - path: pkg/golinters/internal/staticcheck_common.go
@@ -194,7 +194,7 @@ issues:
       text: "SA1019: (.+).(GoVersion|LangVersion) is deprecated: use the global `run.go` instead."
 
     # Based on existing code, the modifications should be limited to make maintenance easier.
-    - path: pkg/golinters/unused.go
+    - path: pkg/golinters/unused/unused.go
       linters: [gocritic]
       text: "rangeValCopy: each iteration copies 160 bytes \\(consider pointers or indexing\\)"
 

Makefile

@@ -33,23 +33,21 @@ test: build
 	GL_TEST_RUN=1 go test -v -parallel 2 ./...
 .PHONY: test
 
-# ex: T=gofmt.go make test_fix
-# the value of `T` is the name of a file from `test/testdata/fix`
-test_fix: build
-	GL_TEST_RUN=1 go test -v ./test -count 1 -run TestFix/$T
-.PHONY: test_fix
-
 test_race: build_race
 	GL_TEST_RUN=1 ./$(BINARY) run -v --timeout=5m
 .PHONY: test_race
 
-test_linters:
+# ex: T=output.go make test_integration
+# the value of `T` is the name of a file from `test/testdata`
+test_integration:
 	GL_TEST_RUN=1 go test -v ./test -count 1 -run TestSourcesFromTestdata/$T
-.PHONY: test_linters
+.PHONY: test_integration
 
-test_linters_sub:
-	GL_TEST_RUN=1 go test -v ./test -count 1 -run TestSourcesFromTestdataSubDir/$T
-.PHONY: test_linters_sub
+# ex: T=multiple-issues-fix.go make test_integration_fix
+# the value of `T` is the name of a file from `test/testdata/fix`
+test_integration_fix: build
+	GL_TEST_RUN=1 go test -v ./test -count 1 -run TestFix/$T
+.PHONY: test_integration_fix
 
 # Maintenance
 

docs/src/docs/contributing/new-linters.mdx

@@ -15,19 +15,16 @@ We don't accept non `go/analysis` linters.
 After that:
 
 1. Implement functional tests for the linter:
-    - Add one file into directory [`test/testdata`](https://github.com/golangci/golangci-lint/tree/master/test/testdata).
-    - Run the test to ensure that test fails:
-      ```bash
-      T=yourlintername.go make test_linters
-      ```
+    - Add one file into directory `pkg/golinters/{yourlintername}/testdata/`.
+    - Run the test to ensure that test fails.
     - Run:
       ```bash
-      go run ./cmd/golangci-lint/ run --no-config --disable-all --enable=yourlintername ./test/testdata/yourlintername.go
+      go run ./cmd/golangci-lint/ run --no-config --disable-all --enable={yourlintername} ./pkg/golinters/{yourlintername}/testdata/{yourlintername}.go
       ```
-2. Add a new file `pkg/golinters/{yourlintername}.go`.
+2. Add a new file `pkg/golinters/{yourlintername}/{yourlintername}.go`.
    Look at other linters in this directory.
    Implement linter integration and check that test passes.
-3. Add the new struct for the linter (which you've implemented in `pkg/golinters/{yourlintername}.go`) to the
+3. Add the new struct for the linter (which you've implemented in `pkg/golinters/{yourlintername}/{yourlintername}.go`) to the
    list of all supported linters in [`pkg/lint/lintersdb/builder_linter.go`](https://github.com/golangci/golangci-lint/blob/master/pkg/lint/lintersdb/builder_linter.go)
    to the method `LinterBuilder.Build`.
     - Add `WithSince("next_version")`, where `next_version` must be replaced by the next minor version. (ex: v1.2.0 if the current version is v1.1.0)

pkg/golinters/asasalint.go renamed to pkg/golinters/asasalint/asasalint.go

@@ -1,4 +1,4 @@
-package golinters
+package asasalint
 
 import (
 	"github.com/alingse/asasalint"
@@ -9,7 +9,7 @@ import (
 	"github.com/golangci/golangci-lint/pkg/golinters/internal"
 )
 
-func NewAsasalint(setting *config.AsasalintSettings) *goanalysis.Linter {
+func New(setting *config.AsasalintSettings) *goanalysis.Linter {
 	cfg := asasalint.LinterSetting{}
 	if setting != nil {
 		cfg.Exclude = setting.Exclude

pkg/golinters/asasalint/asasalint_integration_test.go

@@ -0,0 +1,11 @@
+package asasalint
+
+import (
+	"testing"
+
+	"github.com/golangci/golangci-lint/test/testshared/integration"
+)
+
+func TestFromTestdata(t *testing.T) {
+	integration.RunTestdata(t)
+}

test/testdata/asasalint.go renamed to pkg/golinters/asasalint/testdata/asasalint.go

@@ -1,4 +1,4 @@
-package golinters
+package asciicheck
 
 import (
 	"github.com/tdakkota/asciicheck"
@@ -7,7 +7,7 @@ import (
 	"github.com/golangci/golangci-lint/pkg/goanalysis"
 )
 
-func NewAsciicheck() *goanalysis.Linter {
+func New() *goanalysis.Linter {
 	a := asciicheck.NewAnalyzer()
 
 	return goanalysis.NewLinter(

pkg/golinters/asciicheck.go renamed to pkg/golinters/asciicheck/asciicheck.go

@@ -0,0 +1,11 @@
+package asciicheck
+
+import (
+	"testing"
+
+	"github.com/golangci/golangci-lint/test/testshared/integration"
+)
+
+func TestFromTestdata(t *testing.T) {
+	integration.RunTestdata(t)
+}