🐛 internal: fix a possible temporary directory leak

a temporary directory leak was found when running the tests

Signed-off-by: Li Zhijian <[email protected]>

Author: zhijianli88

pkg/internal/testing/integration/internal/process.go

@@ -87,6 +87,13 @@ func DoDefaulting(
 		defaults.URL = *listenURL
 	}
 
+	if path == "" {
+		if name == "" {
+			return DefaultedProcessInput{}, fmt.Errorf("must have at least one of name or path")
+		}
+		defaults.Path = BinPathFinder(name)
+	}
+
 	if dir == "" {
 		newDir, err := ioutil.TempDir("", "k8s_test_framework_")
 		if err != nil {
@@ -96,13 +103,6 @@ func DoDefaulting(
 		defaults.DirNeedsCleaning = true
 	}
 
-	if path == "" {
-		if name == "" {
-			return DefaultedProcessInput{}, fmt.Errorf("must have at least one of name or path")
-		}
-		defaults.Path = BinPathFinder(name)
-	}
-
 	if startTimeout == 0 {
 		defaults.StartTimeout = 20 * time.Second
 	}