Skip to content

Commit 6c89803

Browse files
wxiaoguangwxiaoguang
committed
add tests
1 parent 480b2dd commit 6c89803

File tree

5 files changed

+167
-120
lines changed

5 files changed

+167
-120
lines changed

‎models/fixtures/comment.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,3 +75,11 @@
7575
content: "comment in private pository"
7676
created_unix: 946684811
7777
updated_unix: 946684811
78+
79+
-
80+
id: 9
81+
type: 22 # review
82+
poster_id: 2
83+
issue_id: 2 # in repo_id 1
84+
review_id: 20
85+
created_unix: 946684810

‎models/fixtures/review.yml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -170,3 +170,12 @@
170170
content: "review request for user15"
171171
updated_unix: 946684835
172172
created_unix: 946684835
173+
174+
-
175+
id: 20
176+
type: 22
177+
reviewer_id: 1
178+
issue_id: 2
179+
content: "Review Comment"
180+
updated_unix: 946684810
181+
created_unix: 946684810

‎routers/web/repo/pull_review_test.go

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@ import (
1313
"code.gitea.io/gitea/modules/context"
1414
"code.gitea.io/gitea/modules/contexttest"
1515
"code.gitea.io/gitea/modules/templates"
16+
"code.gitea.io/gitea/modules/test"
17+
1618
"code.gitea.io/gitea/services/pull"
1719

1820
"github.com/stretchr/testify/assert"
@@ -73,4 +75,16 @@ func TestRenderConversation(t *testing.T) {
7375
renderConversation(ctx, preparedComment, "timeline")
7476
assert.Contains(t, resp.Body.String(), `<div id="code-comments-`)
7577
})
78+
run("diff non-existing review", func(t *testing.T, ctx *context.Context, resp *httptest.ResponseRecorder) {
79+
err := db.TruncateBeans(db.DefaultContext, &issues_model.Review{})
80+
assert.NoError(t, err)
81+
renderConversation(ctx, preparedComment, "diff")
82+
assert.True(t, test.IsNormalPageCompleted(resp.Body.String()))
83+
})
84+
run("timeline non-existing review", func(t *testing.T, ctx *context.Context, resp *httptest.ResponseRecorder) {
85+
err := db.TruncateBeans(db.DefaultContext, &issues_model.Review{})
86+
assert.NoError(t, err)
87+
renderConversation(ctx, preparedComment, "timeline")
88+
assert.True(t, test.IsNormalPageCompleted(resp.Body.String()))
89+
})
7690
}

‎templates/repo/issue/view_content/conversation.tmpl

