mirror of
				https://codeberg.org/forgejo/forgejo.git
				synced 2025-10-25 11:33:11 +00:00 
			
		
		
		
	* API: Added pull review read only endpoints
* Update Structs, move Conversion, Refactor
* refactor
* lint & co
* fix lint + refactor
* add new Review state, rm unessesary, refacotr loadAttributes, convert patch to diff
* add DeletePullReview
* add paggination
* draft1: Create & submit review
* fix lint
* fix lint
* impruve test
* DONT use GhostUser for loadReviewer
* expose comments_count of a PullReview
* infent GetCodeCommentsCount()
* fixes
* fix+impruve
* some nits
* Handle Ghosts 👻
* add TEST for GET apis
* complete TESTS
* add HTMLURL to PullReview responce
* code format as per @lafriks
* update swagger definition
* Update routers/api/v1/repo/pull_review.go
Co-authored-by: David Svantesson <davidsvantesson@gmail.com>
* add comments
Co-authored-by: Thomas Berger <loki@lokis-chaos.de>
Co-authored-by: David Svantesson <davidsvantesson@gmail.com>
		
	
			
		
			
				
	
	
		
			141 lines
		
	
	
	
		
			4.2 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			141 lines
		
	
	
	
		
			4.2 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
| // 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 models
 | |
| 
 | |
| import (
 | |
| 	"testing"
 | |
| 
 | |
| 	"github.com/stretchr/testify/assert"
 | |
| )
 | |
| 
 | |
| func TestGetReviewByID(t *testing.T) {
 | |
| 	assert.NoError(t, PrepareTestDatabase())
 | |
| 	review, err := GetReviewByID(1)
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.Equal(t, "Demo Review", review.Content)
 | |
| 	assert.Equal(t, ReviewTypeApprove, review.Type)
 | |
| 
 | |
| 	_, err = GetReviewByID(23892)
 | |
| 	assert.Error(t, err)
 | |
| 	assert.True(t, IsErrReviewNotExist(err), "IsErrReviewNotExist")
 | |
| }
 | |
| 
 | |
| func TestReview_LoadAttributes(t *testing.T) {
 | |
| 	assert.NoError(t, PrepareTestDatabase())
 | |
| 	review := AssertExistsAndLoadBean(t, &Review{ID: 1}).(*Review)
 | |
| 	assert.NoError(t, review.LoadAttributes())
 | |
| 	assert.NotNil(t, review.Issue)
 | |
| 	assert.NotNil(t, review.Reviewer)
 | |
| 
 | |
| 	invalidReview1 := AssertExistsAndLoadBean(t, &Review{ID: 2}).(*Review)
 | |
| 	assert.Error(t, invalidReview1.LoadAttributes())
 | |
| 
 | |
| 	invalidReview2 := AssertExistsAndLoadBean(t, &Review{ID: 3}).(*Review)
 | |
| 	assert.Error(t, invalidReview2.LoadAttributes())
 | |
| 
 | |
| }
 | |
| 
 | |
| func TestReview_LoadCodeComments(t *testing.T) {
 | |
| 	assert.NoError(t, PrepareTestDatabase())
 | |
| 
 | |
| 	review := AssertExistsAndLoadBean(t, &Review{ID: 4}).(*Review)
 | |
| 	assert.NoError(t, review.LoadAttributes())
 | |
| 	assert.NoError(t, review.LoadCodeComments())
 | |
| 	assert.Len(t, review.CodeComments, 1)
 | |
| 	assert.Equal(t, int64(4), review.CodeComments["README.md"][int64(4)][0].Line)
 | |
| }
 | |
| 
 | |
| func TestReviewType_Icon(t *testing.T) {
 | |
| 	assert.Equal(t, "check", ReviewTypeApprove.Icon())
 | |
| 	assert.Equal(t, "request-changes", ReviewTypeReject.Icon())
 | |
| 	assert.Equal(t, "comment", ReviewTypeComment.Icon())
 | |
| 	assert.Equal(t, "comment", ReviewTypeUnknown.Icon())
 | |
| 	assert.Equal(t, "primitive-dot", ReviewTypeRequest.Icon())
 | |
| 	assert.Equal(t, "comment", ReviewType(6).Icon())
 | |
| }
 | |
| 
 | |
