Merge branch 'fix-expandable-diffs' into 'master'

Fix expandable diffs

## What does this MR do?

Fix expandable diffs and the master build.

## Are there points in the code the reviewer needs to double check?

Nope.

## Why was this MR needed?

https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4776#note_13003494

Doesn't need a CHANGELOG entry etc.

See merge request !5200

1 files changed, 3 insertions, 3 deletions

diff --git a/spec/features/expand_collapse_diffs_spec.rb b/spec/features/expand_collapse_diffs_spec.rb
index 7cff196c8d9..78bc888f2a6 100644
--- a/spec/features/expand_collapse_diffs_spec.rb
+++ b/spec/features/expand_collapse_diffs_spec.rb
@@ -106,7 +106,7 @@ feature 'Expand and collapse diffs', js: true, feature: true do
         let(:comment_text) { 'A comment' }
 
         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('.note-textarea').send_keys comment_text
           large_diff.find_button('Comment').click
@@ -161,7 +161,7 @@ feature 'Expand and collapse diffs', js: true, feature: true do
         end
 
         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
@@ -199,7 +199,7 @@ feature 'Expand and collapse diffs', js: true, feature: true do
         end
 
         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