Merge pull request #6 from bozd4g/bug/context-canceling-problem

Update unmarshal and tests

response.go

@@ -2,7 +2,6 @@ package gohttpclient
 
 import (
 	"encoding/json"
-	"io/ioutil"
 	"net/http"
 )
 
@@ -18,13 +17,7 @@ func (r *Response) Body() []byte {
 }
 
 func (r *Response) Unmarshal(v any) error {
-	defer r.res.Body.Close()
-	body, err := ioutil.ReadAll(r.res.Body)
-	if err != nil {
-		return err
-	}
-
-	return json.Unmarshal(body, &v)
+	return json.Unmarshal(r.body, &v)
 }
 
 func (r *Response) Status() int {

response_test.go

@@ -3,29 +3,13 @@ package gohttpclient
 import (
 	"bytes"
 	"context"
-	"fmt"
 	"io/ioutil"
 	"net/http"
 	"testing"
 
-	"github.com/stretchr/testify/mock"
 	"github.com/stretchr/testify/suite"
 )
 
-type mockReadCloser struct {
-	mock.Mock
-}
-
-func (m *mockReadCloser) Read(p []byte) (n int, err error) {
-	args := m.Called(p)
-	return args.Int(0), args.Error(1)
-}
-
-func (m *mockReadCloser) Close() error {
-	args := m.Called()
-	return args.Error(0)
-}
-
 type TestResponseSuite struct {
 	suite.Suite
 	ctx context.Context
@@ -56,13 +40,10 @@ func (s *TestResponseSuite) Test_Body_ShouldRunSuccesfully() {
 func (s *TestResponseSuite) Test_Unmarshal_ShouldRunSuccesfully() {
 	// Arrange
 	body := []byte(`{"name":"test"}`)
+	res := &http.Response{}
 
 	// Act
-	resp := Response{
-		res: &http.Response{
-			Body: ioutil.NopCloser(bytes.NewBuffer(body)),
-		},
-	}
+	resp := Response{res, body}
 
 	// Assert
 	var response map[string]interface{}
@@ -73,14 +54,8 @@ func (s *TestResponseSuite) Test_Unmarshal_ShouldRunSuccesfully() {
 
 func (s *TestResponseSuite) Test_Unmarshal_WhenBodyIsWrong_ShouldReturnError() {
 	// Arrange
-	mockReadCloser := mockReadCloser{}
-	mockReadCloser.On("Read", mock.AnythingOfType("[]uint8")).Return(0, fmt.Errorf("error reading"))
-	mockReadCloser.On("Close").Return(fmt.Errorf("error closing"))
-
 	resp := Response{
-		res: &http.Response{
-			Body: &mockReadCloser,
-		},
+		body: nil,
 	}
 
 	// Act
@@ -94,13 +69,10 @@ func (s *TestResponseSuite) Test_Unmarshal_WhenBodyIsWrong_ShouldReturnError() {
 func (s *TestResponseSuite) Test_Unmarshal_WhenUnMarshalReturnsError_ShouldReturnError() {
 	// Arrange
 	body := []byte(`{"name":"test"`)
+	res := &http.Response{}
 
 	// Act
-	resp := Response{
-		res: &http.Response{
-			Body: ioutil.NopCloser(bytes.NewBuffer(body)),
-		},
-	}
+	resp := Response{res, body}
 
 	// Assert
 	var response map[string]interface{}