summaryrefslogtreecommitdiff
path: root/spec/features/projects/commits/multi_view_diff_spec.rb
diff options
context:
space:
mode:
Diffstat (limited to 'spec/features/projects/commits/multi_view_diff_spec.rb')
-rw-r--r--spec/features/projects/commits/multi_view_diff_spec.rb85
1 files changed, 85 insertions, 0 deletions
diff --git a/spec/features/projects/commits/multi_view_diff_spec.rb b/spec/features/projects/commits/multi_view_diff_spec.rb
new file mode 100644
index 00000000000..ecdd398c739
--- /dev/null
+++ b/spec/features/projects/commits/multi_view_diff_spec.rb
@@ -0,0 +1,85 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.shared_examples "no multiple viewers" do |commit_ref|
+ let(:ref) { commit_ref }
+
+ it "does not display multiple diff viewers" do
+ expect(page).not_to have_selector '[data-diff-toggle-entity]'
+ end
+end
+
+RSpec.describe 'Multiple view Diffs', :js do
+ let(:user) { create(:user) }
+ let(:project) { create(:project, :repository, visibility_level: Gitlab::VisibilityLevel::PUBLIC) }
+ let(:ref) { '5d6ed1503801ca9dc28e95eeb85a7cf863527aee' }
+ let(:path) { project_commit_path(project, ref) }
+ let(:feature_flag_on) { false }
+
+ before do
+ stub_feature_flags(rendered_diffs_viewer: feature_flag_on ? project : false)
+
+ visit path
+
+ wait_for_all_requests
+ end
+
+ context 'when :rendered_diffs_viewer is off' do
+ context 'and diff does not have ipynb' do
+ include_examples "no multiple viewers", 'ddd0f15ae83993f5cb66a927a28673882e99100b'
+ end
+
+ context 'and diff has ipynb' do
+ include_examples "no multiple viewers", '5d6ed1503801ca9dc28e95eeb85a7cf863527aee'
+
+ it 'shows the transformed diff' do
+ diff = page.find('.diff-file, .file-holder', match: :first)
+
+ expect(diff['innerHTML']).to include('%% Cell type:markdown id:0aac5da7-745c-4eda-847a-3d0d07a1bb9b tags:')
+ end
+ end
+ end
+
+ context 'when :rendered_diffs_viewer is on' do
+ let(:feature_flag_on) { true }
+
+ context 'and diff does not include ipynb' do
+ include_examples "no multiple viewers", 'ddd0f15ae83993f5cb66a927a28673882e99100b'
+ end
+
+ context 'and opening a diff with ipynb' do
+ context 'but the changes are not renderable' do
+ include_examples "no multiple viewers", 'a867a602d2220e5891b310c07d174fbe12122830'
+ end
+
+ it 'loads the rendered diff as hidden' do
+ diff = page.find('.diff-file, .file-holder', match: :first)
+
+ expect(diff).not_to have_selector '[data-diff-toggle-entity="toHide"]'
+ expect(diff).to have_selector '[data-diff-toggle-entity="toShow"]'
+
+ expect(classes_for_element(diff, 'toHide', visible: false)).to include('hidden')
+ expect(classes_for_element(diff, 'toShow')).not_to include('hidden')
+
+ expect(classes_for_element(diff, 'toShowBtn')).to include('selected')
+ expect(classes_for_element(diff, 'toHideBtn')).not_to include('selected')
+ end
+
+ it 'displays the rendered diff and hides after selection changes' do
+ diff = page.find('.diff-file, .file-holder', match: :first)
+ diff.find('[data-diff-toggle-entity="toShowBtn"]').click
+
+ expect(diff).to have_selector '[data-diff-toggle-entity="toShow"]'
+ expect(diff).not_to have_selector '[data-diff-toggle-entity="toHide"]'
+
+ expect(classes_for_element(diff, 'toHideBtn')).not_to include('selected')
+ expect(classes_for_element(diff, 'toShowBtn')).to include('selected')
+ end
+ end
+ end
+
+ def classes_for_element(node, data_diff_entity, visible: true)
+ node.find("[data-diff-toggle-entity=\"#{data_diff_entity}\"]", visible: visible)[:class]
+ end
+end