Closed
Bug 1623699
Opened 5 years ago
Closed 5 years ago
Add tests for Resources.watch against console messages
Categories
(DevTools :: Console, enhancement, P1)
DevTools
Console
Tracking
(Fission Milestone:M6, firefox76 fixed)
Tracking | Status | |
---|---|---|
firefox76 | --- | fixed |
People
(Reporter: ochameau, Assigned: ochameau)
References
Details
(Whiteboard: dt-fission-m2-mvp)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Bug 1620234 is going to add support for console messages listening via the Resources API.
It does that without adding any test. It is still covered by many tests. All frontend tests and also the actor tests, which the resource API is using.
But as we move forward with bug 1620243, we are going to stop using the existing actor APIs (getCachedMessages and startListeners):
- It will be important to keep the coverage of existing tests against getCachedMessages and startListeners
- We are most likely going to followup and remove getCachedMessages and startListeners methods in favor of Resource APIs actor methods.
Assignee | ||
Comment 1•5 years ago
|
||
Assignee | ||
Comment 2•5 years ago
|
||
This copies assertions from existing actor/front tests in order to be able to later
drop startListeners and getCachedMessages on the console actor.
Updated•5 years ago
|
Status: NEW → ASSIGNED
Comment 3•5 years ago
|
||
Tracking Fission DevTools bugs for Fission Nightly (M6) milestone
Fission Milestone: --- → M6
Pushed by apoirot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2963e55978fe
Add tests against Resources.watch for console messages. r=nchevobbe
Comment 5•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox76:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 76
You need to log in
before you can comment on or make changes to this bug.
Description
•