Skip to content

Commit 7e24d9b

Browse files
committed
test: add note about aria-query fix
1 parent d806161 commit 7e24d9b

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/__tests__/element-queries.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -432,13 +432,15 @@ test('queryAllByRole returns semantic html elements', () => {
432432
expect(queryAllByRole(/heading/i)).toHaveLength(6)
433433
expect(queryAllByRole('list')).toHaveLength(2)
434434
expect(queryAllByRole(/listitem/i)).toHaveLength(3)
435-
// TODO: fix upstream in `aria-query`
435+
// TODO: with https://github.com/A11yance/aria-query/pull/42
436+
// the actual value will match `toHaveLength(2)`
436437
expect(queryAllByRole(/textbox/i)).toHaveLength(1)
437438
expect(queryAllByRole(/checkbox/i)).toHaveLength(1)
438439
expect(queryAllByRole(/radio/i)).toHaveLength(1)
439440
expect(queryAllByRole('row')).toHaveLength(3)
440441
expect(queryAllByRole(/rowgroup/i)).toHaveLength(2)
441-
// TODO: fix upstream in `aria-query`
442+
// TODO: with https://github.com/A11yance/aria-query/pull/42
443+
// the actual value will match `toHaveLength(3)`
442444
expect(queryAllByRole(/(table)|(textbox)/i)).toHaveLength(2)
443445
expect(queryAllByRole(/img/i)).toHaveLength(1)
444446
expect(queryAllByRole('meter')).toHaveLength(1)

0 commit comments

Comments
 (0)