test: removed special case for handling testing dirs on windows

golang/go#71544 is still officially open.
However a fix has been committed to go codebase.

This issue is not really blocking: the problem lies on windows not
reliaby cleaning up the temporary directory.

Signed-off-by: Frédéric BIDON <fredbi@yahoo.com>

Author: fredbi

internal/antest/helpers_test.go

@@ -5,7 +5,6 @@ package antest
 
 import (
 	"os"
-	"runtime"
 	"testing"
 
 	"github.com/go-openapi/testify/v2/require"
@@ -50,6 +49,7 @@ func TestLoadSpecErrorCases(t *testing.T) {
 
 func prepareBadDoc(t testing.TB, kind string, invalidFormat bool) (string, func()) {
 	t.Helper()
+	folder := t.TempDir()
 
 	var (
 		file string
@@ -58,9 +58,10 @@ func prepareBadDoc(t testing.TB, kind string, invalidFormat bool) (string, func(
 
 	switch kind {
 	case "yaml", "yml":
-		f, err := os.CreateTemp(workaroundTempDir(t)(), "*.yaml")
+		f, err := os.CreateTemp(folder, "*.yaml")
 		require.NoError(t, err)
 		file = f.Name()
+		f.Close()
 
 		if invalidFormat {
 			data = []byte(`--
@@ -76,9 +77,10 @@ info:
 		}
 
 	case "json":
-		f, err := os.CreateTemp(workaroundTempDir(t)(), "*.json")
+		f, err := os.CreateTemp(folder, "*.json")
 		require.NoError(t, err)
 		file = f.Name()
+		f.Close()
 
 		if invalidFormat {
 			data = []byte(`{
@@ -104,16 +106,3 @@ info:
 
 	return file, func() {}
 }
-
-func workaroundTempDir(t testing.TB) func() string {
-	// Workaround for go testing bug on Windows: https://github.com/golang/go/issues/71544
-	// On windows, t.TempDir() doesn't properly release file handles yet,
-	// se we just leave it unchecked (no cleanup would take place).
-	if runtime.GOOS == "windows" {
-		return func() string {
-			return ""
-		}
-	}
-
-	return t.TempDir
-}

internal/debug/debug.go

@@ -11,11 +11,9 @@ import (
 	"runtime"
 )
 
-var (
-	output = os.Stdout
-)
+var output = os.Stdout //nolint:gochecknoglobals // this is on purpose to be overridable during tests
 
-// GetLogger provides a prefix debug logger
+// GetLogger provides a prefix debug logger.
 func GetLogger(prefix string, debug bool) func(string, ...any) {
 	if debug {
 		logger := log.New(output, prefix+":", log.LstdFlags)

internal/debug/debug_test.go

@@ -5,52 +5,56 @@ package debug
 
 import (
 	"os"
-	"runtime"
 	"testing"
 
 	"github.com/go-openapi/testify/v2/assert"
 	"github.com/go-openapi/testify/v2/require"
 )
 
 func TestDebug(t *testing.T) {
-	tmpFile, err := os.CreateTemp(workaroundTempDir(t)(), "debug-test")
+	folder := t.TempDir()
+
+	tmpFile, err := os.CreateTemp(folder, "debug-test")
 	require.NoError(t, err)
+	tmpFileClosed := false
+	defer func() {
+		if tmpFileClosed {
+			return
+		}
+		_ = tmpFile.Close()
+	}()
 
 	output = tmpFile
 	tmpName := tmpFile.Name()
 	testLogger := GetLogger("test", true)
 
 	testLogger("A debug: %s", "a string")
 	tmpFile.Close()
+	tmpFileClosed = true
 
 	flushed, err := os.ReadFile(tmpName)
 	require.NoError(t, err)
 
 	assert.Contains(t, string(flushed), "A debug: a string")
 
-	tmpEmptyFile, err := os.CreateTemp(workaroundTempDir(t)(), "debug-test")
+	tmpEmptyFile, err := os.CreateTemp(folder, "debug-empty-test")
 	require.NoError(t, err)
+	tmpEmptyFileClosed := false
+	defer func() {
+		if tmpEmptyFileClosed {
+			return
+		}
+		_ = tmpEmptyFile.Close()
+	}()
 	tmpEmpty := tmpEmptyFile.Name()
 	testLogger = GetLogger("test", false)
 
 	testLogger("A debug: %s", "a string")
-	tmpFile.Close()
+	tmpEmptyFile.Close()
+	tmpEmptyFileClosed = true
 
 	flushed, err = os.ReadFile(tmpEmpty)
 	require.NoError(t, err)
 
 	assert.Empty(t, flushed)
 }
-
-func workaroundTempDir(t testing.TB) func() string {
-	// Workaround for go testing bug on Windows: https://github.com/golang/go/issues/71544
-	// On windows, t.TempDir() doesn't properly release file handles yet,
-	// se we just leave it unchecked (no cleanup would take place).
-	if runtime.GOOS == "windows" {
-		return func() string {
-			return ""
-		}
-	}
-
-	return t.TempDir
-}