1659610 - Intermittent browser/components/urlbar/tests/browser/browser_searchMode_setURI.js | This test exceeded the timeout threshold. It should be rewritten or split up. If that's not possible, use requestLongerTimeout(N), but only as a last resort. -

Status: RESOLVED FIXED

(Firefox :: Address Bar, defect, P5)

Description

[Treeherder Bug Filer]

Filed by: csabou [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=313269192&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/HdNsW4lETFCBWIXB7f8AAQ/runs/0/artifacts/public/logs/live_backing.log

---

[task 2020-08-17T22:13:25.138Z] 22:13:25 INFO - TEST-PASS | browser/components/urlbar/tests/browser/browser_searchMode_setURI.js | gURLBar.searchMode not expected - true == true -
[task 2020-08-17T22:13:25.139Z] 22:13:25 INFO - TEST-PASS | browser/components/urlbar/tests/browser/browser_searchMode_setURI.js | Expected placeholder l10n when search mode is inactive - {"args":{"name":"Google"},"id":"urlbar-placeholder-with-name"} deepEqual {"id":"urlbar-placeholder-with-name","args":{"name":"Google"}} -
[task 2020-08-17T22:13:25.139Z] 22:13:25 INFO - Leaving test bound tabSwitch
[task 2020-08-17T22:13:25.139Z] 22:13:25 INFO - Buffered messages finished
[task 2020-08-17T22:13:25.144Z] 22:13:25 INFO - TEST-UNEXPECTED-FAIL | browser/components/urlbar/tests/browser/browser_searchMode_setURI.js | This test exceeded the timeout threshold. It should be rewritten or split up. If that's not possible, use requestLongerTimeout(N), but only as a last resort. -
[task 2020-08-17T22:13:25.145Z] 22:13:25 INFO - GECKO(1657) | MEMORY STAT | vsize 7892MB | residentFast 529MB | heapAllocated 197MB
[task 2020-08-17T22:13:25.145Z] 22:13:25 INFO - TEST-OK | browser/components/urlbar/tests/browser/browser_searchMode_setURI.js | took 45561ms
[task 2020-08-17T22:13:25.145Z] 22:13:25 INFO - checking window state

Comment 1

[Drew Willcoxon :adw]

This test exceeded the timeout threshold. It should be rewritten or split up.

It does take a while and I wondered whether this would happen.

Comment 2

[Drew Willcoxon :adw]

Attachment: Bug 1659610 - Split up browser_searchMode_setURI.js to fix intermittent timeout.

The last line from the test in the failure log is "INFO - Leaving test bound
tabSwitch", which is the final task in the test. I'm pretty sure the test is just
doing too much rather than timing out waiting for something to happen. So let's
split it up.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1659128

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d67e96dc3d46
Split up browser_searchMode_setURI.js to fix intermittent timeout. r=harry

Comment 5

[Intermittent Failures Robot]

16 failures in 811 pushes (0.02 failures/push) were associated with this bug yesterday.

Repository breakdown:

• autoland: 14
• mozilla-central: 2

Platform breakdown:

• macosx1014-64: 14
• macosx1014-64-shippable: 2

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1659610&startday=2020-08-18&endday=2020-08-18&tree=all

Comment 6

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/d67e96dc3d46

Comment 7

[Intermittent Failures Robot]

20 failures in 4100 pushes (0.005 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• autoland: 17
• mozilla-central: 2
• try: 1

Platform breakdown:

• macosx1014-64: 17
• macosx1014-64-shippable: 3

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1659610&startday=2020-08-17&endday=2020-08-23&tree=all