Closed
Bug 1384721
Opened 7 years ago
Closed 7 years ago
fix pocket preffing causing non-local connection test failures
Categories
(Firefox :: New Tab Page, enhancement)
Firefox
New Tab Page
Tracking
()
RESOLVED
FIXED
Firefox 56
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: dmosedale, Assigned: dmosedale)
References
Details
Attachments
(1 file, 1 obsolete file)
Turning on pocket feeds by default in our github repo is causing our mochitests and talos tests on pine to fail. We need to make sure this is fixed before we do our next export to mozilla-central.
Assignee | ||
Comment 1•7 years ago
|
||
This patch should staunch the bleeding. I'm not sure it's the right fix for the slightly longer term (we probably want dummy data for the pocket feeds in the integration tests), but we need to staunch the bleeding ASAP so that we can land the latest activity-stream bits in mozilla-central to get them in nightly for telemetry and other reasons. I'm working on testing this change locally and on pine to see if it fixes the problem. Unfortunately, since artifact builds are busted, I've had to do a fresh debug build, which is taking a while...
Assignee | ||
Comment 2•7 years ago
|
||
This patch fixes talos and mochitests, but, oddly, not reftests.
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Attachment #8890547 -
Attachment is obsolete: true
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8890560 [details] Bug 1384721 - fix pocket network activity during tests, https://reviewboard.mozilla.org/r/161704/#review167006
Attachment #8890560 -
Flags: review?(edilee) → review+
Pushed by edilee@gmail.com: https://hg.mozilla.org/integration/autoland/rev/8798108d6ad4 fix pocket network activity during tests, r=Mardak
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8798108d6ad4
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Updated•5 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
•