modification in the logic and alignment of test according to the curr…

…ent method

metrics-operator/controllers/common/providers/prometheus/common.go

@@ -41,19 +41,12 @@ func (r RoundTripperRetriever) GetRoundTripper(ctx context.Context, provider met
 		return nil, err
 	}
 
-	baseTransport := promapi.DefaultRoundTripper
-
-	if provider.Spec.InsecureSkipTlsVerify {
-		if httpTransport, ok := baseTransport.(*http.Transport); ok {
-			newTransport := httpTransport.Clone()
-			newTransport.TLSClientConfig = &tls.Config{
-				InsecureSkipVerify: true,
-			}
-			baseTransport = newTransport
-		}
+	transport := promapi.DefaultRoundTripper.(*http.Transport).Clone()
+	transport.TLSClientConfig = &tls.Config{
+		InsecureSkipVerify: provider.Spec.InsecureSkipTlsVerify,
 	}
 
-	return config.NewBasicAuthRoundTripper(secret.User, secret.Password, "", "", baseTransport), nil
+	return config.NewBasicAuthRoundTripper(secret.User, secret.Password, "", "", transport), nil
 }
 
 func getPrometheusSecret(ctx context.Context, provider metricsapi.KeptnMetricsProvider, k8sClient client.Client) (*SecretData, error) {

metrics-operator/controllers/common/providers/prometheus/common_test.go

@@ -2,6 +2,7 @@ package prometheus
 
 import (
 	"context"
+	"crypto/tls"
 	"net/http"
 	"net/http/httptest"
 	"reflect"
@@ -137,11 +138,18 @@ func Test_GetRoundtripper(t *testing.T) {
 						Key:      "",
 						Optional: nil,
 					},
+					InsecureSkipTlsVerify: true,
 				},
 			},
 			k8sClient: fake.NewClient(goodsecret),
-			want:      config.NewBasicAuthRoundTripper("myuser", "mytoken", "", "", promapi.DefaultRoundTripper),
-			wantErr:   false,
+			want: func() http.RoundTripper {
+				transport := promapi.DefaultRoundTripper.(*http.Transport).Clone()
+				transport.TLSClientConfig = &tls.Config{
+					InsecureSkipVerify: true,
+				}
+				return config.NewBasicAuthRoundTripper("myuser", "mytoken", "", "", transport)
+			}(),
+			wantErr: false,
 		},
 		{
 			name:      "TestSecretNotDefined",