test: close response body (#1496)

valyala · Feb 15, 2023 · b79a03a · b79a03a
1 parent 9230a3d
commit b79a03a
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -18,4 +18,4 @@ jobs:
       uses: golangci/golangci-lint-action@v3
       with:
         version: v1.51.1
-        args: --enable=nolintlint --enable=gochecknoinits --verbose
+        args: --enable=nolintlint,gochecknoinits,bodyclose --verbose
diff --git a/fasthttputil/inmemory_listener_test.go b/fasthttputil/inmemory_listener_test.go
@@ -146,6 +146,7 @@ func testInmemoryListenerHTTPSingle(t *testing.T, client *http.Client, content s
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)
 	}
+	defer func() { _ = res.Body.Close() }()
 	b, err := io.ReadAll(res.Body)
 	if err != nil {
 		t.Fatalf("unexpected error: %v", err)

diff --git a/header_test.go b/header_test.go
@@ -71,6 +71,7 @@ func TestResponseHeaderMultiLineValue(t *testing.T) {
 	if err != nil {
 		t.Fatalf("parse response using net/http failed, %v", err)
 	}
+	defer func() { _ = response.Body.Close() }()
 
 	if !bytes.Equal(header.StatusMessage(), []byte("SuperOK")) {
 		t.Errorf("parse status line with non-default value failed, got: '%q' want: 'SuperOK'", header.StatusMessage())