mirror of
				https://codeberg.org/forgejo/forgejo.git
				synced 2025-10-31 06:21:11 +00:00 
			
		
		
		
	- Don't show the labels-list element, if no labels are selected.
- The labels-list was taking up vertical space, even if no labels were
selected which caused an inconsistency in how the sidebar looked.
- Adds integration test
(cherry picked from commit 013b89eb13)
		
	
			
		
			
				
	
	
		
			970 lines
		
	
	
	
		
			34 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			970 lines
		
	
	
	
		
			34 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
| // Copyright 2017 The Gitea Authors. All rights reserved.
 | |
| // Copyright 2024 The Forgejo Authors c/o Codeberg e.V.. All rights reserved.
 | |
| // SPDX-License-Identifier: MIT
 | |
| 
 | |
| package integration
 | |
| 
 | |
| import (
 | |
| 	"context"
 | |
| 	"fmt"
 | |
| 	"net/http"
 | |
| 	"net/url"
 | |
| 	"path"
 | |
| 	"strconv"
 | |
| 	"strings"
 | |
| 	"testing"
 | |
| 	"time"
 | |
| 
 | |
| 	"code.gitea.io/gitea/models/db"
 | |
| 	issues_model "code.gitea.io/gitea/models/issues"
 | |
| 	project_model "code.gitea.io/gitea/models/project"
 | |
| 	repo_model "code.gitea.io/gitea/models/repo"
 | |
| 	unit_model "code.gitea.io/gitea/models/unit"
 | |
| 	"code.gitea.io/gitea/models/unittest"
 | |
| 	user_model "code.gitea.io/gitea/models/user"
 | |
| 	"code.gitea.io/gitea/modules/indexer/issues"
 | |
| 	"code.gitea.io/gitea/modules/optional"
 | |
| 	"code.gitea.io/gitea/modules/references"
 | |
| 	"code.gitea.io/gitea/modules/setting"
 | |
| 	api "code.gitea.io/gitea/modules/structs"
 | |
| 	"code.gitea.io/gitea/modules/test"
 | |
| 	files_service "code.gitea.io/gitea/services/repository/files"
 | |
| 	"code.gitea.io/gitea/tests"
 | |
| 
 | |
| 	"github.com/PuerkitoBio/goquery"
 | |
| 	"github.com/stretchr/testify/assert"
 | |
| )
 | |
| 
 | |
| func getIssuesSelection(t testing.TB, htmlDoc *HTMLDoc) *goquery.Selection {
 | |
| 	issueList := htmlDoc.doc.Find("#issue-list")
 | |
| 	assert.EqualValues(t, 1, issueList.Length())
 | |
| 	return issueList.Find(".flex-item").Find(".issue-title")
 | |
| }
 | |
| 
 | |
| func getIssue(t *testing.T, repoID int64, issueSelection *goquery.Selection) *issues_model.Issue {
 | |
| 	href, exists := issueSelection.Attr("href")
 | |
| 	assert.True(t, exists)
 | |
| 	indexStr := href[strings.LastIndexByte(href, '/')+1:]
 | |
| 	index, err := strconv.Atoi(indexStr)
 | |
| 	assert.NoError(t, err, "Invalid issue href: %s", href)
 | |
| 	return unittest.AssertExistsAndLoadBean(t, &issues_model.Issue{RepoID: repoID, Index: int64(index)})
 | |
| }
 | |
| 
 | |
| func assertMatch(t testing.TB, issue *issues_model.Issue, keyword string) {
 | |
| 	matches := strings.Contains(strings.ToLower(issue.Title), keyword) ||
 | |
| 		strings.Contains(strings.ToLower(issue.Content), keyword)
 | |
| 	for _, comment := range issue.Comments {
 | |
| 		matches = matches || strings.Contains(
 | |
| 			strings.ToLower(comment.Content),
 | |
| 			keyword,
 | |
| 		)
 | |
| 	}
 | |
| 	assert.True(t, matches)
 | |
| }
 | |
| 
 | |
| func TestNoLoginViewIssues(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 	req := NewRequest(t, "GET", "/user2/repo1/issues")
 | |
| 	MakeRequest(t, req, http.StatusOK)
 | |
| }
 | |
| 
 | |
| func TestViewIssuesSortByType(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 	user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 1})
 | |
| 	repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
 | |
| 
 | |
| 	session := loginUser(t, user.Name)
 | |
| 	req := NewRequest(t, "GET", repo.Link()+"/issues?type=created_by")
 | |
| 	resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 	htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 	issuesSelection := getIssuesSelection(t, htmlDoc)
 | |
| 	expectedNumIssues := unittest.GetCount(t,
 | |
| 		&issues_model.Issue{RepoID: repo.ID, PosterID: user.ID},
 | |
| 		unittest.Cond("is_closed=?", false),
 | |
| 		unittest.Cond("is_pull=?", false),
 | |
| 	)
 | |
| 	if expectedNumIssues > setting.UI.IssuePagingNum {
 | |
| 		expectedNumIssues = setting.UI.IssuePagingNum
 | |
| 	}
 | |
| 	assert.EqualValues(t, expectedNumIssues, issuesSelection.Length())
 | |
| 
 | |
| 	issuesSelection.Each(func(_ int, selection *goquery.Selection) {
 | |
| 		issue := getIssue(t, repo.ID, selection)
 | |
| 		assert.EqualValues(t, user.ID, issue.PosterID)
 | |
| 	})
 | |
| }
 | |
| 
 | |
