diff options
author | Phil Hughes <me@iamphill.com> | 2018-03-08 17:35:35 +0000 |
---|---|---|
committer | Phil Hughes <me@iamphill.com> | 2018-03-08 17:35:35 +0000 |
commit | 18042714e8e5241d03980abc71ba26bef8535a9c (patch) | |
tree | 6a5f8d4ea9ba5a343f7b8093c845b9f90e9e1f3f | |
parent | adc5aabc2064dc1e187f463f8ebde730502402af (diff) | |
parent | 117ccb8926093ab97ea47e3216370112b2aafce1 (diff) | |
download | gitlab-ce-18042714e8e5241d03980abc71ba26bef8535a9c.tar.gz |
Merge branch 'jivl-fix-delete-milestone-button-showing-up' into 'master'
Fix delete milestone button showing up for all users
Closes #43778
See merge request gitlab-org/gitlab-ce!17576
-rw-r--r-- | app/views/shared/milestones/_milestone.html.haml | 20 | ||||
-rw-r--r-- | spec/features/milestone_spec.rb | 11 |
2 files changed, 21 insertions, 10 deletions
diff --git a/app/views/shared/milestones/_milestone.html.haml b/app/views/shared/milestones/_milestone.html.haml index 9db2a321526..5926867e2d7 100644 --- a/app/views/shared/milestones/_milestone.html.haml +++ b/app/views/shared/milestones/_milestone.html.haml @@ -63,13 +63,13 @@ = link_to 'Close Milestone', project_milestone_path(@project, milestone, milestone: {state_event: :close }), method: :put, remote: true, class: "btn btn-xs btn-close btn-grouped" - %button.js-delete-milestone-button.btn.btn-xs.btn-grouped.btn-danger{ data: { toggle: 'modal', - target: '#delete-milestone-modal', - milestone_id: milestone.id, - milestone_title: markdown_field(milestone, :title), - milestone_url: project_milestone_path(milestone.project, milestone), - milestone_issue_count: milestone.issues.count, - milestone_merge_request_count: milestone.merge_requests.count }, - disabled: true } - = _('Delete') - = icon('spin spinner', class: 'js-loading-icon hidden' ) + %button.js-delete-milestone-button.btn.btn-xs.btn-grouped.btn-danger{ data: { toggle: 'modal', + target: '#delete-milestone-modal', + milestone_id: milestone.id, + milestone_title: markdown_field(milestone, :title), + milestone_url: project_milestone_path(milestone.project, milestone), + milestone_issue_count: milestone.issues.count, + milestone_merge_request_count: milestone.merge_requests.count }, + disabled: true } + = _('Delete') + = icon('spin spinner', class: 'js-loading-icon hidden' ) diff --git a/spec/features/milestone_spec.rb b/spec/features/milestone_spec.rb index cc12a1005ba..19152bf1f0f 100644 --- a/spec/features/milestone_spec.rb +++ b/spec/features/milestone_spec.rb @@ -97,4 +97,15 @@ feature 'Milestone' do end end end + + feature 'Deleting a milestone' do + scenario "The delete milestone button does not show for unauthorized users" do + create(:milestone, project: project, title: 8.7) + sign_out(user) + + visit group_milestones_path(group) + + expect(page).to have_selector('.js-delete-milestone-button', count: 0) + end + end end |