Lines changed: 124 additions & 118 deletions
Original file line numberDiff line numberDiff line change
@@ -1,133 +1,139 @@
1-
{{$invalid := (index .comments 0).Invalidated}}
2-
{{$resolved := (index .comments 0).IsResolved}}
3-
{{$resolveDoer := (index .comments 0).ResolveDoer}}
4-
{{$isNotPending := (not (eq (index .comments 0).Review.Type 0))}}
5-
<div class="ui segments conversation-holder">
6-
<div class="ui segment collapsible-comment-box gt-py-3 gt-df gt-ac gt-sb">
7-
<div class="gt-df gt-ac">
8-
<a href="{{(index .comments 0).CodeCommentLink ctx}}" class="file-comment gt-ml-3 gt-word-break">{{(index .comments 0).TreePath}}</a>
9-
{{if $invalid}}
10-
<span class="ui label basic small gt-ml-3" data-tooltip-content="{{ctx.Locale.Tr "repo.issues.review.outdated_description"}}">
11-
{{ctx.Locale.Tr "repo.issues.review.outdated"}}
12-
</span>
13-
{{end}}
14-
</div>
15-
<div>
16-
{{if or $invalid $resolved}}
17-
<button id="show-outdated-{{(index .comments 0).ID}}" data-comment="{{(index .comments 0).ID}}" class="{{if not $resolved}}gt-hidden {{end}}ui compact labeled button show-outdated gt-df gt-ac">
18-
{{svg "octicon-unfold" 16 "gt-mr-3"}}
19-
{{if $resolved}}
20-
{{ctx.Locale.Tr "repo.issues.review.show_resolved"}}
21-
{{else}}
22-
{{ctx.Locale.Tr "repo.issues.review.show_outdated"}}
23-
{{end}}
24-
</button>
25-
<button id="hide-outdated-{{(index .comments 0).ID}}" data-comment="{{(index .comments 0).ID}}" class="{{if $resolved}}gt-hidden {{end}}ui compact labeled button hide-outdated gt-df gt-ac">
26-
{{svg "octicon-fold" 16 "gt-mr-3"}}
27-
{{if $resolved}}
28-
{{ctx.Locale.Tr "repo.issues.review.hide_resolved"}}
29-
{{else}}
30-
{{ctx.Locale.Tr "repo.issues.review.hide_outdated"}}
31-
{{end}}
32-
</button>
33-
{{end}}
1+
{{if len .comments}}
2+
{{$comment := index .comments 0}}
3+
{{$invalid := $comment.Invalidated}}
4+
{{$resolved := $comment.IsResolved}}
5+
{{$resolveDoer := $comment.ResolveDoer}}
6+
{{$hasReview := and $comment.Review}}
7+
{{$isReviewPending := and $hasReview (eq $comment.Review.Type 0)}}
8+
<div class="ui segments conversation-holder">
9+
<div class="ui segment collapsible-comment-box gt-py-3 gt-df gt-ac gt-sb">
10+
<div class="gt-df gt-ac">
11+
<a href="{{$comment.CodeCommentLink ctx}}" class="file-comment gt-ml-3 gt-word-break">{{$comment.TreePath}}</a>
12+
{{if $invalid}}
13+
<span class="ui label basic small gt-ml-3" data-tooltip-content="{{ctx.Locale.Tr "repo.issues.review.outdated_description"}}">
14+
{{ctx.Locale.Tr "repo.issues.review.outdated"}}
15+
</span>
16+
{{end}}
17+
</div>
18+
<div>
19+
{{if or $invalid $resolved}}
20+
<button id="show-outdated-{{$comment.ID}}" data-comment="{{$comment.ID}}" class="{{if not $resolved}}gt-hidden {{end}}ui compact labeled button show-outdated gt-df gt-ac">
21+
{{svg "octicon-unfold" 16 "gt-mr-3"}}
22+
{{if $resolved}}
23+
{{ctx.Locale.Tr "repo.issues.review.show_resolved"}}
24+
{{else}}
25+
{{ctx.Locale.Tr "repo.issues.review.show_outdated"}}
26+
{{end}}
27+
</button>
28+
<button id="hide-outdated-{{$comment.ID}}" data-comment="{{$comment.ID}}" class="{{if $resolved}}gt-hidden {{end}}ui compact labeled button hide-outdated gt-df gt-ac">
29+
{{svg "octicon-fold" 16 "gt-mr-3"}}
30+
{{if $resolved}}
31+
{{ctx.Locale.Tr "repo.issues.review.hide_resolved"}}
32+
{{else}}
33+
{{ctx.Locale.Tr "repo.issues.review.hide_outdated"}}
34+
{{end}}
35+
</button>
36+
{{end}}
37+
</div>
3438
</div>
35-
</div>
36-
{{$diff := (CommentMustAsDiff (index .comments 0))}}
37-
{{if $diff}}
38-
{{$file := (index $diff.Files 0)}}
39-
<div id="code-preview-{{(index .comments 0).ID}}" class="ui table segment{{if $resolved}} gt-hidden{{end}}">
40-
<div class="diff-file-box diff-box file-content {{TabSizeClass $.Editorconfig $file.Name}}">
41-
<div class="file-body file-code code-view code-diff code-diff-unified unicode-escaped">
42-
<table>
43-
<tbody>
44-
{{template "repo/diff/section_unified" dict "file" $file "root" $}}
45-
</tbody>
46-
</table>
39+
{{$diff := (CommentMustAsDiff $comment)}}
40+
{{if $diff}}
41+
{{$file := (index $diff.Files 0)}}
42+
<div id="code-preview-{{$comment.ID}}" class="ui table segment{{if $resolved}} gt-hidden{{end}}">
43+
<div class="diff-file-box diff-box file-content {{TabSizeClass $.Editorconfig $file.Name}}">
44+
<div class="file-body file-code code-view code-diff code-diff-unified unicode-escaped">
45+
<table>
46+
<tbody>
47+
{{template "repo/diff/section_unified" dict "file" $file "root" $}}
48+
</tbody>
49+
</table>
50+
</div>
4751
</div>
4852
</div>
49-
</div>
50-
{{end}}
51-
<div id="code-comments-{{(index .comments 0).ID}}" class="comment-code-cloud ui segment{{if $resolved}} gt-hidden{{end}}">
52-
<div class="ui comments gt-mb-0">
53-
{{range .comments}}
54-
{{$createdSubStr:= TimeSinceUnix .CreatedUnix ctx.Locale}}
55-
<div class="comment code-comment gt-pb-4" id="{{.HashTag}}">
56-
<div class="content">
57-
<div class="header comment-header">
58-
<div class="comment-header-left gt-df gt-ac">
59-
{{if not .OriginalAuthor}}
60-
<a class="avatar">
61-
{{ctx.AvatarUtils.Avatar .Poster 20}}
62-
</a>
63-
{{end}}
64-
<span class="text grey muted-links">
65-
{{if .OriginalAuthor}}
66-
<span class="text black gt-font-semibold">
67-
{{svg (MigrationIcon $.Repository.GetOriginalURLHostname)}}
68-
{{.OriginalAuthor}}
69-
</span>
70-
<span class="text grey muted-links"> {{if $.Repository.OriginalURL}}</span>
71-
<span class="text migrate">({{ctx.Locale.Tr "repo.migrated_from" ($.Repository.OriginalURL|Escape) ($.Repository.GetOriginalURLHostname|Escape) | Safe}}){{end}}</span>
72-
{{else}}
73-
{{template "shared/user/authorlink" .Poster}}
53+
{{end}}
54+
<div id="code-comments-{{$comment.ID}}" class="comment-code-cloud ui segment{{if $resolved}} gt-hidden{{end}}">
55+
<div class="ui comments gt-mb-0">
56+
{{range .comments}}
57+
{{$createdSubStr:= TimeSinceUnix .CreatedUnix ctx.Locale}}
58+
<div class="comment code-comment gt-pb-4" id="{{.HashTag}}">
59+
<div class="content">
60+
<div class="header comment-header">
61+
<div class="comment-header-left gt-df gt-ac">
62+
{{if not .OriginalAuthor}}
63+
<a class="avatar">
64+
{{ctx.AvatarUtils.Avatar .Poster 20}}
65+
</a>
7466
{{end}}
75-
{{ctx.Locale.Tr "repo.issues.commented_at" (.HashTag|Escape) $createdSubStr | Safe}}
76-
</span>
67+
<span class="text grey muted-links">
68+
{{if .OriginalAuthor}}
69+
<span class="text black gt-font-semibold">
70+
{{svg (MigrationIcon $.Repository.GetOriginalURLHostname)}}
71+
{{.OriginalAuthor}}
72+
</span>
73+
<span class="text grey muted-links"> {{if $.Repository.OriginalURL}}</span>
74+
<span class="text migrate">({{ctx.Locale.Tr "repo.migrated_from" ($.Repository.OriginalURL|Escape) ($.Repository.GetOriginalURLHostname|Escape) | Safe}}){{end}}</span>
75+
{{else}}
76+
{{template "shared/user/authorlink" .Poster}}
77+
{{end}}
78+
{{ctx.Locale.Tr "repo.issues.commented_at" (.HashTag|Escape) $createdSubStr | Safe}}
79+
</span>
80+
</div>
81+
<div class="comment-header-right actions gt-df gt-ac">
82+
{{template "repo/issue/view_content/show_role" dict "ShowRole" .ShowRole}}
83+
{{if not $.Repository.IsArchived}}
84+
{{template "repo/issue/view_content/add_reaction" dict "ctxData" $ "ActionURL" (printf "%s/comments/%d/reactions" $.RepoLink .ID)}}
85+
{{template "repo/issue/view_content/context_menu" dict "ctxData" $ "item" . "delete" true "issue" true "diff" true "IsCommentPoster" (and $.IsSigned (eq $.SignedUserID .PosterID))}}
86+
{{end}}
87+
</div>
7788
</div>
78-
<div class="comment-header-right actions gt-df gt-ac">
79-
{{template "repo/issue/view_content/show_role" dict "ShowRole" .ShowRole}}
80-
{{if not $.Repository.IsArchived}}
81-
{{template "repo/issue/view_content/add_reaction" dict "ctxData" $ "ActionURL" (printf "%s/comments/%d/reactions" $.RepoLink .ID)}}
82-
{{template "repo/issue/view_content/context_menu" dict "ctxData" $ "item" . "delete" true "issue" true "diff" true "IsCommentPoster" (and $.IsSigned (eq $.SignedUserID .PosterID))}}
89+
<div class="text comment-content">
90+
<div class="render-content markup" {{if or $.Permission.IsAdmin $.HasIssuesOrPullsWritePermission (and $.IsSigned (eq $.SignedUserID .PosterID))}}data-can-edit="true"{{end}}>
91+
{{if .RenderedContent}}
92+
{{.RenderedContent|Str2html}}
93+
{{else}}
94+
<span class="no-content">{{ctx.Locale.Tr "repo.issues.no_content"}}</span>
8395
{{end}}
96+
</div>
97+
<div id="issuecomment-{{.ID}}-raw" class="raw-content gt-hidden">{{.Content}}</div>
98+
<div class="edit-content-zone gt-hidden" data-update-url="{{$.RepoLink}}/comments/{{.ID}}" data-context="{{$.RepoLink}}" data-attachment-url="{{$.RepoLink}}/comments/{{.ID}}/attachments"></div>
8499
</div>
85-
</div>
86-
<div class="text comment-content">
87-
<div class="render-content markup" {{if or $.Permission.IsAdmin $.HasIssuesOrPullsWritePermission (and $.IsSigned (eq $.SignedUserID .PosterID))}}data-can-edit="true"{{end}}>
88-
{{if .RenderedContent}}
89-
{{.RenderedContent|Str2html}}
90-
{{else}}
91-
<span class="no-content">{{ctx.Locale.Tr "repo.issues.no_content"}}</span>
100+
{{$reactions := .Reactions.GroupByType}}
101+
{{if $reactions}}
102+
{{template "repo/issue/view_content/reactions" dict "ctxData" $ "ActionURL" (printf "%s/comments/%d/reactions" $.RepoLink .ID) "Reactions" $reactions}}
92103
{{end}}
93-
</div>
94-
<div id="issuecomment-{{.ID}}-raw" class="raw-content gt-hidden">{{.Content}}</div>
95-
<div class="edit-content-zone gt-hidden" data-update-url="{{$.RepoLink}}/comments/{{.ID}}" data-context="{{$.RepoLink}}" data-attachment-url="{{$.RepoLink}}/comments/{{.ID}}/attachments"></div>
96104
</div>
97-
{{$reactions := .Reactions.GroupByType}}
98-
{{if $reactions}}
99-
{{template "repo/issue/view_content/reactions" dict "ctxData" $ "ActionURL" (printf "%s/comments/%d/reactions" $.RepoLink .ID) "Reactions" $reactions}}
100-
{{end}}
101-
</div>
102-
</div>
103-
{{end}}
104-
</div>
105-
<div class="code-comment-buttons gt-df gt-ac gt-fw gt-mt-3 gt-mb-2 gt-mx-3">
106-
<div class="gt-f1">
107-
{{if $resolved}}
108-
<div class="ui grey text">
109-
{{svg "octicon-check" 16 "gt-mr-2"}}
110-
<b>{{$resolveDoer.Name}}</b> {{ctx.Locale.Tr "repo.issues.review.resolved_by"}}
111105
</div>
112106
{{end}}
113107
</div>
114-
<div class="code-comment-buttons-buttons">
115-
{{if and $.CanMarkConversation $isNotPending}}
116-
<button class="ui tiny basic button resolve-conversation" data-origin="timeline" data-action="{{if not $resolved}}Resolve{{else}}UnResolve{{end}}" data-comment-id="{{(index .comments 0).ID}}" data-update-url="{{$.RepoLink}}/issues/resolve_conversation">
117-
{{if $resolved}}
118-
{{ctx.Locale.Tr "repo.issues.review.un_resolve_conversation"}}
119-
{{else}}
120-
{{ctx.Locale.Tr "repo.issues.review.resolve_conversation"}}
121-
{{end}}
122-
</button>
123-
{{end}}
124-
{{if and $.SignedUserID (not $.Repository.IsArchived)}}
125-
<button class="comment-form-reply ui primary tiny labeled icon button gt-ml-2 gt-mr-0">
126-
{{svg "octicon-reply" 16 "reply icon gt-mr-2"}}{{ctx.Locale.Tr "repo.diff.comment.reply"}}
127-
</button>
128-
{{end}}
108+
<div class="code-comment-buttons gt-df gt-ac gt-fw gt-mt-3 gt-mb-2 gt-mx-3">
109+
<div class="gt-f1">
110+
{{if $resolved}}
111+
<div class="ui grey text">
112+
{{svg "octicon-check" 16 "gt-mr-2"}}
113+
<b>{{$resolveDoer.Name}}</b> {{ctx.Locale.Tr "repo.issues.review.resolved_by"}}
114+
</div>
115+
{{end}}
116+
</div>
117+
<div class="code-comment-buttons-buttons">
118+
{{if and $.CanMarkConversation $hasReview (not $isReviewPending)}}
119+
<button class="ui tiny basic button resolve-conversation" data-origin="timeline" data-action="{{if not $resolved}}Resolve{{else}}UnResolve{{end}}" data-comment-id="{{$comment.ID}}" data-update-url="{{$.RepoLink}}/issues/resolve_conversation">
120+
{{if $resolved}}
121+
{{ctx.Locale.Tr "repo.issues.review.un_resolve_conversation"}}
122+
{{else}}
123+
{{ctx.Locale.Tr "repo.issues.review.resolve_conversation"}}
124+
{{end}}
125+
</button>
126+
{{end}}
127+
{{if and $.SignedUserID (not $.Repository.IsArchived)}}
128+
<button class="comment-form-reply ui primary tiny labeled icon button gt-ml-2 gt-mr-0">
129+
{{svg "octicon-reply" 16 "reply icon gt-mr-2"}}{{ctx.Locale.Tr "repo.diff.comment.reply"}}
130+
</button>
131+
{{end}}
132+
</div>
129133
</div>
134+
{{template "repo/diff/comment_form_datahandler" dict "hidden" true "reply" $comment.ReviewID "root" $ "comment" $comment}}
130135
</div>
131-
{{template "repo/diff/comment_form_datahandler" dict "hidden" true "reply" (index .comments 0).ReviewID "root" $ "comment" (index .comments 0)}}
132136
</div>
133-
</div>
137+
{{else}}
138+
{{template "repo/diff/conversation_outdated"}}
139+
{{end}}