| func TestViewIssuesKeyword(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 	repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
 | |
| 	issue := unittest.AssertExistsAndLoadBean(t, &issues_model.Issue{
 | |
| 		RepoID: repo.ID,
 | |
| 		Index:  1,
 | |
| 	})
 | |
| 	issues.UpdateIssueIndexer(context.Background(), issue.ID)
 | |
| 	time.Sleep(time.Second * 1)
 | |
| 	const keyword = "first"
 | |
| 	req := NewRequestf(t, "GET", "%s/issues?q=%s", repo.Link(), keyword)
 | |
| 	resp := MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 	htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 	issuesSelection := getIssuesSelection(t, htmlDoc)
 | |
| 	assert.EqualValues(t, 1, issuesSelection.Length())
 | |
| 	issuesSelection.Each(func(_ int, selection *goquery.Selection) {
 | |
| 		issue := getIssue(t, repo.ID, selection)
 | |
| 		assert.False(t, issue.IsClosed)
 | |
| 		assert.False(t, issue.IsPull)
 | |
| 		assertMatch(t, issue, keyword)
 | |
| 	})
 | |
| }
 | |
| 
 | |
| func TestViewIssuesSearchOptions(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 	repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
 | |
| 
 | |
| 	// there are two issues in repo1, both bound to a project. Add one
 | |
| 	// that is not bound to any project.
 | |
| 	_, issueNoProject := testIssueWithBean(t, "user2", 1, "Title", "Description")
 | |
| 
 | |
| 	t.Run("All issues", func(t *testing.T) {
 | |
| 		req := NewRequestf(t, "GET", "%s/issues?state=all", repo.Link())
 | |
| 		resp := MakeRequest(t, req, http.StatusOK)
 | |
| 		htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 		issuesSelection := getIssuesSelection(t, htmlDoc)
 | |
| 		assert.EqualValues(t, 3, issuesSelection.Length())
 | |
| 	})
 | |
| 
 | |
| 	t.Run("Issues with no project", func(t *testing.T) {
 | |
| 		req := NewRequestf(t, "GET", "%s/issues?state=all&project=-1", repo.Link())
 | |
| 		resp := MakeRequest(t, req, http.StatusOK)
 | |
| 		htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 		issuesSelection := getIssuesSelection(t, htmlDoc)
 | |
| 		assert.EqualValues(t, 1, issuesSelection.Length())
 | |
| 		issuesSelection.Each(func(_ int, selection *goquery.Selection) {
 | |
| 			issue := getIssue(t, repo.ID, selection)
 | |
| 			assert.Equal(t, issueNoProject.ID, issue.ID)
 | |
| 		})
 | |
| 	})
 | |
| 
 | |
| 	t.Run("Issues with a specific project", func(t *testing.T) {
 | |
| 		project := unittest.AssertExistsAndLoadBean(t, &project_model.Project{ID: 1})
 | |
| 
 | |
| 		req := NewRequestf(t, "GET", "%s/issues?state=all&project=%d", repo.Link(), project.ID)
 | |
| 		resp := MakeRequest(t, req, http.StatusOK)
 | |
| 		htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 		issuesSelection := getIssuesSelection(t, htmlDoc)
 | |
| 		assert.EqualValues(t, 2, issuesSelection.Length())
 | |
| 		found := map[int64]bool{
 | |
| 			1: false,
 | |
| 			5: false,
 | |
| 		}
 | |
| 		issuesSelection.Each(func(_ int, selection *goquery.Selection) {
 | |
| 			issue := getIssue(t, repo.ID, selection)
 | |
| 			found[issue.ID] = true
 | |
| 		})
 | |
| 		assert.EqualValues(t, 2, len(found))
 | |
| 		assert.True(t, found[1])
 | |
| 		assert.True(t, found[5])
 | |
| 	})
 | |
| }
 | |
| 
 | |
| func TestNoLoginViewIssue(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 	req := NewRequest(t, "GET", "/user2/repo1/issues/1")
 | |
| 	MakeRequest(t, req, http.StatusOK)
 | |
| }
 | |
| 
 | |
| func testNewIssue(t *testing.T, session *TestSession, user, repo, title, content string) string {
 | |
| 	req := NewRequest(t, "GET", path.Join(user, repo, "issues", "new"))
 | |
| 	resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 	htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 	link, exists := htmlDoc.doc.Find("form.ui.form").Attr("action")
 | |
| 	assert.True(t, exists, "The template has changed")
 | |
| 	req = NewRequestWithValues(t, "POST", link, map[string]string{
 | |
| 		"_csrf":   htmlDoc.GetCSRF(),
 | |
| 		"title":   title,
 | |
| 		"content": content,
 | |
| 	})
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 	issueURL := test.RedirectURL(resp)
 | |
| 	req = NewRequest(t, "GET", issueURL)
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 	htmlDoc = NewHTMLParser(t, resp.Body)
 | |
| 	val := htmlDoc.doc.Find("#issue-title").Text()
 | |
| 	assert.Contains(t, val, title)
 | |
| 	val = htmlDoc.doc.Find(".comment .render-content p").First().Text()
 | |
| 	assert.Equal(t, content, val)
 | |
| 
 | |
| 	return issueURL
 | |
| }
 | |
| 
 | |
| func testIssueAddComment(t *testing.T, session *TestSession, issueURL, content, status string) int64 {
 | |
| 	req := NewRequest(t, "GET", issueURL)
 | |
| 	resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 	htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 	link, exists := htmlDoc.doc.Find("#comment-form").Attr("action")
 | |
| 	assert.True(t, exists, "The template has changed")
 | |
| 
 | |
| 	commentCount := htmlDoc.doc.Find(".comment-list .comment .render-content").Length()
 | |
| 
 | |
| 	req = NewRequestWithValues(t, "POST", link, map[string]string{
 | |
| 		"_csrf":   htmlDoc.GetCSRF(),
 | |
| 		"content": content,
 | |
| 		"status":  status,
 | |
| 	})
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 	req = NewRequest(t, "GET", test.RedirectURL(resp))
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 	htmlDoc = NewHTMLParser(t, resp.Body)
 | |
| 
 | |
| 	val := htmlDoc.doc.Find(".comment-list .comment .render-content p").Eq(commentCount).Text()
 | |
| 	assert.Equal(t, content, val)
 | |
| 
 | |
| 	idAttr, has := htmlDoc.doc.Find(".comment-list .comment").Eq(commentCount).Attr("id")
 | |
| 	idStr := idAttr[strings.LastIndexByte(idAttr, '-')+1:]
 | |
| 	assert.True(t, has)
 | |
| 	id, err := strconv.Atoi(idStr)
 | |
| 	assert.NoError(t, err)
 | |
| 	return int64(id)
 | |
| }
 | |
| 
 | |
| func TestNewIssue(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 	session := loginUser(t, "user2")
 | |
| 	testNewIssue(t, session, "user2", "repo1", "Title", "Description")
 | |
| }
 | |
| 
 | |
| func TestIssueCommentClose(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 	session := loginUser(t, "user2")
 | |
| 	issueURL := testNewIssue(t, session, "user2", "repo1", "Title", "Description")
 | |
| 	testIssueAddComment(t, session, issueURL, "Test comment 1", "")
 | |
| 	testIssueAddComment(t, session, issueURL, "Test comment 2", "")
 | |
| 	testIssueAddComment(t, session, issueURL, "Test comment 3", "close")
 | |
| 
 | |
| 	// Validate that issue content has not been updated
 | |
| 	req := NewRequest(t, "GET", issueURL)
 | |
| 	resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 	htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 	val := htmlDoc.doc.Find(".comment-list .comment .render-content p").First().Text()
 | |
| 	assert.Equal(t, "Description", val)
 | |
| }
 | |
| 
 | |
| func TestIssueCommentDelete(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 	session := loginUser(t, "user2")
 | |
| 	issueURL := testNewIssue(t, session, "user2", "repo1", "Title", "Description")
 | |
| 	comment1 := "Test comment 1"
 | |
| 	commentID := testIssueAddComment(t, session, issueURL, comment1, "")
 | |
| 	comment := unittest.AssertExistsAndLoadBean(t, &issues_model.Comment{ID: commentID})
 | |
| 	assert.Equal(t, comment1, comment.Content)
 | |
| 
 | |
| 	// Using the ID of a comment that does not belong to the repository must fail
 | |
| 	req := NewRequestWithValues(t, "POST", fmt.Sprintf("/%s/%s/comments/%d/delete", "user5", "repo4", commentID), map[string]string{
 | |
| 		"_csrf": GetCSRF(t, session, issueURL),
 | |
| 	})
 | |
| 	session.MakeRequest(t, req, http.StatusNotFound)
 | |
| 	req = NewRequestWithValues(t, "POST", fmt.Sprintf("/%s/%s/comments/%d/delete", "user2", "repo1", commentID), map[string]string{
 | |
| 		"_csrf": GetCSRF(t, session, issueURL),
 | |
| 	})
 | |
| 	session.MakeRequest(t, req, http.StatusOK)
 | |
| 	unittest.AssertNotExistsBean(t, &issues_model.Comment{ID: commentID})
 | |
| }
 | |
| 
 | |
| func TestIssueCommentAttachment(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 	const repoURL = "user2/repo1"
 | |
| 	const content = "Test comment 4"
 | |
| 	const status = ""
 | |
| 	session := loginUser(t, "user2")
 | |
| 	issueURL := testNewIssue(t, session, "user2", "repo1", "Title", "Description")
 | |
| 
 | |
| 	req := NewRequest(t, "GET", issueURL)
 | |
| 	resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 	htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 	link, exists := htmlDoc.doc.Find("#comment-form").Attr("action")
 | |
| 	assert.True(t, exists, "The template has changed")
 | |
| 
 | |
| 	uuid := createAttachment(t, session, repoURL, "image.png", generateImg(), http.StatusOK)
 | |
| 
 | |
| 	commentCount := htmlDoc.doc.Find(".comment-list .comment .render-content").Length()
 | |
| 
 | |
| 	req = NewRequestWithValues(t, "POST", link, map[string]string{
 | |
| 		"_csrf":   htmlDoc.GetCSRF(),
 | |
| 		"content": content,
 | |
| 		"status":  status,
 | |
| 		"files":   uuid,
 | |
| 	})
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 	req = NewRequest(t, "GET", test.RedirectURL(resp))
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 	htmlDoc = NewHTMLParser(t, resp.Body)
 | |
| 
 | |
| 	val := htmlDoc.doc.Find(".comment-list .comment .render-content p").Eq(commentCount).Text()
 | |
| 	assert.Equal(t, content, val)
 | |
| 
 | |
| 	idAttr, has := htmlDoc.doc.Find(".comment-list .comment").Eq(commentCount).Attr("id")
 | |
| 	idStr := idAttr[strings.LastIndexByte(idAttr, '-')+1:]
 | |
| 	assert.True(t, has)
 | |
| 	id, err := strconv.Atoi(idStr)
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.NotEqual(t, 0, id)
 | |
| 
 | |
| 	req = NewRequest(t, "GET", fmt.Sprintf("/%s/%s/comments/%d/attachments", "user2", "repo1", id))
 | |
| 	session.MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 	// Using the ID of a comment that does not belong to the repository must fail
 | |
| 	req = NewRequest(t, "GET", fmt.Sprintf("/%s/%s/comments/%d/attachments", "user5", "repo4", id))
 | |
| 	session.MakeRequest(t, req, http.StatusNotFound)
 | |
| }
 | |
| 
 | |
