Skip to content

Commit 1aff787

Browse files
committed
Add test for all types of menuitems
1 parent 5b06d14 commit 1aff787

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

test/test.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -248,13 +248,17 @@ describe('details-menu element', function() {
248248
document.body.innerHTML = ''
249249
})
250250

251-
it('manages checked state', function() {
251+
it('manages checked state and fires events', function() {
252252
const details = document.querySelector('details')
253253
const item = details.querySelector('button')
254+
let eventCounter = 0
255+
document.addEventListener('details-menu-selected', () => eventCounter++, true)
256+
254257
assert.equal(item.getAttribute('aria-checked'), 'false')
255258
item.dispatchEvent(new MouseEvent('click', {bubbles: true}))
256259
assert.equal(item.getAttribute('aria-checked'), 'true')
257260
assert.equal(details.querySelectorAll('[aria-checked="true"]').length, 1)
261+
assert.equal(eventCounter, 1, 'selected event is fired twice')
258262
})
259263
})
260264

@@ -288,7 +292,6 @@ describe('details-menu element', function() {
288292
item.dispatchEvent(new MouseEvent('click', {bubbles: true}))
289293
assert.equal(item.getAttribute('aria-checked'), 'true')
290294
assert.equal(details.querySelectorAll('[aria-checked="true"]').length, 1)
291-
292295
assert.equal(eventCounter, 1, 'selected event is fired twice')
293296
})
294297
})

0 commit comments

Comments
 (0)