From 94d4fa3b100352cba4c03b23b229e137d520a13a Mon Sep 17 00:00:00 2001 From: XhmikosR Date: Thu, 9 Dec 2021 16:01:29 +0200 Subject: Fix tests fixture type (#35501) Previously we were adding an Array instead of a String --- js/tests/unit/dom/event-handler.spec.js | 2 +- js/tests/unit/dom/selector-engine.spec.js | 28 ++++++++++++++++------------ js/tests/unit/dropdown.spec.js | 12 ++++++------ js/tests/unit/tooltip.spec.js | 2 +- js/tests/unit/util/index.spec.js | 18 ++++++++---------- 5 files changed, 32 insertions(+), 30 deletions(-) (limited to 'js') diff --git a/js/tests/unit/dom/event-handler.spec.js b/js/tests/unit/dom/event-handler.spec.js index 4772d892e8..601af7409e 100644 --- a/js/tests/unit/dom/event-handler.spec.js +++ b/js/tests/unit/dom/event-handler.spec.js @@ -86,7 +86,7 @@ describe('EventHandler', () => { '', '
', '' - ] + ].join('') const outer = fixtureEl.querySelector('.outer') const inner = fixtureEl.querySelector('.inner') diff --git a/js/tests/unit/dom/selector-engine.spec.js b/js/tests/unit/dom/selector-engine.spec.js index 005e92704c..38eb3be54d 100644 --- a/js/tests/unit/dom/selector-engine.spec.js +++ b/js/tests/unit/dom/selector-engine.spec.js @@ -30,13 +30,15 @@ describe('SelectorEngine', () => { }) it('should handle :scope selectors', () => { - fixtureEl.innerHTML = `` + fixtureEl.innerHTML = [ + '' + ].join('') const listEl = fixtureEl.querySelector('ul') const aActive = fixtureEl.querySelector('.active') @@ -57,11 +59,13 @@ describe('SelectorEngine', () => { describe('children', () => { it('should find children', () => { - fixtureEl.innerHTML = `` + fixtureEl.innerHTML = [ + '' + ].join('') const list = fixtureEl.querySelector('ul') const liList = [].concat(...fixtureEl.querySelectorAll('li')) diff --git a/js/tests/unit/dropdown.spec.js b/js/tests/unit/dropdown.spec.js index 91e783a233..6ee58fe71a 100644 --- a/js/tests/unit/dropdown.spec.js +++ b/js/tests/unit/dropdown.spec.js @@ -1649,7 +1649,7 @@ describe('Dropdown', () => { ' ', '', '' - ] + ].join('') const triggerDropdown = fixtureEl.querySelector('[data-bs-toggle="dropdown"]') const input = fixtureEl.querySelector('input') @@ -1735,7 +1735,7 @@ describe('Dropdown', () => { ' ', ' ', '' - ] + ].join('') const triggerDropdown = fixtureEl.querySelector('[data-bs-toggle="dropdown"]') const dropdown = new Dropdown(triggerDropdown) @@ -1767,7 +1767,7 @@ describe('Dropdown', () => { ' ', ' ', '' - ] + ].join('') const parent = fixtureEl.querySelector('.parent') const toggle = fixtureEl.querySelector('[data-bs-toggle="dropdown"]') @@ -1826,7 +1826,7 @@ describe('Dropdown', () => { ' Dropdown item', ' ', '' - ] + ].join('') const dropdownToggle = fixtureEl.querySelector('[data-bs-toggle="dropdown"]') const dropdownMenu = fixtureEl.querySelector('.dropdown-menu') @@ -1857,7 +1857,7 @@ describe('Dropdown', () => { ' Dropdown item', ' ', '' - ] + ].join('') const dropdownToggle = fixtureEl.querySelector('[data-bs-toggle="dropdown"]') const dropdownMenu = fixtureEl.querySelector('.dropdown-menu') @@ -1888,7 +1888,7 @@ describe('Dropdown', () => { ' Dropdown item', ' ', '' - ] + ].join('') const dropdownToggle = fixtureEl.querySelector('[data-bs-toggle="dropdown"]') const dropdownMenu = fixtureEl.querySelector('.dropdown-menu') diff --git a/js/tests/unit/tooltip.spec.js b/js/tests/unit/tooltip.spec.js index 9054c0f642..3feacf7b44 100644 --- a/js/tests/unit/tooltip.spec.js +++ b/js/tests/unit/tooltip.spec.js @@ -722,7 +722,7 @@ describe('Tooltip', () => { 'Trigger', 'the tooltip', '' - ] + ].join('') const tooltipEl = fixtureEl.querySelector('a') const tooltip = new Tooltip(tooltipEl) diff --git a/js/tests/unit/util/index.spec.js b/js/tests/unit/util/index.spec.js index a9e446c9df..ef6647e921 100644 --- a/js/tests/unit/util/index.spec.js +++ b/js/tests/unit/util/index.spec.js @@ -171,11 +171,10 @@ describe('Util', () => { describe('isElement', () => { it('should detect if the parameter is an element or not and return Boolean', () => { - fixtureEl.innerHTML = - [ - '
', - '
' - ].join('') + fixtureEl.innerHTML = [ + '
', + '
' + ].join('') const el = fixtureEl.querySelector('#foo') @@ -199,11 +198,10 @@ describe('Util', () => { describe('getElement', () => { it('should try to parse element', () => { - fixtureEl.innerHTML = - [ - '
', - '
' - ].join('') + fixtureEl.innerHTML = [ + '
', + '
' + ].join('') const el = fixtureEl.querySelector('div') -- cgit v1.2.3