| func TestIssueCommentUpdate(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 	session := loginUser(t, "user2")
 | |
| 	issueURL := testNewIssue(t, session, "user2", "repo1", "Title", "Description")
 | |
| 	comment1 := "Test comment 1"
 | |
| 	commentID := testIssueAddComment(t, session, issueURL, comment1, "")
 | |
| 
 | |
| 	comment := unittest.AssertExistsAndLoadBean(t, &issues_model.Comment{ID: commentID})
 | |
| 	assert.Equal(t, comment1, comment.Content)
 | |
| 
 | |
| 	modifiedContent := comment.Content + "MODIFIED"
 | |
| 
 | |
| 	// Using the ID of a comment that does not belong to the repository must fail
 | |
| 	req := NewRequestWithValues(t, "POST", fmt.Sprintf("/%s/%s/comments/%d", "user5", "repo4", commentID), map[string]string{
 | |
| 		"_csrf":   GetCSRF(t, session, issueURL),
 | |
| 		"content": modifiedContent,
 | |
| 	})
 | |
| 	session.MakeRequest(t, req, http.StatusNotFound)
 | |
| 
 | |
| 	req = NewRequestWithValues(t, "POST", fmt.Sprintf("/%s/%s/comments/%d", "user2", "repo1", commentID), map[string]string{
 | |
| 		"_csrf":   GetCSRF(t, session, issueURL),
 | |
| 		"content": modifiedContent,
 | |
| 	})
 | |
| 	session.MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 	comment = unittest.AssertExistsAndLoadBean(t, &issues_model.Comment{ID: commentID})
 | |
| 	assert.Equal(t, modifiedContent, comment.Content)
 | |
| }
 | |
| 
 | |
| func TestIssueReaction(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 	session := loginUser(t, "user2")
 | |
| 	issueURL := testNewIssue(t, session, "user2", "repo1", "Title", "Description")
 | |
| 
 | |
| 	req := NewRequest(t, "GET", issueURL)
 | |
| 	resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 	htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 
 | |
| 	req = NewRequestWithValues(t, "POST", path.Join(issueURL, "/reactions/react"), map[string]string{
 | |
| 		"_csrf":   htmlDoc.GetCSRF(),
 | |
| 		"content": "8ball",
 | |
| 	})
 | |
| 	session.MakeRequest(t, req, http.StatusInternalServerError)
 | |
| 	req = NewRequestWithValues(t, "POST", path.Join(issueURL, "/reactions/react"), map[string]string{
 | |
| 		"_csrf":   htmlDoc.GetCSRF(),
 | |
| 		"content": "eyes",
 | |
| 	})
 | |
| 	session.MakeRequest(t, req, http.StatusOK)
 | |
| 	req = NewRequestWithValues(t, "POST", path.Join(issueURL, "/reactions/unreact"), map[string]string{
 | |
| 		"_csrf":   htmlDoc.GetCSRF(),
 | |
| 		"content": "eyes",
 | |
| 	})
 | |
| 	session.MakeRequest(t, req, http.StatusOK)
 | |
| }
 | |
| 
 | |
| func TestIssueCrossReference(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 	// Issue that will be referenced
 | |
| 	_, issueBase := testIssueWithBean(t, "user2", 1, "Title", "Description")
 | |
| 
 | |
| 	// Ref from issue title
 | |
| 	issueRefURL, issueRef := testIssueWithBean(t, "user2", 1, fmt.Sprintf("Title ref #%d", issueBase.Index), "Description")
 | |
| 	unittest.AssertExistsAndLoadBean(t, &issues_model.Comment{
 | |
| 		IssueID:      issueBase.ID,
 | |
| 		RefRepoID:    1,
 | |
| 		RefIssueID:   issueRef.ID,
 | |
| 		RefCommentID: 0,
 | |
| 		RefIsPull:    false,
 | |
| 		RefAction:    references.XRefActionNone,
 | |
| 	})
 | |
| 
 | |
| 	// Edit title, neuter ref
 | |
| 	testIssueChangeInfo(t, "user2", issueRefURL, "title", "Title no ref")
 | |
| 	unittest.AssertExistsAndLoadBean(t, &issues_model.Comment{
 | |
| 		IssueID:      issueBase.ID,
 | |
| 		RefRepoID:    1,
 | |
| 		RefIssueID:   issueRef.ID,
 | |
| 		RefCommentID: 0,
 | |
| 		RefIsPull:    false,
 | |
| 		RefAction:    references.XRefActionNeutered,
 | |
| 	})
 | |
| 
 | |
| 	// Ref from issue content
 | |
| 	issueRefURL, issueRef = testIssueWithBean(t, "user2", 1, "TitleXRef", fmt.Sprintf("Description ref #%d", issueBase.Index))
 | |
| 	unittest.AssertExistsAndLoadBean(t, &issues_model.Comment{
 | |
| 		IssueID:      issueBase.ID,
 | |
| 		RefRepoID:    1,
 | |
| 		RefIssueID:   issueRef.ID,
 | |
| 		RefCommentID: 0,
 | |
| 		RefIsPull:    false,
 | |
| 		RefAction:    references.XRefActionNone,
 | |
| 	})
 | |
| 
 | |
| 	// Edit content, neuter ref
 | |
| 	testIssueChangeInfo(t, "user2", issueRefURL, "content", "Description no ref")
 | |
| 	unittest.AssertExistsAndLoadBean(t, &issues_model.Comment{
 | |
| 		IssueID:      issueBase.ID,
 | |
| 		RefRepoID:    1,
 | |
| 		RefIssueID:   issueRef.ID,
 | |
| 		RefCommentID: 0,
 | |
| 		RefIsPull:    false,
 | |
| 		RefAction:    references.XRefActionNeutered,
 | |
| 	})
 | |
| 
 | |
| 	// Ref from a comment
 | |
| 	session := loginUser(t, "user2")
 | |
| 	commentID := testIssueAddComment(t, session, issueRefURL, fmt.Sprintf("Adding ref from comment #%d", issueBase.Index), "")
 | |
| 	comment := &issues_model.Comment{
 | |
| 		IssueID:      issueBase.ID,
 | |
| 		RefRepoID:    1,
 | |
| 		RefIssueID:   issueRef.ID,
 | |
| 		RefCommentID: commentID,
 | |
| 		RefIsPull:    false,
 | |
| 		RefAction:    references.XRefActionNone,
 | |
| 	}
 | |
| 	unittest.AssertExistsAndLoadBean(t, comment)
 | |
| 
 | |
| 	// Ref from a different repository
 | |
| 	_, issueRef = testIssueWithBean(t, "user12", 10, "TitleXRef", fmt.Sprintf("Description ref user2/repo1#%d", issueBase.Index))
 | |
| 	unittest.AssertExistsAndLoadBean(t, &issues_model.Comment{
 | |
| 		IssueID:      issueBase.ID,
 | |
| 		RefRepoID:    10,
 | |
| 		RefIssueID:   issueRef.ID,
 | |
| 		RefCommentID: 0,
 | |
| 		RefIsPull:    false,
 | |
| 		RefAction:    references.XRefActionNone,
 | |
| 	})
 | |
| }
 | |
