Skip to content

Commit

Permalink
fix: golangci-lint run ./...
Browse files Browse the repository at this point in the history
  • Loading branch information
peaceiris committed Aug 28, 2021
1 parent 9aa874c commit 816cb02
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 4 deletions.
16 changes: 13 additions & 3 deletions cmd/benchmark_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ package cmd
import (
"bytes"
"io"
"log"
"testing"
"time"
)
Expand All @@ -24,7 +25,10 @@ func BenchmarkCopy(b *testing.B) {
b.ResetTimer()

for i := 0; i < b.N; i++ {
CopyTime(buf, lr, time.Now().Add(-50*time.Millisecond))
_, err := CopyTime(buf, lr, time.Now().Add(-50*time.Millisecond))
if err != nil {
log.Fatal(err)
}
buf.Reset()
rd.Reset(bs)
}
Expand Down Expand Up @@ -60,7 +64,10 @@ func BenchmarkWriter(b *testing.B) {
b.ResetTimer()

for i := 0; i < b.N; i++ {
io.Copy(w, lr)
_, err := io.Copy(w, lr)
if err != nil {
log.Fatal(err)
}
buf.Reset()
rd.Reset(bs)
}
Expand All @@ -83,7 +90,10 @@ func BenchmarkWriterBig(b *testing.B) {
b.ResetTimer()

for i := 0; i < b.N; i++ {
io.Copy(w, rd)
_, err := io.Copy(w, rd)
if err != nil {
log.Fatal(err)
}
buf.Reset()
rd.Reset(bs)
}
Expand Down
5 changes: 4 additions & 1 deletion cmd/version_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ package cmd
import (
"bytes"
"fmt"
"log"
"runtime"
"strings"
"testing"
Expand Down Expand Up @@ -33,7 +34,9 @@ func TestBuildEnvsString(t *testing.T) {
cmdArgs := strings.Split(command, " ")
fmt.Printf("cmdArgs %+v\n", cmdArgs)
cmd.SetArgs(cmdArgs[1:])
cmd.Execute()
if err := cmd.Execute(); err != nil {
log.Fatal(err)
}
out := buf.String()

assert.Equal(t, want, out, "they should be equal")
Expand Down

0 comments on commit 816cb02

Please sign in to comment.