Closed Bug 803850 Opened 7 years ago Closed 7 years ago

B2G: marionette dies at testing JS cases

Categories

(Testing :: Marionette, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(firefox18 fixed, firefox19 fixed)

RESOLVED FIXED
B2G C1 (to 19nov)
Tracking Status
firefox18 --- fixed
firefox19 --- fixed

People

(Reporter: vicamo, Assigned: vicamo)

References

Details

Attachments

(2 files, 1 obsolete file)

Attached file console error message
Bug 802227 add testvar support but didn't fix MarionetteJSTestCase at the same time.
Attached patch WIP (obsolete) — Splinter Review
This patch simply adds the omitted arguement back. I'm not quite sure what should the testvar do, but it just works.
Component: DOM: Device Interfaces → Marionette
Product: Core → Testing
Assignee: dburns → vyang
Comment on attachment 673823 [details] [diff] [review]
sync MarionetteJSTestCase.add_tests_to_suite arguements with others

looks good to me. We can speak to jgriffin if this is needed/wanted for the JS tests
Attachment #673823 - Flags: review?(dburns) → review+
https://hg.mozilla.org/mozilla-central/rev/9c8640b0a486
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
Whiteboard: [automation-needed-in-aurora]
Target Milestone: mozilla19 → B2G C1 (to 19nov)
You need to log in before you can comment on or make changes to this bug.