diff options
| author | Sean McGivern <sean@gitlab.com> | 2018-03-02 15:03:03 +0300 |
|---|---|---|
| committer | Sean McGivern <sean@gitlab.com> | 2018-03-02 19:16:13 +0300 |
| commit | daeeb7f8480d747d500ea3aeddb479a29e890562 (patch) | |
| tree | 9c9a9f02a4f91ab51b8bf60ea6ca3b94f3cced0a /spec/features/issues/user_uses_slash_commands_spec.rb | |
| parent | 40c61acb6a9ba84928cebcbce8b57630bd439615 (diff) | |
Fix quick actions for users who cannot update issues and MRs
There are several quick actions now that don't need this access - /todo and
/unsubscribe for instance - but when these were first added, there
weren't. Quick actions are now responsible for checking their own permissions.
Diffstat (limited to 'spec/features/issues/user_uses_slash_commands_spec.rb')
| -rw-r--r-- | spec/features/issues/user_uses_slash_commands_spec.rb | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/spec/features/issues/user_uses_slash_commands_spec.rb b/spec/features/issues/user_uses_slash_commands_spec.rb index e711a191db2..ea7a97d02a0 100644 --- a/spec/features/issues/user_uses_slash_commands_spec.rb +++ b/spec/features/issues/user_uses_slash_commands_spec.rb @@ -59,7 +59,6 @@ feature 'Issues > User uses quick actions', :js do it 'does not create a note, and sets the due date accordingly' do write_note("/due 2016-08-28") - expect(page).to have_content '/due 2016-08-28' expect(page).not_to have_content 'Commands applied' issue.reload @@ -99,7 +98,6 @@ feature 'Issues > User uses quick actions', :js do it 'does not create a note, and sets the due date accordingly' do write_note("/remove_due_date") - expect(page).to have_content '/remove_due_date' expect(page).not_to have_content 'Commands applied' issue.reload @@ -147,7 +145,6 @@ feature 'Issues > User uses quick actions', :js do it 'does not create a note, and does not mark the issue as a duplicate' do write_note("/duplicate ##{original_issue.to_reference}") - expect(page).to have_content "/duplicate ##{original_issue.to_reference}" expect(page).not_to have_content 'Commands applied' expect(page).not_to have_content "marked this issue as a duplicate of #{original_issue.to_reference}" |