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)
Firefox
New Tab Page
Tracking
()
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)
Assignee | ||
Updated•7 years ago
|
Iteration: --- → 61.1 - Mar 26
Priority: -- → P1
Assignee | ||
Comment 1•7 years ago
|
||
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)
Assignee | ||
Comment 2•7 years ago
|
||
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Updated•6 years ago
|
Component: Activity Streams: Newtab → New Tab Page
You need to log in
before you can comment on or make changes to this bug.
Description
•