| 
 | |
| func testIssueWithBean(t *testing.T, user string, repoID int64, title, content string) (string, *issues_model.Issue) {
 | |
| 	session := loginUser(t, user)
 | |
| 	issueURL := testNewIssue(t, session, user, fmt.Sprintf("repo%d", repoID), title, content)
 | |
| 	indexStr := issueURL[strings.LastIndexByte(issueURL, '/')+1:]
 | |
| 	index, err := strconv.Atoi(indexStr)
 | |
| 	assert.NoError(t, err, "Invalid issue href: %s", issueURL)
 | |
| 	issue := &issues_model.Issue{RepoID: repoID, Index: int64(index)}
 | |
| 	unittest.AssertExistsAndLoadBean(t, issue)
 | |
| 	return issueURL, issue
 | |
| }
 | |
| 
 | |
| func testIssueChangeInfo(t *testing.T, user, issueURL, info, value string) {
 | |
| 	session := loginUser(t, user)
 | |
| 
 | |
| 	req := NewRequest(t, "GET", issueURL)
 | |
| 	resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 	htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 
 | |
| 	req = NewRequestWithValues(t, "POST", path.Join(issueURL, info), map[string]string{
 | |
| 		"_csrf": htmlDoc.GetCSRF(),
 | |
| 		info:    value,
 | |
| 	})
 | |
| 	_ = session.MakeRequest(t, req, http.StatusOK)
 | |
| }
 | |
| 
 | |
| func TestIssueRedirect(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 	session := loginUser(t, "user2")
 | |
| 
 | |
| 	// Test external tracker where style not set (shall default numeric)
 | |
| 	req := NewRequest(t, "GET", path.Join("org26", "repo_external_tracker", "issues", "1"))
 | |
| 	resp := session.MakeRequest(t, req, http.StatusSeeOther)
 | |
| 	assert.Equal(t, "https://tracker.com/org26/repo_external_tracker/issues/1", test.RedirectURL(resp))
 | |
| 
 | |
| 	// Test external tracker with numeric style
 | |
| 	req = NewRequest(t, "GET", path.Join("org26", "repo_external_tracker_numeric", "issues", "1"))
 | |
| 	resp = session.MakeRequest(t, req, http.StatusSeeOther)
 | |
| 	assert.Equal(t, "https://tracker.com/org26/repo_external_tracker_numeric/issues/1", test.RedirectURL(resp))
 | |
| 
 | |
| 	// Test external tracker with alphanumeric style (for a pull request)
 | |
| 	req = NewRequest(t, "GET", path.Join("org26", "repo_external_tracker_alpha", "issues", "1"))
 | |
| 	resp = session.MakeRequest(t, req, http.StatusSeeOther)
 | |
| 	assert.Equal(t, "/"+path.Join("org26", "repo_external_tracker_alpha", "pulls", "1"), test.RedirectURL(resp))
 | |
| }
 | |
| 
 | |
| func TestSearchIssues(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 	session := loginUser(t, "user2")
 | |
| 
 | |
| 	expectedIssueCount := 20 // from the fixtures
 | |
| 	if expectedIssueCount > setting.UI.IssuePagingNum {
 | |
| 		expectedIssueCount = setting.UI.IssuePagingNum
 | |
| 	}
 | |
| 
 | |
| 	link, _ := url.Parse("/issues/search")
 | |
| 	req := NewRequest(t, "GET", link.String())
 | |
| 	resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 	var apiIssues []*api.Issue
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, expectedIssueCount)
 | |
| 
 | |
| 	since := "2000-01-01T00:50:01+00:00" // 946687801
 | |
| 	before := time.Unix(999307200, 0).Format(time.RFC3339)
 | |
| 	query := url.Values{}
 | |
| 	query.Add("since", since)
 | |
| 	query.Add("before", before)
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, 11)
 | |
| 	query.Del("since")
 | |
| 	query.Del("before")
 | |
| 
 | |
| 	query.Add("state", "closed")
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, 2)
 | |
| 
 | |
| 	query.Set("state", "all")
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.EqualValues(t, "22", resp.Header().Get("X-Total-Count"))
 | |
| 	assert.Len(t, apiIssues, 20)
 | |
| 
 | |
| 	query.Add("limit", "5")
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.EqualValues(t, "22", resp.Header().Get("X-Total-Count"))
 | |
| 	assert.Len(t, apiIssues, 5)
 | |
| 
 | |
| 	query = url.Values{"assigned": {"true"}, "state": {"all"}}
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, 2)
 | |
| 
 | |
| 	query = url.Values{"milestones": {"milestone1"}, "state": {"all"}}
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, 1)
 | |
| 
 | |
| 	query = url.Values{"milestones": {"milestone1,milestone3"}, "state": {"all"}}
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, 2)
 | |
| 
 | |
| 	query = url.Values{"owner": {"user2"}} // user
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, 8)
 | |
| 
 | |
| 	query = url.Values{"owner": {"org3"}} // organization
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, 5)
 | |
| 
 | |
| 	query = url.Values{"owner": {"org3"}, "team": {"team1"}} // organization + team
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, 2)
 | |
| }
 | |
| 
 | |
