Skip to content
Snippets Groups Projects
Commit cdc031b6 authored by Sean McGivern's avatar Sean McGivern
Browse files

Fix expandable diffs

parent 3999b80e
No related branches found
No related tags found
No related merge requests found
class @Diff class @Diff
UNFOLD_COUNT = 20 UNFOLD_COUNT = 20
constructor: -> constructor: ->
$('.files .diff-file').singleFileDiff()
@filesCommentButton = $('.files .diff-file').filesCommentButton() @filesCommentButton = $('.files .diff-file').filesCommentButton()
   
$(document).off('click', '.js-unfold') $(document).off('click', '.js-unfold')
Loading
Loading
Loading
@@ -106,7 +106,7 @@ feature 'Expand and collapse diffs', js: true, feature: true do
Loading
@@ -106,7 +106,7 @@ feature 'Expand and collapse diffs', js: true, feature: true do
let(:comment_text) { 'A comment' } let(:comment_text) { 'A comment' }
   
before do before do
large_diff.find('.line_holder', match: :prefer_exact).hover large_diff.find('.diff-line-num', match: :prefer_exact).hover
large_diff.find('.add-diff-note').click large_diff.find('.add-diff-note').click
large_diff.find('.note-textarea').send_keys comment_text large_diff.find('.note-textarea').send_keys comment_text
large_diff.find_button('Comment').click large_diff.find_button('Comment').click
Loading
@@ -161,7 +161,7 @@ feature 'Expand and collapse diffs', js: true, feature: true do
Loading
@@ -161,7 +161,7 @@ feature 'Expand and collapse diffs', js: true, feature: true do
end end
   
it 'does not make a new HTTP request' do it 'does not make a new HTTP request' do
expect(evaluate_script('ajaxUris')).to be_empty expect(evaluate_script('ajaxUris')).not_to include(a_string_matching('small_diff.md'))
end end
end end
end end
Loading
@@ -199,7 +199,7 @@ feature 'Expand and collapse diffs', js: true, feature: true do
Loading
@@ -199,7 +199,7 @@ feature 'Expand and collapse diffs', js: true, feature: true do
end end
   
it 'does not make a new HTTP request' do it 'does not make a new HTTP request' do
expect(evaluate_script('ajaxUris')).to be_empty expect(evaluate_script('ajaxUris')).not_to include(a_string_matching('small_diff.md'))
end end
end end
end end
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment