Skip to content

Commit 050830e

Browse files
committed
fine tune comments
1 parent 5004bb0 commit 050830e

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

tests/integration/manual_merge_test.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ func TestManualMergeAutodetect(t *testing.T) {
3030
user1 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 1})
3131
user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
3232

33+
// Create a repo owned by user2
3334
repoName := "manual-merge-autodetect"
3435
defaultBranch := setting.Repository.DefaultBranch
3536
user2Ctx := NewAPITestContext(t, user2.Name, repoName, auth_model.AccessTokenScopeWriteRepository, auth_model.AccessTokenScopeWriteUser)
@@ -45,7 +46,7 @@ func TestManualMergeAutodetect(t *testing.T) {
4546
repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{Name: repoName})
4647
user1Ctx := NewAPITestContext(t, user1.Name, repoName, auth_model.AccessTokenScopeWriteRepository)
4748

48-
// multiple PRs should be able to be closed together if a commit contains their branch commits.
49+
// multiple PRs should be able to be closed together if a push contains their branch commits.
4950
branchNames := []string{"fix-1", "fix-2"}
5051
apiPulls := make([]api.PullRequest, len(branchNames))
5152
for i, branchName := range branchNames {
@@ -60,7 +61,7 @@ func TestManualMergeAutodetect(t *testing.T) {
6061

6162
// user1 clones, then merges every branch sequentially, then pushes once.
6263
// The first merge fast-forwards; the rest produce real merge commits,
63-
// which generates multiple commits for "git ancestry-path --merges --reverse".
64+
// which generates multiple commits for "git rev-list --ancestry-path --merges ...".
6465
dstPath := t.TempDir()
6566
u, _ := url.Parse(giteaURL.String())
6667
u.Path = fmt.Sprintf("%s/%s.git", user2.Name, repoName)

0 commit comments

Comments
 (0)