| func TestSearchIssuesWithLabels(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 	expectedIssueCount := 20 // from the fixtures
 | |
| 	if expectedIssueCount > setting.UI.IssuePagingNum {
 | |
| 		expectedIssueCount = setting.UI.IssuePagingNum
 | |
| 	}
 | |
| 
 | |
| 	session := loginUser(t, "user1")
 | |
| 	link, _ := url.Parse("/issues/search")
 | |
| 	query := url.Values{}
 | |
| 	var apiIssues []*api.Issue
 | |
| 
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req := NewRequest(t, "GET", link.String())
 | |
| 	resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, expectedIssueCount)
 | |
| 
 | |
| 	query.Add("labels", "label1")
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, 2)
 | |
| 
 | |
| 	// multiple labels
 | |
| 	query.Set("labels", "label1,label2")
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, 2)
 | |
| 
 | |
| 	// an org label
 | |
| 	query.Set("labels", "orglabel4")
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, 1)
 | |
| 
 | |
| 	// org and repo label
 | |
| 	query.Set("labels", "label2,orglabel4")
 | |
| 	query.Add("state", "all")
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, 2)
 | |
| 
 | |
| 	// org and repo label which share the same issue
 | |
| 	query.Set("labels", "label1,orglabel4")
 | |
| 	link.RawQuery = query.Encode()
 | |
| 	req = NewRequest(t, "GET", link.String())
 | |
| 	resp = session.MakeRequest(t, req, http.StatusOK)
 | |
| 	DecodeJSON(t, resp, &apiIssues)
 | |
| 	assert.Len(t, apiIssues, 2)
 | |
| }
 | |
| 
 | |
| func TestGetIssueInfo(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 	issue := unittest.AssertExistsAndLoadBean(t, &issues_model.Issue{ID: 10})
 | |
| 	repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: issue.RepoID})
 | |
| 	owner := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: repo.OwnerID})
 | |
| 	assert.NoError(t, issue.LoadAttributes(db.DefaultContext))
 | |
| 	assert.Equal(t, int64(1019307200), int64(issue.DeadlineUnix))
 | |
| 	assert.Equal(t, api.StateOpen, issue.State())
 | |
| 
 | |
| 	session := loginUser(t, owner.Name)
 | |
| 
 | |
| 	urlStr := fmt.Sprintf("/%s/%s/issues/%d/info", owner.Name, repo.Name, issue.Index)
 | |
| 	req := NewRequest(t, "GET", urlStr)
 | |
| 	resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 	var apiIssue api.Issue
 | |
| 	DecodeJSON(t, resp, &apiIssue)
 | |
| 
 | |
| 	assert.EqualValues(t, issue.ID, apiIssue.ID)
 | |
| }
 | |
| 
 | |
| func TestIssuePinMove(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 	session := loginUser(t, "user2")
 | |
| 	issueURL, issue := testIssueWithBean(t, "user2", 1, "Title", "Content")
 | |
| 	assert.EqualValues(t, 0, issue.PinOrder)
 | |
| 
 | |
| 	req := NewRequestWithValues(t, "POST", fmt.Sprintf("%s/pin", issueURL), map[string]string{
 | |
| 		"_csrf": GetCSRF(t, session, issueURL),
 | |
| 	})
 | |
| 	session.MakeRequest(t, req, http.StatusOK)
 | |
| 	issue = unittest.AssertExistsAndLoadBean(t, &issues_model.Issue{ID: issue.ID})
 | |
| 
 | |
| 	position := 1
 | |
| 	assert.EqualValues(t, position, issue.PinOrder)
 | |
| 
 | |
| 	newPosition := 2
 | |
| 
 | |
| 	// Using the ID of an issue that does not belong to the repository must fail
 | |
| 	{
 | |
| 		session5 := loginUser(t, "user5")
 | |
| 		movePinURL := "/user5/repo4/issues/move_pin?_csrf=" + GetCSRF(t, session5, issueURL)
 | |
| 		req = NewRequestWithJSON(t, "POST", movePinURL, map[string]any{
 | |
| 			"id":       issue.ID,
 | |
| 			"position": newPosition,
 | |
| 		})
 | |
| 		session5.MakeRequest(t, req, http.StatusNotFound)
 | |
| 
 | |
| 		issue = unittest.AssertExistsAndLoadBean(t, &issues_model.Issue{ID: issue.ID})
 | |
| 		assert.EqualValues(t, position, issue.PinOrder)
 | |
| 	}
 | |
| 
 | |
| 	movePinURL := issueURL[:strings.LastIndexByte(issueURL, '/')] + "/move_pin?_csrf=" + GetCSRF(t, session, issueURL)
 | |
| 	req = NewRequestWithJSON(t, "POST", movePinURL, map[string]any{
 | |
| 		"id":       issue.ID,
 | |
| 		"position": newPosition,
 | |
| 	})
 | |
| 	session.MakeRequest(t, req, http.StatusNoContent)
 | |
| 
 | |
| 	issue = unittest.AssertExistsAndLoadBean(t, &issues_model.Issue{ID: issue.ID})
 | |
| 	assert.EqualValues(t, newPosition, issue.PinOrder)
 | |
| }
 | |
| 
 | |
| func TestUpdateIssueDeadline(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 	issueBefore := unittest.AssertExistsAndLoadBean(t, &issues_model.Issue{ID: 10})
 | |
| 	repoBefore := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: issueBefore.RepoID})
 | |
| 	owner := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: repoBefore.OwnerID})
 | |
| 	assert.NoError(t, issueBefore.LoadAttributes(db.DefaultContext))
 | |
| 	assert.Equal(t, int64(1019307200), int64(issueBefore.DeadlineUnix))
 | |
| 	assert.Equal(t, api.StateOpen, issueBefore.State())
 | |
| 
 | |
| 	session := loginUser(t, owner.Name)
 | |
| 
 | |
| 	issueURL := fmt.Sprintf("%s/%s/issues/%d", owner.Name, repoBefore.Name, issueBefore.Index)
 | |
| 	req := NewRequest(t, "GET", issueURL)
 | |
| 	resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 	htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 
 | |
