Closed Bug 1775917 Opened 2 years ago Closed 11 months ago

Intermittent browser/components/search/test/browser/browser_search_telemetry_sources.js | single tracking bug

Categories

(Firefox :: Search, defect, P5)

defect

Tracking

()

RESOLVED FIXED
115 Branch
Tracking Status
firefox115 --- fixed

People

(Reporter: jmaher, Assigned: jteow)

References

Details

(Keywords: intermittent-failure, intermittent-testcase, Whiteboard: [stockwell disable-recommended])

Attachments

(1 file)

No description provided.

Additional information about this bug failures and frequency patterns can be found by running: ./mach test-info failure-report --bug 1775917

Severity: -- → S4
Priority: P3 → P5
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INCOMPLETE
Status: RESOLVED → REOPENED
Resolution: INCOMPLETE → ---
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → INCOMPLETE
Status: RESOLVED → REOPENED
Resolution: INCOMPLETE → ---
Status: REOPENED → RESOLVED
Closed: 2 years ago1 year ago
Resolution: --- → INCOMPLETE
Status: RESOLVED → REOPENED
Resolution: INCOMPLETE → ---
Status: REOPENED → RESOLVED
Closed: 1 year ago1 year ago
Resolution: --- → INCOMPLETE
Status: RESOLVED → REOPENED
Resolution: INCOMPLETE → ---

James, please can you take a look?

Flags: needinfo?(jteow)

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.

Flags: needinfo?(jteow)
Whiteboard: [stockwell needswork:owner]

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.

Assignee: nobody → jteow
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
Status: REOPENED → RESOLVED
Closed: 1 year ago11 months ago
Resolution: --- → FIXED
Target Milestone: --- → 115 Branch
See Also: → 1842609
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: