Skip to content

Commit 94fff1f

Browse files
committed
Fix test
1 parent af01ce2 commit 94fff1f

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

models/issues/review_test.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -122,13 +122,20 @@ func TestGetReviewersByIssueID(t *testing.T) {
122122
assert.NoError(t, unittest.PrepareTestDatabase())
123123

124124
issue := unittest.AssertExistsAndLoadBean(t, &issues_model.Issue{ID: 3})
125+
user1 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 1})
125126
user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
126127
org3 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 3})
127128
user4 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 4})
128129
user5 := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 5})
129130

130131
expectedReviews := []*issues_model.Review{}
131132
expectedReviews = append(expectedReviews,
133+
&issues_model.Review{
134+
ID: 5,
135+
Reviewer: user1,
136+
Type: issues_model.ReviewTypeComment,
137+
UpdatedUnix: 946684810,
138+
},
132139
&issues_model.Review{
133140
ID: 7,
134141
Reviewer: org3,
@@ -169,6 +176,7 @@ func TestGetReviewersByIssueID(t *testing.T) {
169176
}
170177
if assert.Len(t, allReviews, 6) {
171178
for i, review := range allReviews {
179+
assert.Equal(t, expectedReviews[i].ID, review.ID)
172180
assert.Equal(t, expectedReviews[i].Reviewer, review.Reviewer)
173181
assert.Equal(t, expectedReviews[i].Type, review.Type)
174182
assert.Equal(t, expectedReviews[i].UpdatedUnix, review.UpdatedUnix)

0 commit comments

Comments
 (0)