Skip to content

Commit 5ccfd17

Browse files
authored
ci(deps): fix ineffassign and bodyclose in ".*_test.go$" (#6777)
Signed-off-by: Matthieu MOREL <[email protected]>
1 parent 03fc534 commit 5ccfd17

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

.golangci.yaml

-2
Original file line numberDiff line numberDiff line change
@@ -109,10 +109,8 @@ issues:
109109
exclude-rules:
110110
- path: ".*_test.go$"
111111
linters:
112-
- bodyclose
113112
- goconst
114113
- gosec
115-
- ineffassign
116114
- misspell
117115
- unused
118116
- path: ".*_test.go$"

pkg/fanal/analyzer/executable/executable_test.go

+1
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ func Test_executableAnalyzer_Analyze(t *testing.T) {
4747
Content: f,
4848
Info: stat,
4949
})
50+
require.NoError(t, err)
5051
assert.Equal(t, tt.want, got)
5152
})
5253
}

pkg/rpc/server/listen_test.go

+4-3
Original file line numberDiff line numberDiff line change
@@ -262,18 +262,19 @@ func Test_newServeMux(t *testing.T) {
262262
url := ts.URL + tt.path
263263
if tt.header == nil {
264264
resp, err = http.Get(url)
265+
require.NoError(t, err)
266+
defer resp.Body.Close()
265267
} else {
266268
req, err := http.NewRequest(http.MethodPost, url, http.NoBody)
267269
require.NoError(t, err)
268270

269271
req.Header = tt.header
270272
client := new(http.Client)
271273
resp, err = client.Do(req)
274+
require.NoError(t, err)
275+
defer resp.Body.Close()
272276
}
273-
274-
require.NoError(t, err)
275277
assert.Equal(t, tt.want, resp.StatusCode)
276-
defer resp.Body.Close()
277278
})
278279
}
279280
}

0 commit comments

Comments
 (0)