Merge pull request #194 from Tantalor93/errortests

add more cases to cover IO errors
Tantalor93 · Nov 21, 2023 · e6ec30d · e6ec30d
2 parents dffa323 + 64df9a9
commit e6ec30d
Show file tree

Hide file tree

Showing 5 changed files with 269 additions and 116 deletions.
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -13,8 +13,9 @@ jobs:
     - name: Install Go
       uses: actions/setup-go@v4
       with:
+        cache: false
         go-version-file: go.mod
     - name: Lint
       uses: golangci/golangci-lint-action@v3
       with:
-        version: v1.51.2
+        version: v1.55.2
diff --git a/cmd/benchmark.go b/cmd/benchmark.go
@@ -451,14 +451,10 @@ func (b *Benchmark) Run(ctx context.Context) ([]*ResultStats, error) {
 						start := time.Now()
 
 						reqTimeoutCtx, cancel := context.WithTimeout(ctx, b.RequestTimeout)
-						if resp, err = query(reqTimeoutCtx, b.Server, &m); err != nil {
-							cancel()
-							st.Counters.IOError++
-							st.Errors = append(st.Errors, err)
-						} else {
-							cancel()
-							st.record(&m, resp, start, time.Since(start))
-						}
+						resp, err = query(reqTimeoutCtx, b.Server, &m)
+						cancel()
+						st.record(&m, resp, err, start, time.Since(start))
+
 						if incrementBar {
 							bar.Add(1)
 						}