ayo/lion
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: fixed failing accordion test

Browse source
This commit is contained in:
Danny Moerkerke 2022-08-10 14:18:32 +02:00
parent 8a74e33625
commit 74201816eb
2 changed files with 11 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
packages/accordion/src/LionAccordion.js
View file

@ -157,9 +157,11 @@ export class LionAccordion extends LitElement {
* @private
*/
__setupSlots() {
const invokerSlot = this.shadowRoot?.querySelector('slot[name=invoker]');
const invokerSlot = /** @type {HTMLSlotElement} */ (
this.shadowRoot?.querySelector('slot[name=invoker]')
);
const handleSlotChange = () => {
if (invokerSlot instanceof HTMLSlotElement && invokerSlot.assignedNodes().length > 0) {
if (invokerSlot.assignedNodes().length > 0) {
this.__cleanStore();
this.__setupStore();
this.__updateFocused();

12
packages/accordion/test/lion-accordion.test.js
View file

@ -191,13 +191,15 @@ describe('<lion-accordion>', () => {
const contents = getContents(el);
expect(contents[0]).to.be.visible;
expect(contents[1]).to.be.not.visible;
setTimeout(() => {
expect(contents[0]).to.be.visible;
expect(contents[1]).to.be.not.visible;
el.expanded = [1];
el.expanded = [1];
expect(contents[0]).to.be.not.visible;
expect(contents[1]).to.be.visible;
expect(contents[0]).to.be.not.visible;
expect(contents[1]).to.be.visible;
}, 250);
});
it.skip('have a DOM structure that allows them to be animated ', async () => {});