Open Bug 1586001 Opened 2 years ago Updated 2 years ago

Tests required for autocomplete item labels

Categories

(Core :: Disability Access APIs, task, P2)

task

Tracking

()

People

(Reporter: morgan, Unassigned)

References

Details

Recently, bug 1568453 landed with changes to prevent autofill options from being read twice by screen readers. There are no tests for this change, and they should be added :)

Summary: Tests required for autofill → Tests required for autofill item labels

Simple test case:
data:text/html,<input list="datalist"><datalist id="datalist"><option>a</option><option>b</option></datalist>

Bug 1566299 also needs automated testing. We actually have tests for that one (accessible/tests/mochitest/events/test_focus_autocomplete.xul), but they were disabled, which is why that regressed in the first place. That test is super complicated and I think we should just write a new, promise-based test rather than trying to fix that one.

The question is: should we cover the label testing in the same test? Normally, we like to keep tree and event tests separate. In this case, you can't test an autofill without opening it, so I think it probably makes sense to just test the labels in the event test, rather than having two nearly identical tests just so we can categorise our tests nicely.

Type: defect → task
Priority: -- → P2
See Also: → 1566299
Summary: Tests required for autofill item labels → Tests required for autocomplete item labels
You need to log in before you can comment on or make changes to this bug.