Bug 1083237 - Wrong test suite duration when using mozlog as test reporter. r=mdas
☠☠ backed out by a074be8d7cfc ☠ ☠
authorJulien Pagès <j.parkouss@gmail.com>
Sun, 19 Oct 2014 11:20:00 +0200
changeset 237815 8c0cf60141dc549a7042b00742761a27e0bedf66
parent 237814 776061abb9be3feedade5a31c84a99f869bbab45
child 237816 75e8a2121a08c629fc77799d71532d870d1cac67
push id660
push userraliiev@mozilla.com
push dateWed, 18 Feb 2015 20:30:48 +0000
treeherdermozilla-release@49e493494178 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmdas
bugs1083237
milestone36.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1083237 - Wrong test suite duration when using mozlog as test reporter. r=mdas
testing/marionette/client/marionette/runner/base.py
--- a/testing/marionette/client/marionette/runner/base.py
+++ b/testing/marionette/client/marionette/runner/base.py
@@ -607,17 +607,22 @@ class BaseMarionetteTestRunner(object):
             })
         return kwargs
 
     def start_marionette(self):
         self.marionette = Marionette(**self._build_kwargs())
 
     def run_tests(self, tests):
         self.reset_test_stats()
+
+        for test in tests:
+            self.add_test(test)
+
         self.start_time = time.time()
+        self.logger.suite_start(self.tests)
 
         need_external_ip = True
         if not self.marionette:
             self.start_marionette()
             if self.emulator:
                 self.marionette.emulator.wait_for_homescreen(self.marionette)
             # Retrieve capabilities for later use
             if not self._capabilities:
@@ -626,21 +631,16 @@ class BaseMarionetteTestRunner(object):
             # an external ip
             if self._capabilities['device'] == "desktop":
                 need_external_ip = False
 
         if not self.httpd:
             self.logger.info("starting httpd")
             self.start_httpd(need_external_ip)
 
-        for test in tests:
-            self.add_test(test)
-
-        self.logger.suite_start(self.tests)
-
         for test in self.manifest_skipped_tests:
             name = os.path.basename(test['path'])
             self.logger.test_start(name)
             self.logger.test_end(name,
                                  'SKIP',
                                  message=test['disabled'])
             self.todo += 1