Skip to content

Commit 45d8921

Browse files
committed
extend Test for TestAPIEditIssue
1 parent 6642286 commit 45d8921

File tree

3 files changed

+35
-3
lines changed

3 files changed

+35
-3
lines changed

integrations/api_issue_test.go

Lines changed: 26 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -70,15 +70,27 @@ func TestAPIEditIssue(t *testing.T) {
7070
repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: issueBefore.RepoID}).(*models.Repository)
7171
owner := models.AssertExistsAndLoadBean(t, &models.User{ID: repo.OwnerID}).(*models.User)
7272
assert.NoError(t, issueBefore.LoadAttributes())
73+
assert.Equal(t, int64(1019307200), int64(issueBefore.DeadlineUnix))
74+
assert.Equal(t, api.StateOpen, issueBefore.State())
7375

7476
session := loginUser(t, owner.Name)
7577
token := getTokenForLoggedInUser(t, session)
7678

79+
// update values of issue
7780
issueState := "closed"
81+
removeDeadline := true
82+
milestone := int64(4)
83+
body := "new content!"
84+
title := "new title from api set"
7885

7986
urlStr := fmt.Sprintf("/api/v1/repos/%s/%s/issues/%d?token=%s", owner.Name, repo.Name, issueBefore.Index, token)
8087
req := NewRequestWithJSON(t, "PATCH", urlStr, api.EditIssueOption{
81-
State: &issueState,
88+
State: &issueState,
89+
RemoveDeadline: &removeDeadline,
90+
Milestone: &milestone,
91+
Body: &body,
92+
Title: title,
93+
8294
// ToDo change more
8395
})
8496
resp := session.MakeRequest(t, req, http.StatusCreated)
@@ -87,13 +99,24 @@ func TestAPIEditIssue(t *testing.T) {
8799

88100
issueAfter := models.AssertExistsAndLoadBean(t, &models.Issue{ID: 10}).(*models.Issue)
89101

90-
assert.Equal(t, api.StateOpen, issueBefore.State())
91-
assert.Equal(t, api.StateClosed, issueAfter.State())
92102
// check deleted user
93103
assert.Equal(t, int64(500), issueAfter.PosterID)
94104
assert.NoError(t, issueAfter.LoadAttributes())
95105
assert.Equal(t, int64(-1), issueAfter.PosterID)
96106
assert.Equal(t, int64(-1), issueBefore.PosterID)
97107
assert.Equal(t, int64(-1), apiIssue.Poster.ID)
98108

109+
// API response
110+
assert.Equal(t, api.StateClosed, apiIssue.State)
111+
assert.Equal(t, milestone, apiIssue.Milestone.ID)
112+
assert.Equal(t, body, apiIssue.Body)
113+
assert.True(t, apiIssue.Deadline == nil)
114+
assert.Equal(t, title, apiIssue.Title)
115+
116+
// in database
117+
assert.Equal(t, api.StateClosed, issueAfter.State())
118+
assert.Equal(t, milestone, issueAfter.MilestoneID)
119+
assert.Equal(t, int64(0), int64(issueAfter.DeadlineUnix))
120+
assert.Equal(t, body, issueAfter.Content)
121+
assert.Equal(t, title, issueAfter.Title)
99122
}

models/fixtures/issue.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,3 +121,4 @@
121121
is_pull: false
122122
created_unix: 946684830
123123
updated_unix: 999307200
124+
deadline_unix: 1019307200

models/fixtures/milestone.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,3 +21,11 @@
2121
content: content3
2222
is_closed: true
2323
num_issues: 0
24+
25+
-
26+
id: 4
27+
repo_id: 42
28+
name: milestone of repo42
29+
content: content random
30+
is_closed: false
31+
num_issues: 0

0 commit comments

Comments
 (0)