diff --git a/app/assets/javascripts/merge_request_tabs.js.es6 b/app/assets/javascripts/merge_request_tabs.js.es6 index 771803edb7c123cefdf7e794f7f7d03d969fd822..3ec0f1fd613e553341e2820641621232f591dc23 100644 --- a/app/assets/javascripts/merge_request_tabs.js.es6 +++ b/app/assets/javascripts/merge_request_tabs.js.es6 @@ -225,7 +225,7 @@ // We extract pathname for the current Changes tab anchor href // some pages like MergeRequestsController#new has query parameters on that anchor - var urlPathname = gl.utils.parseUrlPathname(source); + const urlPathname = gl.utils.parseUrlPathname(source); this.ajaxGet({ url: `${urlPathname}.json${location.search}`, diff --git a/spec/javascripts/merge_request_tabs_spec.js b/spec/javascripts/merge_request_tabs_spec.js index 4facc42c5b4e735220ec78b481f9842a96ffcac7..130d391bfab30b15136b3982ef1058b7985e73a5 100644 --- a/spec/javascripts/merge_request_tabs_spec.js +++ b/spec/javascripts/merge_request_tabs_spec.js @@ -29,7 +29,7 @@ describe('#activateTab', function () { beforeEach(function () { - spyOn($, 'ajax').and.callFake(function() {}); + spyOn($, 'ajax').and.callFake(function () {}); fixture.load('merge_request_tabs.html'); this.subject = this.class.activateTab; }); @@ -53,7 +53,7 @@ describe('#setCurrentAction', function () { beforeEach(function () { - spyOn($, 'ajax').and.callFake(function() {}); + spyOn($, 'ajax').and.callFake(function () {}); this.subject = this.class.setCurrentAction; }); it('changes from commits', function () { @@ -110,9 +110,9 @@ expect(this.subject('show')).toBe('/foo/bar/merge_requests/1'); }); }); - describe('#loadDiff', function() { - it('requires an absolute pathname', function() { - spyOn($, 'ajax').and.callFake(function(options) { + describe('#loadDiff', function () { + it('requires an absolute pathname', function () { + spyOn($, 'ajax').and.callFake(function (options) { expect(options.url).toEqual('/foo/bar/merge_requests/1/diffs.json'); }); this.class.loadDiff('/foo/bar/merge_requests/1/diffs');