Closed Bug 793322 Opened 7 years ago Closed 7 years ago

Troubleshoot.jsm test should fail immediately instead of timing out on snapshot data validation errors

Categories

(Toolkit :: General, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla18

People

(Reporter: adw, Assigned: adw)

References

Details

Attachments

(1 file)

Attached patch patchSplinter Review
validateObject throws an exception on validation failure.  When that happens, done is never called and the test ends up timing out.  The exception should be caught and reported as a failure, and then the test should continue.
Attachment #663576 - Flags: review?(felipc)
Attachment #663576 - Flags: review?(felipc) → review+
Thanks for the quick review.

https://hg.mozilla.org/integration/mozilla-inbound/rev/046968d6f256
Flags: in-testsuite+
https://hg.mozilla.org/mozilla-central/rev/046968d6f256
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
You need to log in before you can comment on or make changes to this bug.