Closed Bug 1446656 Opened 7 years ago Closed 7 years ago

Fix AboutPreferences.jsm and/or preferences tests

Categories

(Firefox :: New Tab Page, defect, P1)

defect

Tracking

()

RESOLVED FIXED
Firefox 61
Iteration:
61.1 - Mar 26
Tracking Status
firefox61 --- fixed

People

(Reporter: Mardak, Assigned: Mardak)

References

Details

jaws, any ideas for what would cause the missing "nothing found" -> "Should be in search results" failure in browser_search_within_preferences_1.js below? I.e., what would make it find nothing but fail to display the "nothing found"? Bug 1446186 and bug 1446392 were most likely caused by the export of bug 1404890 (attempted to land in m-c in 1446053). In the export, we did fix up a test in browser_search_within_preferences_1.js by adding "homeContentsGroup" to the expected id visible list: https://hg.mozilla.org/mozilla-central/diff/b0af4c10b8bb/browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js Looks like browser_search_within_preferences_1.js has an intermittent test when it tries to search for "coach" expecting to find the "nothing found", but the screenshot from bug 1446186 shows nothing appears below the search header. TEST-UNEXPECTED-FAIL | browser/components/preferences/in-content/tests/browser_search_within_preferences_1.js | Should be in search results - Separately bug 1446392 has unexpected network to getpocket.cdn.mozilla.net from browser_privacypane_8/4.js that seems to match up with the href added for the sponsored learn more link: https://hg.mozilla.org/mozilla-central/diff/b0af4c10b8bb/browser/extensions/activity-stream/lib/AboutPreferences.jsm#l1.239 Although I thought sponsored shouldn't be set during these tests anyway, so maybe it's another link triggering it..
Flags: needinfo?(jaws)
Iteration: --- → 61.1 - Mar 26
Priority: -- → P1
Looks like the "nothing found" failures were actually caused by bug 1445084. And network was from topstories custom false test pref being reset to the firefox default.
Flags: needinfo?(jaws)
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.