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

RESOLVED FIXED

Status

RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mozmill-test-failure])

Attachments

(4 attachments)

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.
Created attachment 509428 [details] [diff] [review]
Patch v1 [checked-in]
Attachment #509428 - Flags: review?(aaron.train)
Attachment #509428 - Flags: review?(aaron.train) → review+
Attachment #509428 - Attachment description: Patch v1 → Patch v1 [checked-in]
Created attachment 509449 [details] [diff] [review]
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.
Attachment #509449 - Flags: review?(aaron.train)
Created attachment 509450 [details] [diff] [review]
Backport (older branches)
Attachment #509450 - Flags: review?(aaron.train)
Attachment #509449 - Flags: review?(aaron.train) → review+
Attachment #509450 - Flags: review?(aaron.train) → review+
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)
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Sorry have to reopen. We missed something small.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Created attachment 509602 [details] [diff] [review]
quote fix

Small fix to report the real value in the logs/report.
Attachment #509602 - Flags: review?(aaron.train)
Comment on attachment 509602 [details] [diff] [review]
quote fix

Yep, saw this on the archive report. Thanks.
Attachment #509602 - Flags: review?(aaron.train) → review+
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.
Status: REOPENED → RESOLVED
Last Resolved: 8 years ago8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.