Status
People
(Reporter: whimboo, Assigned: whimboo)
Tracking
Firefox Tracking Flags
(Not tracked)
Details
(Whiteboard: [mozmill-2.0])
Attachments
(1 attachment)
Right now we add Components.stack for both fail and pass states in assertions.js. We should really only do this for test failures. There is no need for pass states. That will drastically limit the amount of data we have to pass through JSBridge.
|
(Assignee) | |
Comment 1•7 years ago
|
||
Created attachment 650359 [details] Pointer to Github pull request: https://github.com/mozilla/mozmill/pull/84 Pointer to Github pull-request
|
|
(Assignee) | |
Updated•7 years ago
|
Attachment #650359 -
Flags: review?(jhammel)
|
||
Comment 2•7 years ago
|
||
Comment on attachment 650359 [details] Pointer to Github pull request: https://github.com/mozilla/mozmill/pull/84 Assuming this doesn't break anything, sounds good to me
Attachment #650359 -
Flags: review?(jhammel) → review+
|
|
(Assignee) | |
Comment 3•7 years ago
|
||
Nothing on our side depends on it. Stacks are only useful when you really have a failure. So we will be fine with this patch. Landed as: https://github.com/mozilla/mozmill/commit/aa922595402de0a1b564c05f4448ab66cbec18fb
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Whiteboard: [mozmill-2.0]
|
|
(Assignee) | |
Comment 4•7 years ago
|
||
Just for comparison, the numbers speak themselves: -rw-r--r--@ 1 henrik staff 264310 Aug 9 11:26 new.txt -rw-r--r--@ 1 henrik staff 728381 Aug 9 11:27 old.txt It's a reduction of 64% in size of a report.
|
||
Updated•3 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•