1129665 - Convert Mozmill test 'functional/testTabbedBrowsing/testOpenInBackground.js' to Marionette

Status: RESOLVED WONTFIX

(Testing :: Firefox UI Tests, defect, P2)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

This bug will cover the necessary work to do to convert the following test into Marionette.

http://hg.mozilla.org/qa/mozmill-tests/file/default/firefox/tests/functional/testTabbedBrowsing/testOpenInBackground.js

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Here my feedback after looking at this test:

* This test is similar to the test we cover in bug 1129658. It only opens the same tabs in the background.

* I see this test valuable given that our unit tests do not cover the usage of
middle click on a link and nor using its context menu.

* I don't see what this specific close tab test is helping us here, which
indeed not even fit into this test. So we want to remove those lines.

So what to do in raw steps:

* Ensure the pref to load tabs in the background is set to true
* Open the test page 
* Check that opening a new tab via the context menu works and it opens in the background
* Switch to the new tab and back to the original tab
* Check that opening two more tabs via the context menu works (lets not keep the
for loop for all)
* Check that the tabs have been opened in the right order by checking the id
element in the web page

Comment 2

[Barbara Miller (:galgeek)]

Attachment: github pull request

I don't find examples of other Firefox ui tests accessing a context menu, and I have yet to figure it out. Is this otherwise a good start?

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+1]]

(In reply to Barbara Miller (:galgeek) from comment #2)
> I don't find examples of other Firefox ui tests accessing a context menu,

This is something Tareq currently is working on via bug 1121710. So maybe lets wait some days for its inclusion in this test.

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Comment on attachment 8681670 [details] [review]
github pull request

I made some comments on the PR but as a starter it looks fine to me. I would suggest doing it more data driven to reduce the amount of redundant (duplicated) lines.

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+1]]

There is good coverage in browser chrome tests so this is not needed, and we are also going to remove the firefox-ui tests.