Bug 1323770 - Marionette should not take screenshots for skipped tests.
Taking screenshots for skipped tests is useless and should be avoided to
reduce the size of the test logs.
MozReview-Commit-ID: 9HPH7pSXTj9
--- a/testing/marionette/harness/marionette_harness/runner/base.py
+++ b/testing/marionette/harness/marionette_harness/runner/base.py
@@ -547,16 +547,20 @@ class BaseMarionetteTestRunner(object):
self.workspace = workspace
# If no workspace is set, default location for gecko.log is .
# and default location for profile is TMP
self.workspace_path = workspace or os.getcwd()
self.verbose = verbose
self.e10s = e10s
def gather_debug(test, status):
+ # No screenshots and page source for skipped tests
+ if status == "SKIP":
+ return
+
rv = {}
marionette = test._marionette_weakref()
# In the event we're gathering debug without starting a session,
# skip marionette commands
if marionette.session is not None:
try:
with marionette.using_context(marionette.CONTEXT_CHROME):
--- a/testing/marionette/harness/requirements.txt
+++ b/testing/marionette/harness/requirements.txt
@@ -4,11 +4,11 @@ marionette-driver >= 2.2.0
mozcrash >= 0.5
mozdevice >= 0.44
mozinfo >= 0.8
mozlog >= 3.0
moznetwork >= 0.21
mozprocess >= 0.9
mozprofile >= 0.7
mozrunner >= 6.13
-moztest >= 0.7
+moztest >= 0.8
mozversion >= 1.1
wptserve >= 1.3.0