diff options
author | Sean McGivern <sean@mcgivern.me.uk> | 2017-07-19 16:19:32 +0000 |
---|---|---|
committer | Sean McGivern <sean@mcgivern.me.uk> | 2017-07-19 16:19:32 +0000 |
commit | c6e979587af7cf8d85b4e97a8771d44aa04f3ef4 (patch) | |
tree | cd33f2ed61a0fb375228025de16f4065d0c233e9 /spec/controllers/projects/issues_controller_spec.rb | |
parent | b042765d2f56e6a7c64c2563bba2f795fde439bf (diff) | |
parent | a9d940bffcf7447f8d62012bd1c8f866697a12d6 (diff) | |
download | gitlab-ce-c6e979587af7cf8d85b4e97a8771d44aa04f3ef4.tar.gz |
Merge branch '34930-fix-edited-by' into 'master'
Fix issue detail if user who last edited an issue was deleted
Closes #34930
See merge request !12933
Diffstat (limited to 'spec/controllers/projects/issues_controller_spec.rb')
-rw-r--r-- | spec/controllers/projects/issues_controller_spec.rb | 30 |
1 files changed, 30 insertions, 0 deletions
diff --git a/spec/controllers/projects/issues_controller_spec.rb b/spec/controllers/projects/issues_controller_spec.rb index 1f9ca765233..18d0be3c103 100644 --- a/spec/controllers/projects/issues_controller_spec.rb +++ b/spec/controllers/projects/issues_controller_spec.rb @@ -516,6 +516,36 @@ describe Projects::IssuesController do end end + describe 'GET #realtime_changes' do + it_behaves_like 'restricted action', success: 200 + + def go(id:) + get :realtime_changes, + namespace_id: project.namespace.to_param, + project_id: project, + id: id + end + + context 'when an issue was edited by a deleted user' do + let(:deleted_user) { create(:user) } + + before do + project.team << [user, :developer] + + issue.update!(last_edited_by: deleted_user, last_edited_at: Time.now) + + deleted_user.destroy + sign_in(user) + end + + it 'returns 200' do + go(id: issue.iid) + + expect(response).to have_http_status(200) + end + end + end + describe 'GET #edit' do it_behaves_like 'restricted action', success: 200 |