From fc5b25aca17b00be789b5c64b0e9ffe993940fae Mon Sep 17 00:00:00 2001
From: Moritz Clasmeier <mclasmeier@redhat.com>
Date: Tue, 24 Mar 2026 14:15:33 +0100
Subject: [PATCH] Fix test, don't delete docker config

---
 internal/dockerauth/dockerauth_test.go | 26 ++++----------------------
 1 file changed, 4 insertions(+), 22 deletions(-)

diff --git a/internal/dockerauth/dockerauth_test.go b/internal/dockerauth/dockerauth_test.go
index aca6be89..939dc29e 100644
--- a/internal/dockerauth/dockerauth_test.go
+++ b/internal/dockerauth/dockerauth_test.go
@@ -12,12 +12,8 @@ import (
 
 func TestGetAndVerifyCredentialsFromEnv(t *testing.T) {
 	// Set environment variables for test
-	os.Setenv("REGISTRY_USERNAME", "user")
-	os.Setenv("REGISTRY_PASSWORD", "pass")
-	defer func() {
-		os.Unsetenv("REGISTRY_USERNAME")
-		os.Unsetenv("REGISTRY_PASSWORD")
-	}()
+	t.Setenv("REGISTRY_USERNAME", "user")
+	t.Setenv("REGISTRY_PASSWORD", "pass")
 
 	log := logger.New()
 	da := New(log)
@@ -87,22 +83,8 @@ func TestGetAndVerifyCredentialsNoCredentials(t *testing.T) {
 	os.Unsetenv("REGISTRY_USERNAME")
 	os.Unsetenv("REGISTRY_PASSWORD")
 
-	// Remove docker config if it exists (temporarily)
-	homeDir, _ := os.UserHomeDir()
-	dockerConfigPath := homeDir + "/.docker/config.json"
-	var backupConfig []byte
-	var configExisted bool
-
-	if data, err := os.ReadFile(dockerConfigPath); err == nil {
-		backupConfig = data
-		configExisted = true
-		os.Remove(dockerConfigPath)
-	}
-	defer func() {
-		if configExisted {
-			os.WriteFile(dockerConfigPath, backupConfig, 0644)
-		}
-	}()
+	// Use a temporary home directory to simulate missing credentials.
+	t.Setenv("HOME", t.TempDir())
 
 	log := logger.New()
 	da := New(log)