More integration tests for comment API (#2156)
This commit is contained in:
parent
c4ccf16c43
commit
dd758ad8dc
1 changed files with 90 additions and 1 deletions
|
@ -5,6 +5,7 @@
|
||||||
package integrations
|
package integrations
|
||||||
|
|
||||||
import (
|
import (
|
||||||
|
"fmt"
|
||||||
"net/http"
|
"net/http"
|
||||||
"testing"
|
"testing"
|
||||||
|
|
||||||
|
@ -14,7 +15,31 @@ import (
|
||||||
"github.com/stretchr/testify/assert"
|
"github.com/stretchr/testify/assert"
|
||||||
)
|
)
|
||||||
|
|
||||||
func TestAPIListComments(t *testing.T) {
|
func TestAPIListRepoComments(t *testing.T) {
|
||||||
|
prepareTestEnv(t)
|
||||||
|
|
||||||
|
comment := models.AssertExistsAndLoadBean(t, &models.Comment{},
|
||||||
|
models.Cond("type = ?", models.CommentTypeComment)).(*models.Comment)
|
||||||
|
issue := models.AssertExistsAndLoadBean(t, &models.Issue{ID: comment.IssueID}).(*models.Issue)
|
||||||
|
repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: issue.RepoID}).(*models.Repository)
|
||||||
|
repoOwner := models.AssertExistsAndLoadBean(t, &models.User{ID: repo.OwnerID}).(*models.User)
|
||||||
|
|
||||||
|
session := loginUser(t, repoOwner.Name)
|
||||||
|
req := NewRequestf(t, "GET", "/api/v1/repos/%s/%s/issues/comments",
|
||||||
|
repoOwner.Name, repo.Name)
|
||||||
|
resp := session.MakeRequest(t, req, http.StatusOK)
|
||||||
|
|
||||||
|
var apiComments []*api.Comment
|
||||||
|
DecodeJSON(t, resp, &apiComments)
|
||||||
|
for _, apiComment := range apiComments {
|
||||||
|
c := &models.Comment{ID: apiComment.ID}
|
||||||
|
models.AssertExistsAndLoadBean(t, c,
|
||||||
|
models.Cond("type = ?", models.CommentTypeComment))
|
||||||
|
models.AssertExistsAndLoadBean(t, &models.Issue{ID: c.IssueID, RepoID: repo.ID})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestAPIListIssueComments(t *testing.T) {
|
||||||
prepareTestEnv(t)
|
prepareTestEnv(t)
|
||||||
|
|
||||||
comment := models.AssertExistsAndLoadBean(t, &models.Comment{},
|
comment := models.AssertExistsAndLoadBean(t, &models.Comment{},
|
||||||
|
@ -34,3 +59,67 @@ func TestAPIListComments(t *testing.T) {
|
||||||
models.Cond("type = ?", models.CommentTypeComment))
|
models.Cond("type = ?", models.CommentTypeComment))
|
||||||
assert.EqualValues(t, expectedCount, len(comments))
|
assert.EqualValues(t, expectedCount, len(comments))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestAPICreateComment(t *testing.T) {
|
||||||
|
prepareTestEnv(t)
|
||||||
|
const commentBody = "Comment body"
|
||||||
|
|
||||||
|
issue := models.AssertExistsAndLoadBean(t, &models.Issue{}).(*models.Issue)
|
||||||
|
repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: issue.RepoID}).(*models.Repository)
|
||||||
|
repoOwner := models.AssertExistsAndLoadBean(t, &models.User{ID: repo.OwnerID}).(*models.User)
|
||||||
|
|
||||||
|
session := loginUser(t, repoOwner.Name)
|
||||||
|
urlStr := fmt.Sprintf("/api/v1/repos/%s/%s/issues/%d/comments",
|
||||||
|
repoOwner.Name, repo.Name, issue.Index)
|
||||||
|
req := NewRequestWithValues(t, "POST", urlStr, map[string]string{
|
||||||
|
"body": commentBody,
|
||||||
|
})
|
||||||
|
resp := session.MakeRequest(t, req, http.StatusCreated)
|
||||||
|
|
||||||
|
var updatedComment api.Comment
|
||||||
|
DecodeJSON(t, resp, &updatedComment)
|
||||||
|
assert.EqualValues(t, commentBody, updatedComment.Body)
|
||||||
|
models.AssertExistsAndLoadBean(t, &models.Comment{ID: updatedComment.ID, IssueID: issue.ID, Content: commentBody})
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestAPIEditComment(t *testing.T) {
|
||||||
|
prepareTestEnv(t)
|
||||||
|
const newCommentBody = "This is the new comment body"
|
||||||
|
|
||||||
|
comment := models.AssertExistsAndLoadBean(t, &models.Comment{},
|
||||||
|
models.Cond("type = ?", models.CommentTypeComment)).(*models.Comment)
|
||||||
|
issue := models.AssertExistsAndLoadBean(t, &models.Issue{ID: comment.IssueID}).(*models.Issue)
|
||||||
|
repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: issue.RepoID}).(*models.Repository)
|
||||||
|
repoOwner := models.AssertExistsAndLoadBean(t, &models.User{ID: repo.OwnerID}).(*models.User)
|
||||||
|
|
||||||
|
session := loginUser(t, repoOwner.Name)
|
||||||
|
urlStr := fmt.Sprintf("/api/v1/repos/%s/%s/issues/%d/comments/%d",
|
||||||
|
repoOwner.Name, repo.Name, issue.Index, comment.ID)
|
||||||
|
req := NewRequestWithValues(t, "PATCH", urlStr, map[string]string{
|
||||||
|
"body": newCommentBody,
|
||||||
|
})
|
||||||
|
resp := session.MakeRequest(t, req, http.StatusOK)
|
||||||
|
|
||||||
|
var updatedComment api.Comment
|
||||||
|
DecodeJSON(t, resp, &updatedComment)
|
||||||
|
assert.EqualValues(t, comment.ID, updatedComment.ID)
|
||||||
|
assert.EqualValues(t, newCommentBody, updatedComment.Body)
|
||||||
|
models.AssertExistsAndLoadBean(t, &models.Comment{ID: comment.ID, IssueID: issue.ID, Content: newCommentBody})
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestAPIDeleteComment(t *testing.T) {
|
||||||
|
prepareTestEnv(t)
|
||||||
|
|
||||||
|
comment := models.AssertExistsAndLoadBean(t, &models.Comment{},
|
||||||
|
models.Cond("type = ?", models.CommentTypeComment)).(*models.Comment)
|
||||||
|
issue := models.AssertExistsAndLoadBean(t, &models.Issue{ID: comment.IssueID}).(*models.Issue)
|
||||||
|
repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: issue.RepoID}).(*models.Repository)
|
||||||
|
repoOwner := models.AssertExistsAndLoadBean(t, &models.User{ID: repo.OwnerID}).(*models.User)
|
||||||
|
|
||||||
|
session := loginUser(t, repoOwner.Name)
|
||||||
|
req := NewRequestf(t, "DELETE", "/api/v1/repos/%s/%s/issues/%d/comments/%d",
|
||||||
|
repoOwner.Name, repo.Name, issue.Index, comment.ID)
|
||||||
|
session.MakeRequest(t, req, http.StatusNoContent)
|
||||||
|
|
||||||
|
models.AssertNotExistsBean(t, &models.Comment{ID: comment.ID})
|
||||||
|
}
|
||||||
|
|
Reference in a new issue