-
Notifications
You must be signed in to change notification settings - Fork 9
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #1853 from exadel-inc/fix/incorrect-activator-on-c…
…lose fix(esl-toggleable): fix activator when close-on inner trigger click handled (#1852)
- Loading branch information
Showing
2 changed files
with
110 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
102 changes: 102 additions & 0 deletions
102
src/modules/esl-toggleable/test/toggleable.close-on.test.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,102 @@ | ||
import {ESLToggleable} from '../core/esl-toggleable'; | ||
|
||
describe('ESLToggleable: close-on trigger handler', () => { | ||
const $template = document.createElement('template'); | ||
$template.innerHTML = ` | ||
<esl-toggleable class='toggleable-close-on-test' close-on='.trigger-close' defaultParams='{delay: -1}'> | ||
<button type='button' class='trigger trigger-close'><span>Close<span></button> | ||
<button type='button' class='trigger trigger-additional'><span>Trigger<span></button> | ||
</esl-toggleable> | ||
<button type='button' class='trigger-close trigger-outside toggleable-close-on-test'> | ||
<span>Close<span> | ||
</button> | ||
`; | ||
const elements = { | ||
get $el() { | ||
return document.querySelector('esl-toggleable.toggleable-close-on-test') as ESLToggleable; | ||
}, | ||
get $trigger() { | ||
return document.querySelector('.trigger-close') as HTMLElement; | ||
}, | ||
get $triggerOutside() { | ||
return document.querySelector('.trigger-outside') as HTMLElement; | ||
}, | ||
get $triggerAdditional() { | ||
return document.querySelector('.trigger-additional') as HTMLElement; | ||
} | ||
}; | ||
|
||
beforeAll(() => ESLToggleable.register()); | ||
beforeEach(() => document.body.appendChild($template.content.cloneNode(true))); | ||
afterEach(() => [...document.querySelectorAll('.toggleable-close-on-test')].forEach(($el) => $el.remove())); | ||
|
||
test('Close-on trigger click leads to toggleable hide', () => { | ||
const {$el, $trigger} = elements; | ||
|
||
$el.show(); | ||
expect($el.open).toBe(true); | ||
|
||
$trigger.click(); | ||
expect($el.open).toBe(false); | ||
}); | ||
|
||
test('Click on inner trigger content leads to toggleable hide', () => { | ||
const {$el, $trigger} = elements; | ||
|
||
$el.show(); | ||
expect($el.open).toBe(true); | ||
|
||
$trigger.querySelector('span')?.click(); | ||
expect($el.open).toBe(false); | ||
}); | ||
|
||
test('Close-on trigger click outside toggleable does not leads to toggleable hide', () => { | ||
const {$el, $triggerOutside} = elements; | ||
|
||
$el.show(); | ||
expect($el.open).toBe(true); | ||
|
||
$triggerOutside.click(); | ||
expect($el.open).toBe(true); | ||
}); | ||
|
||
test('Close-on trigger click on additional trigger does not leads to toggleable hide', () => { | ||
const {$el, $triggerAdditional} = elements; | ||
|
||
$el.show(); | ||
expect($el.open).toBe(true); | ||
|
||
$triggerAdditional.click(); | ||
expect($el.open).toBe(true); | ||
}); | ||
|
||
test('Dynamic change of close-on attribute processed correctly', () => { | ||
const {$el, $triggerAdditional} = elements; | ||
|
||
$el.show(); | ||
$el.setAttribute('close-on', '.trigger'); | ||
|
||
$triggerAdditional.click(); | ||
expect($el.open).toBe(false); | ||
}); | ||
|
||
test('ESLToggleableActionParams contains correct activator details', () => { | ||
const {$el, $trigger} = elements; | ||
|
||
const $fn = jest.fn((e) => console.log(e)); | ||
|
||
$el.show(); | ||
$el.addEventListener('esl:hide', $fn); | ||
$trigger.querySelector('span')?.click(); | ||
|
||
expect($fn).lastCalledWith(expect.objectContaining({ | ||
type: 'esl:hide', | ||
detail: expect.objectContaining({ | ||
params: expect.objectContaining({ | ||
initiator: 'close', | ||
activator: $trigger | ||
}) | ||
}) | ||
})); | ||
}); | ||
}); |