Closed Bug 1829062 Opened 1 years ago Closed 1 years ago

Add "doh-rollout.doneFirstRun" to ignorePrefs.json, since it gets toggled in the background while tests are running

Categories

(Testing :: General, task)

Default
task

Tracking

(firefox114 fixed)

RESOLVED FIXED
114 Branch
Tracking Status
firefox114 --- fixed

People

(Reporter: dholbert, Assigned: dholbert)

References

Details

Attachments

(1 file)

I just did a Try run with some Test-Verify tasks for (slightly-modified) test_bug1261673.html, and I got this failure back from one of the tasks:

TEST-UNEXPECTED-FAIL | /tests/dom/html/test/test_bug1261673.html | changed preference: doh-rollout.doneFirstRun

Here's the log with the failure:
https://treeherder.mozilla.org/logviewer?job_id=413088341&repo=try&lineNumber=3986

This test does not in fact toggle that pref (or have anything to do with that pref); instead, this pref apparently gets toggled in the background, lazily, on the first time we do some DoH (DNS-over-HTTPS) activity, or something like that.

So: this isn't really a legit test failure. We should add this pref to testing/mochitest/ignorePrefs.json

This pref gets toggled lazily in the background, without tests explicitly
setting it, so it's appropriate for us to ignore it when auditing for prefs
that have been persistently modified during a test run.

Pushed by dholbert@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/88638acd2532 Add "doh-rollout.doneFirstRun" to our list of prefs that are allowed to be persistently changed during test-verify testruns. r=jmaher DONTBUILD
Status: ASSIGNED → RESOLVED
Closed: 1 years ago
Resolution: --- → FIXED
Target Milestone: --- → 114 Branch
Duplicate of this bug: 1827193
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: