diff options
Diffstat (limited to 'spec/features/merge_request/user_sees_versions_spec.rb')
-rw-r--r-- | spec/features/merge_request/user_sees_versions_spec.rb | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/spec/features/merge_request/user_sees_versions_spec.rb b/spec/features/merge_request/user_sees_versions_spec.rb index 5abf4e2f5ad..2b856811e02 100644 --- a/spec/features/merge_request/user_sees_versions_spec.rb +++ b/spec/features/merge_request/user_sees_versions_spec.rb @@ -3,6 +3,8 @@ require 'spec_helper' RSpec.describe 'Merge request > User sees versions', :js do + include MergeRequestDiffHelpers + let(:merge_request) do create(:merge_request).tap do |mr| mr.merge_request_diff.destroy! @@ -27,8 +29,12 @@ RSpec.describe 'Merge request > User sees versions', :js do diff_file_selector = ".diff-file[id='#{file_id}']" line_code = "#{file_id}_#{line_code}" - page.within(diff_file_selector) do - first("[id='#{line_code}']").hover + page.within find_by_scrolling(diff_file_selector) do + line_code_element = first("[id='#{line_code}']") + # scrolling to element's bottom is required in order for .hover action to work + # otherwise, the element could be hidden underneath a sticky header + scroll_to_elements_bottom(line_code_element) + line_code_element.hover first("[id='#{line_code}'] [role='button']").click page.within("form[data-line-code='#{line_code}']") do |