Bug 1294427 - Fix case in condition on appName in BaseMarionetteTestRunner; r?AutomatedTester draft
authorMaja Frydrychowicz <mjzffr@gmail.com>
Thu, 11 Aug 2016 19:14:38 -0400
changeset 399736 8c7ca039bd4b3558a8e08a8dd3bcaf5d2d5f3584
parent 398604 6cf0089510fad8deb866136f5b92bbced9498447
child 528048 0a6bd15ff1862139752c425ae1cd077b8ba64a20
push id25969
push usermjzffr@gmail.com
push dateThu, 11 Aug 2016 23:25:27 +0000
reviewersAutomatedTester
bugs1294427, 1288863
milestone51.0a1
Bug 1294427 - Fix case in condition on appName in BaseMarionetteTestRunner; r?AutomatedTester This fixes regression introduced in Bug 1288863 MozReview-Commit-ID: CGYeVBIbZb1
testing/marionette/harness/marionette/runner/base.py
--- a/testing/marionette/harness/marionette/runner/base.py
+++ b/testing/marionette/harness/marionette/runner/base.py
@@ -803,17 +803,17 @@ setReq.onerror = function() {
         self.reset_test_stats()
 
     def _start_marionette(self):
         need_external_ip = True
         if not self.marionette:
             self.marionette = self.driverclass(**self._build_kwargs())
             # if we're working against a desktop version, we usually don't need
             # an external ip
-            if self.appName != 'Fennec':
+            if self.appName != 'fennec':
                 need_external_ip = False
         self.logger.info('Initial Profile Destination is '
                          '"{}"'.format(self.marionette.profile_path))
         return need_external_ip
 
     def _set_baseurl(self, need_external_ip):
         # Gaia sets server_root and that means we shouldn't spin up our own httpd
         if not self.httpd: