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

RESOLVED FIXED

Status

Mozilla QA
Mozmill Tests
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mozmill-test-failure])

Attachments

(4 attachments)

(Assignee)

Description

7 years ago
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.
(Assignee)

Comment 1

7 years ago
Created attachment 509428 [details] [diff] [review]
Patch v1 [checked-in]
Attachment #509428 - Flags: review?(aaron.train)

Updated

7 years ago
Attachment #509428 - Flags: review?(aaron.train) → review+
(Assignee)

Updated

7 years ago
Attachment #509428 - Attachment description: Patch v1 → Patch v1 [checked-in]
(Assignee)

Comment 2

7 years ago
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)
(Assignee)

Comment 3

7 years ago
Created attachment 509450 [details] [diff] [review]
Backport (older branches)
Attachment #509450 - Flags: review?(aaron.train)

Updated

7 years ago
Attachment #509449 - Flags: review?(aaron.train) → review+

Updated

7 years ago
Attachment #509450 - Flags: review?(aaron.train) → review+
(Assignee)

Comment 4

7 years ago
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: 7 years ago
Resolution: --- → FIXED
(Assignee)

Comment 5

7 years ago
Sorry have to reopen. We missed something small.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 6

7 years ago
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+
(Assignee)

Comment 8

7 years ago
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: 7 years ago7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.