fix small bugs

Author: golangcidev

Gopkg.lock

@@ -153,7 +153,7 @@ func buildLintCtx(ctx context.Context, linters []pkg.Linter, cfg *config.Config)
 		args = []string{"./..."}
 	}
 
-	paths, err := fsutils.GetPathsForAnalysis(args)
+	paths, err := fsutils.GetPathsForAnalysis(ctx, args)
 	if err != nil {
 		return nil, err
 	}

internal/commands/run.go

@@ -1,4 +1,4 @@
-package golinters
+package pkg
 
 import (
 	"bytes"
@@ -19,11 +19,15 @@ func runGoErrchk(c *exec.Cmd, t *testing.T) {
 }
 
 const testdataDir = "testdata"
+
+var testdataWithIssuesDir = filepath.Join(testdataDir, "with_issues")
+var testdataNotCompilingDir = filepath.Join(testdataDir, "not_compiles")
+
 const binName = "golangci-lint"
 
-func TestSourcesFromTestdataDir(t *testing.T) {
-	t.Log(filepath.Join(testdataDir, "*.go"))
-	sources, err := filepath.Glob(filepath.Join(testdataDir, "*.go"))
+func TestSourcesFromTestdataWithIssuesDir(t *testing.T) {
+	t.Log(filepath.Join(testdataWithIssuesDir, "*.go"))
+	sources, err := filepath.Glob(filepath.Join(testdataWithIssuesDir, "*.go"))
 	assert.NoError(t, err)
 	assert.NotEmpty(t, sources)
 
@@ -39,7 +43,7 @@ func TestSourcesFromTestdataDir(t *testing.T) {
 }
 
 func installBinary(t *testing.T) {
-	cmd := exec.Command("go", "install", filepath.Join("..", "..", "cmd", binName))
+	cmd := exec.Command("go", "install", filepath.Join("..", "cmd", binName))
 	assert.NoError(t, cmd.Run(), "Can't go install %s", binName)
 }
 
@@ -48,3 +52,9 @@ func testOneSource(t *testing.T, sourcePath string) {
 	cmd := exec.Command(goErrchkBin, binName, "run", "--enable-all", "--gocyclo.min-complexity", "20", sourcePath)
 	runGoErrchk(cmd, t)
 }
+
+func TestNotCompilingProgram(t *testing.T) {
+	installBinary(t)
+	err := exec.Command(binName, "run", "--enable-all", testdataNotCompilingDir).Run()
+	assert.NoError(t, err)
+}

pkg/golinters/linters_test.go renamed to pkg/enabled_linters_test.go

@@ -8,6 +8,7 @@ import (
 	"path"
 	"path/filepath"
 	"strings"
+	"time"
 
 	"github.com/golangci/golangci-shared/pkg/analytics"
 )
@@ -54,14 +55,20 @@ func processPaths(root string, paths []string, maxPaths int) ([]string, error) {
 	return ret, nil
 }
 
-func GetPathsForAnalysis(inputPaths []string) (*ProjectPaths, error) {
+func GetPathsForAnalysis(ctx context.Context, inputPaths []string) (ret *ProjectPaths, err error) {
+	defer func(startedAt time.Time) {
+		if ret != nil {
+			analytics.Log(ctx).Infof("Found paths for analysis for %s: %s", time.Since(startedAt), ret.MixedPaths())
+		}
+	}(time.Now())
+
 	for _, path := range inputPaths {
 		if strings.HasSuffix(path, ".go") && len(inputPaths) != 1 {
 			return nil, fmt.Errorf("Specific files for analysis are allowed only if one file is set")
 		}
 	}
 
-	excludeDirs := []string{"vendor", "testdata", "examples", "Godeps"}
+	excludeDirs := []string{"vendor", "testdata", "examples", "Godeps", "builtin"}
 	pr := NewPathResolver(excludeDirs, []string{".go"})
 	paths, err := pr.Resolve(inputPaths...)
 	if err != nil {

pkg/fsutils/fsutils.go

@@ -37,8 +37,8 @@ func runLinter(ctx context.Context, linter Linter, lintCtx *golinters.Context, i
 	}()
 	startedAt := time.Now()
 	res, err = linter.Run(ctx, lintCtx)
-	analytics.Log(ctx).Infof("worker #%d: linter %s took %s for paths %s", i, linter.Name(),
-		time.Since(startedAt), lintCtx.Paths.MixedPaths())
+	analytics.Log(ctx).Infof("worker #%d: linter %s took %s", i, linter.Name(),
+		time.Since(startedAt))
 	return
 }
 

pkg/runner.go

@@ -0,0 +1,3 @@
+package p
+
+func F {

pkg/testdata/not_compiles/main.go

@@ -9,7 +9,7 @@ func (DuplLogger) level() int {
 func (DuplLogger) Debug(args ...interface{}) {}
 func (DuplLogger) Info(args ...interface{})  {}
 
-func (logger *DuplLogger) First(args ...interface{}) { // ERROR "12-21 lines are duplicate of `testdata/dupl.go:23-32`"
+func (logger *DuplLogger) First(args ...interface{}) { // ERROR "12-21 lines are duplicate of `testdata/with_issues/dupl.go:23-32`"
 	if logger.level() >= 0 {
 		logger.Debug(args...)
 		logger.Debug(args...)
@@ -20,7 +20,7 @@ func (logger *DuplLogger) First(args ...interface{}) { // ERROR "12-21 lines are
 	}
 }
 
-func (logger *DuplLogger) Second(args ...interface{}) { // ERROR "23-32 lines are duplicate of `testdata/dupl.go:12-21`"
+func (logger *DuplLogger) Second(args ...interface{}) { // ERROR "23-32 lines are duplicate of `testdata/with_issues/dupl.go:12-21`"
 	if logger.level() >= 1 {
 		logger.Info(args...)
 		logger.Info(args...)