Closed Bug 975312 Opened 8 years ago Closed 8 years ago

[mozrunner] Runner.check_for_crashes() should support save_path and quiet parameters


(Testing :: Mozbase, defect)

Not set


(Not tracked)



(Reporter: whimboo, Assigned: whimboo)




(1 file, 1 obsolete file)

Right now the check_for_crashes() method of the Runner class does not support the save_path parameter of mozcrash. We have to get this added so that Mozmill can use it on bug 619204.
Test frameworks might not always want to let the information printed to stdout, but handle it on their own. So also forwarding the quiet parameter sounds important to me.
Summary: [mozrunner] Runner.check_for_crashes() should support save_path for storing minidump files → [mozrunner] Runner.check_for_crashes() should support save_path and quiet parameters
Attached patch Patch v1 (obsolete) — Splinter Review
I also pushed this to try:
Attachment #8379648 - Flags: review?(ahalberstadt)
Attached patch Patch v1.1Splinter Review
Included the mozrunner version bump too.
Attachment #8379648 - Attachment is obsolete: true
Attachment #8379648 - Flags: review?(ahalberstadt)
Attachment #8379660 - Flags: review?(ahalberstadt)
Comment on attachment 8379660 [details] [diff] [review]
Patch v1.1

Review of attachment 8379660 [details] [diff] [review]:

Looks good, thanks!
Attachment #8379660 - Flags: review?(ahalberstadt) → review+
Damn. Somehow I pushed the wrong patch to inbound which didn't include the version bump. :/ I will most likely do it later or wait until I'm back.
(In reply to Henrik Skupin (:whimboo) [away 02/24 - 02/28] from comment #8)
> Pushed version bump to 5.35:

Not sure what went wrong here, but it ended-up as a no-op. :( So I pushed it again:

Given that we haven't seen any problems with the former patch, I'm going to release mozrunner 5.35 now.
Submitting dist/mozrunner-5.35.tar.gz to
Server response (200): OK
Keywords: leave-open
You need to log in before you can comment on or make changes to this bug.