Closed Bug 481732 Opened 11 years ago Closed 11 years ago
Have unit tests (automation
.py ones) dump stacks on crashes
This patch may be enough to have automation.py-style unit tests dump crash stacks. Requirements: * should have already run `make buildsymbols` * should have MINIDUMP_STACKWALK=/path/to/minidump_stackwalk set in the environment I'm not sure how/whether it will work with packaged tests... Ted is there a MDC doc on packaged tests?
No, but bug 421611 comment 36 explains how to do it with Mochitest.
Minor path fixup discovered during tinderbox testing
Attachment #366015 - Flags: review?(ted.mielczarek) → review+
Comment on attachment 366015 [details] [diff] [review] Run minidump_stackwalk from automation.py, under the right conditions, rev. 1.1 + mkdir $(DIST)/crashreporter-symbols Should probably be $(NSINSTALL) -D while you're here. + cd $(DIST)/crashreporter-symbols && \ zip -r9D ../../$(SYMBOL_ARCHIVE_BASENAME).zip . You need to take out one ../ in the zip file name there. + log.info("TEST-UNEXPECTED-FAIL | Browser crashed (minidump found)") can you make this say: TEST-UNEXPECTED-FAIL | automation.py | Browser crashed (minidump found) In the tinderbox summary, this should work out ok, right? the crash stack will wind up right below this, and both should wind up in the short log? It'd be nice if you added cmdline args to runtests.py / runreftest.py to specify the symbol path, since we're gonna need that eventually anyway.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
Version: unspecified → Trunk
Pushed to 1.9.1: http://hg.mozilla.org/releases/mozilla-1.9.1/rev/6d48903f134b
You need to log in before you can comment on or make changes to this bug.