Open
Bug 1884982
Opened 2 months ago
Updated 2 months ago
browser-blocking tests are taking too long to run in debug mode, investigate and re-enable
Categories
(Core :: Privacy: Anti-Tracking, defect, P3)
Core
Privacy: Anti-Tracking
Tracking
()
REOPENED
125 Branch
Tracking | Status | |
---|---|---|
firefox125 | --- | fixed |
People
(Reporter: jmaher, Assigned: jmaher)
References
(Blocks 1 open bug)
Details
(Keywords: leave-open)
Attachments
(2 files)
No description provided.
Assignee | ||
Comment 1•2 months ago
|
||
runtime is in the 20 minutes for debug. for 17 tests, that is a bit too much. The outliers are (3 minutes each on windows):
browser_blockingSessionStorage.js
browser_blockingSharedWorkers.js
Assignee | ||
Comment 2•2 months ago
|
||
Updated•2 months ago
|
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Pushed by jmaher@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/72f2d6b322a3 disable toolkit/components/antitracking/test/browser/browser-blocking on debug due to extremely long runtime. r=aryx,anti-tracking-reviewers,timhuang
Comment 4•2 months ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
status-firefox125:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 125 Branch
Assignee | ||
Updated•2 months ago
|
Assignee | ||
Comment 5•2 months ago
|
||
Updated•2 months ago
|
Severity: -- → S3
Priority: -- → P3
Pushed by jmaher@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c5e567a16b9b disable rest of anti tracking browser test cases on debug for 25-40 minute runtimes. r=aryx,anti-tracking-reviewers,timhuang
Assignee | ||
Comment 7•2 months ago
|
||
some options to consider:
- run fewer tests- maybe there are tests that are very similar- cutting the number of tests in half could look better
- maybe this is important on a single platform and that might be a compromise- if we do re-enable, I would like these to be in <20 minute groupings on debug, so that would mean splitting up the manifest into multiple parts.
Comment 8•2 months ago
|
||
bugherder |
You need to log in
before you can comment on or make changes to this bug.
Description
•