Intermittent browser/components/search/test/browser/browser_search_telemetry_sources.js | single tracking bug
Categories
(Firefox :: Search, defect, P5)
Tracking
()
Tracking | Status | |
---|---|---|
firefox115 | --- | fixed |
People
(Reporter: jmaher, Assigned: jteow)
References
Details
(Keywords: intermittent-failure, intermittent-testcase, Whiteboard: [stockwell disable-recommended])
Attachments
(1 file)
Reporter | ||
Comment 1•2 years ago
|
||
Additional information about this bug failures and frequency patterns can be found by running: ./mach test-info failure-report --bug 1775917
Updated•2 years ago
|
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 5•2 years ago
|
||
https://wiki.mozilla.org/Bug_Triage#Intermittent_Test_Failure_Cleanup
For more information, please visit auto_nag documentation.
Comment 6•2 years ago
|
||
treeherder |
New failure instance: https://treeherder.mozilla.org/logviewer?job_id=387317528&repo=autoland
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 10•2 years ago
|
||
https://wiki.mozilla.org/Bug_Triage#Intermittent_Test_Failure_Cleanup
For more information, please visit auto_nag documentation.
Comment 11•2 years ago
|
||
treeherder |
New failure instance: https://treeherder.mozilla.org/logviewer?job_id=396026497&repo=autoland
Comment hidden (Intermittent Failures Robot) |
Comment 13•1 year ago
|
||
https://wiki.mozilla.org/Bug_Triage#Intermittent_Test_Failure_Cleanup
For more information, please visit auto_nag documentation.
Comment 14•1 year ago
|
||
treeherder |
New failure instance: https://treeherder.mozilla.org/logviewer?job_id=402532757&repo=autoland
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 17•1 year ago
|
||
https://wiki.mozilla.org/Bug_Triage#Intermittent_Test_Failure_Cleanup
For more information, please visit auto_nag documentation.
Comment 18•11 months ago
|
||
treeherder |
New failure instance: https://treeherder.mozilla.org/logviewer?job_id=416823612&repo=autoland
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 23•11 months ago
•
|
||
I think it's because the Persist Search tip is appearing. Kind of strange that it only popped up recently.
Here is the log of the results of the urlbar from getFirstSuggestionIndex
when I get a test failure on TH:
[task 2023-05-29T12:26:31.940Z] 12:26:31 INFO - GECKO(20387) | console.log: Result: {"result":{"type":7,"source":5,"rowIndex":0,"heuristic":false,"exposureResultType":"","exposureResultHidden":false,"payload":{"type":"searchTip_persist","buttons":[{"l10n":{"id":"urlbar-search-tips-confirm-short"}}],"icon":"chrome://global/skin/icons/lightbulb.svg","titleL10n":{"id":"urlbar-search-tips-persist"}},"payloadHighlights":{"type":[],"buttons":[],"icon":[]},"providerName":"UrlbarProviderSearchTips","providerType":2},"url":null,"postData":null,"type":7,"source":5,"heuristic":false,"autofill":false,"image":"chrome://global/skin/icons/lightbulb.svg","title":"","tags":[],"displayed":{"title":"Searching just got simpler. Try making your search more specific here in the address bar. To show the URL instead, visit Search, in settings.","action":"","url":"","typeIcon":"none"},"element":{"action":{},"row":{"_elements":{},"_buttons":{},"result":{"type":7,"source":5,"rowIndex":0,"heuristic":false,"exposureResultType":"","exposureResultHidden":false,"payload":{"type":"searchTip_persist","buttons":[{"l10n":{"id":"urlbar-search-tips-confirm-short"}}],"icon":"chrome://global/skin/icons/lightbulb.svg","titleL10n":{"id":"urlbar-search-tips-persist"}},"payloadHighlights":{"type":[],"buttons":[],"icon":[]},"providerName":"UrlbarProviderSearchTips","providerType":2},"_content":{}},"separator":{},"title":{"_tooltip":""},"url":{"_tooltip":""}}}
It's strange that it's appearing in this situation when we haven't hit a search page yet.
Will continue investigating.
Updated•11 months ago
|
Assignee | ||
Comment 24•11 months ago
|
||
The tests enter "searchSuggestion" in the Urlbar and wait for the Urlbar to open, at which point it queries the list of results and looks for a search suggestion. If no search results are available, it'll return -1.
For reference, here is a commit here where I add some logging to help understand the failures.
The reason why it's failing is because the Persist Search Tip is the result in the Urlbar: https://treeherder.mozilla.org/logviewer?job_id=417409193&repo=try&lineNumber=42488 . This is strange because we haven't landed on a SERP and this is the first test in this file. I wonder if it's caused from another test?
I added a log right before we commit to showing a search tip. In the same job, the search tip is schedule to show in browser/components/search/test/browser/browser_rich_suggestions.js
. Locally, I am able to confirm it'll pass the checks to generate a search tip. That test gets run several tests before browser_search_telemetry_sources.js
, I not expecting it to remain in the Urlbar so far after.
I believe that test is causing the issue because I haven't touched the persist search tip in months and I don't think anything in browser_search_telemetry_sources.js
would trigger this issue.
I noticed browser_rich_suggestions.js
had a commit on May 25th at 10:50 and the surge of failures started on May 25th at 14:14 onward. The change indeed does a search with a term suggested in the Urlbar, and then immediately asserts telemetry is recorded properly.
I'll try testing disabling the persist search tip in in the rich suggestions test to see if it alleviates the issue, since I can regularly hit this error in running these tests on Try. It might be the case that we should avoid enabling the persist search tip on all search tests since it can cause issues such as these, but I'd like to know why one test is affecting another, that seems iffy.
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 26•11 months ago
|
||
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 29•11 months ago
|
||
Pushed by jteow@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9718fca142ae Disable the persist search tip from appearing in browser_rich_suggestions.js - r=daleharvey
Comment hidden (Intermittent Failures Robot) |
Comment 31•11 months ago
|
||
bugherder |
Comment hidden (Intermittent Failures Robot) |
Description
•