Resolve eslint violations

spec/javascripts/bootstrap_linked_tabs_spec.js

@@ -13,8 +13,8 @@ describe('Linked Tabs', () => {
     });
     it('should activate the tab correspondent to the given action', () => {
-      const linkedTabs = new LinkedTabs({
-        // eslint-disable-line
+      // eslint-disable-next-line no-new
+      new LinkedTabs({
         action: 'tab1',
         defaultAction: 'tab1',
         parentEl: '.linked-tabs',
@@ -24,8 +24,8 @@ describe('Linked Tabs', () => {
     });
     it('should active the default tab action when the action is show', () => {
-      const linkedTabs = new LinkedTabs({
-        // eslint-disable-line
+      // eslint-disable-next-line no-new
+      new LinkedTabs({
         action: 'show',
         defaultAction: 'tab1',
         parentEl: '.linked-tabs',

spec/javascripts/ci_variable_list/native_form_variable_list_spec.js

@@ -23,6 +23,7 @@ describe('NativeFormVariableList', () => {
       expect($row.find('.js-ci-variable-input-key').attr('name')).toBe(
         'schedule[variables_attributes][][key]',
       );
+
       expect($row.find('.js-ci-variable-input-value').attr('name')).toBe(
         'schedule[variables_attributes][][secret_value]',
       );

spec/javascripts/clusters/components/application_row_spec.js

@@ -239,6 +239,7 @@ describe('Application Row', () => {
       expect(generalErrorMessage.textContent.trim()).toEqual(
         `Something went wrong while installing ${DEFAULT_APPLICATION_STATE.title}`,
       );
+
       expect(statusErrorMessage.textContent.trim()).toEqual(statusReason);
     });
@@ -260,6 +261,7 @@ describe('Application Row', () => {
       expect(generalErrorMessage.textContent.trim()).toEqual(
         `Something went wrong while installing ${DEFAULT_APPLICATION_STATE.title}`,
       );
+
       expect(requestErrorMessage.textContent.trim()).toEqual(requestReason);
     });
   });

spec/javascripts/environments/environment_actions_spec.js

@@ -43,6 +43,7 @@ describe('Actions Component', () => {
     expect(
       component.$el.querySelector('.dropdown-new').getAttribute('data-original-title'),
     ).toEqual('Deploy to...');
+
     expect(component.$el.querySelector('.dropdown-new').getAttribute('aria-label')).toEqual(
       'Deploy to...',
     );

spec/javascripts/filtered_search/filtered_search_visual_tokens_spec.js

@@ -138,6 +138,7 @@ describe('Filtered Search Visual Tokens', () => {
       expect(subject.getEndpointWithQueryParams(endpoint, singleQueryParams)).toBe(
         `${endpoint}?foo=true`,
       );
+
       expect(subject.getEndpointWithQueryParams(endpoint, multipleQueryParams)).toBe(
         `${endpoint}?foo=true&bar=true`,
       );

spec/javascripts/image_diff/replaced_image_diff_spec.js

@@ -90,9 +90,11 @@ describe('ReplacedImageDiff', () => {
       expect(imageFrameEls[viewTypes.TWO_UP]).toEqual(
         element.querySelector('.two-up .js-image-frame'),
       );
+
       expect(imageFrameEls[viewTypes.SWIPE]).toEqual(
         element.querySelector('.swipe .js-image-frame'),
       );
+
       expect(imageFrameEls[viewTypes.ONION_SKIN]).toEqual(
         element.querySelector('.onion-skin .js-image-frame'),
       );
@@ -105,9 +107,11 @@ describe('ReplacedImageDiff', () => {
       expect(viewModesEls[viewTypes.TWO_UP]).toEqual(
         element.querySelector('.view-modes-menu .two-up'),
       );
+
       expect(viewModesEls[viewTypes.SWIPE]).toEqual(
         element.querySelector('.view-modes-menu .swipe'),
       );
+
       expect(viewModesEls[viewTypes.ONION_SKIN]).toEqual(
         element.querySelector('.view-modes-menu .onion-skin'),
       );

spec/javascripts/integrations/integration_settings_form_spec.js

@@ -164,6 +164,7 @@ describe('IntegrationSettingsForm', () => {
               .text()
               .trim(),
           ).toEqual('Test failed. some error');
+
           expect($flashContainer.find('.flash-action')).toBeDefined();
           expect(
             $flashContainer
@@ -197,6 +198,7 @@ describe('IntegrationSettingsForm', () => {
               .text()
               .trim(),
           ).toEqual('Validations failed. some error');
+
           expect($flashContainer.find('.flash-action')).toBeDefined();
           expect(
             $flashContainer

spec/javascripts/issue_show/components/app_spec.js

@@ -74,6 +74,7 @@ describe('Issuable output', () => {
         expect(vm.$el.querySelector('.js-task-list-field').value).toContain(
           'this is a description',
         );
+
         expect(formatText(editedText.innerText)).toMatch(/Edited[\s\S]+?by Some User/);
         expect(editedText.querySelector('.author-link').href).toMatch(/\/some_user$/);
         expect(editedText.querySelector('time')).toBeTruthy();
@@ -91,6 +92,7 @@ describe('Issuable output', () => {
         expect(formatText(vm.$el.querySelector('.edited-text').innerText)).toMatch(
           /Edited[\s\S]+?by Other User/,
         );
+
         expect(editedText.querySelector('.author-link').href).toMatch(/\/other_user$/);
         expect(editedText.querySelector('time')).toBeTruthy();
       })
@@ -294,8 +296,9 @@ describe('Issuable output', () => {
   it('opens recaptcha modal if update rejected as spam', done => {
     function mockScriptSrc() {
       const recaptchaChild = vm.$children.find(
+        // eslint-disable-next-line no-underscore-dangle
         child => child.$options._componentTag === 'recaptcha-modal',
-      ); // eslint-disable-line no-underscore-dangle
+      );
       recaptchaChild.scriptSrc = '//scriptsrc';
     }

spec/javascripts/issue_show/components/description_spec.js

@@ -54,8 +54,9 @@ describe('Description component', () => {
   it('opens recaptcha dialog if update rejected as spam', done => {
     let modal;
     const recaptchaChild = vm.$children.find(
+      // eslint-disable-next-line no-underscore-dangle
       child => child.$options._componentTag === 'recaptcha-modal',
-    ); // eslint-disable-line no-underscore-dangle
+    );
     recaptchaChild.scriptSrc = '//scriptsrc';

spec/javascripts/jobs/components/trigger_block_spec.js

@@ -51,10 +51,12 @@ describe('Trigger block', () => {
             expect(vm.$el.querySelector('.js-build-variables').textContent).toContain(
               'UPLOAD_TO_GCS',
             );
+
             expect(vm.$el.querySelector('.js-build-variables').textContent).toContain('false');
             expect(vm.$el.querySelector('.js-build-variables').textContent).toContain(
               'UPLOAD_TO_S3',
             );
+
             expect(vm.$el.querySelector('.js-build-variables').textContent).toContain('true');
           })
           .then(done)

spec/javascripts/monitoring/dashboard_state_spec.js

@@ -59,9 +59,11 @@ describe('EmptyState', () => {
     expect(getTextFromNode(component, '.state-title')).toEqual(
       component.states.gettingStarted.title,
     );
+
     expect(getTextFromNode(component, '.state-description')).toEqual(
       component.states.gettingStarted.description,
     );
+
     expect(getTextFromNode(component, '.btn-success')).toEqual(
       component.states.gettingStarted.buttonText,
     );
@@ -77,6 +79,7 @@ describe('EmptyState', () => {
     expect(getTextFromNode(component, '.state-description')).toEqual(
       component.states.loading.description,
     );
+
     expect(getTextFromNode(component, '.btn-success')).toEqual(component.states.loading.buttonText);
   });
@@ -89,6 +92,7 @@ describe('EmptyState', () => {
     expect(getTextFromNode(component, '.state-title')).toEqual(
       component.states.unableToConnect.title,
     );
+
     expect(component.$el.querySelector('.state-description a')).toBeDefined();
     expect(getTextFromNode(component, '.btn-success')).toEqual(
       component.states.unableToConnect.buttonText,

spec/javascripts/new_branch_spec.js

-/* eslint-disable one-var, no-var, no-return-assign */
-
 import $ from 'jquery';
 import NewBranchForm from '~/new_branch_form';
 describe('Branch', function() {
   describe('create a new branch', function() {
-    var expectToHaveError, fillNameWith;
     preloadFixtures('branches/new_branch.html.raw');
-    fillNameWith = function(value) {
-      return $('.js-branch-name')
+
+    function fillNameWith(value) {
+      $('.js-branch-name')
         .val(value)
         .trigger('blur');
-    };
-    expectToHaveError = function(error) {
+    }
+
+    function expectToHaveError(error) {
       expect($('.js-branch-name-error span').text()).toEqual(error);
-    };
+    }
+
     beforeEach(function() {
       loadFixtures('branches/new_branch.html.raw');
       $('form').on('submit', function(e) {
         return e.preventDefault();
       });
-      return (this.form = new NewBranchForm($('.js-create-branch-form'), []));
+      this.form = new NewBranchForm($('.js-create-branch-form'), []);
     });
     it("can't start with a dot", function() {
       fillNameWith('.foo');
-      return expectToHaveError("can't start with '.'");
+      expectToHaveError("can't start with '.'");
     });
     it("can't start with a slash", function() {
       fillNameWith('/foo');
-      return expectToHaveError("can't start with '/'");
+      expectToHaveError("can't start with '/'");
     });
     it("can't have two consecutive dots", function() {
       fillNameWith('foo..bar');
-      return expectToHaveError("can't contain '..'");
+      expectToHaveError("can't contain '..'");
     });
     it("can't have spaces anywhere", function() {
@@ -44,7 +44,7 @@ describe('Branch', function() {
       fillNameWith('foo bar');
       expectToHaveError("can't contain spaces");
       fillNameWith('foo ');
-      return expectToHaveError("can't contain spaces");
+      expectToHaveError("can't contain spaces");
     });
     it("can't have ~ anywhere", function() {
@@ -53,7 +53,7 @@ describe('Branch', function() {
       fillNameWith('foo~bar');
       expectToHaveError("can't contain '~'");
       fillNameWith('foo~');
-      return expectToHaveError("can't contain '~'");
+      expectToHaveError("can't contain '~'");
     });
     it("can't have tilde anwhere", function() {
@@ -62,7 +62,7 @@ describe('Branch', function() {
       fillNameWith('foo~bar');
       expectToHaveError("can't contain '~'");
       fillNameWith('foo~');
-      return expectToHaveError("can't contain '~'");
+      expectToHaveError("can't contain '~'");
     });
     it("can't have caret anywhere", function() {
@@ -71,7 +71,7 @@ describe('Branch', function() {
       fillNameWith('foo^bar');
       expectToHaveError("can't contain '^'");
       fillNameWith('foo^');
-      return expectToHaveError("can't contain '^'");
+      expectToHaveError("can't contain '^'");
     });
     it("can't have : anywhere", function() {
@@ -80,7 +80,7 @@ describe('Branch', function() {
       fillNameWith('foo:bar');
       expectToHaveError("can't contain ':'");
       fillNameWith(':foo');
-      return expectToHaveError("can't contain ':'");
+      expectToHaveError("can't contain ':'");
     });
     it("can't have question mark anywhere", function() {
@@ -89,7 +89,7 @@ describe('Branch', function() {
       fillNameWith('foo?bar');
       expectToHaveError("can't contain '?'");
       fillNameWith('foo?');
-      return expectToHaveError("can't contain '?'");
+      expectToHaveError("can't contain '?'");
     });
     it("can't have asterisk anywhere", function() {
@@ -98,7 +98,7 @@ describe('Branch', function() {
       fillNameWith('foo*bar');
       expectToHaveError("can't contain '*'");
       fillNameWith('foo*');
-      return expectToHaveError("can't contain '*'");
+      expectToHaveError("can't contain '*'");
     });
     it("can't have open bracket anywhere", function() {
@@ -107,7 +107,7 @@ describe('Branch', function() {
       fillNameWith('foo[bar');
       expectToHaveError("can't contain '['");
       fillNameWith('foo[');
-      return expectToHaveError("can't contain '['");
+      expectToHaveError("can't contain '['");
     });
     it("can't have a backslash anywhere", function() {
@@ -116,7 +116,7 @@ describe('Branch', function() {
       fillNameWith('foo\\bar');
       expectToHaveError("can't contain '\\'");
       fillNameWith('foo\\');
-      return expectToHaveError("can't contain '\\'");
+      expectToHaveError("can't contain '\\'");
     });
     it("can't contain a sequence @{ anywhere", function() {
@@ -125,44 +125,42 @@ describe('Branch', function() {
       fillNameWith('foo@{bar');
       expectToHaveError("can't contain '@{'");
       fillNameWith('foo@{');
-      return expectToHaveError("can't contain '@{'");
+      expectToHaveError("can't contain '@{'");
     });
     it("can't have consecutive slashes", function() {
       fillNameWith('foo//bar');
-      return expectToHaveError("can't contain consecutive slashes");
+      expectToHaveError("can't contain consecutive slashes");
     });
     it("can't end with a slash", function() {
       fillNameWith('foo/');
-      return expectToHaveError("can't end in '/'");
+      expectToHaveError("can't end in '/'");
     });
     it("can't end with a dot", function() {
       fillNameWith('foo.');
-      return expectToHaveError("can't end in '.'");
+      expectToHaveError("can't end in '.'");
     });
     it("can't end with .lock", function() {
       fillNameWith('foo.lock');
-      return expectToHaveError("can't end in '.lock'");
+      expectToHaveError("can't end in '.lock'");
     });
     it("can't be the single character @", function() {
       fillNameWith('@');
-      return expectToHaveError("can't be '@'");
+      expectToHaveError("can't be '@'");
     });
     it('concatenates all error messages', function() {
       fillNameWith('/foo bar?~.');
-      return expectToHaveError(
-        "can't start with '/', can't contain spaces, '?', '~', can't end in '.'",
-      );
+      expectToHaveError("can't start with '/', can't contain spaces, '?', '~', can't end in '.'");
     });
     it("doesn't duplicate error messages", function() {
       fillNameWith('?foo?bar?zoo?');
-      return expectToHaveError("can't contain '?'");
+      expectToHaveError("can't contain '?'");
     });
     it('removes the error message when is a valid name', function() {

spec/javascripts/notebook/cells/output/html_sanitize_tests.js

@@ -50,7 +50,7 @@ export default {
     output: '<a>foo</a>',
   },
   'protocol-based JS injection: invalid URL char': {
-    input: '<img src=javascript:alert("XSS")>', // eslint-disable-line no-useless-escape
+    input: '<img src=javascript:alert("XSS")>',
     output: '<img>',
   },
   'protocol-based JS injection: Unicode': {

spec/javascripts/oauth_remember_me_spec.js

@@ -16,6 +16,7 @@ describe('OAuthRememberMe', () => {
     expect($('#oauth-container .oauth-login.twitter').attr('href')).toBe(
       'http://example.com/?remember_me=1',
     );
+
     expect($('#oauth-container .oauth-login.github').attr('href')).toBe(
       'http://example.com/?remember_me=1',
     );

spec/javascripts/pipelines/empty_state_spec.js

@@ -50,6 +50,7 @@ describe('Pipelines Empty State', () => {
     expect(component.$el.querySelector('.js-get-started-pipelines').getAttribute('href')).toEqual(
       'foo',
     );
+
     expect(component.$el.querySelector('.js-get-started-pipelines').textContent).toContain(
       'Get started with Pipelines',
     );

spec/javascripts/pipelines/pipelines_spec.js

@@ -171,6 +171,7 @@ describe('Pipelines', () => {
         expect(vm.$el.querySelector('.js-empty-state h4').textContent.trim()).toEqual(
           'Build with confidence',
         );
+
         expect(vm.$el.querySelector('.js-get-started-pipelines').getAttribute('href')).toEqual(
           paths.helpPagePath,
         );
@@ -207,6 +208,7 @@ describe('Pipelines', () => {
         expect(vm.$el.querySelector('.js-run-pipeline').getAttribute('href')).toEqual(
           paths.newPipelinePath,
         );
+
         expect(vm.$el.querySelector('.js-ci-lint').getAttribute('href')).toEqual(paths.ciLintPath);
         expect(vm.$el.querySelector('.js-clear-cache').textContent.trim()).toEqual(
           'Clear Runner Caches',
@@ -324,6 +326,7 @@ describe('Pipelines', () => {
         expect(vm.$el.querySelector('.js-empty-state').textContent.trim()).toEqual(
           'This project is not currently set up to run pipelines.',
         );
+
         expect(vm.$el.querySelector('.js-get-started-pipelines')).toBeNull();
       });

spec/javascripts/pipelines/pipelines_table_spec.js

@@ -41,12 +41,15 @@ describe('Pipelines Table', () => {
       expect(
         component.$el.querySelector('.table-section.js-pipeline-status').textContent.trim(),
       ).toEqual('Status');
+
       expect(
         component.$el.querySelector('.table-section.js-pipeline-info').textContent.trim(),
       ).toEqual('Pipeline');
+
       expect(
         component.$el.querySelector('.table-section.js-pipeline-commit').textContent.trim(),
       ).toEqual('Commit');
+
       expect(
         component.$el.querySelector('.table-section.js-pipeline-stages').textContent.trim(),
       ).toEqual('Stages');

spec/javascripts/pipelines_spec.js

@@ -13,7 +13,7 @@ describe('Pipelines', () => {
   it('should create a `Pipelines` instance without options', () => {
     expect(() => {
-      new Pipelines();
-    }).not.toThrow(); //eslint-disable-line
+      new Pipelines(); // eslint-disable-line no-new
+    }).not.toThrow();
   });
 });

spec/javascripts/prometheus_metrics/prometheus_metrics_spec.js

@@ -88,6 +88,7 @@ describe('PrometheusMetrics', () => {
       expect(prometheusMetrics.$monitoredMetricsCount.text()).toEqual(
         '3 exporters with 12 metrics were found',
       );
+
       expect($metricsListLi.length).toEqual(metrics.length);
       expect(
         $metricsListLi

spec/javascripts/reports/components/modal_spec.js

@@ -36,6 +36,7 @@ describe('Grouped Test Reports Modal', () => {
     expect(vm.$el.querySelector('.js-modal-link').getAttribute('href')).toEqual(
       modalDataStructure.class.value,
     );
+
     expect(trimText(vm.$el.querySelector('.js-modal-link').textContent)).toEqual(
       modalDataStructure.class.value,
     );