Created attachment 673570 [details] console error message Bug 802227 add testvar support but didn't fix MarionetteJSTestCase at the same time.
Created attachment 673572 [details] [diff] [review] WIP 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
Created attachment 673823 [details] [diff] [review] sync MarionetteJSTestCase.add_tests_to_suite arguements with others
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/integration/mozilla-inbound/rev/0234f58b6103 - backout previous commit https://hg.mozilla.org/integration/mozilla-inbound/rev/9c8640b0a486 - re-commit with r=automatedtester
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
status-firefox18: --- → fixed
status-firefox19: --- → fixed
You need to log in before you can comment on or make changes to this bug.