557799 - [mozmill] Cannot validate tab id in testOpenInForegroundTab (testOpenInForeground.js)

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

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

Looks like opening the last tab with a middle click doesn't work.

could not validate element ID: id with text 3

Comment 1

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

Kenny, would you like to check what's wrong with this test? But only if you are able to reproduce it.

Comment 2

[Kenny Wong [:redgroup]]

I'm on it.

Comment 3

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

Thanks Kenny!

Comment 4

[Aaron Train [:aaronmt]]

Attachment: Patch v1 - waitForPageLoad

Hi Kenny,

If you don't mind I will look at this bug. 

This bug is a result of failing an assert on the last opened tab looking at the textContent of the linkId. The textContent value has an empty string. As it's the last tab we open in the foreground before the check, we are missing a waitForPageLoad as waitForElement is not enough time to wait for the textContent property to have a value. The following patch was written against 1.9.2 and works against this branch along with 1.9.1.

Comment 5

[Aaron Train [:aaronmt]]

Attachment: Patch v1 - waitForPageLoad (1.9.2)

As per Henrik's request to place the check inside the for loop.

Comment 6

[Aaron Train [:aaronmt]]

Attachment: Patch v1 - waitForPageLoad (1.9.1)

Same patch for 1.9.1

Comment 7

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

I don't think that we need those patches anymore. Given my research on bug 552594 the patch there should also fix this test failure.

Comment 8

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

Should be fixed now with bug 552594. Please keep watching the results the next days and reopen if necessary.

Comment 9

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

Mass move of Mozmill Test related project bugs to newly created components. You can filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.