diff options
author | Phil Hughes <me@iamphill.com> | 2016-07-29 15:22:26 +0100 |
---|---|---|
committer | Phil Hughes <me@iamphill.com> | 2016-07-29 15:22:26 +0100 |
commit | d147894b84d1978b967488cdc87716e0a524bfbc (patch) | |
tree | 879ca78159a528e3fca190ba15145735aa1141c3 /spec/features/merge_requests/diff_notes_resolve_spec.rb | |
parent | 9878356dbcf7100786c29a5effd0d9cd0f30434f (diff) | |
download | gitlab-ce-d147894b84d1978b967488cdc87716e0a524bfbc.tar.gz |
Hides jump to next button if only one discussion visible
Diffstat (limited to 'spec/features/merge_requests/diff_notes_resolve_spec.rb')
-rw-r--r-- | spec/features/merge_requests/diff_notes_resolve_spec.rb | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/spec/features/merge_requests/diff_notes_resolve_spec.rb b/spec/features/merge_requests/diff_notes_resolve_spec.rb index b991817bee0..a98f795b195 100644 --- a/spec/features/merge_requests/diff_notes_resolve_spec.rb +++ b/spec/features/merge_requests/diff_notes_resolve_spec.rb @@ -168,6 +168,12 @@ feature 'Diff notes resolve', feature: true, js: true do expect(page).to have_content("Resolved by #{user.name}") end + + it 'hides jump to next discussion button' do + page.within '.discussion-reply-holder' do + expect(page).not_to have_selector('.discussion-next-btn') + end + end end context 'multiple notes' do @@ -265,6 +271,12 @@ feature 'Diff notes resolve', feature: true, js: true do expect(page).to have_content("Resolved by #{user.name}") end + + it 'shows jump to next discussion button' do + page.all('.discussion-reply-holder').each do |holder| + expect(holder).to have_selector('.discussion-next-btn') + end + end end context 'changes tab' do |