CR fixes

jfrog · Oct 20, 2024 · b6ead79 · b6ead79
1 parent 8d77882
commit b6ead79
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 20 deletions.
diff --git a/integrationutils.go b/integrationutils.go
@@ -80,18 +80,18 @@ func setIntegrationTestEnvs(t *testing.T, testDetails *IntegrationTestDetails) f
 		useLocalRepo = "true"
 	}
 	unsetEnvs := utils.SetEnvsAndAssertWithCallback(t, map[string]string{
-		utils.RequirementsFileEnv:             "requirements.txt",
-		utils.GitPullRequestIDEnv:             testDetails.PullRequestID,
-		utils.GitProvider:                     testDetails.GitProvider,
-		utils.GitTokenEnv:                     testDetails.GitToken,
-		utils.GitRepoEnv:                      testDetails.RepoName,
-		utils.GitRepoOwnerEnv:                 testDetails.RepoOwner,
-		utils.BranchNameTemplateEnv:           testDetails.CustomBranchName,
-		utils.GitApiEndpointEnv:               testDetails.ApiEndpoint,
-		utils.GitProjectEnv:                   testDetails.GitProject,
-		utils.GitUsernameEnv:                  testDetails.GitUsername,
-		utils.GitBaseBranchEnv:                mainBranch,
-		utils.GitUseCurrentLocalRepositoryEnv: useLocalRepo,
+		utils.RequirementsFileEnv:      "requirements.txt",
+		utils.GitPullRequestIDEnv:      testDetails.PullRequestID,
+		utils.GitProvider:              testDetails.GitProvider,
+		utils.GitTokenEnv:              testDetails.GitToken,
+		utils.GitRepoEnv:               testDetails.RepoName,
+		utils.GitRepoOwnerEnv:          testDetails.RepoOwner,
+		utils.BranchNameTemplateEnv:    testDetails.CustomBranchName,
+		utils.GitApiEndpointEnv:        testDetails.ApiEndpoint,
+		utils.GitProjectEnv:            testDetails.GitProject,
+		utils.GitUsernameEnv:           testDetails.GitUsername,
+		utils.GitBaseBranchEnv:         mainBranch,
+		utils.GitUseLocalRepositoryEnv: useLocalRepo,
 	})
 	return func() {
 		envRestoreFunc()

diff --git a/utils/consts.go b/utils/consts.go
@@ -34,12 +34,12 @@ const (
 	JfrogConfigProfileEnv  = "JF_CONFIG_PROFILE"
 
 	// Git environment variables
-	GitProvider                     = "JF_GIT_PROVIDER"
-	GitRepoOwnerEnv                 = "JF_GIT_OWNER"
-	GitRepoEnv                      = "JF_GIT_REPO"
-	GitProjectEnv                   = "JF_GIT_PROJECT"
-	GitUsernameEnv                  = "JF_GIT_USERNAME"
-	GitUseCurrentLocalRepositoryEnv = "JF_USE_CURRENT_LOCAL_REPOSITORY"
+	GitProvider              = "JF_GIT_PROVIDER"
+	GitRepoOwnerEnv          = "JF_GIT_OWNER"
+	GitRepoEnv               = "JF_GIT_REPO"
+	GitProjectEnv            = "JF_GIT_PROJECT"
+	GitUsernameEnv           = "JF_GIT_USERNAME"
+	GitUseLocalRepositoryEnv = "JF_USE_LOCAL_REPOSITORY"
 
 	// Git naming template environment variables
 	BranchNameTemplateEnv       = "JF_BRANCH_NAME_TEMPLATE"

diff --git a/utils/git.go b/utils/git.go
@@ -119,7 +119,7 @@ func (gm *GitManager) SetLocalRepositoryAndRemoteName() (*GitManager, error) {
 	var err error
 	// Re-initialize the repository and update remoteName
 	gm.remoteName = vcsutils.RemoteName
-	gm.localGitRepository, err = git.PlainOpen(".")
+	err = gm.SetLocalRepository()
 	return gm, err
 }
 

diff --git a/utils/params.go b/utils/params.go
@@ -375,7 +375,7 @@ func (g *Git) extractScanRepositoryEnvParams(gitParamsFromEnv *Git) (err error)
 		}
 	}
 	if !g.UseLocalRepository {
-		if g.UseLocalRepository, err = getBoolEnv(GitUseCurrentLocalRepositoryEnv, false); err != nil {
+		if g.UseLocalRepository, err = getBoolEnv(GitUseLocalRepositoryEnv, false); err != nil {
 			return
 		}
 	}