diff options
author | XhmikosR <xhmikosr@gmail.com> | 2021-12-09 17:01:29 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-12-09 17:01:29 +0300 |
commit | 94d4fa3b100352cba4c03b23b229e137d520a13a (patch) | |
tree | e5bdb442dc900b7b3004ccfe228a321296230ab7 /js | |
parent | 28a5a72ed56df3cc5efb1d5164376c9c9541e4f0 (diff) |
Fix tests fixture type (#35501)
Previously we were adding an Array instead of a String
Diffstat (limited to 'js')
-rw-r--r-- | js/tests/unit/dom/event-handler.spec.js | 2 | ||||
-rw-r--r-- | js/tests/unit/dom/selector-engine.spec.js | 28 | ||||
-rw-r--r-- | js/tests/unit/dropdown.spec.js | 12 | ||||
-rw-r--r-- | js/tests/unit/tooltip.spec.js | 2 | ||||
-rw-r--r-- | js/tests/unit/util/index.spec.js | 18 |
5 files changed, 32 insertions, 30 deletions
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', () => { '</div>', '<div class="sibling"></div>', '</div>' - ] + ].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 = `<ul> - <li></li> - <li> - <a href="#" class="active">link</a> - </li> - <li></li> - </ul>` + fixtureEl.innerHTML = [ + '<ul>', + ' <li></li>', + ' <li>', + ' <a href="#" class="active">link</a>', + ' </li>', + ' <li></li>', + '</ul>' + ].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 = `<ul> - <li></li> - <li></li> - <li></li> - </ul>` + fixtureEl.innerHTML = [ + '<ul>', + ' <li></li>', + ' <li></li>', + ' <li></li>', + '</ul>' + ].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', () => { ' </div>', '</div>', '<input type="text">' - ] + ].join('') const triggerDropdown = fixtureEl.querySelector('[data-bs-toggle="dropdown"]') const input = fixtureEl.querySelector('input') @@ -1735,7 +1735,7 @@ describe('Dropdown', () => { ' </div>', ' </div>', '</div>' - ] + ].join('') const triggerDropdown = fixtureEl.querySelector('[data-bs-toggle="dropdown"]') const dropdown = new Dropdown(triggerDropdown) @@ -1767,7 +1767,7 @@ describe('Dropdown', () => { ' </div>', ' </div>', '</div>' - ] + ].join('') const parent = fixtureEl.querySelector('.parent') const toggle = fixtureEl.querySelector('[data-bs-toggle="dropdown"]') @@ -1826,7 +1826,7 @@ describe('Dropdown', () => { ' <a class="dropdown-item" href="#">Dropdown item</a>', ' </div>', '</div>' - ] + ].join('') const dropdownToggle = fixtureEl.querySelector('[data-bs-toggle="dropdown"]') const dropdownMenu = fixtureEl.querySelector('.dropdown-menu') @@ -1857,7 +1857,7 @@ describe('Dropdown', () => { ' <a class="dropdown-item" href="#">Dropdown item</a>', ' </div>', '</div>' - ] + ].join('') const dropdownToggle = fixtureEl.querySelector('[data-bs-toggle="dropdown"]') const dropdownMenu = fixtureEl.querySelector('.dropdown-menu') @@ -1888,7 +1888,7 @@ describe('Dropdown', () => { ' <a class="dropdown-item" href="#">Dropdown item</a>', ' </div>', '</div>' - ] + ].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', () => { '<b>Trigger</b>', 'the tooltip', '</a>' - ] + ].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 = - [ - '<div id="foo" class="test"></div>', - '<div id="bar" class="test"></div>' - ].join('') + fixtureEl.innerHTML = [ + '<div id="foo" class="test"></div>', + '<div id="bar" class="test"></div>' + ].join('') const el = fixtureEl.querySelector('#foo') @@ -199,11 +198,10 @@ describe('Util', () => { describe('getElement', () => { it('should try to parse element', () => { - fixtureEl.innerHTML = - [ - '<div id="foo" class="test"></div>', - '<div id="bar" class="test"></div>' - ].join('') + fixtureEl.innerHTML = [ + '<div id="foo" class="test"></div>', + '<div id="bar" class="test"></div>' + ].join('') const el = fixtureEl.querySelector('div') |