From 6c97107d37771522d1deec6007a791332270ba6f Mon Sep 17 00:00:00 2001 From: Phil Hughes Date: Wed, 4 Oct 2017 14:44:43 +0100 Subject: [PATCH] fixed eslint --- app/assets/javascripts/flash.js | 16 ++++++++-------- spec/javascripts/flash_spec.js | 5 ++--- 2 files changed, 10 insertions(+), 11 deletions(-) diff --git a/app/assets/javascripts/flash.js b/app/assets/javascripts/flash.js index d745483c93f..8ac2b96a22d 100644 --- a/app/assets/javascripts/flash.js +++ b/app/assets/javascripts/flash.js @@ -44,14 +44,14 @@ const createFlashEl = (message, type) => ` * along with ability to provide actionConfig which can be used to show * additional action or link on banner next to message * - * @param {String} message Flash message text - * @param {String} type Type of Flash, it can be `notice` or `alert` (default) - * @param {Object} parent Reference to parent element under which Flash needs to appear - * @param {Object} actonConfig Map of config to show action on banner - * @param {String} href URL to which action config should point to (default: '#') - * @param {String} title Title of action - * @param {Function} clickHandler Method to call when action is clicked on - * @param {Boolean} fadeTransition Boolean to determine whether to fade the alert out + * @param {String} message Flash message text + * @param {String} type Type of Flash, it can be `notice` or `alert` (default) + * @param {Object} parent Reference to parent element under which Flash needs to appear + * @param {Object} actonConfig Map of config to show action on banner + * @param {String} href URL to which action config should point to (default: '#') + * @param {String} title Title of action + * @param {Function} clickHandler Method to call when action is clicked on + * @param {Boolean} fadeTransition Boolean to determine whether to fade the alert out */ const createFlash = function createFlash( message, diff --git a/spec/javascripts/flash_spec.js b/spec/javascripts/flash_spec.js index 1feb31f5c4f..66cc76ee626 100644 --- a/spec/javascripts/flash_spec.js +++ b/spec/javascripts/flash_spec.js @@ -135,7 +135,6 @@ describe('Flash', () => { describe('createFlash', () => { describe('no flash-container', () => { it('does not add to the DOM', () => { - const el = flash('test'); const flashEl = flash('testing'); expect( @@ -185,7 +184,7 @@ describe('Flash', () => { expect( document.querySelector('.flash-text').className, - ).toBe('flash-text container-fluid container-limited') + ).toBe('flash-text container-fluid container-limited'); }); it('does not add container when outside of content-wrapper', () => { @@ -194,7 +193,7 @@ describe('Flash', () => { expect( document.querySelector('.flash-text').className, - ).toBe('flash-text') + ).toBe('flash-text'); }); it('removes element after clicking', () => {