Skip to content

Commit 16e596c

Browse files
Copilotgaby
andcommitted
Add nolint directives for benchmark error handling
Co-authored-by: gaby <835733+gaby@users.noreply.github.com>
1 parent c037939 commit 16e596c

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

router_test.go

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2046,7 +2046,7 @@ func Benchmark_App_MethodNotAllowed_Parallel(b *testing.B) {
20462046
appHandler(c)
20472047
}
20482048
})
2049-
2049+
20502050
// Single-threaded verification on a fresh context to preserve correctness checks
20512051
verifyCtx := &fasthttp.RequestCtx{}
20522052
verifyCtx.Request.Header.SetMethod("DELETE")
@@ -2166,6 +2166,7 @@ func Benchmark_Router_Next_Parallel(b *testing.B) {
21662166
c := acquireDefaultCtxForRouterBenchmark(b, app, request)
21672167
for pb.Next() {
21682168
c.indexRoute = -1
2169+
//nolint:errcheck // Benchmark hot path - error checked in verification
21692170
_, _ = app.next(c)
21702171
}
21712172
})
@@ -2212,7 +2213,7 @@ func Benchmark_Route_Match_Parallel(b *testing.B) {
22122213
_ = route.match("/user/keys/1337", "/user/keys/1337", &params)
22132214
}
22142215
})
2215-
2216+
22162217
// Single-threaded verification to preserve correctness checks
22172218
var verifyParams [maxParams]string
22182219
match := route.match("/user/keys/1337", "/user/keys/1337", &verifyParams)
@@ -2316,11 +2317,12 @@ func Benchmark_Router_GitHub_API_Parallel(b *testing.B) {
23162317
for pb.Next() {
23172318
c.URI().SetPath(routesFixture.TestRoutes[i].Path)
23182319
ctx := acquireDefaultCtxForRouterBenchmark(b, app, c)
2320+
//nolint:errcheck // Benchmark hot path - error checked in verification
23192321
_, _ = app.next(ctx)
23202322
app.ReleaseCtx(ctx)
23212323
}
23222324
})
2323-
2325+
23242326
// Single-threaded verification on a fresh context to preserve correctness checks
23252327
verifyC := &fasthttp.RequestCtx{}
23262328
verifyC.Request.Header.SetMethod(routesFixture.TestRoutes[i].Method)

0 commit comments

Comments
 (0)