| func TestFindReviews(t *testing.T) {
 | |
| 	assert.NoError(t, PrepareTestDatabase())
 | |
| 	reviews, err := FindReviews(FindReviewOptions{
 | |
| 		Type:       ReviewTypeApprove,
 | |
| 		IssueID:    2,
 | |
| 		ReviewerID: 1,
 | |
| 	})
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.Len(t, reviews, 1)
 | |
| 	assert.Equal(t, "Demo Review", reviews[0].Content)
 | |
| }
 | |
| 
 | |
| func TestGetCurrentReview(t *testing.T) {
 | |
| 	assert.NoError(t, PrepareTestDatabase())
 | |
| 	issue := AssertExistsAndLoadBean(t, &Issue{ID: 2}).(*Issue)
 | |
| 	user := AssertExistsAndLoadBean(t, &User{ID: 1}).(*User)
 | |
| 
 | |
| 	review, err := GetCurrentReview(user, issue)
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.NotNil(t, review)
 | |
| 	assert.Equal(t, ReviewTypePending, review.Type)
 | |
| 	assert.Equal(t, "Pending Review", review.Content)
 | |
| 
 | |
| 	user2 := AssertExistsAndLoadBean(t, &User{ID: 7}).(*User)
 | |
| 	review2, err := GetCurrentReview(user2, issue)
 | |
| 	assert.Error(t, err)
 | |
| 	assert.True(t, IsErrReviewNotExist(err))
 | |
| 	assert.Nil(t, review2)
 | |
| }
 | |
| 
 | |
| func TestCreateReview(t *testing.T) {
 | |
| 	assert.NoError(t, PrepareTestDatabase())
 | |
| 
 | |
| 	issue := AssertExistsAndLoadBean(t, &Issue{ID: 2}).(*Issue)
 | |
| 	user := AssertExistsAndLoadBean(t, &User{ID: 1}).(*User)
 | |
| 
 | |
| 	review, err := CreateReview(CreateReviewOptions{
 | |
| 		Content:  "New Review",
 | |
| 		Type:     ReviewTypePending,
 | |
| 		Issue:    issue,
 | |
| 		Reviewer: user,
 | |
| 	})
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.Equal(t, "New Review", review.Content)
 | |
| 	AssertExistsAndLoadBean(t, &Review{Content: "New Review"})
 | |
| }
 | |
| 
 | |
| func TestGetReviewersByIssueID(t *testing.T) {
 | |
| 	assert.NoError(t, PrepareTestDatabase())
 | |
| 
 | |
| 	issue := AssertExistsAndLoadBean(t, &Issue{ID: 3}).(*Issue)
 | |
| 	user2 := AssertExistsAndLoadBean(t, &User{ID: 2}).(*User)
 | |
| 	user3 := AssertExistsAndLoadBean(t, &User{ID: 3}).(*User)
 | |
| 	user4 := AssertExistsAndLoadBean(t, &User{ID: 4}).(*User)
 | |
| 
 | |
| 	expectedReviews := []*Review{}
 | |
| 	expectedReviews = append(expectedReviews,
 | |
| 		&Review{
 | |
| 			Reviewer:    user3,
 | |
| 			Type:        ReviewTypeReject,
 | |
| 			UpdatedUnix: 946684812,
 | |
| 		},
 | |
| 		&Review{
 | |
| 			Reviewer:    user4,
 | |
| 			Type:        ReviewTypeApprove,
 | |
| 			UpdatedUnix: 946684813,
 | |
| 		},
 | |
| 		&Review{
 | |
| 			Reviewer:    user2,
 | |
| 			Type:        ReviewTypeReject,
 | |
| 			UpdatedUnix: 946684814,
 | |
| 		})
 | |
| 
 | |
| 	allReviews, err := GetReviewersByIssueID(issue.ID)
 | |
| 	assert.NoError(t, err)
 | |
| 	if assert.Len(t, allReviews, 3) {
 | |
| 		for i, review := range allReviews {
 | |
| 			assert.Equal(t, expectedReviews[i].Reviewer, review.Reviewer)
 | |
| 			assert.Equal(t, expectedReviews[i].Type, review.Type)
 | |
| 			assert.Equal(t, expectedReviews[i].UpdatedUnix, review.UpdatedUnix)
 | |
| 		}
 | |
| 	}
 | |
| }
 |