🧪 Improve test readability by reformatting code for better clarity and consistency in BaseModal unit tests
This commit is contained in:
@@ -121,7 +121,9 @@ describe('BaseModal', () => {
|
|||||||
|
|
||||||
await nextTick();
|
await nextTick();
|
||||||
const container = document.querySelector('.modal-container');
|
const container = document.querySelector('.modal-container');
|
||||||
expect(container.classList.contains(`modal-container--${size}`)).toBe(true);
|
expect(container.classList.contains(`modal-container--${size}`)).toBe(
|
||||||
|
true
|
||||||
|
);
|
||||||
wrapper.unmount();
|
wrapper.unmount();
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
@@ -288,7 +290,8 @@ describe('BaseModal', () => {
|
|||||||
modelValue: true
|
modelValue: true
|
||||||
},
|
},
|
||||||
slots: {
|
slots: {
|
||||||
default: '<button id="first-btn">First</button><button>Second</button>'
|
default:
|
||||||
|
'<button id="first-btn">First</button><button>Second</button>'
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
@@ -305,7 +308,8 @@ describe('BaseModal', () => {
|
|||||||
modelValue: true
|
modelValue: true
|
||||||
},
|
},
|
||||||
slots: {
|
slots: {
|
||||||
default: '<button id="btn1">Button 1</button><button id="btn2">Button 2</button>'
|
default:
|
||||||
|
'<button id="btn1">Button 1</button><button id="btn2">Button 2</button>'
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
@@ -320,7 +324,10 @@ describe('BaseModal', () => {
|
|||||||
expect(document.activeElement).toBe(btn2);
|
expect(document.activeElement).toBe(btn2);
|
||||||
|
|
||||||
// Press Tab - should cycle to first button
|
// Press Tab - should cycle to first button
|
||||||
const tabEvent = new KeyboardEvent('keydown', { key: 'Tab', bubbles: true });
|
const tabEvent = new KeyboardEvent('keydown', {
|
||||||
|
key: 'Tab',
|
||||||
|
bubbles: true
|
||||||
|
});
|
||||||
document.dispatchEvent(tabEvent);
|
document.dispatchEvent(tabEvent);
|
||||||
await nextTick();
|
await nextTick();
|
||||||
|
|
||||||
@@ -442,7 +449,10 @@ describe('BaseModal', () => {
|
|||||||
await nextTick();
|
await nextTick();
|
||||||
wrapper.unmount();
|
wrapper.unmount();
|
||||||
|
|
||||||
expect(removeEventListenerSpy).toHaveBeenCalledWith('keydown', expect.any(Function));
|
expect(removeEventListenerSpy).toHaveBeenCalledWith(
|
||||||
|
'keydown',
|
||||||
|
expect.any(Function)
|
||||||
|
);
|
||||||
});
|
});
|
||||||
|
|
||||||
it('restores body overflow on unmount', async () => {
|
it('restores body overflow on unmount', async () => {
|
||||||
|
|||||||
Reference in New Issue
Block a user