From b9e0a59a8bd99ffc2815ee68912c2dc778da49e2 Mon Sep 17 00:00:00 2001
From: Phodal Huang <h@phodal.com>
Date: Fri, 8 Dec 2023 17:16:29 +0800
Subject: [PATCH] fix: fix typos for alla

---
 .../action/checkout/ExecuteGitCheckout.kt     | 21 +++++++++++--------
 .../action/checkout/GitCommandManager.kt      |  2 +-
 .../action/checkout/GitSourceSettings.kt      |  2 +-
 .../action/checkout/GitSourceSettingsTest.kt  |  4 +---
 4 files changed, 15 insertions(+), 14 deletions(-)

diff --git a/pipeline/checkout/src/main/kotlin/org/archguard/action/checkout/ExecuteGitCheckout.kt b/pipeline/checkout/src/main/kotlin/org/archguard/action/checkout/ExecuteGitCheckout.kt
index 4e5a379..97f590c 100644
--- a/pipeline/checkout/src/main/kotlin/org/archguard/action/checkout/ExecuteGitCheckout.kt
+++ b/pipeline/checkout/src/main/kotlin/org/archguard/action/checkout/ExecuteGitCheckout.kt
@@ -4,9 +4,18 @@ import org.archguard.action.checkout.helper.GitAuthHelper
 import org.archguard.action.checkout.helper.RefHelper
 import org.archguard.action.io.FileExt
 import java.io.File
+import kotlin.io.path.Path
 
 fun executeGitCheckout(settings: GitSourceSettings) {
-    val git = GitCommandManager(settings.workdir + File.separator + settings.repositoryPath)
+    val workingDirectory = settings.workdir + File.separator + settings.repositoryPath
+    val git = GitCommandManager(workingDirectory)
+
+    logger.info("Working directory: {}", Path(workingDirectory).toAbsolutePath())
+    if (File(workingDirectory).exists()) {
+        FileExt.rmdir(workingDirectory)
+    }
+
+    FileExt.mkdir(workingDirectory)
 
     doCheckout(git, settings)
 }
@@ -14,12 +23,6 @@ fun executeGitCheckout(settings: GitSourceSettings) {
 fun doCheckout(git: GitCommandManager, settings: GitSourceSettings) {
     val authHelper = GitAuthHelper(git, settings)
 
-    if (File(git.workingDirectory).exists()) {
-        FileExt.rmdir(git.workingDirectory)
-    }
-
-    FileExt.mkdir(git.workingDirectory)
-
     authHelper.configureTempGlobalConfig()
 
     git.config("safe.directory", git.workingDirectory, true, true)
@@ -36,8 +39,8 @@ fun doCheckout(git: GitCommandManager, settings: GitSourceSettings) {
     logger.info("Disabling automatic garbage collection")
     git.tryDisableAutomaticGarbageCollection()
 
-    settings.ref = git.getDefaultBranch(git.workingDirectory)
-    logger.info("Determining default branch for repository: ${git.workingDirectory}, default branch: ${settings.ref}")
+    settings.ref = git.getDefaultBranch(settings.repository)
+    logger.info("Determining default branch for repository: ${settings.repository}, default branch: ${settings.ref}")
 
     val refHelper = RefHelper()
     val refSpec = refHelper.getRefSpecForAllHistory(settings.ref, settings.branch)
diff --git a/pipeline/checkout/src/main/kotlin/org/archguard/action/checkout/GitCommandManager.kt b/pipeline/checkout/src/main/kotlin/org/archguard/action/checkout/GitCommandManager.kt
index 4ab0fb7..e8b35fe 100644
--- a/pipeline/checkout/src/main/kotlin/org/archguard/action/checkout/GitCommandManager.kt
+++ b/pipeline/checkout/src/main/kotlin/org/archguard/action/checkout/GitCommandManager.kt
@@ -20,7 +20,7 @@ class GitCommandManager(var workingDirectory: String = ".", private var lfs: Boo
     private val exec = Command()
 
     fun init(): GitOutput {
-        return execGit(listOf("init", workingDirectory))
+        return execGit(listOf("init", "."))
     }
 
     fun remoteAdd(remoteName: String, remoteUrl: String) {
diff --git a/pipeline/checkout/src/main/kotlin/org/archguard/action/checkout/GitSourceSettings.kt b/pipeline/checkout/src/main/kotlin/org/archguard/action/checkout/GitSourceSettings.kt
index 2c754ea..255c03a 100644
--- a/pipeline/checkout/src/main/kotlin/org/archguard/action/checkout/GitSourceSettings.kt
+++ b/pipeline/checkout/src/main/kotlin/org/archguard/action/checkout/GitSourceSettings.kt
@@ -13,7 +13,7 @@ class GitSourceSettings(
     val serverSide: Boolean = false,
     val submodule: Boolean = false,
     var fetchDepth: Int = 0,
-    var workdir: String = "",
+    var workdir: String = ".tmp",
 ) : CommandSetting {
     val gitServerUrl: String = "https://github.com"
     val nestedSubmodules: Boolean = false
diff --git a/pipeline/checkout/src/test/kotlin/org/archguard/action/checkout/GitSourceSettingsTest.kt b/pipeline/checkout/src/test/kotlin/org/archguard/action/checkout/GitSourceSettingsTest.kt
index e9b585d..f30617c 100644
--- a/pipeline/checkout/src/test/kotlin/org/archguard/action/checkout/GitSourceSettingsTest.kt
+++ b/pipeline/checkout/src/test/kotlin/org/archguard/action/checkout/GitSourceSettingsTest.kt
@@ -12,10 +12,8 @@ class GitSourceSettingsTest {
         val settings = GitSourceSettings(
             repository = repository,
             branch = branch,
-            workdir = ".tmp",
         )
 
-        val commandManager = GitCommandManager(settings.repositoryPath)
-        doCheckout(commandManager, settings)
+        executeGitCheckout(settings)
     }
 }