fix: Use more defer function

disk.go

@@ -28,22 +28,22 @@ func newDiskspaceService(s *Service) *DiskspaceService {
 // Get return Radarr disk space info
 func (s *DiskspaceService) Get() (*Diskspaces, error) {
 	diskspaceURL := fmt.Sprintf("%s/api%s", s.s.url, diskspaceURI)
-	response, err := s.s.client.Get(diskspaceURL)
+	resp, err := s.s.client.Get(diskspaceURL)
 	if err != nil {
 		return nil, err
 	}
+	defer resp.Body.Close()
 
-	err = parseRadarrResponse(response)
+	err = parseRadarrResponse(resp)
 	if err != nil {
 		return nil, err
 	}
 
 	var diskspaces Diskspaces
-	err = json.NewDecoder(response.Body).Decode(&diskspaces)
+	err = json.NewDecoder(resp.Body).Decode(&diskspaces)
 	if err != nil {
 		return nil, err
 	}
 
-	_ = response.Body.Close()
 	return &diskspaces, nil
 }

dummy_test.go

@@ -26,6 +26,7 @@ func helperLoadBytes(name string) []byte {
 	if err != nil {
 		panic(err)
 	}
+
 	return bytes
 }
 

history.go

@@ -125,19 +125,22 @@ func (s *HistoryService) paginate(page int) (*History, error) {
 	q.Add("pageSize", "50")
 
 	request.URL.RawQuery = q.Encode()
-	response, err := s.s.client.Do(request)
+	resp, err := s.s.client.Do(request)
 	if err != nil {
 		return nil, err
 	}
-	err = parseRadarrResponse(response)
+	defer resp.Body.Close()
+
+	err = parseRadarrResponse(resp)
 	if err != nil {
 		return nil, err
 	}
 
 	var history History
-	err = json.NewDecoder(response.Body).Decode(&history)
+	err = json.NewDecoder(resp.Body).Decode(&history)
 	if err != nil {
 		return nil, err
 	}
+
 	return &history, nil
 }

lookup.go

@@ -39,6 +39,7 @@ func (l *LookupService) Plain(term string) (Movies, error) {
 	if err != nil {
 		return nil, err
 	}
+	defer resp.Body.Close()
 
 	err = parseRadarrResponse(resp)
 	if err != nil {
@@ -51,7 +52,6 @@ func (l *LookupService) Plain(term string) (Movies, error) {
 		return nil, err
 	}
 
-	_ = resp.Body.Close()
 	return movies, nil
 }
 
@@ -72,6 +72,7 @@ func (l *LookupService) Tmdb(TMDBID int) (*Movie, error) {
 	if err != nil {
 		return nil, err
 	}
+	defer resp.Body.Close()
 
 	err = parseRadarrResponse(resp)
 	if err != nil {
@@ -84,7 +85,6 @@ func (l *LookupService) Tmdb(TMDBID int) (*Movie, error) {
 		return nil, err
 	}
 
-	_ = resp.Body.Close()
 	return &movie, nil
 }
 
@@ -110,6 +110,7 @@ func (l *LookupService) Imdb(IMDBID string) (*Movie, error) {
 	if err != nil {
 		return nil, err
 	}
+	defer resp.Body.Close()
 
 	err = parseRadarrResponse(resp)
 	if err != nil {
@@ -122,6 +123,5 @@ func (l *LookupService) Imdb(IMDBID string) (*Movie, error) {
 		return nil, err
 	}
 
-	_ = resp.Body.Close()
 	return &movie, nil
 }

movie.go

@@ -167,47 +167,47 @@ type UpcomingOptions struct {
 // https://github.com/Radarr/Radarr/wiki/API:Movie#getid
 func (m *MovieService) Get(movieID int) (*Movie, error) {
 	movieURL := fmt.Sprintf("%s/api%s/%d", m.s.url, movieURI, movieID)
-	response, err := m.s.client.Get(movieURL)
+	resp, err := m.s.client.Get(movieURL)
 	if err != nil {
 		return nil, err
 	}
+	defer resp.Body.Close()
 
-	err = parseRadarrResponse(response)
+	err = parseRadarrResponse(resp)
 	if err != nil {
 		return nil, err
 	}
 
 	var movie Movie
-	err = json.NewDecoder(response.Body).Decode(&movie)
+	err = json.NewDecoder(resp.Body).Decode(&movie)
 	if err != nil {
 		return nil, err
 	}
 
-	_ = response.Body.Close()
 	return &movie, nil
 }
 
 // List Returns the movie with the matching ID or eerror if no matching movie is found
 // https://github.com/Radarr/Radarr/wiki/API:Movie#get
 func (m *MovieService) List() (Movies, error) {
 	moviesURL := fmt.Sprintf("%s/api%s", m.s.url, movieURI)
-	response, err := m.s.client.Get(moviesURL)
+	resp, err := m.s.client.Get(moviesURL)
 	if err != nil {
 		return nil, err
 	}
+	defer resp.Body.Close()
 
-	err = parseRadarrResponse(response)
+	err = parseRadarrResponse(resp)
 	if err != nil {
 		return nil, err
 	}
 
 	var movies Movies
-	err = json.NewDecoder(response.Body).Decode(&movies)
+	err = json.NewDecoder(resp.Body).Decode(&movies)
 	if err != nil {
 		return nil, err
 	}
 
-	_ = response.Body.Close()
 	return movies, nil
 }
 
@@ -245,23 +245,23 @@ func (m *MovieService) Upcoming(opts ...*UpcomingOptions) (Movies, error) {
 		}
 	}
 
-	response, err := m.s.client.Do(req)
+	resp, err := m.s.client.Do(req)
 	if err != nil {
 		return nil, err
 	}
+	defer resp.Body.Close()
 
-	err = parseRadarrResponse(response)
+	err = parseRadarrResponse(resp)
 	if err != nil {
 		return nil, err
 	}
 
 	var movies Movies
-	err = json.NewDecoder(response.Body).Decode(&movies)
+	err = json.NewDecoder(resp.Body).Decode(&movies)
 	if err != nil {
 		return nil, err
 	}
 
-	_ = response.Body.Close()
 	return movies, nil
 }
 
@@ -286,9 +286,9 @@ func (m *MovieService) Delete(movie *Movie, opts ...*DeleteMovieOptions) error {
 	if err != nil {
 		return err
 	}
+	defer resp.Body.Close()
 
 	err = parseRadarrResponse(resp)
-	_ = resp.Body.Close()
 	return err
 }
 
@@ -299,6 +299,7 @@ func (m *MovieService) Excluded() (ExcludedMovies, error) {
 	if err != nil {
 		return nil, err
 	}
+	defer resp.Body.Close()
 
 	err = parseRadarrResponse(resp)
 	if err != nil {
@@ -311,6 +312,5 @@ func (m *MovieService) Excluded() (ExcludedMovies, error) {
 		return nil, err
 	}
 
-	_ = resp.Body.Close()
 	return movies, nil
 }

status.go

@@ -44,22 +44,22 @@ func newSystemStatusService(s *Service) *SystemStatusService {
 // Get https://github.com/Radarr/Radarr/wiki/API:System-Status#get
 func (s *SystemStatusService) Get() (*SystemStatus, error) {
 	statusURL := fmt.Sprintf("%s/api%s", s.s.url, statusURI)
-	response, err := s.s.client.Get(statusURL)
+	resp, err := s.s.client.Get(statusURL)
 	if err != nil {
 		return nil, err
 	}
+	defer resp.Body.Close()
 
-	err = parseRadarrResponse(response)
+	err = parseRadarrResponse(resp)
 	if err != nil {
 		return nil, err
 	}
 
 	var status SystemStatus
-	err = json.NewDecoder(response.Body).Decode(&status)
+	err = json.NewDecoder(resp.Body).Decode(&status)
 	if err != nil {
 		return nil, err
 	}
 
-	_ = response.Body.Close()
 	return &status, nil
 }