Open Bug 1758142 Opened 2 years ago Updated 2 years ago

Fix dummy task workaround in test_clear_form_expiry_select_element

Categories

(Toolkit :: Form Autofill, task, P3)

task

Tracking

()

People

(Reporter: tgiles, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [formautofill-test-refactoring])

As part of Bug 1740001, we discovered some timing issues and other irregularities in our test infrastructure. In order to get the previously mentioned bug landed, I added a dummy task that checks an address entry first before trying the same kind of test with a credit card entry. This particular setup causes the test to pass consistently. If you happen to run a credit card test first in this case, the test will fail in one of two ways:

  • Consistent timeout due to setting up the popup listener after the popup has already appeared. This appears to be due to how triggerPopupAndHoverItem is laid out, but changing this control flow causes new issues to appear.
  • Consistent exception of "JavaScript error: , line 0: uncaught exception: Checking selected index - timed out after 50 tries." due to the selected index being 1 while the expected index was 0. I'm not sure how the test is getting out of sync here with the satchel_common.js notifySelectedIndex call but this is another thing that will need to be investigated.

I've spent a lot of time trying to figure these issues out and I haven't got a resolution yet, but I'll update the ticket with my findings as I go along.

Blocks: 1757806
Status: ASSIGNED → NEW
Priority: P2 → P3
Whiteboard: [formautofill-test-refactoring]
Assignee: tgiles → nobody
You need to log in before you can comment on or make changes to this bug.