Closed Bug 1129658 Opened 7 years ago Closed 2 years ago

Convert Mozmill test 'functional/testTabbedBrowsing/testOpenInForeground.js' to Marionette

Categories

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

defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: whimboo, Unassigned)

References

Details

(Whiteboard: [blocked])

Here my feedback after looking at this test:

* 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 false
* Open the test page 
* Check that opening a new tab via middle click works
* Select Tab 1
* Check that opening a new tab via the context menu works (lets not keep the for loop)
* Check that the tabs have been opened in the right order by checking the id element in the web page
It looks like this is blocked until middle click is implemented in bug 1097705
Depends on: 1097705
Thank you Rebecca for checking this! Lets hope we can get this dependency fixed soon.
Whiteboard: [blocked]
Priority: P1 → P2
Product: Mozilla QA → Testing

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

Status: NEW → RESOLVED
Closed: 2 years ago
QA Contact: hskupin
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.