diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2019-11-21 12:06:16 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2019-11-21 12:06:16 +0300 |
commit | a048261403ea7e12992ccffe704f0779235712d7 (patch) | |
tree | 59254549db6d39a4da824379a7bf354e7c8e7e67 /spec/javascripts/line_highlighter_spec.js | |
parent | 80e5134020483299c039114e76b734436f006c66 (diff) |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec/javascripts/line_highlighter_spec.js')
-rw-r--r-- | spec/javascripts/line_highlighter_spec.js | 41 |
1 files changed, 14 insertions, 27 deletions
diff --git a/spec/javascripts/line_highlighter_spec.js b/spec/javascripts/line_highlighter_spec.js index f8f835ffdef..45b10fc3bd8 100644 --- a/spec/javascripts/line_highlighter_spec.js +++ b/spec/javascripts/line_highlighter_spec.js @@ -1,12 +1,11 @@ -/* eslint-disable no-var, no-else-return, dot-notation, no-return-assign, no-new, no-underscore-dangle */ +/* eslint-disable no-else-return, dot-notation, no-return-assign, no-new, no-underscore-dangle */ import $ from 'jquery'; import LineHighlighter from '~/line_highlighter'; describe('LineHighlighter', function() { - var clickLine; preloadFixtures('static/line_highlighter.html'); - clickLine = function(number, eventData = {}) { + const clickLine = function(number, eventData = {}) { if ($.isEmptyObject(eventData)) { return $(`#L${number}`).click(); } else { @@ -39,34 +38,30 @@ describe('LineHighlighter', function() { }); it('highlights a range of lines given in the URL hash', function() { - var line; new LineHighlighter({ hash: '#L5-25' }); expect($(`.${this.css}`).length).toBe(21); - for (line = 5; line <= 25; line += 1) { + for (let line = 5; line <= 25; line += 1) { expect($(`#LC${line}`)).toHaveClass(this.css); } }); it('scrolls to the first highlighted line on initial load', function() { - var spy; - spy = spyOn($, 'scrollTo'); + const spy = spyOn($, 'scrollTo'); new LineHighlighter({ hash: '#L5-25' }); expect(spy).toHaveBeenCalledWith('#L5', jasmine.anything()); }); it('discards click events', function() { - var spy; - spy = spyOnEvent('a[data-line-number]', 'click'); + const spy = spyOnEvent('a[data-line-number]', 'click'); clickLine(13); expect(spy).toHaveBeenPrevented(); }); it('handles garbage input from the hash', function() { - var func; - func = function() { + const func = function() { return new LineHighlighter({ fileHolderSelector: '#blob-content-holder' }); }; @@ -76,8 +71,7 @@ describe('LineHighlighter', function() { describe('clickHandler', function() { it('handles clicking on a child icon element', function() { - var spy; - spy = spyOn(this['class'], 'setHash').and.callThrough(); + const spy = spyOn(this['class'], 'setHash').and.callThrough(); $('#L13 i') .mousedown() .click(); @@ -102,8 +96,7 @@ describe('LineHighlighter', function() { }); it('sets the hash', function() { - var spy; - spy = spyOn(this['class'], 'setHash').and.callThrough(); + const spy = spyOn(this['class'], 'setHash').and.callThrough(); clickLine(13); expect(spy).toHaveBeenCalledWith(13); @@ -112,8 +105,7 @@ describe('LineHighlighter', function() { describe('with shiftKey', function() { it('sets the hash', function() { - var spy; - spy = spyOn(this['class'], 'setHash').and.callThrough(); + const spy = spyOn(this['class'], 'setHash').and.callThrough(); clickLine(13); clickLine(20, { shiftKey: true, @@ -134,8 +126,7 @@ describe('LineHighlighter', function() { }); it('sets the hash', function() { - var spy; - spy = spyOn(this['class'], 'setHash'); + const spy = spyOn(this['class'], 'setHash'); clickLine(13, { shiftKey: true, }); @@ -146,27 +137,25 @@ describe('LineHighlighter', function() { describe('with existing single-line highlight', function() { it('uses existing line as last line when target is lesser', function() { - var line; clickLine(20); clickLine(15, { shiftKey: true, }); expect($(`.${this.css}`).length).toBe(6); - for (line = 15; line <= 20; line += 1) { + for (let line = 15; line <= 20; line += 1) { expect($(`#LC${line}`)).toHaveClass(this.css); } }); it('uses existing line as first line when target is greater', function() { - var line; clickLine(5); clickLine(10, { shiftKey: true, }); expect($(`.${this.css}`).length).toBe(6); - for (line = 5; line <= 10; line += 1) { + for (let line = 5; line <= 10; line += 1) { expect($(`#LC${line}`)).toHaveClass(this.css); } }); @@ -183,25 +172,23 @@ describe('LineHighlighter', function() { }); it('uses target as first line when it is less than existing first line', function() { - var line; clickLine(5, { shiftKey: true, }); expect($(`.${this.css}`).length).toBe(6); - for (line = 5; line <= 10; line += 1) { + for (let line = 5; line <= 10; line += 1) { expect($(`#LC${line}`)).toHaveClass(this.css); } }); it('uses target as last line when it is greater than existing first line', function() { - var line; clickLine(15, { shiftKey: true, }); expect($(`.${this.css}`).length).toBe(6); - for (line = 10; line <= 15; line += 1) { + for (let line = 10; line <= 15; line += 1) { expect($(`#LC${line}`)).toHaveClass(this.css); } }); |