python: adding support to tunnelOwner/tunnelName

Author: waggledans

python/saucebindings/configs.py

@@ -7,6 +7,8 @@ class Configs:
         'custom_data': 'customData',
         'public': 'public',
         'tunnel_identifier': 'tunnelIdentifier',
+        'tunnel_name': 'tunnelName',
+        'tunnel_owner': 'tunnelOwner',
         'parent_tunnel': 'parentTunnel'
     }
 

python/saucebindings/options.py

@@ -31,6 +31,7 @@
     'iedriver_version': 'iedriverVersion',
     'max_duration': 'maxDuration',
     'name': 'name',
+    # parent_tunnel is deprecated and being replaced with tunnel_owner.
     'parent_tunnel': 'parentTunnel',
     'prerun': 'prerun',
     'priority': 'priority',
@@ -42,7 +43,10 @@
     'selenium_version': 'seleniumVersion',
     'tags': 'tags',
     'time_zone': 'timeZone',
+    # tunnel_identifier is deprecated and being replaced with tunnel_name.
     'tunnel_identifier': 'tunnelIdentifier',
+    'tunnel_name': 'tunnelName',
+    'tunnel_owner': 'tunnelOwner',
     'video_upload_on_pass': 'videoUploadOnPass',
     'capture_performance': 'capturePerformance'
 }

python/tests/unit/test_chrome.py

@@ -117,6 +117,20 @@ def test_accepts_sauce_values_with_dict(self):
         assert sauce.tunnel_identifier == 'foobar'
         assert sauce.video_upload_on_pass is False
 
+    def test_accepts_tunnel(self):
+        options = {'build': 'bar',
+                   'idleTimeout': 3,
+                   'name': 'foo',
+                   'tunnelOwner': 'bar',
+                   'tunnelName': 'foobar'}
+
+        sauce = SauceOptions.chrome(**options)
+
+        assert sauce.build == 'bar'
+        assert sauce.name == 'foo'
+        assert sauce.tunnel_owner == 'bar'
+        assert sauce.tunnel_name == 'foobar'
+
     def test_accepts_sauce_values_as_params(self):
         custom_data = {'foo': 'foo',
                        'bar': 'bar'}

python/tests/unit/test_edge.py

@@ -293,6 +293,20 @@ def test_sauce_options(self):
         assert options.tunnel_identifier == 'tunnelname'
         assert options.video_upload_on_pass is False
 
+    def test_accepts_tunnel(self):
+        options = {'build': 'bar',
+                   'idleTimeout': 3,
+                   'name': 'foo',
+                   'tunnelOwner': 'bar',
+                   'tunnelName': 'foobar'}
+
+        sauce = SauceOptions.edge(**options)
+
+        assert sauce.build == 'bar'
+        assert sauce.name == 'foo'
+        assert sauce.tunnel_owner == 'bar'
+        assert sauce.tunnel_name == 'foobar'
+
     def test_setting_browser_name(self):
         options = SauceOptions.edge()
 

python/tests/unit/test_firefox.py

@@ -352,7 +352,7 @@ def test_setting_capabilities(self):
         assert options.build == 'Sample Build Name'
         assert options.command_timeout == 2
         assert options.custom_data == custom_data
-        assert options.extended_debugging == True
+        assert options.extended_debugging is True
         assert options.idle_timeout == 3
         assert options.geckodriver_version == '0.23'
         assert options.max_duration == 300
@@ -487,3 +487,17 @@ def test_capabilities_for_selenium(self):
                                  }
 
         assert options.to_capabilities() == expected_capabilities
+
+    def test_accepts_tunnel(self):
+        options = {'build': 'bar',
+                   'idleTimeout': 3,
+                   'name': 'foo',
+                   'tunnelOwner': 'bar',
+                   'tunnelName': 'foobar'}
+
+        sauce = SauceOptions.firefox(**options)
+
+        assert sauce.build == 'bar'
+        assert sauce.name == 'foo'
+        assert sauce.tunnel_owner == 'bar'
+        assert sauce.tunnel_name == 'foobar'