add get timeouts for w3c (#46)

* add get timeouts for w3c

* add testcode for timeouts

* fix rubocop

Author: KazuCocoa

lib/appium_lib_core/common/base/bridge/w3c.rb

@@ -34,6 +34,10 @@ def action(async = false)
           end
           alias actions action
 
+          def get_timeouts
+            execute :get_timeouts
+          end
+
           # For Appium
           # override
           def page_source

lib/appium_lib_core/common/base/driver.rb

@@ -68,6 +68,18 @@ def back
         def logs
           @logs ||= Logs.new(@bridge)
         end
+
+        # For W3C.
+        # Get the timeout related settings on the server side.
+        #
+        # @return [Hash]
+        #
+        # @example
+        #   @driver.get_timeouts
+        #
+        def get_timeouts
+          @bridge.get_timeouts
+        end
       end # class Driver
     end # class Base
   end # module Core

lib/appium_lib_core/common/command.rb

@@ -89,6 +89,8 @@ module Commands
           # The fix will be included in selenium-3.8.2
           get_page_source: [:get, 'session/:session_id/source'.freeze],
 
+          get_timeouts: [:get, 'session/:session_id/timeouts'.freeze],
+
           ## Add OSS commands to W3C commands
           ### rotatable
           get_screen_orientation: [:get, 'session/:session_id/orientation'.freeze],

lib/appium_lib_core/common/device.rb

@@ -403,11 +403,6 @@ def extended(_mod)
             add_endpoint_method method
           end
 
-          # Don't define selenium-side methods. We pick up from them.
-          # ::Appium::Core::Base::Commands::MJSONWP.each_key do |method|
-          #   add_endpoint_method method
-          # end
-
           add_endpoint_method(:available_contexts) do
             def available_contexts
               # return empty array instead of nil on failure

test/unit/android/webdriver_w3c_test.rb

@@ -90,6 +90,15 @@ def test_active_element
 
         assert_requested(:get, "#{SESSION}/element/active", times: 1)
       end
+
+      def test_get_timeouts
+        stub_request(:get, "#{SESSION}/timeouts")
+          .to_return(headers: HEADER, status: 200, body: { value: 'xxxx' }.to_json)
+
+        @driver.get_timeouts
+
+        assert_requested(:get, "#{SESSION}/timeouts", times: 1)
+      end
     end
   end
 end