Closed Bug 736517 Opened 12 years ago Closed 6 years ago

HTML parse errors should cause mochitests to fail

Categories

(Testing :: Mochitest, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: ayg, Unassigned)

References

Details

See bug 735805 comment 11.  In fixing that bug, I found two tests that ran no checks because a > or " was omitted in the source code, causing the test contents to be eaten silently by the parser.  Any HTML files used by mochitest should ideally be checked for parse errors before running.  An opt-out mechanism might be desired for tests that intend to test parse errors, but we have separate parser tests as far as I understand it.  Obviously, any such fix will need lots and lots of tests to be updated.

Henri says this would be hard, so I'm not expecting this to happen anytime soon, but it seems like a good idea to file it anyway.
Blocks: SmTestFail
Depends on: 736886
Version: unspecified → Trunk
No longer blocks: SmTestFail
Mass closing mochitest bugs that haven't had activity in the past 5 years. Please re-open or file a new bug with modern context if this is still relevant.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.