Closed Bug 1307918 Opened 8 years ago Closed 4 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: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.