rb: get HasNetworkConnection (and specs) to work

Signed-off-by: Andreas Tolfsen <ato@mozilla.com>
jleyba · Mar 2, 2015 · 51d4486 · 51d4486
1 parent 29ea618
commit 51d4486
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/rb/lib/selenium/webdriver/remote/bridge.rb b/rb/lib/selenium/webdriver/remote/bridge.rb
@@ -85,6 +85,8 @@ def driver_extensions
             DriverExtensions::HasSessionId,
             DriverExtensions::Rotatable,
             DriverExtensions::HasTouchScreen,
+            DriverExtensions::HasLocation,
+            DriverExtensions::HasNetworkConnection,
             DriverExtensions::HasRemoteStatus
           ]
         end
@@ -326,7 +328,7 @@ def getNetworkConnection
         end
 
         def setNetworkConnection(type)
-          execute :setNetworkConnection, {}, :type => type
+          execute :setNetworkConnection, {}, :parameters => {:type => type}
         end
 
         #

diff --git a/rb/spec/integration/selenium/webdriver/network_connection_spec.rb b/rb/spec/integration/selenium/webdriver/network_connection_spec.rb
@@ -2,14 +2,13 @@
 
 module Selenium::WebDriver::DriverExtensions
   describe HasNetworkConnection do
-    compliant_on :browser => nil do
+    compliant_on :browser => :android do
       it "can return the network connection type" do
         expect(driver.network_connection_type).to eq :all
       end
 
       it "can set the network connection type" do
-        expect { driver.network_connection_type = :airplane_mode }.to
-          change { driver.network_connection_type }.from(:all).to(:airplane_mode)
+        expect { driver.network_connection_type = :airplane_mode }.to change { driver.network_connection_type }.from(:all).to(:airplane_mode)
       end
     end
   end