Refactor Marionette testrunner to make Gaia testing easier

RESOLVED FIXED in mozilla18

Status

defect
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: jgriffin, Assigned: jgriffin)

Tracking

unspecified
mozilla18
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

This patch does a couple of things:

 - it adds a console entry point for Marionette ("marionette") that allows you to run Marionette tests from anywhere; you don't need to be in the directory where runtests.py lives
 - it makes it possible to subclass the testrunner to provide a custom test class

I've already tested that these changes are harmless to current automation, and that they work for the target use case; that is, I've created a GaiaTestClass which exposes some unique methods applicable to Gaia testing, and verified I could run tests with them independent of Marionette's pre-existing test classes.  I'll submit the Gaia-related changes in a separate patch.
Realized I should have bumped the version here as well
Attachment #660251 - Flags: review?(mdas)
Attachment #660250 - Attachment is obsolete: true
Attachment #660250 - Flags: review?(mdas)
Blocks: 790469
Assignee: nobody → jgriffin
Comment on attachment 660251 [details] [diff] [review]
Refactor Marionette testrunner to make it easier to write Gaia tests,

Review of attachment 660251 [details] [diff] [review]:
-----------------------------------------------------------------

This looks great, thanks!

::: testing/marionette/client/marionette/marionette_test.py
@@ +52,1 @@
>      def kill_gaia_app(self, url):

We should probably remove this and kill_gaia_apps from the CommonTestCase class
Attachment #660251 - Flags: review?(mdas) → review+
bug 787114 was created to deal with the gaia helper functions, but I think we can change that here and close this bug
Duplicate of this bug: 787114
(In reply to Malini Das [:mdas] from comment #4)
> bug 787114 was created to deal with the gaia helper functions, but I think
> we can change that here and close this bug

I agree, I duped that bug and will remove those functions with this patch.
Attachment #660251 - Attachment is obsolete: true
Comment on attachment 661383 [details] [diff] [review]
Refactor Marionette testrunner to make it easier to write Gaia tests,

Carrying r+ forward
Attachment #661383 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/8d96a5392ff4
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.