Closed
Bug 1341983
Opened 7 years ago
Closed 7 years ago
Disable services/common/tests/unit/test_blocklist_certificates.js in Thunderbird after bug 1224528
Categories
(Cloud Services :: Firefox: Common, defect)
Cloud Services
Firefox: Common
Tracking
(firefox54 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox54 | --- | fixed |
People
(Reporter: jorgk-bmo, Assigned: jorgk-bmo)
Details
(Whiteboard: [Thunderbird-testfailure: X all])
Attachments
(1 file)
904 bytes,
patch
|
leplatrem
:
review+
|
Details | Diff | Splinter Review |
Preparing patch now
Assignee | ||
Comment 1•7 years ago
|
||
Actually ... Bug 1224528 added this: https://hg.mozilla.org/mozilla-central/rev/735f81d9fd96c9d19d3f2fdfda29a14c1a066d22#l1.12 ; [Default Preferences] ; All the pref files must be part of base to prevent migration bugs @RESPATH@/browser/@PREF_DIR@/firefox.js @RESPATH@/browser/@PREF_DIR@/firefox-branding.js @RESPATH@/greprefs.js @RESPATH@/defaults/autoconfig/prefcalls.js @RESPATH@/browser/defaults/permissions +@RESPATH@/browser/defaults/blocklists Is this only applicable to Firefox? SM has @RESPATH@/defaults/permissions but TB/IM have not. So perhaps better to disable the test?
Flags: needinfo?(frgrahl)
Assignee | ||
Comment 2•7 years ago
|
||
Patch to disable test.
Attachment #8840306 -
Flags: review?(mgoodwin)
Assignee | ||
Comment 3•7 years ago
|
||
Comment on attachment 8840306 [details] [diff] [review] 1341983-skip-test_blocklist_certificates.patch Mathieu, perhaps you can approve this half-line change or advise us otherwise. We're currently seeing test failures all over our tree which I'd like to resolve.
Attachment #8840306 -
Flags: review?(mathieu)
Comment 4•7 years ago
|
||
Comment on attachment 8840306 [details] [diff] [review] 1341983-skip-test_blocklist_certificates.patch Disabling this test is one way to go. The other way would be to ship the JSON files on Thunderbird too (see services/blocklists/moz.build). Note: In the long term (Bug 1257565) the JSON diff-based system is going to replace the blocklist XML daily download.
Attachment #8840306 -
Flags: review?(mathieu) → review+
Assignee | ||
Comment 5•7 years ago
|
||
(In reply to Mathieu Leplatre (:leplatrem) from comment #4) > Disabling this test is one way to go. The other way would be to ship the > JSON files on Thunderbird too (see services/blocklists/moz.build). Can you please advise us on what is best? Sorry about the our lack of knowledge, we Thunderbird developers have to interface with many parts of Mozilla Core we have little or no knowledge of. Would there be a security benefit of porting the Firefox changes instead of disabling the test? I see that in services/blocklists/moz.build you added +if CONFIG['MOZ_BUILD_APP'] == 'browser': + DIST_SUBDIR = 'browser' which may relate to +@RESPATH@/browser/defaults/blocklists Sadly my forte is also not build configuration, but we could get some of our build guys to look into porting this.
Assignee | ||
Updated•7 years ago
|
Attachment #8840306 -
Flags: review?(mgoodwin)
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(frgrahl) → needinfo?(mathieu)
Comment 6•7 years ago
|
||
> Can you please advise us on what is best? I would suggest to skip the test to get your green build back. And maybe open a new issue to mimic the delivery of those JSON files. > Sorry about the our lack of knowledge Please, don't be sorry! Basically, the Bug 1224528 consists in shipping initial JSON files for the blocklist settings, so that clients do not have to download the whole list on a cold start (first use). > Would there be a security benefit of porting the Firefox changes instead of disabling the test? AFAIK, no.
Flags: needinfo?(mathieu)
Assignee | ||
Updated•7 years ago
|
Component: General → Firefox: Common
Keywords: checkin-needed
Product: Thunderbird → Cloud Services
Summary: Port Bug 1224528 to C-C: Load initial JSON files for blocklist - TEST-UNEXPECTED-FAIL | services/common/tests/unit/test_blocklist_certificates.js → Disable services/common/tests/unit/test_blocklist_certificates.js in Thunderbird after bug 1224528
Assignee | ||
Comment 7•7 years ago
|
||
Thanks. We've already been trough bug 1266794, bug 1247662, bug 1248557 and bug 1312827 in this area. Hopefully one day we'll have someone knowable enough on the TB team to decide TB's way forward on this.
Updated•7 years ago
|
Assignee: nobody → jorgk
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/185d93157773 Skip test_blocklist_certificates.js for Thunderbird. r=leplatrem
Keywords: checkin-needed
Comment 9•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/185d93157773
You need to log in
before you can comment on or make changes to this bug.
Description
•