diff options
author | Luke Duncalfe <lduncalfe@eml.cc> | 2019-07-09 13:58:57 +1200 |
---|---|---|
committer | Luke Duncalfe <lduncalfe@eml.cc> | 2019-08-21 11:39:41 +1200 |
commit | 926bf71e511e084fa033b4e96f5b4067e0eeca0a (patch) | |
tree | a9015ce749112eba53fdba3bb3bba145615942e4 /spec/controllers/projects/issues_controller_spec.rb | |
parent | 15b02da69f1b3bfac859d076728a46586b53db6b (diff) | |
download | gitlab-ce-926bf71e511e084fa033b4e96f5b4067e0eeca0a.tar.gz |
Improve specs for Issues and Notes controllers63372-award-emoji-services
This adds test that Todos are completed.
https://gitlab.com/gitlab-org/gitlab-ce/issues/63372
Diffstat (limited to 'spec/controllers/projects/issues_controller_spec.rb')
-rw-r--r-- | spec/controllers/projects/issues_controller_spec.rb | 33 |
1 files changed, 27 insertions, 6 deletions
diff --git a/spec/controllers/projects/issues_controller_spec.rb b/spec/controllers/projects/issues_controller_spec.rb index fab47aa4701..187c7864ad7 100644 --- a/spec/controllers/projects/issues_controller_spec.rb +++ b/spec/controllers/projects/issues_controller_spec.rb @@ -1104,18 +1104,39 @@ describe Projects::IssuesController do project.add_developer(user) end + subject do + post(:toggle_award_emoji, params: { + namespace_id: project.namespace, + project_id: project, + id: issue.iid, + name: emoji_name + }) + end + let(:emoji_name) { 'thumbsup' } + it "toggles the award emoji" do expect do - post(:toggle_award_emoji, params: { - namespace_id: project.namespace, - project_id: project, - id: issue.iid, - name: "thumbsup" - }) + subject end.to change { issue.award_emoji.count }.by(1) expect(response).to have_gitlab_http_status(200) end + + it "removes the already awarded emoji" do + create(:award_emoji, awardable: issue, name: emoji_name, user: user) + + expect { subject }.to change { AwardEmoji.count }.by(-1) + + expect(response).to have_gitlab_http_status(200) + end + + it 'marks Todos on the Issue as done' do + todo = create(:todo, target: issue, project: project, user: user) + + subject + + expect(todo.reload).to be_done + end end describe 'POST create_merge_request' do |