chore: fix failing tests

Signed-off-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>

cliv2/internal/httpauth/httpauth.go

@@ -2,6 +2,7 @@ package httpauth
 
 import (
 	"fmt"
+	"io"
 	"log"
 	"net/url"
 )
@@ -55,6 +56,7 @@ func NewHandler(mechanism AuthenticationMechanism) AuthenticationHandlerInterfac
 		spnegoProvider: SpnegoProviderInstance(),
 		Mechanism:      mechanism,
 		state:          Initial,
+		logger:         log.New(io.Discard, "", 0),
 	}
 	return a
 }

cliv2/internal/httpauth/httpauth_test.go

@@ -13,9 +13,7 @@ func Test_DisableAuthentication(t *testing.T) {
 	proxyAddr, _ := url.Parse("http://127.0.0.1")
 	expectedValue := ""
 
-	authHandler := httpauth.AuthenticationHandler{
-		Mechanism: httpauth.NoAuth,
-	}
+	authHandler := httpauth.NewHandler(httpauth.NoAuth)
 
 	actualValue, err := authHandler.GetAuthorizationValue(proxyAddr, "")
 	assert.Nil(t, err)
@@ -29,9 +27,7 @@ func Test_EnabledAuthentication_Mock(t *testing.T) {
 	proxyAddr, _ := url.Parse("http://127.0.0.1")
 	expectedValue := "Mock"
 
-	authHandler := httpauth.AuthenticationHandler{
-		Mechanism: httpauth.Mock,
-	}
+	authHandler := httpauth.NewHandler(httpauth.Mock)
 
 	actualValue, err := authHandler.GetAuthorizationValue(proxyAddr, "")
 	assert.Nil(t, err)
@@ -59,3 +55,20 @@ func Test_AuthenticationMechanismFromAndToString(t *testing.T) {
 	}
 
 }
+
+func Test_LookupSchemeFromAddress(t *testing.T) {
+	defaultValue := "none"
+
+	input := map[string]string{
+		"snyk.io:443":     "https",
+		"snyk.io:80":      "http",
+		"snyk.io:1080":    "socks5",
+		"snyk.io":         defaultValue,
+		"snyk.io:443:das": defaultValue,
+	}
+
+	for addr, expected := range input {
+		actual := httpauth.LookupSchemeFromCannonicalAddress(addr, defaultValue)
+		assert.Equal(t, expected, actual)
+	}
+}

cliv2/internal/proxy/proxy_test.go

@@ -210,20 +210,3 @@ func Test_SetUpstreamProxy(t *testing.T) {
 	assert.Nil(t, err)
 	assert.Equal(t, expectedUrl, actualUrl)
 }
-
-func Test_LookupSchemeFromAddress(t *testing.T) {
-	defaultValue := "none"
-
-	input := map[string]string{
-		"snyk.io:443":     "https",
-		"snyk.io:80":      "http",
-		"snyk.io:1080":    "socks5",
-		"snyk.io":         defaultValue,
-		"snyk.io:443:das": defaultValue,
-	}
-
-	for addr, expected := range input {
-		actual := proxy.LookupSchemeFromCannonicalAddress(addr, defaultValue)
-		assert.Equal(t, expected, actual)
-	}
-}