Closed
Bug 1132055
Opened 9 years ago
Closed 9 years ago
[marionette-js-runner] Parametrize marionette desired capabilities for marionette-mocha.
Categories
(Testing Graveyard :: JSMarionette, defect)
Testing Graveyard
JSMarionette
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: yzen, Assigned: yzen, NeedInfo)
Details
(Keywords: pi-marionette-runner, Whiteboard: [affects=a11y])
Attachments
(1 file)
In order to enable capabilities without restarting or modifying the client it would be nice to parametrize them for marionette-mocha as follows: marionette-mocha --marionette-capabilities=file.json Some context from IRC: [15:49:55] <yzen> lightsofapollo thanks! btw do you think there could be a way to enable some capabilities based on some environment variables or something else of some sort at runtime of the test (in addition to be able to create a new client with capabilities in the test itself explicitly) ? [15:49:58] <lightsofapollo> yzen: okay just waiting on this https://github.com/mozilla-b2g/gaia/pull/27793 [15:50:16] <yzen> lightsofapollo nice [15:50:31] <lightsofapollo> yzen: yeah env variable is probably easiest and most hacky [15:50:58] <lightsofapollo> yzen: maybe we could do a json config + an env variable to select the config? [15:51:06] <lightsofapollo> (or better yet a cli flag...) [15:51:42] <yzen> lightsofapollo where im coming from : i want to try running existing marionette tests for gaia and without touching the actual body of the test [15:52:16] <yzen> yzen i see [15:53:22] <lightsofapollo> yzen: so maybe... MARIONETTE_GLOBAL_CAPS=do_l10n_test_magic.json make test-integration [15:53:28] lightsofapollo cringes at env varaibles / make [15:53:48] <lightsofapollo> marionette-mocha --marionette-capabilities=file.json ? [15:54:07] <yzen> interesting yeah i could play around with it
Assignee | ||
Comment 1•9 years ago
|
||
Hopefully this is close to what you've suggested, James. Thanks!
Attachment #8562800 -
Flags: review?(jlal)
Comment 2•9 years ago
|
||
Comment on attachment 8562800 [details] [review] Github pull request. https://github.com/mozilla-b2g/marionette-js-runner/commit/13608a6c0c8dbc58e10d174e296726695ad15501
Attachment #8562800 -
Flags: review?(jlal) → review+
Comment 3•9 years ago
|
||
++ thanks! I am going to NI Gareth so we can make sure this work gets merged into the crashme branch...
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(gaye)
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•