123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297 |
- // Copyright 2019 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 repofiles
-
- import (
- "testing"
-
- "code.gitea.io/gitea/models"
- "code.gitea.io/gitea/modules/repository"
- "code.gitea.io/gitea/modules/setting"
-
- "github.com/stretchr/testify/assert"
- )
-
- func TestUpdateIssuesCommit(t *testing.T) {
- assert.NoError(t, models.PrepareTestDatabase())
- pushCommits := []*repository.PushCommit{
- {
- Sha1: "abcdef1",
- CommitterEmail: "user2@example.com",
- CommitterName: "User Two",
- AuthorEmail: "user4@example.com",
- AuthorName: "User Four",
- Message: "start working on #FST-1, #1",
- },
- {
- Sha1: "abcdef2",
- CommitterEmail: "user2@example.com",
- CommitterName: "User Two",
- AuthorEmail: "user2@example.com",
- AuthorName: "User Two",
- Message: "a plain message",
- },
- {
- Sha1: "abcdef2",
- CommitterEmail: "user2@example.com",
- CommitterName: "User Two",
- AuthorEmail: "user2@example.com",
- AuthorName: "User Two",
- Message: "close #2",
- },
- }
-
- user := models.AssertExistsAndLoadBean(t, &models.User{ID: 2}).(*models.User)
- repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 1}).(*models.Repository)
- repo.Owner = user
-
- commentBean := &models.Comment{
- Type: models.CommentTypeCommitRef,
- CommitSHA: "abcdef1",
- PosterID: user.ID,
- IssueID: 1,
- }
- issueBean := &models.Issue{RepoID: repo.ID, Index: 4}
-
- models.AssertNotExistsBean(t, commentBean)
- models.AssertNotExistsBean(t, &models.Issue{RepoID: repo.ID, Index: 2}, "is_closed=1")
- assert.NoError(t, UpdateIssuesCommit(user, repo, pushCommits, repo.DefaultBranch))
- models.AssertExistsAndLoadBean(t, commentBean)
- models.AssertExistsAndLoadBean(t, issueBean, "is_closed=1")
- models.CheckConsistencyFor(t, &models.Action{})
-
- // Test that push to a non-default branch closes no issue.
- pushCommits = []*repository.PushCommit{
- {
- Sha1: "abcdef1",
- CommitterEmail: "user2@example.com",
- CommitterName: "User Two",
- AuthorEmail: "user4@example.com",
- AuthorName: "User Four",
- Message: "close #1",
- },
- }
- repo = models.AssertExistsAndLoadBean(t, &models.Repository{ID: 3}).(*models.Repository)
- commentBean = &models.Comment{
- Type: models.CommentTypeCommitRef,
- CommitSHA: "abcdef1",
- PosterID: user.ID,
- IssueID: 6,
- }
- issueBean = &models.Issue{RepoID: repo.ID, Index: 1}
-
- models.AssertNotExistsBean(t, commentBean)
- models.AssertNotExistsBean(t, &models.Issue{RepoID: repo.ID, Index: 1}, "is_closed=1")
- assert.NoError(t, UpdateIssuesCommit(user, repo, pushCommits, "non-existing-branch"))
- models.AssertExistsAndLoadBean(t, commentBean)
- models.AssertNotExistsBean(t, issueBean, "is_closed=1")
- models.CheckConsistencyFor(t, &models.Action{})
-
- pushCommits = []*repository.PushCommit{
- {
- Sha1: "abcdef3",
- CommitterEmail: "user2@example.com",
- CommitterName: "User Two",
- AuthorEmail: "user2@example.com",
- AuthorName: "User Two",
- Message: "close " + setting.AppURL + repo.FullName() + "/pulls/1",
- },
- }
- repo = models.AssertExistsAndLoadBean(t, &models.Repository{ID: 3}).(*models.Repository)
- commentBean = &models.Comment{
- Type: models.CommentTypeCommitRef,
- CommitSHA: "abcdef3",
- PosterID: user.ID,
- IssueID: 6,
- }
- issueBean = &models.Issue{RepoID: repo.ID, Index: 1}
-
- models.AssertNotExistsBean(t, commentBean)
- models.AssertNotExistsBean(t, &models.Issue{RepoID: repo.ID, Index: 1}, "is_closed=1")
- assert.NoError(t, UpdateIssuesCommit(user, repo, pushCommits, repo.DefaultBranch))
- models.AssertExistsAndLoadBean(t, commentBean)
- models.AssertExistsAndLoadBean(t, issueBean, "is_closed=1")
- models.CheckConsistencyFor(t, &models.Action{})
- }
-
- func TestUpdateIssuesCommit_Colon(t *testing.T) {
- assert.NoError(t, models.PrepareTestDatabase())
- pushCommits := []*repository.PushCommit{
- {
- Sha1: "abcdef2",
- CommitterEmail: "user2@example.com",
- CommitterName: "User Two",
- AuthorEmail: "user2@example.com",
- AuthorName: "User Two",
- Message: "close: #2",
- },
- }
-
- user := models.AssertExistsAndLoadBean(t, &models.User{ID: 2}).(*models.User)
- repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 1}).(*models.Repository)
- repo.Owner = user
-
- issueBean := &models.Issue{RepoID: repo.ID, Index: 4}
-
- models.AssertNotExistsBean(t, &models.Issue{RepoID: repo.ID, Index: 2}, "is_closed=1")
- assert.NoError(t, UpdateIssuesCommit(user, repo, pushCommits, repo.DefaultBranch))
- models.AssertExistsAndLoadBean(t, issueBean, "is_closed=1")
- models.CheckConsistencyFor(t, &models.Action{})
- }
-
- func TestUpdateIssuesCommit_Issue5957(t *testing.T) {
- assert.NoError(t, models.PrepareTestDatabase())
- user := models.AssertExistsAndLoadBean(t, &models.User{ID: 2}).(*models.User)
-
- // Test that push to a non-default branch closes an issue.
- pushCommits := []*repository.PushCommit{
- {
- Sha1: "abcdef1",
- CommitterEmail: "user2@example.com",
- CommitterName: "User Two",
- AuthorEmail: "user4@example.com",
- AuthorName: "User Four",
- Message: "close #2",
- },
- }
-
- repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 2}).(*models.Repository)
- commentBean := &models.Comment{
- Type: models.CommentTypeCommitRef,
- CommitSHA: "abcdef1",
- PosterID: user.ID,
- IssueID: 7,
- }
-
- issueBean := &models.Issue{RepoID: repo.ID, Index: 2, ID: 7}
-
- models.AssertNotExistsBean(t, commentBean)
- models.AssertNotExistsBean(t, issueBean, "is_closed=1")
- assert.NoError(t, UpdateIssuesCommit(user, repo, pushCommits, "non-existing-branch"))
- models.AssertExistsAndLoadBean(t, commentBean)
- models.AssertExistsAndLoadBean(t, issueBean, "is_closed=1")
- models.CheckConsistencyFor(t, &models.Action{})
- }
-
- func TestUpdateIssuesCommit_AnotherRepo(t *testing.T) {
- assert.NoError(t, models.PrepareTestDatabase())
- user := models.AssertExistsAndLoadBean(t, &models.User{ID: 2}).(*models.User)
-
- // Test that a push to default branch closes issue in another repo
- // If the user also has push permissions to that repo
- pushCommits := []*repository.PushCommit{
- {
- Sha1: "abcdef1",
- CommitterEmail: "user2@example.com",
- CommitterName: "User Two",
- AuthorEmail: "user2@example.com",
- AuthorName: "User Two",
- Message: "close user2/repo1#1",
- },
- }
-
- repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 2}).(*models.Repository)
- commentBean := &models.Comment{
- Type: models.CommentTypeCommitRef,
- CommitSHA: "abcdef1",
- PosterID: user.ID,
- IssueID: 1,
- }
-
- issueBean := &models.Issue{RepoID: 1, Index: 1, ID: 1}
-
- models.AssertNotExistsBean(t, commentBean)
- models.AssertNotExistsBean(t, issueBean, "is_closed=1")
- assert.NoError(t, UpdateIssuesCommit(user, repo, pushCommits, repo.DefaultBranch))
- models.AssertExistsAndLoadBean(t, commentBean)
- models.AssertExistsAndLoadBean(t, issueBean, "is_closed=1")
- models.CheckConsistencyFor(t, &models.Action{})
- }
-
- func TestUpdateIssuesCommit_AnotherRepo_FullAddress(t *testing.T) {
- assert.NoError(t, models.PrepareTestDatabase())
- user := models.AssertExistsAndLoadBean(t, &models.User{ID: 2}).(*models.User)
-
- // Test that a push to default branch closes issue in another repo
- // If the user also has push permissions to that repo
- pushCommits := []*repository.PushCommit{
- {
- Sha1: "abcdef1",
- CommitterEmail: "user2@example.com",
- CommitterName: "User Two",
- AuthorEmail: "user2@example.com",
- AuthorName: "User Two",
- Message: "close " + setting.AppURL + "user2/repo1/issues/1",
- },
- }
-
- repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 2}).(*models.Repository)
- commentBean := &models.Comment{
- Type: models.CommentTypeCommitRef,
- CommitSHA: "abcdef1",
- PosterID: user.ID,
- IssueID: 1,
- }
-
- issueBean := &models.Issue{RepoID: 1, Index: 1, ID: 1}
-
- models.AssertNotExistsBean(t, commentBean)
- models.AssertNotExistsBean(t, issueBean, "is_closed=1")
- assert.NoError(t, UpdateIssuesCommit(user, repo, pushCommits, repo.DefaultBranch))
- models.AssertExistsAndLoadBean(t, commentBean)
- models.AssertExistsAndLoadBean(t, issueBean, "is_closed=1")
- models.CheckConsistencyFor(t, &models.Action{})
- }
-
- func TestUpdateIssuesCommit_AnotherRepoNoPermission(t *testing.T) {
- assert.NoError(t, models.PrepareTestDatabase())
- user := models.AssertExistsAndLoadBean(t, &models.User{ID: 10}).(*models.User)
-
- // Test that a push with close reference *can not* close issue
- // If the commiter doesn't have push rights in that repo
- pushCommits := []*repository.PushCommit{
- {
- Sha1: "abcdef3",
- CommitterEmail: "user10@example.com",
- CommitterName: "User Ten",
- AuthorEmail: "user10@example.com",
- AuthorName: "User Ten",
- Message: "close user3/repo3#1",
- },
- {
- Sha1: "abcdef4",
- CommitterEmail: "user10@example.com",
- CommitterName: "User Ten",
- AuthorEmail: "user10@example.com",
- AuthorName: "User Ten",
- Message: "close " + setting.AppURL + "user3/repo3/issues/1",
- },
- }
-
- repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 6}).(*models.Repository)
- commentBean := &models.Comment{
- Type: models.CommentTypeCommitRef,
- CommitSHA: "abcdef3",
- PosterID: user.ID,
- IssueID: 6,
- }
- commentBean2 := &models.Comment{
- Type: models.CommentTypeCommitRef,
- CommitSHA: "abcdef4",
- PosterID: user.ID,
- IssueID: 6,
- }
-
- issueBean := &models.Issue{RepoID: 3, Index: 1, ID: 6}
-
- models.AssertNotExistsBean(t, commentBean)
- models.AssertNotExistsBean(t, commentBean2)
- models.AssertNotExistsBean(t, issueBean, "is_closed=1")
- assert.NoError(t, UpdateIssuesCommit(user, repo, pushCommits, repo.DefaultBranch))
- models.AssertNotExistsBean(t, commentBean)
- models.AssertNotExistsBean(t, commentBean2)
- models.AssertNotExistsBean(t, issueBean, "is_closed=1")
- models.CheckConsistencyFor(t, &models.Action{})
- }
|