‎tests/integration/pull_review_test.go

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ import (
1313
issues_model "code.gitea.io/gitea/models/issues"
1414
"code.gitea.io/gitea/models/unittest"
1515
user_model "code.gitea.io/gitea/models/user"
16+
"code.gitea.io/gitea/modules/test"
1617
repo_service "code.gitea.io/gitea/services/repository"
1718
files_service "code.gitea.io/gitea/services/repository/files"
1819
"code.gitea.io/gitea/tests"
@@ -25,10 +26,19 @@ func TestPullView_ReviewerMissed(t *testing.T) {
2526
session := loginUser(t, "user1")
2627

2728
req := NewRequest(t, "GET", "/pulls")
28-
session.MakeRequest(t, req, http.StatusOK)
29+
resp := session.MakeRequest(t, req, http.StatusOK)
30+
assert.True(t, test.IsNormalPageCompleted(resp.Body.String()))
2931

3032
req = NewRequest(t, "GET", "/user2/repo1/pulls/3")
31-
session.MakeRequest(t, req, http.StatusOK)
33+
resp = session.MakeRequest(t, req, http.StatusOK)
34+
assert.True(t, test.IsNormalPageCompleted(resp.Body.String()))
35+
36+
// if some reviews are missing, the page shouldn't fail
37+
err := db.TruncateBeans(db.DefaultContext, &issues_model.Review{})
38+
assert.NoError(t, err)
39+
req = NewRequest(t, "GET", "/user2/repo1/pulls/2")
40+
resp = session.MakeRequest(t, req, http.StatusOK)
41+
assert.True(t, test.IsNormalPageCompleted(resp.Body.String()))
3242
}
3343

3444
func TestPullView_CodeOwner(t *testing.T) {

0 commit comments

Comments
 (0)