test

Author: wxiaoguang

modules/git/repo_commit_test.go

@@ -124,7 +124,7 @@ func TestGetRefCommitID(t *testing.T) {
 	}
 
 	for _, testCase := range testCases {
-		commitID, err := bareRepo1.GetRefCommitIDNew(testCase.Ref)
+		commitID, err := bareRepo1.GetRefCommitIDOld(testCase.Ref)
 		if assert.NoError(t, err) {
 			assert.Equal(t, testCase.ExpectedCommitID, commitID)
 		}

modules/git/repo_compare_test.go

@@ -96,7 +96,7 @@ func TestReadWritePullHead(t *testing.T) {
 	defer repo.Close()
 
 	// Try to open non-existing Pull
-	_, err = repo.GetRefCommitIDNew(PullPrefix + "0/head")
+	_, err = repo.GetRefCommitIDOld(PullPrefix + "0/head")
 	assert.Error(t, err)
 
 	// Write a fake sha1 with only 40 zeros
@@ -111,7 +111,7 @@ func TestReadWritePullHead(t *testing.T) {
 	}
 
 	// Read the file created
-	headContents, err := repo.GetRefCommitIDNew(PullPrefix + "1/head")
+	headContents, err := repo.GetRefCommitIDOld(PullPrefix + "1/head")
 	if err != nil {
 		assert.NoError(t, err)
 		return

tests/integration/git_general_test.go

@@ -844,7 +844,7 @@ func doCreateAgitFlowPull(dstPath string, ctx *APITestContext, headBranch string
 				Message: "Testing commit 1",
 			})
 			assert.NoError(t, err)
-			commit, err = gitRepo.GetRefCommitIDNew("HEAD")
+			commit, err = gitRepo.GetRefCommitIDOld("HEAD")
 			assert.NoError(t, err)
 		})
 
@@ -916,7 +916,7 @@ func doCreateAgitFlowPull(dstPath string, ctx *APITestContext, headBranch string
 				Message: "Testing commit 2",
 			})
 			assert.NoError(t, err)
-			commit, err = gitRepo.GetRefCommitIDNew("HEAD")
+			commit, err = gitRepo.GetRefCommitIDOld("HEAD")
 			assert.NoError(t, err)
 		})
 

tests/integration/git_misc_test.go

@@ -226,7 +226,7 @@ func TestAgitReviewStaleness(t *testing.T) {
 		assert.NoError(t, err)
 		defer baseGitRepo.Close()
 
-		updatedCommitID, err := baseGitRepo.GetRefCommitIDNew(pr.GetGitHeadRefName())
+		updatedCommitID, err := baseGitRepo.GetRefCommitIDOld(pr.GetGitHeadRefName())
 		assert.NoError(t, err)
 		t.Logf("Updated commit ID: %s", updatedCommitID)
 

tests/integration/pull_merge_test.go

@@ -842,7 +842,7 @@ func TestPullAutoMergeAfterCommitStatusSucceed(t *testing.T) {
 		// update commit status to success, then it should be merged automatically
 		baseGitRepo, err := gitrepo.OpenRepository(t.Context(), baseRepo)
 		assert.NoError(t, err)
-		sha, err := baseGitRepo.GetRefCommitIDNew(pr.GetGitHeadRefName())
+		sha, err := baseGitRepo.GetRefCommitIDOld(pr.GetGitHeadRefName())
 		assert.NoError(t, err)
 		masterCommitID, err := baseGitRepo.GetBranchCommitID("master")
 		assert.NoError(t, err)
@@ -924,7 +924,7 @@ func TestPullAutoMergeAfterCommitStatusSucceedAndApproval(t *testing.T) {
 		// update commit status to success, then it should be merged automatically
 		baseGitRepo, err := gitrepo.OpenRepository(t.Context(), baseRepo)
 		assert.NoError(t, err)
-		sha, err := baseGitRepo.GetRefCommitIDNew(pr.GetGitHeadRefName())
+		sha, err := baseGitRepo.GetRefCommitIDOld(pr.GetGitHeadRefName())
 		assert.NoError(t, err)
 		masterCommitID, err := baseGitRepo.GetBranchCommitID("master")
 		assert.NoError(t, err)
@@ -1055,7 +1055,7 @@ func TestPullAutoMergeAfterCommitStatusSucceedAndApprovalForAgitFlow(t *testing.
 		// update commit status to success, then it should be merged automatically
 		baseGitRepo, err := gitrepo.OpenRepository(t.Context(), baseRepo)
 		assert.NoError(t, err)
-		sha, err := baseGitRepo.GetRefCommitIDNew(pr.GetGitHeadRefName())
+		sha, err := baseGitRepo.GetRefCommitIDOld(pr.GetGitHeadRefName())
 		assert.NoError(t, err)
 		masterCommitID, err := baseGitRepo.GetBranchCommitID("master")
 		assert.NoError(t, err)