Add linting and fix a linter warning and a flaky test

Author: mihaitodor

.golangci.yml

@@ -0,0 +1,39 @@
+run:
+  timeout: 30s
+
+issues:
+  max-issues-per-linter: 0
+  max-same-issues: 0
+
+linters-settings:
+  gocritic:
+    enabled-tags:
+      - diagnostic
+      - experimental
+      - opinionated
+      - performance
+      - style
+
+linters:
+  disable-all: true
+  enable:
+    # Default linters reported by golangci-lint help linters` in v1.39.0
+    - deadcode
+    - errcheck
+    - gosimple
+    - govet
+    - gosimple
+    - ineffassign
+    - staticcheck
+    - structcheck
+    - stylecheck
+    - typecheck
+    - unused
+    - varcheck
+    # Extra linters:
+    - gofmt
+    - goimports
+    - gocritic
+    - revive
+    - bodyclose
+    - gosec

tunny_test.go

@@ -187,7 +187,7 @@ func TestTimedJobsAfterClose(t *testing.T) {
 	})
 	pool.Close()
 
-	_, act := pool.ProcessTimed(10, time.Duration(1))
+	_, act := pool.ProcessTimed(10, time.Duration(10*time.Millisecond))
 	if exp := ErrPoolNotRunning; exp != act {
 		t.Errorf("Wrong error returned: %v != %v", act, exp)
 	}

worker.go

@@ -104,7 +104,7 @@ func (w *workerWrapper) run() {
 				case <-w.interruptChan:
 					w.interruptChan = make(chan struct{})
 				}
-			case _, _ = <-w.interruptChan:
+			case <-w.interruptChan:
 				w.interruptChan = make(chan struct{})
 			}
 		case <-w.closeChan: