Skip to content

Commit 5ca56f4

Browse files
committed
Fix linting
Signed-off-by: Xabier Larrakoetxea <me@slok.dev>
1 parent dbd44e6 commit 5ca56f4

File tree

3 files changed

+6
-7
lines changed

3 files changed

+6
-7
lines changed

middleware/iris/example_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ func Example_irisMiddleware() {
2525

2626
// Add our handler and middleware
2727
h := func(ctx iris.Context) {
28-
ctx.WriteString("Hello world")
28+
ctx.WriteString("Hello world") // nolint: errcheck
2929
}
3030
app.Get("/", irismiddleware.Handler("", mdlw), h)
3131

middleware/iris/iris.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import (
99
"github.com/slok/go-http-metrics/middleware"
1010
)
1111

12-
// Handler returns a Gin measuring middleware.
12+
// Handler returns a Iris measuring middleware.
1313
func Handler(handlerID string, m middleware.Middleware) iris.Handler {
1414
return func(ctx iris.Context) {
1515
r := &reporter{ctx: ctx}

middleware/iris/iris_test.go

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ func TestMiddleware(t *testing.T) {
5151
handler: func() iris.Handler {
5252
return func(ctx iris.Context) {
5353
ctx.StatusCode(iris.StatusAccepted)
54-
ctx.WriteString("test1")
54+
_, _ = ctx.WriteString("test1")
5555
}
5656
},
5757
expRespCode: 202,
@@ -82,7 +82,7 @@ func TestMiddleware(t *testing.T) {
8282
handler: func() iris.Handler {
8383
return func(ctx iris.Context) {
8484
ctx.StatusCode(iris.StatusAccepted)
85-
ctx.JSON(map[string]string{"test": "one"})
85+
ctx.JSON(map[string]string{"test": "one"}) // nolint: errcheck
8686
}
8787
},
8888
expRespCode: 202,
@@ -109,9 +109,8 @@ func TestMiddleware(t *testing.T) {
109109

110110
// Make the request.
111111
resp := httptest.NewRecorder()
112-
if err := app.Build(); err != nil {
113-
t.Fatal(err)
114-
}
112+
err := app.Build()
113+
require.NoError(err)
115114
app.ServeHTTP(resp, req)
116115

117116
// Check.

0 commit comments

Comments
 (0)