813300 - A lot of B2G specific code sneaking into marionette client

Status: RESOLVED DUPLICATE of bug 997244

(Remote Protocol :: Marionette, defect)

Description

[Andrew Halberstadt [:ahal]]

See http://mxr.mozilla.org/mozilla-central/search?string=b2g&find=%2Ftesting%2Fmarionette%2F.*\.py&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

Yes, much of this stuff is ok. But there are other things (like install_gecko) that seem a little too b2g specific for marionette.

Figuring out how to structure everything is probably not the highest priority, just wanted to file so it would remain in the back of my mind.

Comment 1

[Jonathan Griffin (:jgriffin)]

I think the right solution here is to make a B2G-specific subclass of Marionette (similar to what we did for GaiaUnitTests) and move the B2G-specific code there.  We could almost move most of the code from mozb2g into this new class.

I'm not sure when anyone will have the time to do this, though.  Maybe an intern this summer?

Comment 2

[Andrew Halberstadt [:ahal]]

Most of the b2g specific stuff this bug was referencing was in emulator.py. I've filed bug 997244 to remove emulator.py from marionette and into mozrunner. As a part of that, I'm devising a way to keep b2g specific stuff out of emulator.py which essentially fixes this bug. Therefore resolving duplicate.