From 72110e5c4427284f09f03701280f37c60ae31574 Mon Sep 17 00:00:00 2001 From: Phodal Huang Date: Fri, 8 Dec 2023 15:49:40 +0800 Subject: [PATCH] fix: update for config --- pipeline/checkout/.gitignore | 1 + .../org/archguard/action/checkout/GitCommandManager.kt | 3 ++- .../org/archguard/action/checkout/GitSourceSettings.kt | 4 ++-- .../archguard/action/checkout/GitSourceSettingsTest.kt | 8 ++++++-- 4 files changed, 11 insertions(+), 5 deletions(-) create mode 100644 pipeline/checkout/.gitignore diff --git a/pipeline/checkout/.gitignore b/pipeline/checkout/.gitignore new file mode 100644 index 0000000..d36977d --- /dev/null +++ b/pipeline/checkout/.gitignore @@ -0,0 +1 @@ +.tmp 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 828dab5..3d2b98c 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", ".")) + return execGit(listOf("init", workingDirectory)) } fun remoteAdd(remoteName: String, remoteUrl: String) { @@ -208,6 +208,7 @@ class GitCommandManager(var workingDirectory: String = ".", private var lfs: Boo } else { listOf("log", "-1") } + val silent = format != null return execGit(args, false, silent) } 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 2b18fc0..d28adfb 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 @@ -2,6 +2,7 @@ package org.archguard.action.checkout import org.archguard.action.exec.CommandArgs import org.archguard.action.exec.CommandSetting +import java.io.File.separator class GitSourceSettings( val repository: String, @@ -17,8 +18,7 @@ class GitSourceSettings( val gitServerUrl: String = "https://github.com" val nestedSubmodules: Boolean = false val commit: String = "" - val repositoryPath: String get() = repository.substringAfterLast("/") - + var repositoryPath = workdir + separator + repository.substringAfterLast("/") val persistCredentials: Boolean = false val sshStrict: Boolean = false val sshKnownHosts: String = "" 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 ef57928..f38d8f3 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 @@ -10,8 +10,12 @@ class GitSourceSettingsTest { val branch = "master" val settings = GitSourceSettings.fromArgs(arrayOf("--repository", repository, "--branch", branch)) + settings.repositoryPath = ".tmp/codedb" - settings.repository shouldBe repository - settings.branch shouldBe branch +// settings.repository shouldBe repository +// settings.branch shouldBe branch + + val commandManager = GitCommandManager(settings.repositoryPath) + doCheckout(commandManager, settings) } }