spec/features/merge_request/user_sees_merge_request_file_tree_sidebar_spec.rb


1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75

# frozen_string_literal: true

require 'spec_helper'

RSpec.describe 'Merge request > User sees merge request file tree sidebar', :js, feature_category: :code_review_workflow do
  include MergeRequestDiffHelpers

  let_it_be(:project) { create(:project, :public, :repository) }
  let_it_be(:merge_request) { create(:merge_request, source_project: project) }
  let(:user) { project.creator }
  let(:sidebar) { find('.diff-tree-list') }
  let(:sidebar_scroller) { sidebar.find('.vue-recycle-scroller') }

  before do
    sign_in(user)
    visit diffs_project_merge_request_path(project, merge_request)
    wait_for_requests
  end

  it 'sees file tree sidebar' do
    expect(page).to have_selector('.file-row[role=button]')
  end

  shared_examples 'last entry clickable' do
    specify do
      sidebar_scroller.execute_script('this.scrollBy(0,99999)')
      button = find_all('.file-row[role=button]').last
      title = button.find('[data-testid=file-row-name-container]')[:title]
      expect(button.obscured?).to be_falsy
      button.click
      expect(page).to have_selector(".file-title-name[title*=\"#{title}\"]")
    end
  end

  it_behaves_like 'last entry clickable'

  context 'when has started a review' do
    before do
      add_diff_line_draft_comment('foo', find('.line_holder', match: :first))
      # wait for review bar to appear
      find_by_testid('review_bar_component')
      # wait for sidebar to adjust
      sleep(1)
    end

    it_behaves_like 'last entry clickable'

    context 'when scrolled into full view' do
      before do
        sidebar.execute_script("this.scrollIntoView({ block: 'end' })")
      end

      it_behaves_like 'last entry clickable'
    end
  end

  context 'when viewing using file-by-file mode' do
    let(:user) { create(:user, view_diffs_file_by_file: true) }

    it_behaves_like 'last entry clickable'

    context 'when navigating to the next file' do
      before do
        click_link 'Next'
        wait_for_requests
        # when we click the Next button the viewport will be scrolled a bit into the diffs view
        # this will cause for the file tree sidebar height to be recalculated
        # because this logic is async and debounced twice we have to wait for the layout to stabilize
        sleep(1)
      end

      it_behaves_like 'last entry clickable'
    end
  end
end