| 	urlStr := issueURL + "/deadline?_csrf=" + htmlDoc.GetCSRF()
 | |
| 	req = NewRequestWithJSON(t, "POST", urlStr, map[string]string{
 | |
| 		"due_date": "2022-04-06T00:00:00.000Z",
 | |
| 	})
 | |
| 
 | |
| 	resp = session.MakeRequest(t, req, http.StatusCreated)
 | |
| 	var apiIssue api.IssueDeadline
 | |
| 	DecodeJSON(t, resp, &apiIssue)
 | |
| 
 | |
| 	assert.EqualValues(t, "2022-04-06", apiIssue.Deadline.Format("2006-01-02"))
 | |
| }
 | |
| 
 | |
| func TestIssueReferenceURL(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 	session := loginUser(t, "user2")
 | |
| 
 | |
| 	issue := unittest.AssertExistsAndLoadBean(t, &issues_model.Issue{ID: 1})
 | |
| 	repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: issue.RepoID})
 | |
| 
 | |
| 	req := NewRequest(t, "GET", fmt.Sprintf("%s/issues/%d", repo.FullName(), issue.Index))
 | |
| 	resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 	htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 
 | |
| 	// the "reference" uses relative URLs, then JS code will convert them to absolute URLs for current origin, in case users are using multiple domains
 | |
| 	ref, _ := htmlDoc.Find(`.timeline-item.comment.first .reference-issue`).Attr("data-reference")
 | |
| 	assert.EqualValues(t, "/user2/repo1/issues/1#issue-1", ref)
 | |
| 
 | |
| 	ref, _ = htmlDoc.Find(`.timeline-item.comment:not(.first) .reference-issue`).Attr("data-reference")
 | |
| 	assert.EqualValues(t, "/user2/repo1/issues/1#issuecomment-2", ref)
 | |
| }
 | |
| 
 | |
| func TestGetContentHistory(t *testing.T) {
 | |
| 	defer tests.AddFixtures("tests/integration/fixtures/TestGetContentHistory/")()
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 	issue := unittest.AssertExistsAndLoadBean(t, &issues_model.Issue{ID: 1})
 | |
| 	repo := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: issue.RepoID})
 | |
| 	issueURL := fmt.Sprintf("%s/issues/%d", repo.FullName(), issue.Index)
 | |
| 	contentHistory := unittest.AssertExistsAndLoadBean(t, &issues_model.ContentHistory{ID: 2, IssueID: issue.ID})
 | |
| 	contentHistoryURL := fmt.Sprintf("%s/issues/%d/content-history/detail?comment_id=%d&history_id=%d", repo.FullName(), issue.Index, contentHistory.CommentID, contentHistory.ID)
 | |
| 
 | |
| 	type contentHistoryResp struct {
 | |
| 		CanSoftDelete bool `json:"canSoftDelete"`
 | |
| 		HistoryID     int  `json:"historyId"`
 | |
| 		PrevHistoryID int  `json:"prevHistoryId"`
 | |
| 	}
 | |
| 
 | |
| 	testCase := func(t *testing.T, session *TestSession, canDelete bool) {
 | |
| 		t.Helper()
 | |
| 		contentHistoryURL := contentHistoryURL + "&_csrf=" + GetCSRF(t, session, issueURL)
 | |
| 
 | |
| 		req := NewRequest(t, "GET", contentHistoryURL)
 | |
| 		resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 
 | |
| 		var respJSON contentHistoryResp
 | |
| 		DecodeJSON(t, resp, &respJSON)
 | |
| 
 | |
| 		assert.EqualValues(t, canDelete, respJSON.CanSoftDelete)
 | |
| 		assert.EqualValues(t, contentHistory.ID, respJSON.HistoryID)
 | |
| 		assert.EqualValues(t, contentHistory.ID-1, respJSON.PrevHistoryID)
 | |
| 	}
 | |
| 
 | |
| 	t.Run("Anonymous", func(t *testing.T) {
 | |
| 		defer tests.PrintCurrentTest(t)()
 | |
| 		testCase(t, emptyTestSession(t), false)
 | |
| 	})
 | |
| 
 | |
| 	t.Run("Another user", func(t *testing.T) {
 | |
| 		defer tests.PrintCurrentTest(t)()
 | |
| 		testCase(t, loginUser(t, "user8"), false)
 | |
| 	})
 | |
| 
 | |
| 	t.Run("Repo owner", func(t *testing.T) {
 | |
| 		defer tests.PrintCurrentTest(t)()
 | |
| 		testCase(t, loginUser(t, "user2"), true)
 | |
| 	})
 | |
| 
 | |
| 	t.Run("Poster", func(t *testing.T) {
 | |
| 		defer tests.PrintCurrentTest(t)()
 | |
| 		testCase(t, loginUser(t, "user5"), true)
 | |
| 	})
 | |
| }
 | |
| 
 | |
| func TestCommitRefComment(t *testing.T) {
 | |
| 	defer tests.AddFixtures("tests/integration/fixtures/TestCommitRefComment/")()
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 	t.Run("Pull request", func(t *testing.T) {
 | |
| 		defer tests.PrintCurrentTest(t)()
 | |
| 
 | |
| 		req := NewRequest(t, "GET", "/user2/repo1/pulls/2")
 | |
| 		resp := MakeRequest(t, req, http.StatusOK)
 | |
| 		htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 
 | |
| 		event := htmlDoc.Find("#issuecomment-1000 .text").Text()
 | |
| 		assert.Contains(t, event, "referenced this pull request")
 | |
| 	})
 | |
| 
 | |
| 	t.Run("Issue", func(t *testing.T) {
 | |
| 		defer tests.PrintCurrentTest(t)()
 | |
| 
 | |
| 		req := NewRequest(t, "GET", "/user2/repo1/issues/1")
 | |
| 		resp := MakeRequest(t, req, http.StatusOK)
 | |
| 		htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 
 | |
| 		event := htmlDoc.Find("#issuecomment-1001 .text").Text()
 | |
| 		assert.Contains(t, event, "referenced this issue")
 | |
| 	})
 | |
| }
 | |
