968818 - Create mozmill test for link context menu functionality

Status: RESOLVED INVALID

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

[Andreea Matei [:AndreeaMatei]]

This will test the context menu when you longpress a link: Copy, Bookmark, Open in new tab.
Here is the testcase for this test:
https://bug831971.bugzilla.mozilla.org/attachment.cgi?id=708621

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Why do you think that this has to be a Mozmill test?

Comment 2

[Andreea Matei [:AndreeaMatei]]

There's this test:
* http://mxr.mozilla.org/mozilla-central/source/browser/metro/base/tests/mochitest/browser_context_menu_tests.js#119
which checks all kinds of context menus, but from what I see it only checks for the visibility of those context items from a link, not the functionality.

Jim, could you please clarify if there is such a test covered or if it's in plan to be created? Thanks

Comment 3

[Jim Mathies [:jimm]]

(In reply to Andreea Matei [:AndreeaMatei] from comment #2)
> There's this test:
> *
> http://mxr.mozilla.org/mozilla-central/source/browser/metro/base/tests/
> mochitest/browser_context_menu_tests.js#119
> which checks all kinds of context menus, but from what I see it only checks
> for the visibility of those context items from a link, not the functionality.
> 
> Jim, could you please clarify if there is such a test covered or if it's in
> plan to be created? Thanks

Not currently. We can file a bug though to create some tests like this.

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

(In reply to Andreea Matei [:AndreeaMatei] from comment #2)
> which checks all kinds of context menus, but from what I see it only checks
> for the visibility of those context items from a link, not the functionality.

Are access keys and / or shortcuts involved here? If yes, that would qualify for a Mozmill test beside basic functional tests in mochitest.

Kamil, could you followup here?

Comment 5

[Kamil Jozwiak [:kjozwiak]]

I'm pretty sure there's no way of retrieving a context menu via access keys/keyboard shortcuts so this most likely qualifies as a mochitest rather than a Mozmill test. Henrik, would you like me to create a new ticket for the new mochitest?

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Thanks for checking that Kamil. So there is no need for a new bug. Lets move this into the right component and update the summary. That should be all to get started with.

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Metro is dead. So nothing to do here.