Closed Bug 1162349 Opened 9 years ago Closed 9 years ago

integration tests for context menu

Categories

(Firefox OS Graveyard :: Gaia::TV::System, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: johnhu, Assigned: johnhu)

References

Details

Attachments

(1 file)

Since the context menu is operated by keyboard, we should create a integration tests to operate it with keyboard and check if the focus and style is updated.
Comment on attachment 8602581 [details] [review]
[gaia] huchengtw-moz:bug-1162349-integration-tests-context-menu > mozilla-b2g:master

Hi,

Since this is the first integration test, it would be nice to be reviewed by two people. Please review it. Thanks.
Attachment #8602581 - Flags: review?(rexboy)
Attachment #8602581 - Flags: review?(chens)
Comment on attachment 8602581 [details] [review]
[gaia] huchengtw-moz:bug-1162349-integration-tests-context-menu > mozilla-b2g:master

Looks good to me, left some comments on github.
Attachment #8602581 - Flags: review?(chens) → review+
Comment on attachment 8602581 [details] [review]
[gaia] huchengtw-moz:bug-1162349-integration-tests-context-menu > mozilla-b2g:master

Smart-system part looks good to me.
I'm not familiar with marionette test so I just read the test once and they seems ok to me.
Thanks a lot!
Attachment #8602581 - Flags: review?(rexboy) → review+
Assignee: nobody → im
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: