ae325d4bec
Signed-off-by: Martijn van der Kleijn <martijn.niji@gmail.com>
217 lines
7 KiB
Go
217 lines
7 KiB
Go
// Copyright 2024 The Forgejo Authors. All rights reserved.
|
|
// Use of this source code is governed by a MIT-style
|
|
// license that can be found in the LICENSE file.
|
|
|
|
// Copyright 2020 The Gitea Authors. All rights reserved.
|
|
// Use of this source code is governed by a MIT-style
|
|
// license that can be found in the LICENSE file.
|
|
|
|
package forgejo
|
|
|
|
import (
|
|
"log"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
)
|
|
|
|
func TestPull(t *testing.T) {
|
|
log.Println("== TestPull ==")
|
|
c := newTestClient()
|
|
user, _, err := c.GetMyUserInfo()
|
|
assert.NoError(t, err)
|
|
|
|
repoName := "repo_pull_test"
|
|
forkOrg := "ForkOrg"
|
|
if !preparePullTest(t, c, repoName, forkOrg) {
|
|
return
|
|
}
|
|
|
|
// ListRepoPullRequests list PRs of one repository
|
|
pulls, _, err := c.ListRepoPullRequests(user.UserName, repoName, ListPullRequestsOptions{})
|
|
assert.NoError(t, err)
|
|
assert.Len(t, pulls, 0)
|
|
|
|
pullUpdateFile, _, err := c.CreatePullRequest(c.username, repoName, CreatePullRequestOption{
|
|
Base: "main",
|
|
Head: forkOrg + ":overwrite_licence",
|
|
Title: "overwrite a file",
|
|
})
|
|
assert.NoError(t, err)
|
|
assert.NotNil(t, pullUpdateFile)
|
|
|
|
pullNewFile, _, err := c.CreatePullRequest(c.username, repoName, CreatePullRequestOption{
|
|
Base: "main",
|
|
Head: forkOrg + ":new_file",
|
|
Title: "create a file",
|
|
})
|
|
assert.NoError(t, err)
|
|
assert.NotNil(t, pullNewFile)
|
|
|
|
pullConflict, _, err := c.CreatePullRequest(c.username, repoName, CreatePullRequestOption{
|
|
Base: "main",
|
|
Head: forkOrg + ":will_conflict",
|
|
Title: "this pull will conflict",
|
|
})
|
|
assert.NoError(t, err)
|
|
assert.NotNil(t, pullConflict)
|
|
|
|
pulls, _, err = c.ListRepoPullRequests(user.UserName, repoName, ListPullRequestsOptions{})
|
|
assert.NoError(t, err)
|
|
assert.Len(t, pulls, 3)
|
|
|
|
diff, _, err := c.GetPullRequestDiff(c.username, repoName, pullUpdateFile.Index, PullRequestDiffOptions{
|
|
Binary: true,
|
|
})
|
|
assert.NoError(t, err)
|
|
assert.True(t, len(diff) > 1100 && len(diff) < 1300)
|
|
patch, _, err := c.GetPullRequestPatch(c.username, repoName, pullUpdateFile.Index)
|
|
assert.NoError(t, err)
|
|
assert.True(t, len(patch) > len(diff))
|
|
|
|
commits, _, err := c.ListPullRequestCommits(c.username, repoName, pullUpdateFile.Index, ListPullRequestCommitsOptions{})
|
|
assert.NoError(t, err)
|
|
if assert.Len(t, commits, 1) && assert.Len(t, commits[0].Files, 1) {
|
|
assert.EqualValues(t, "LICENSE", commits[0].Files[0].Filename)
|
|
}
|
|
|
|
files, _, err := c.ListPullRequestFiles(c.username, repoName, pullUpdateFile.Index, ListPullRequestFilesOptions{})
|
|
assert.NoError(t, err)
|
|
assert.Len(t, files, 1)
|
|
file := files[0]
|
|
assert.EqualValues(t, "LICENSE", file.Filename)
|
|
assert.EqualValues(t, "changed", file.Status)
|
|
assert.EqualValues(t, 3, file.Additions)
|
|
assert.EqualValues(t, 9, file.Deletions)
|
|
assert.EqualValues(t, 12, file.Changes)
|
|
|
|
// test Update pull
|
|
pr, _, err := c.GetPullRequest(user.UserName, repoName, pullUpdateFile.Index)
|
|
assert.NoError(t, err)
|
|
assert.NotNil(t, pr)
|
|
assert.False(t, pullUpdateFile.HasMerged)
|
|
assert.True(t, pullUpdateFile.Mergeable)
|
|
merged, _, err := c.MergePullRequest(user.UserName, repoName, pullUpdateFile.Index, MergePullRequestOption{
|
|
Style: MergeStyleSquash,
|
|
Title: pullUpdateFile.Title,
|
|
Message: "squash: " + pullUpdateFile.Title,
|
|
})
|
|
assert.NoError(t, err)
|
|
assert.True(t, merged)
|
|
merged, _, err = c.IsPullRequestMerged(user.UserName, repoName, pullUpdateFile.Index)
|
|
assert.NoError(t, err)
|
|
assert.True(t, merged)
|
|
pr, _, err = c.GetPullRequest(user.UserName, repoName, pullUpdateFile.Index)
|
|
assert.NoError(t, err)
|
|
assert.EqualValues(t, pullUpdateFile.Head.Name, pr.Head.Name)
|
|
assert.EqualValues(t, pullUpdateFile.Base.Name, pr.Base.Name)
|
|
assert.NotEqual(t, pullUpdateFile.Base.Sha, pr.Base.Sha)
|
|
assert.Len(t, *pr.MergedCommitID, 40)
|
|
assert.True(t, pr.HasMerged)
|
|
|
|
// test conflict pull
|
|
pr, _, err = c.GetPullRequest(user.UserName, repoName, pullConflict.Index)
|
|
assert.NoError(t, err)
|
|
assert.False(t, pr.HasMerged)
|
|
assert.False(t, pr.Mergeable)
|
|
merged, _, err = c.MergePullRequest(user.UserName, repoName, pullConflict.Index, MergePullRequestOption{
|
|
Style: MergeStyleMerge,
|
|
Title: "pullConflict",
|
|
Message: "pullConflict Msg",
|
|
})
|
|
assert.NoError(t, err)
|
|
assert.False(t, merged)
|
|
merged, _, err = c.IsPullRequestMerged(user.UserName, repoName, pullConflict.Index)
|
|
assert.NoError(t, err)
|
|
assert.False(t, merged)
|
|
pr, _, err = c.GetPullRequest(user.UserName, repoName, pullConflict.Index)
|
|
assert.NoError(t, err)
|
|
assert.Nil(t, pr.MergedCommitID)
|
|
assert.False(t, pr.HasMerged)
|
|
|
|
state := StateClosed
|
|
pr, _, err = c.EditPullRequest(user.UserName, repoName, pullConflict.Index, EditPullRequestOption{
|
|
Title: "confl",
|
|
State: &state,
|
|
})
|
|
assert.NoError(t, err)
|
|
assert.EqualValues(t, state, pr.State)
|
|
|
|
pulls, _, err = c.ListRepoPullRequests(user.UserName, repoName, ListPullRequestsOptions{
|
|
State: StateClosed,
|
|
Sort: "leastupdate",
|
|
})
|
|
assert.NoError(t, err)
|
|
assert.Len(t, pulls, 2)
|
|
}
|
|
|
|
func preparePullTest(t *testing.T, c *Client, repoName, forkOrg string) bool {
|
|
_, _ = c.DeleteRepo(forkOrg, repoName)
|
|
_, _ = c.DeleteRepo(c.username, repoName)
|
|
_, _ = c.DeleteOrg(forkOrg)
|
|
|
|
origRepo, err := createTestRepo(t, repoName, c)
|
|
if !assert.NoError(t, err) {
|
|
return false
|
|
}
|
|
org, _, err := c.CreateOrg(CreateOrgOption{Name: forkOrg})
|
|
assert.NoError(t, err)
|
|
forkRepo, _, err := c.CreateFork(origRepo.Owner.UserName, origRepo.Name, CreateForkOption{Organization: &org.UserName})
|
|
assert.NoError(t, err)
|
|
assert.NotNil(t, forkRepo)
|
|
|
|
mainLicense, _, err := c.GetContents(forkRepo.Owner.UserName, forkRepo.Name, "main", "LICENSE")
|
|
if !assert.NoError(t, err) || !assert.NotNil(t, mainLicense) {
|
|
return false
|
|
}
|
|
|
|
updatedFile, _, err := c.UpdateFile(forkRepo.Owner.UserName, forkRepo.Name, "LICENSE", UpdateFileOptions{
|
|
FileOptions: FileOptions{
|
|
Message: "Overwrite",
|
|
BranchName: "main",
|
|
NewBranchName: "overwrite_licence",
|
|
},
|
|
SHA: mainLicense.SHA,
|
|
Content: "Tk9USElORyBJUyBIRVJFIEFOWU1PUkUKSUYgWU9VIExJS0UgVE8gRklORCBTT01FVEhJTkcKV0FJVCBGT1IgVEhFIEZVVFVSRQo=",
|
|
})
|
|
if !assert.NoError(t, err) || !assert.NotNil(t, updatedFile) {
|
|
return false
|
|
}
|
|
|
|
newFile, _, err := c.CreateFile(forkRepo.Owner.UserName, forkRepo.Name, "WOW-file", CreateFileOptions{
|
|
Content: "QSBuZXcgRmlsZQo=",
|
|
FileOptions: FileOptions{
|
|
Message: "creat a new file",
|
|
BranchName: "main",
|
|
NewBranchName: "new_file",
|
|
},
|
|
})
|
|
if !assert.NoError(t, err) || !assert.NotNil(t, newFile) {
|
|
return false
|
|
}
|
|
|
|
conflictFile1, _, err := c.CreateFile(origRepo.Owner.UserName, origRepo.Name, "bad-file", CreateFileOptions{
|
|
Content: "U3RhcnQgQ29uZmxpY3QK",
|
|
FileOptions: FileOptions{
|
|
Message: "Start Conflict",
|
|
BranchName: "main",
|
|
},
|
|
})
|
|
if !assert.NoError(t, err) || !assert.NotNil(t, conflictFile1) {
|
|
return false
|
|
}
|
|
|
|
conflictFile2, _, err := c.CreateFile(forkRepo.Owner.UserName, forkRepo.Name, "bad-file", CreateFileOptions{
|
|
Content: "V2lsbEhhdmUgQ29uZmxpY3QK",
|
|
FileOptions: FileOptions{
|
|
Message: "creat a new file witch will conflict",
|
|
BranchName: "main",
|
|
NewBranchName: "will_conflict",
|
|
},
|
|
})
|
|
if !assert.NoError(t, err) || !assert.NotNil(t, conflictFile2) {
|
|
return false
|
|
}
|
|
|
|
return true
|
|
}
|