| 
 | |
| func TestIssueFilterNoFollow(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 	req := NewRequest(t, "GET", "/user2/repo1/issues")
 | |
| 	resp := MakeRequest(t, req, http.StatusOK)
 | |
| 	htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 
 | |
| 	// Check that every link in the filter list has rel="nofollow".
 | |
| 	filterLinks := htmlDoc.Find(".issue-list-toolbar-right a[href*=\"?q=\"]")
 | |
| 	assert.True(t, filterLinks.Length() > 0)
 | |
| 	filterLinks.Each(func(i int, link *goquery.Selection) {
 | |
| 		rel, has := link.Attr("rel")
 | |
| 		assert.True(t, has)
 | |
| 		assert.Equal(t, "nofollow", rel)
 | |
| 	})
 | |
| }
 | |
| 
 | |
| func TestIssueForm(t *testing.T) {
 | |
| 	onGiteaRun(t, func(t *testing.T, u *url.URL) {
 | |
| 		user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
 | |
| 		session := loginUser(t, user2.Name)
 | |
| 		repo, _, f := CreateDeclarativeRepo(t, user2, "",
 | |
| 			[]unit_model.Type{unit_model.TypeCode, unit_model.TypeIssues}, nil,
 | |
| 			[]*files_service.ChangeRepoFile{
 | |
| 				{
 | |
| 					Operation: "create",
 | |
| 					TreePath:  ".forgejo/issue_template/test.yaml",
 | |
| 					ContentReader: strings.NewReader(`name: Test
 | |
| about: Hello World
 | |
| body:
 | |
|   - type: checkboxes
 | |
|     id: test
 | |
|     attributes:
 | |
|       label: Test
 | |
|       options:
 | |
|         - label: This is a label
 | |
| `),
 | |
| 				},
 | |
| 			},
 | |
| 		)
 | |
| 		defer f()
 | |
| 
 | |
| 		t.Run("Choose list", func(t *testing.T) {
 | |
| 			defer tests.PrintCurrentTest(t)()
 | |
| 
 | |
| 			req := NewRequest(t, "GET", repo.Link()+"/issues/new/choose")
 | |
| 			resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 			htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 
 | |
| 			htmlDoc.AssertElement(t, "a[href$='/issues/new?template=.forgejo%2fissue_template%2ftest.yaml']", true)
 | |
| 		})
 | |
| 
 | |
| 		t.Run("Issue template", func(t *testing.T) {
 | |
| 			defer tests.PrintCurrentTest(t)()
 | |
| 
 | |
| 			req := NewRequest(t, "GET", repo.Link()+"/issues/new?template=.forgejo%2fissue_template%2ftest.yaml")
 | |
| 			resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 			htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 
 | |
| 			htmlDoc.AssertElement(t, "#new-issue .field .ui.checkbox input[name='form-field-test-0']", true)
 | |
| 			checkboxLabel := htmlDoc.Find("#new-issue .field .ui.checkbox label").Text()
 | |
| 			assert.Contains(t, checkboxLabel, "This is a label")
 | |
| 		})
 | |
| 	})
 | |
| }
 | |
| 
 | |
| func TestIssueUnsubscription(t *testing.T) {
 | |
| 	onGiteaRun(t, func(t *testing.T, u *url.URL) {
 | |
| 		defer tests.PrepareTestEnv(t)()
 | |
| 
 | |
| 		user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 1})
 | |
| 		repo, _, f := CreateDeclarativeRepoWithOptions(t, user, DeclarativeRepoOptions{
 | |
| 			AutoInit: optional.Some(false),
 | |
| 		})
 | |
| 		defer f()
 | |
| 		session := loginUser(t, user.Name)
 | |
| 
 | |
| 		issueURL := testNewIssue(t, session, user.Name, repo.Name, "Issue title", "Description")
 | |
| 		req := NewRequestWithValues(t, "POST", fmt.Sprintf("%s/watch", issueURL), map[string]string{
 | |
| 			"_csrf": GetCSRF(t, session, issueURL),
 | |
| 			"watch": "0",
 | |
| 		})
 | |
| 		session.MakeRequest(t, req, http.StatusOK)
 | |
| 	})
 | |
| }
 | |
| 
 | |
| func TestIssueLabelList(t *testing.T) {
 | |
| 	defer tests.PrepareTestEnv(t)()
 | |
| 	labelListSelector := ".labels.list .labels-list"
 | |
| 	hiddenClass := "tw-hidden"
 | |
| 
 | |
| 	t.Run("Show label list", func(t *testing.T) {
 | |
| 		defer tests.PrintCurrentTest(t)()
 | |
| 
 | |
| 		req := NewRequest(t, "GET", "/user2/repo1/issues/1")
 | |
| 		resp := MakeRequest(t, req, http.StatusOK)
 | |
| 		htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 
 | |
| 		htmlDoc.AssertElement(t, labelListSelector, true)
 | |
| 		htmlDoc.AssertElement(t, ".labels.list .no-select.item."+hiddenClass, true)
 | |
| 	})
 | |
| 
 | |
| 	t.Run("Show no label list", func(t *testing.T) {
 | |
| 		defer tests.PrintCurrentTest(t)()
 | |
| 		session := loginUser(t, "user2")
 | |
| 
 | |
| 		req := NewRequest(t, "GET", "/user2/repo2/issues/1")
 | |
| 		resp := session.MakeRequest(t, req, http.StatusOK)
 | |
| 		htmlDoc := NewHTMLParser(t, resp.Body)
 | |
| 
 | |
| 		htmlDoc.AssertElement(t, labelListSelector, false)
 | |
| 		htmlDoc.AssertElement(t, ".labels.list .no-select.item:not([class*='"+hiddenClass+"'])", true)
 | |
| 	})
 | |
| }
 |