Bug 1283459 - Rename pacUrl to proxyAutoconfigUrl; r=automatedtester, a=test-only
authorAndreas Tolfsen <ato@mozilla.com>
Thu, 30 Jun 2016 14:41:45 +0100
changeset 339822 5e44121bee428b73385b8127a6c80a820e6b58c0
parent 339821 128eff0a581ff318eb3629005a871d5095b4f430
child 339823 c6b972226566ca6230a11b58577178f5992c1064
push id6249
push userjlund@mozilla.com
push dateMon, 01 Aug 2016 13:59:36 +0000
treeherdermozilla-beta@bad9d4f5bf7e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersautomatedtester, test-only
bugs1283459
milestone49.0a2
Bug 1283459 - Rename pacUrl to proxyAutoconfigUrl; r=automatedtester, a=test-only In order to not have to manage compatibility in this area, we use the field name proxyAutoconfigUrl that existing Selenium implementations use. MozReview-Commit-ID: JRk6uvEbJQi
testing/marionette/driver.js
testing/marionette/harness/marionette/tests/unit/test_proxy.py
--- a/testing/marionette/driver.js
+++ b/testing/marionette/driver.js
@@ -657,17 +657,17 @@ GeckoDriver.prototype.setUpProxy = funct
           if (proxy.socksVersion) {
             Preferences.set("network.proxy.socks_version", proxy.socksVersion);
           }
         }
         break;
 
       case "PAC":
         Preferences.set("network.proxy.type", 2);
-        Preferences.set("network.proxy.autoconfig_url", proxy.pacUrl);
+        Preferences.set("network.proxy.autoconfig_url", proxy.proxyAutoconfigUrl);
         break;
 
       case "AUTODETECT":
         Preferences.set("network.proxy.type", 4);
         break;
 
       case "SYSTEM":
         Preferences.set("network.proxy.type", 5);
--- a/testing/marionette/harness/marionette/tests/unit/test_proxy.py
+++ b/testing/marionette/harness/marionette/tests/unit/test_proxy.py
@@ -61,31 +61,31 @@ class TestProxy(MarionetteTestCase):
         self.assertEqual(result["proxyType"], 5)
 
     def test_we_can_set_a_pac_proxy(self):
         url = "http://marionette.test"
         capabilities = {"requiredCapabilities":
                             {
                                 "proxy":{
                                     "proxyType": "pac",
-                                    "pacUrl": url,
+                                    "proxyAutoconfigUrl": url,
                                 }
                             }
                         }
         self.marionette.start_session(capabilities)
         result = None
         with self.marionette.using_context('chrome'):
             result = self.marionette.execute_script("""return {
                 "proxyType" : Services.prefs.getIntPref('network.proxy.type'),
-                "pacUrl" : Services.prefs.getCharPref('network.proxy.autoconfig_url'),
+                "proxyAutoconfigUrl" : Services.prefs.getCharPref('network.proxy.autoconfig_url'),
                 }
             """)
 
         self.assertEqual(result["proxyType"], 2)
-        self.assertEqual(result["pacUrl"], url, 'pacUrl was not set')
+        self.assertEqual(result["proxyAutoconfigUrl"], url, 'proxyAutoconfigUrl was not set')
 
     def test_that_we_can_set_a_manual_proxy(self):
         port = 4444
         url = "http://marionette.test"
         capabilities = {"requiredCapabilities":
                             {
                                 "proxy":{
                                     "proxyType": "manual",