Closed Bug 1544042 Opened 5 months ago Closed 5 months ago

Adding menu keyboard navigation test, issue with window-manager-less test environment

Categories

(Firefox :: Keyboard Navigation, defect, P5)

defect

Tracking

()

RESOLVED INVALID
Tracking Status
firefox68 --- affected

People

(Reporter: samuel.thibault, Unassigned)

Details

Attachments

(2 obsolete files)

Hello,

As part of adding UI regression testing in thunderbird, discussed on http://lists.thunderbird.net/pipermail/maildev_lists.thunderbird.net/2019-March/001469.html , we will need to use menus. I have prepared a unit test doing menu navigation tests, which I'll post on this bug report.

One issue I have with this unit test, however, is that if I run it in a Linux environment which does not have a window manager, such as xvfb-run (xvfb-run ./mach marionette-test), the menus do not remain open, they close shortly after opening. It seems that in the try server environment the problem arises. If it is thus ok to add such test that fails in xvfb-run, we can close this bug along with landing the test. If it is not, that bug will prevent us from testing many features of Thunderbird if we can't really open menus, and we will have to fix it along landing the test.

Samuel

Attachment #9057932 - Attachment description: Bug 1544042 marionette: add tests for opening/closing menus r=ato,whimboo → Bug 1544042 firefox-ui: add tests for opening/closing menus r=flod,Gijs

While looking at the existing test_menubar.py, I realize that it directly clicks on menu items without even opening the menu. That is something that thunderbird UI regression testing could use too indeed.

The proposed unit test addition however still holds to make sure that keyboard navigation works as expected.

Priority: -- → P5
Attachment #9057932 - Attachment is obsolete: true

From comments in https://phabricator.services.mozilla.com/D27324 , the keyboard navigation in menus is supposed to be already tested by mochitests.

Status: UNCONFIRMED → RESOLVED
Closed: 5 months ago
Resolution: --- → INVALID
Attachment #9060052 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.