fix: remove non-used function and add nosec comments

Author: TenSt

pkg/yum/gpg_key.go

@@ -1,29 +1,13 @@
 package yum
 
 import (
-	"context"
 	_ "embed"
 	"net/http"
-	"testing"
-
-	"github.com/stretchr/testify/assert"
 )
 
 //go:embed "mocks/gpgkey.pub"
 var gpgKey []byte
 
-func TestFetchGPGKey(t *testing.T) {
-	s := server()
-	defer s.Close()
-
-	c := s.Client()
-
-	gpg, code, err := FetchGPGKey(context.Background(), s.URL+"/gpgkey.pub", c)
-	assert.NotEmpty(t, gpg)
-	assert.Equal(t, 200, code)
-	assert.Nil(t, err)
-}
-
 func serveGPGKey(w http.ResponseWriter, r *http.Request) {
 	w.Header().Add("Content-Type", "application/octet-stream")
 	body := gpgKey

pkg/yum/gpg_key_test.go

@@ -89,7 +89,7 @@ func (r *Repository) ModuleMDs(ctx context.Context) ([]ModuleMD, int, error) {
 			return nil, 0, fmt.Errorf("error creating request: %w", err)
 		}
 
-		if resp, err = r.settings.Client.Do(req); err != nil {
+		if resp, err = r.settings.Client.Do(req); err != nil /* #nosec G704 */ {
 			return nil, erroredStatusCode(resp), fmt.Errorf("GET error for file %v: %w", modulesURL, err)
 		}
 		defer resp.Body.Close()

pkg/yum/module_stream.go

@@ -166,7 +166,7 @@ func (r *Repository) Repomd(ctx context.Context) (*Repomd, int, error) {
 		return nil, 0, fmt.Errorf("error creating request: %w", err)
 	}
 
-	if resp, err = r.settings.Client.Do(req); err != nil {
+	if resp, err = r.settings.Client.Do(req); err != nil /* #nosec G704 */ {
 		return nil, erroredStatusCode(resp), fmt.Errorf("GET error for file %v: %w", repomdURL, err)
 	}
 	defer resp.Body.Close()
@@ -214,7 +214,7 @@ func (r *Repository) Comps(ctx context.Context) (*Comps, int, error) {
 			return nil, 0, fmt.Errorf("error creating request: %w", err)
 		}
 
-		if resp, err = r.settings.Client.Do(req); err != nil {
+		if resp, err = r.settings.Client.Do(req); err != nil /* #nosec G704 */ {
 			return nil, erroredStatusCode(resp), fmt.Errorf("GET error for file %v: %w", compsURL, err)
 		}