Closed Bug 1307918 Opened 8 years ago Closed 5 years ago

Split old console pageError tests into backend and frontend unit tests

Categories

(DevTools :: Console, enhancement, P2)

enhancement

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: linclark, Unassigned)

References

Details

Originally posted by:linclark see https://github.com/devtools-html/gecko-dev/issues/230 Currently pageErrors are tested with integration tests. We can convert these into unit tests. The backend test would still be a mochitest. It should load the test page and ensure the correct packet is sent, comparing to a stub packet. The frontend can then use that stub in a mocha test. Tests to convert (list in progress): - [ ] browser_webconsole_bug_737873_mixedcontent.js - [ ] browser_webconsole_trackingprotection_errors.js - [ ] browser_webconsole_bug_770099_violation.js - [ ] browser_webconsole_bug_762593_insecure_passwords_web_console_warning.js - [ ] browser_webconsole_bug_762593_insecure_passwords_about_blank_web_console_warning.js - [ ] browser_bug1045902_console_csp_ignore_reflected_xss_message.js
Priority: -- → P2
Whiteboard: new-console
Flags: qe-verify-
Whiteboard: new-console → [new-console]
Priority: P2 → P3
Whiteboard: [new-console] → [reserve-new-console]
Whiteboard: [reserve-new-console] → [reserve-console-html]
Priority: P3 → P4
Flags: qe-verify-
Priority: P4 → P2
Whiteboard: [reserve-console-html] → [newconsole-mvp]
Whiteboard: [newconsole-mvp]
Product: Firefox → DevTools

we do have unit tests for errors now

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.