update (#27)

.github/workflows/golanglint-ci.yml

@@ -11,8 +11,8 @@ jobs:
     name: lint
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v2
+        uses: golangci/golangci-lint-action@v3
         with:
-          version: v1.45
+          version: v1.49

.github/workflows/test.yml

@@ -15,14 +15,14 @@ jobs:
       - name: Setup Go
         uses: actions/setup-go@v2
         with:
-          go-version: 1.17
+          go-version: 1.19
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
       - name: Test
         run: go test -v ./... -coverprofile cover.out
 
       - name: Convert coverage to lcov
-        uses: jandelgado/gcov2lcov-action@v1.0.8
+        uses: jandelgado/gcov2lcov-action@v1.0.9
         if: github.ref == 'refs/heads/master'
         with:
           infile: cover.out

.golangci.yml

@@ -1,6 +1,33 @@
 linters:
-  enable:
-    - gofmt
-    - goimports
+  enable-all: true
+  disable:
+    - golint # WARN [runner] The linter 'golint' is deprecated (since v1.41.0) due to: The repository of the linter has been archived by the owner.  Replaced by revive.
+    - maligned # WARN [runner] The linter 'maligned' is deprecated (since v1.38.0) due to: The repository of the linter has been archived by the owner.  Replaced by govet 'fieldalignment'.
+    - interfacer # WARN [runner] The linter 'interfacer' is deprecated (since v1.38.0) due to: The repository of the linter has been archived by the owner.
+    - scopelint # WARN [runner] The linter 'scopelint' is deprecated (since v1.39.0) due to: The repository of the linter has been deprecated by the owner.  Replaced by exportloopref.
+    - tagliatelle
+    - wsl
+    - goerr113
+    - nlreturn
+    - lll
+    - godot
+    - exhaustivestruct
+    - exhaustruct
+    - varnamelen
+    - ireturn
+    - dupl
+    - wrapcheck
+    - testpackage
+    - godox
+    - nestif
+    - funlen
+    - gocognit
+    - gocyclo
+    - gomnd
+    - nonamedreturns
+    - gochecknoglobals
+    - revive
+    - stylecheck
+    - gocritic
 run:
   timeout: 3m

go.mod

@@ -1,6 +1,6 @@
 module github.com/hirosassa/zerodriver
 
-go 1.17
+go 1.19
 
 require (
 	bou.ke/monkey v1.0.2

http_test.go

@@ -2,7 +2,7 @@ package zerodriver
 
 import (
 	"bytes"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"net/url"
 	"strings"
@@ -53,7 +53,7 @@ func TestHTTP(t *testing.T) {
 func TestNewHTTP(t *testing.T) {
 	t.Parallel()
 
-	var tests = map[string]struct {
+	tests := map[string]struct {
 		req  *http.Request
 		res  *http.Response
 		want *HTTPPayload
@@ -69,20 +69,22 @@ func TestNewHTTP(t *testing.T) {
 			&HTTPPayload{RequestURL: "https://example.com"},
 		},
 		"RequestSize": {
-			&http.Request{ContentLength: 5, Body: ioutil.NopCloser(strings.NewReader("12345"))},
+			&http.Request{ContentLength: 5, Body: io.NopCloser(strings.NewReader("12345"))},
 			nil,
 			&HTTPPayload{RequestSize: "5"},
 		},
 
 		"ResponseSize": {
 			nil,
-			&http.Response{ContentLength: 5, Body: ioutil.NopCloser(strings.NewReader("12345"))},
+			&http.Response{ContentLength: 5, Body: io.NopCloser(strings.NewReader("12345"))},
 			&HTTPPayload{ResponseSize: "5"},
 		},
 	}
 
 	for name, tt := range tests {
+		tt := tt
 		t.Run(name, func(t *testing.T) {
+			t.Parallel()
 			if diff := cmp.Diff(tt.want, NewHTTP(tt.req, tt.res)); diff != "" {
 				t.Errorf("HTTPPayload differs (-got +want)\n%s", diff)
 			}
@@ -92,7 +94,7 @@ func TestNewHTTP(t *testing.T) {
 
 func TestMakeLatency(t *testing.T) {
 	t.Parallel()
-	var tests = map[string]struct {
+	tests := map[string]struct {
 		d     time.Duration
 		isGKE bool
 		want  Latency
@@ -110,17 +112,18 @@ func TestMakeLatency(t *testing.T) {
 	}
 
 	for name, tt := range tests {
+		tt := tt
 		t.Run(name, func(t *testing.T) {
+			t.Parallel()
 			assert.Equal(t, tt.want, MakeLatency(tt.d, tt.isGKE))
 		})
 	}
-
 }
 
 func TestMakeGAELatency(t *testing.T) {
 	t.Parallel()
 
-	var tests = map[string]struct {
+	tests := map[string]struct {
 		d    time.Duration
 		want GAELatency
 	}{
@@ -139,7 +142,9 @@ func TestMakeGAELatency(t *testing.T) {
 	}
 
 	for name, tt := range tests {
+		tt := tt
 		t.Run(name, func(t *testing.T) {
+			t.Parallel()
 			assert.Equal(t, tt.want, makeGAELatency(tt.d))
 		})
 	}
@@ -148,7 +153,7 @@ func TestMakeGAELatency(t *testing.T) {
 func TestRemoteIP(t *testing.T) {
 	t.Parallel()
 
-	var tests = map[string]struct {
+	tests := map[string]struct {
 		req  *http.Request
 		want string
 	}{
@@ -172,7 +177,9 @@ func TestRemoteIP(t *testing.T) {
 		},
 	}
 	for name, tt := range tests {
+		tt := tt
 		t.Run(name, func(t *testing.T) {
+			t.Parallel()
 			assert.Equal(t, tt.want, remoteIP(tt.req))
 		})
 	}

logger_test.go

@@ -13,11 +13,13 @@ import (
 )
 
 func TestNewProduction(t *testing.T) {
+	t.Parallel()
 	logger := NewProductionLogger()
 	assert.IsType(t, &Logger{}, logger)
 }
 
 func TestNewDevelopment(t *testing.T) {
+	t.Parallel()
 	logger := NewDevelopmentLogger()
 	assert.IsType(t, &Logger{}, logger)
 }
@@ -66,7 +68,9 @@ func TestLoggers(t *testing.T) {
 		},
 	}
 	for name, tt := range tests {
+		tt := tt
 		t.Run(name, func(t *testing.T) {
+			t.Parallel()
 			assert.Equal(t, tt.want, tt.res)
 		})
 	}
@@ -124,6 +128,8 @@ func TestPanic(t *testing.T) {
 }
 
 func TestPrint(t *testing.T) {
+	t.Parallel()
+
 	// replace writer
 	log := NewDevelopmentLogger()
 	out := &bytes.Buffer{}
@@ -141,6 +147,8 @@ func TestPrint(t *testing.T) {
 }
 
 func TestPrintf(t *testing.T) {
+	t.Parallel()
+
 	// replace writer
 	log := NewDevelopmentLogger()
 	out := &bytes.Buffer{}