fix: reset config in tests (#1516)

* fix: reset config in tests * fix: another reset
testcontainers · Aug 23, 2023 · b914c67 · b914c67
1 parent 864dbb3
commit b914c67
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 0 deletions.
diff --git a/config_test.go b/config_test.go
@@ -17,6 +17,9 @@ func resetTestEnv(t *testing.T) {
 
 func TestReadConfig(t *testing.T) {
 	resetTestEnv(t)
+	t.Cleanup(func() {
+		config.Reset()
+	})
 
 	t.Run("Config is read just once", func(t *testing.T) {
 		t.Setenv("HOME", "")

diff --git a/internal/config/config_test.go b/internal/config/config_test.go
@@ -26,6 +26,10 @@ func TestReadConfig(t *testing.T) {
 	resetTestEnv(t)
 
 	t.Run("Config is read just once", func(t *testing.T) {
+		t.Cleanup(func() {
+			Reset()
+		})
+
 		t.Setenv("HOME", "")
 		t.Setenv("USERPROFILE", "") // Windows support
 		t.Setenv("DOCKER_HOST", "")

diff --git a/logconsumer_test.go b/logconsumer_test.go
@@ -14,6 +14,7 @@ import (
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 
+	"github.com/testcontainers/testcontainers-go/internal/config"
 	"github.com/testcontainers/testcontainers-go/wait"
 )
 
@@ -253,6 +254,10 @@ func Test_StartStop(t *testing.T) {
 }
 
 func TestContainerLogWithErrClosed(t *testing.T) {
+	t.Cleanup(func() {
+		config.Reset()
+	})
+
 	if providerType == ProviderPodman {
 		t.Skip("Docker-in-Docker does not work with rootless Podman")
 	}