631235 - Timeout exceeded for waitForElement XPath: /html/body/div[2]/div/table/tbody/tr[1]/td in testGoogleSuggestions.js

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

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

Module:    /testGeneral/testGoogleSuggestions.js
Test:      testGoogleSuggestedTerms    
Failure:   Timeout exceeded for waitForElement XPath: /html/body/div[2]/div/table/tbody/tr[1]/td 
Branches:  ALL
Platforms: ALL

Using the XPath string in the autocomplete test is wonky in some areas around the globe. We should get rid of it and replace it with a nodeCollector instance.

Patch upcoming.

Comment 1

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

Attachment: Patch v1 [checked-in]

Comment 2

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

Attachment: Follow-up

I should have checked the older branches first. We get a security error when clicking one of the autocomplete results. Instead of click we should use KeyDown + Return which is even closer to an user scenario.

Comment 3

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

Attachment: Backport (older branches)

Comment 4

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

Landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/d3fbdc5b71b1 (default)
http://hg.mozilla.org/qa/mozmill-tests/rev/5241646f0eac (1.9.2)
http://hg.mozilla.org/qa/mozmill-tests/rev/7fc6c7b322ab (1.9.1)

Comment 5

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

Sorry have to reopen. We missed something small.

Comment 6

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

Attachment: quote fix

Small fix to report the real value in the logs/report.

Comment 7

[Aaron Train [:aaronmt]]

Comment on attachment 509602 [details] [diff] [review]
quote fix

Yep, saw this on the archive report. Thanks.

Comment 8

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

Landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/6a13621ad288 (default)
http://hg.mozilla.org/qa/mozmill-tests/rev/0bd3830206e2 (1.9.2)
http://hg.mozilla.org/qa/mozmill-tests/rev/8846a2059862 (1.9.1)

Not sure why the test failed on horus in my test-run. But lets see tomorrows results. We should then file a new bug if it persists.