diff options
Diffstat (limited to 'spec/features/merge_request/user_sees_avatar_on_diff_notes_spec.rb')
-rw-r--r-- | spec/features/merge_request/user_sees_avatar_on_diff_notes_spec.rb | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/spec/features/merge_request/user_sees_avatar_on_diff_notes_spec.rb b/spec/features/merge_request/user_sees_avatar_on_diff_notes_spec.rb index 64cd5aa2bb1..33c5a936b8d 100644 --- a/spec/features/merge_request/user_sees_avatar_on_diff_notes_spec.rb +++ b/spec/features/merge_request/user_sees_avatar_on_diff_notes_spec.rb @@ -1,9 +1,12 @@ # frozen_string_literal: true require 'spec_helper' +include Spec::Support::Helpers::ModalHelpers # rubocop:disable Style/MixinUsage RSpec.describe 'Merge request > User sees avatars on diff notes', :js do include NoteInteractionHelpers + include Spec::Support::Helpers::ModalHelpers + include MergeRequestDiffHelpers let(:project) { create(:project, :public, :repository) } let(:user) { project.creator } @@ -121,8 +124,8 @@ RSpec.describe 'Merge request > User sees avatars on diff notes', :js do it 'removes avatar when note is deleted' do open_more_actions_dropdown(note) - page.within find(".note-row-#{note.id}") do - accept_confirm { find('.js-note-delete').click } + accept_gl_confirm(button_text: 'Delete Comment') do + find(".note-row-#{note.id} .js-note-delete").click end wait_for_requests @@ -133,6 +136,7 @@ RSpec.describe 'Merge request > User sees avatars on diff notes', :js do end it 'adds avatar when commenting' do + find_by_scrolling('[data-discussion-id]', match: :first) find_field('Reply…', match: :first).click page.within '.js-discussion-note-form' do @@ -152,6 +156,7 @@ RSpec.describe 'Merge request > User sees avatars on diff notes', :js do it 'adds multiple comments' do 3.times do + find_by_scrolling('[data-discussion-id]', match: :first) find_field('Reply…', match: :first).click page.within '.js-discussion-note-form' do @@ -190,7 +195,7 @@ RSpec.describe 'Merge request > User sees avatars on diff notes', :js do end def find_line(line_code) - line = find("[id='#{line_code}']") + line = find_by_scrolling("[id='#{line_code}']") line = line.find(:xpath, 'preceding-sibling::*[1][self::td]/preceding-sibling::*[1][self::td]') if line.tag_name == 'td' line end |