appium_lib_core 3.2.0 → 3.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4ffb1cbef0e8af1c0cf3b8dba42764a0decc3f9b5938ebff6f95f09c7438e516
-  data.tar.gz: baaeb26f3d7333f9aeccfcb25e84a81caadae91ff90b95ce3b8f884c14fb7872
+  metadata.gz: 3ff75e0359bf70b322aa12cd173ef3fea0df195997233905a017e6fe4cf5f38c
+  data.tar.gz: 114ac1d61568e2cbe0325f8d1b286e9ae1955cb1a48bd9400c7c3e153c859ee9
 SHA512:
-  metadata.gz: 7b102f1ee71cb033a2763b14f390fde773da2c02dc0c5ff6869c366a25067afd1c7448b228111c7117f1c6f5efd9e88e55f534b3c48dd1b3a1769a40bfb720c2
-  data.tar.gz: 70d55ecdd3cc250c6d504954006dfef47b7e89db80fcc05379ea7e71362ad572a61786651239c5f5cbd462efb4e0e8776aa594bef1540a0148278cf0789f9a4c
+  metadata.gz: d5787feb8efa19498e62c01a0458801c69979a32325b61f92367a4dca60c29dfcf7510d88e25d237196332ac63db22593177e6523981b960e28bc6e7c25e7e8a
+  data.tar.gz: e81f4f201e7fe44c4f97da21738c9399823ca8cdafe86f9f10e81c7d55f7847e37b9471745fbe180cb69aa2a59614d20cf5995835af0001b519aed7013654438

data/.rubocop.yml

@@ -21,6 +21,8 @@ Lint/NestedMethodDefinition:
 # TODO: Replace <<- with <<~ after dropping Ruby 2.2
 Layout/IndentHeredoc:
   Enabled: false
+Style/ZeroLengthPredicate:
+  Enabled: false
 Style/Documentation:
   Enabled: false
 Style/CommentedKeyword:

data/CHANGELOG.md

@@ -8,7 +8,21 @@ Read `release_notes.md` for commit level details.
 
 ### Bug fixes
 
+
+## [3.2.1] - 2019-07-19
+
+### Enhancements
+- Add `video_filters` argument for `start_recording_screen` iOS
+    - It is available over Appium 1.15.0
+
+### Bug fixes
+- Fix wrong warning message in driver detection
+
 ### Deprecations
+- `Driver#set_immediate_value(element, *value)`
+    - Use `Element#immediate_value(*value)` instead
+- `Driver#replace_value(element, *value)`
+    - Use `Element#replace_value(*value)` instead
 
 ## [3.2.0] - 2019-06-27
 

data/README.md

@@ -4,7 +4,7 @@
 
 | Travis, Ubuntu | Azure, Windows and functional tests |
 |:---:|:---:|
-|[![Build Status](https://travis-ci.org/appium/ruby_lib_core.svg?branch=master)](https://travis-ci.org/appium/ruby_lib_core)|[![Build Status](https://dev.azure.com/kazucocoa/ruby_lib_core_windows/_apis/build/status/appium.ruby_lib_core)](https://dev.azure.com/kazucocoa/ruby_lib_core_windows/_build/latest?definitionId=4)|
+|[![Build Status](https://travis-ci.org/appium/ruby_lib_core.svg?branch=master)](https://travis-ci.org/appium/ruby_lib_core)|[![Build Status](https://dev.azure.com/kazucocoa/ruby_lib_core/_apis/build/status/ruby_lib_core?branchName=master)](https://dev.azure.com/kazucocoa/ruby_lib_core/_build/latest?definitionId=9&branchName=master)|
 
 This library is a Ruby client for Appium.  The gem is available via [appium_lib_core](https://rubygems.org/gems/appium_lib_core).
 
@@ -76,6 +76,16 @@ $ PARALLEL=1 bundle exec parallel_test test/functional/android -n 3
 $ PARALLEL=1 bundle exec parallel_test test/functional/ios -n 2
 ```
 
+## CI
+
+- Runs on CI environment (on Azure)
+    - Non `IGNORE_VERSION_SKIP` or `IGNORE_VERSION_SKIP=true` runs all tests ignoring `skip` them by Appium versions
+    - `IGNORE_VERSION_SKIP=false` skips the following tests if the Appium version is lower than the requirement  
+
+```
+$ IGNORE_VERSION_SKIP=true CI=true bundle exec rake test:func:android
+```
+
 ## Run a test case
 1. Launch the Appium server locally.
 2. Run the following script.

data/azure-pipelines.yml

@@ -24,3 +24,12 @@ jobs:
       - template: ./ci-jobs/functional/publish_test_result.yml
         parameters:
           xcodeVersion: 'Run_unit_tests_on_Windows_environment'
+
+# Runs tests nightly to make sure they works against appium@beta
+schedules:
+  - cron: "0 0 * * *"
+    displayName: Daily Nightly build (UTC)
+    branches:
+      include:
+        - master
+    always: true

data/ci-jobs/functional/run_appium.yml

@@ -1,12 +1,14 @@
 steps:
 - task: NodeTool@0
   inputs:
-    versionSpec: 11.x
-  displayName: Install Node 11.x
-- script: npm install -g appium@beta --chromedriver_version='2.44'
+    versionSpec: 10.x
+  displayName: Install Node 10.x
+- script: npm install -g appium@${APPIUM_VERSION} --chromedriver_version='2.44'
   displayName: Install appium beta
 - script: npm install -g mjpeg-consumer
   displayName: Install MJPEG Consumer
+- script: npm list --depth 2 -g || echo 'ok'
+  displayName: Installed node dependencies
 - task: UseRubyVersion@0
   inputs:
     versionSpec: '2.4' # so far, bundler does not work over 2.5 since it raises openssl error

data/ci-jobs/functional_test.yml

@@ -1,18 +1,27 @@
 # jobs for functional test
 parameters:
   vmImage: 'macOS-10.14'
+  vmImageForIOS: 'macOS-10.13' # Not sure the reason, but macOS 10.14 instance raises no info.plist error
+  xcodeForIOS: 10.1
+  xcodeForTVOS: 10.2
+  androidSDK: 28
+  appiumVersion: 'beta'
+  ignoreVersionSkip: true
+  CI: true
 
 jobs:
   # Run unit tests on different Node versions
   - job: func_test_ios_base
     pool:
-      vmImage: ${{ parameters.vmImage }}
+      vmImage: ${{ parameters.vmImageForIOS }}
     variables:
-      CI: true
+      CI: ${{ parameters.ci }}
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
     - template: ./functional/ios_setup.yml
       parameters:
-        xcodeVersion: 10.1
+        xcodeVersion: ${{ parameters.xcodeForIOS }}
     - template: ./functional/run_appium.yml
     - script: bundle exec rake test:func:ios TESTS=test/functional/ios/driver_test.rb,test/functional/ios/patch_test.rb
       displayName: Run tests
@@ -22,13 +31,15 @@ jobs:
 
   - job: func_test_ios_webdriver1
     pool:
-      vmImage: ${{ parameters.vmImage }}
+      vmImage: ${{ parameters.vmImageForIOS }}
     variables:
-      CI: true
+      CI: ${{ parameters.ci }}
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
     - template: ./functional/ios_setup.yml
       parameters:
-        xcodeVersion: 10.1
+        xcodeVersion: ${{ parameters.xcodeForIOS }}
     - template: ./functional/run_appium.yml
     - script: bundle exec rake test:func:ios TESTS=test/functional/ios/webdriver/create_session_test.rb,test/functional/ios/webdriver/w3c_actions_test.rb
       displayName: Run tests
@@ -38,13 +49,15 @@ jobs:
 
   - job: func_test_ios_webdriver2
     pool:
-      vmImage: ${{ parameters.vmImage }}
+      vmImage: ${{ parameters.vmImageForIOS }}
     variables:
-      CI: true
+      CI: ${{ parameters.ci }}
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
     - template: ./functional/ios_setup.yml
       parameters:
-        xcodeVersion: 10.1
+        xcodeVersion: ${{ parameters.xcodeForIOS }}
     - template: ./functional/run_appium.yml
     - script: bundle exec rake test:func:ios TESTS=test/functional/ios/webdriver/device_test.rb
       displayName: Run tests
@@ -54,13 +67,15 @@ jobs:
 
   - job: func_test_ios_ios1
     pool:
-      vmImage: ${{ parameters.vmImage }}
+      vmImage: ${{ parameters.vmImageForIOS }}
     variables:
-      CI: true
+      CI: ${{ parameters.ci }}
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
     - template: ./functional/ios_setup.yml
       parameters:
-        xcodeVersion: 10.1
+        xcodeVersion: ${{ parameters.xcodeForIOS }}
     - template: ./functional/run_appium.yml
     - script: bundle exec rake test:func:ios TESTS=test/functional/ios/ios/device_test.rb
       displayName: Run tests
@@ -70,14 +85,18 @@ jobs:
 
   - job: func_test_ios_ios2
     pool:
-      vmImage: ${{ parameters.vmImage }}
+      vmImage: ${{ parameters.vmImageForIOS }}
     variables:
-      CI: true
+      CI: ${{ parameters.ci }}
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
     - template: ./functional/ios_setup.yml
       parameters:
-        xcodeVersion: 10.1
+        xcodeVersion: ${{ parameters.xcodeForIOS }}
     - template: ./functional/run_appium.yml
+    - script: npm install -g appium opencv4nodejs@4.17.0
+      displayName: Install opencv4nodejs@4.17.0
     - script: bundle exec rake test:func:ios TESTS=test/functional/ios/ios/device_wda_attachment_test.rb,test/functional/ios/ios/image_comparison_test.rb
       displayName: Run tests
     - template: ./functional/publish_test_result.yml
@@ -86,13 +105,15 @@ jobs:
 
   - job: func_test_ios_ios3
     pool:
-      vmImage: ${{ parameters.vmImage }}
+      vmImage: ${{ parameters.vmImageForIOS }}
     variables:
-      CI: true
+      CI: ${{ parameters.ci }}
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
     - template: ./functional/ios_setup.yml
       parameters:
-        xcodeVersion: 10.1
+        xcodeVersion: ${{ parameters.xcodeForIOS }}
     - script: brew install ffmpeg && brew tap wix/brew && brew install wix/brew/applesimutils
       displayName: Install ffmpeg and applesimutils
     - template: ./functional/run_appium.yml
@@ -106,11 +127,13 @@ jobs:
     pool:
       vmImage: ${{ parameters.vmImage }}
     variables:
-      CI: true
+      CI: ${{ parameters.ci }}
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
     - template: ./functional/ios_setup.yml
       parameters:
-        xcodeVersion: 10.2
+        xcodeVersion: ${{ parameters.xcodeForTVOS }}
     - template: ./functional/run_appium.yml
     - script: bundle exec rake test:func:ios TESTS=test/functional/ios/tv_driver_test.rb
       displayName: Run tests
@@ -122,8 +145,10 @@ jobs:
     pool:
       vmImage: ${{ parameters.vmImage }}
     variables:
-      CI: true
-      ANDROID_SDK_VERSION: 28
+      CI: ${{ parameters.ci }}
+      ANDROID_SDK_VERSION: ${{ parameters.androidSDK }}
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
     - template: ./functional/android_setup.yml
     - template: ./functional/run_appium.yml
@@ -137,8 +162,10 @@ jobs:
     pool:
       vmImage: ${{ parameters.vmImage }}
     variables:
-      CI: true
-      ANDROID_SDK_VERSION: 28
+      CI: ${{ parameters.ci }}
+      ANDROID_SDK_VERSION: ${{ parameters.androidSDK }}
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
       - template: ./functional/android_setup.yml
       - template: ./functional/run_appium.yml
@@ -152,8 +179,10 @@ jobs:
     pool:
       vmImage: ${{ parameters.vmImage }}
     variables:
-      CI: true
-      ANDROID_SDK_VERSION: 28
+      CI: ${{ parameters.ci }}
+      ANDROID_SDK_VERSION: ${{ parameters.androidSDK }}
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
       - template: ./functional/android_setup.yml
       - template: ./functional/run_appium.yml
@@ -167,11 +196,15 @@ jobs:
     pool:
       vmImage: ${{ parameters.vmImage }}
     variables:
-      CI: true
-      ANDROID_SDK_VERSION: 28
+      CI: ${{ parameters.ci }}
+      ANDROID_SDK_VERSION: ${{ parameters.androidSDK }}
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
       - template: ./functional/android_setup.yml
       - template: ./functional/run_appium.yml
+      - script: npm install -g appium opencv4nodejs@4.17.0
+        displayName: Install opencv4nodejs@4.17.0
       - script: bundle exec rake test:func:android TESTS=test/functional/android/android/image_comparison_test.rb,test/functional/android/android/mjpeg_server_test.rb
         displayName: Run tests
       - template: ./functional/publish_test_result.yml
@@ -182,8 +215,10 @@ jobs:
     pool:
       vmImage: ${{ parameters.vmImage }}
     variables:
-      CI: true
-      ANDROID_SDK_VERSION: 28
+      CI: ${{ parameters.ci }}
+      ANDROID_SDK_VERSION: ${{ parameters.androidSDK }}
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
       - template: ./functional/android_setup.yml
       - template: ./functional/run_appium.yml
@@ -197,8 +232,10 @@ jobs:
     pool:
       vmImage: ${{ parameters.vmImage }}
     variables:
-      CI: true
-      ANDROID_SDK_VERSION: 28
+      CI: ${{ parameters.ci }}
+      ANDROID_SDK_VERSION: ${{ parameters.androidSDK }}
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
       - template: ./functional/android_setup.yml
       - template: ./functional/run_appium.yml
@@ -212,9 +249,11 @@ jobs:
     pool:
       vmImage: ${{ parameters.vmImage }}
     variables:
-      CI: true
-      ANDROID_SDK_VERSION: 28
+      CI: ${{ parameters.ci }}
+      ANDROID_SDK_VERSION: ${{ parameters.androidSDK }}
       AUTOMATION_NAME_DROID: espresso
+      IGNORE_VERSION_SKIP: ${{ parameters.ignoreVersionSkip }}
+      APPIUM_VERSION: ${{ parameters.appiumVersion }}
     steps:
       - template: ./functional/android_setup.yml
       - template: ./functional/run_appium.yml

data/lib/appium_lib_core/android/device.rb

@@ -77,7 +77,7 @@ module Appium
         #   Same as #network_connection_type in selenium-webdriver.
         #
         #   Returns a key of <code>{:airplane_mode: 1, wifi: 2, data: 4, all: 6, none: 0}</code> in #network_connection_type
-        #   Returns a number of the mode in `#get_network_connection`
+        #   Returns a number of the mode in +#get_network_connection+
         #
         # @example
         #
@@ -129,7 +129,7 @@ module Appium
         #   Defaults to 'Done' for iOS(except for XCUITest).
         # @param [Symbol] strategy The symbol of the strategy which closes the keyboard.
         #   XCUITest ignore this argument.
-        #   Default for iOS is `:pressKey`. Default for Android is `:tapOutside`.
+        #   Default for iOS is +:pressKey+. Default for Android is +:tapOutside+.
         #
         # @example
         #
@@ -154,12 +154,12 @@ module Appium
         # @option opts [String] :app_activity The target activity [required]
         # @option opts [String] :app_wait_package The package to start before the target package [optional]
         # @option opts [String] :app_wait_activity The activity to start before the target activity [optional]
-        # @option opts [String] :intent_action The intent action to give it when start the target activity (`-a`) [optional]
-        # @option opts [String] :intent_category The intent category to give it when start the target activity (`-c`) [optional]
-        # @option opts [String] :intent_flags The intent flag to give it when start the target activity (`-f`) [optional]
+        # @option opts [String] :intent_action The intent action to give it when start the target activity (+-a+) [optional]
+        # @option opts [String] :intent_category The intent category to give it when start the target activity (+-c+) [optional]
+        # @option opts [String] :intent_flags The intent flag to give it when start the target activity (+-f+) [optional]
         # @option opts [String] :optional_intent_arguments The optional intent action to give it when start the target activity [optional]
         #                                                  You can set arbitrary arguments with space as string.
-        #                                                  e.g. `'--ez your_extra_bool bool --ei your_extra_int 1'`
+        #                                                  e.g. +'--ez your_extra_bool bool --ei your_extra_int 1'+
         # @option opts [bool] :dont_stop_app_on_reset Do not stop the app when the reset is called in Appium create/delete session [optional]
         #
         # @example
@@ -170,7 +170,7 @@ module Appium
 
         # @!method set_network_connection(mode)
         # Set the device network connection mode
-        # Same as `#network_connection_type` in selenium-webdriver.
+        # Same as +#network_connection_type+ in selenium-webdriver.
         #
         # @param [String] mode Bit mask that represent the network mode
         # Or the key matched with <code>{:airplane_mode: 1, wifi: 2, data: 4, all: 6, none: 0}</code>
@@ -204,7 +204,7 @@ module Appium
         #   Get the resource usage information of the application.
         #   https://github.com/appium/appium-base-driver/blob/be29aec2318316d12b5c3295e924a5ba8f09b0fb/lib/mjsonwp/routes.js#L303
         # @param [String] package_name: Package name
-        # @param [String] data_type: Data type get with `get_performance_data_types`
+        # @param [String] data_type: Data type get with +get_performance_data_types+
         # @param [String] data_read_timeout: Command timeout. Default is 2.
         #
         # @example
@@ -220,13 +220,13 @@ module Appium
         #                             An exception will be thrown if the generated media file is too big to
         #                             fit into the available process memory.
         #                             This option only has an effect if there is screen recording process in progress
-        #                             and `forceRestart` parameter is not set to `true`.
+        #                             and +forceRestart+ parameter is not set to +true+.
         # @param [String] user The name of the user for the remote authentication.
         # @param [String] pass The password for the remote authentication.
         # @param [String] method The http multipart upload method name. The 'PUT' one is used by default.
         # @param [Boolean] force_restart Whether to try to catch and upload/return the currently running screen recording
-        #                                 (`false`, the default setting on server) or ignore the result of it
-        #                                 and start a new recording immediately (`true`).
+        #                                 (+false+, the default setting on server) or ignore the result of it
+        #                                 and start a new recording immediately (+true+).
         #
         # @param [String] video_size The format is widthxheight.
         #                            The default value is the device's native display resolution (if supported),
@@ -241,7 +241,7 @@ module Appium
         #                            present then the most recent screen recording chunk is going to be returned as the result.
         # @param [String] bit_rate The video bit rate for the video, in megabits per second.
         #                          4 Mbp/s(4000000) is by default for Android API level below 27. 20 Mb/s(20000000) for API level 27 and above.
-        # @param [Boolean] bug_report Set it to `true` in order to display additional information on the video overlay,
+        # @param [Boolean] bug_report Set it to +true+ in order to display additional information on the video overlay,
         #                             such as a timestamp, that is helpful in videos captured to illustrate bugs.
         #                             This option is only supported since API level 27 (Android P).
         #

data/lib/appium_lib_core/android/uiautomator2/device.rb

@@ -30,7 +30,7 @@ module Appium
           #                 Battery level in range [0.0, 1.0], where 1.0 means 100% charge. -1 is returned
           #                 if the actual value cannot be retrieved from the system.
           #                 Battery state. The following symbols are possible
-          #                 `:unknown, :charging, :discharging, :not_charging, :full`
+          #                 +:unknown, :charging, :discharging, :not_charging, :full+
           #
           # @example
           #

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

@@ -58,10 +58,10 @@ module Appium
 
         # Override
         # Creates session handling both OSS and W3C dialects.
-        # Copy from Selenium::WebDriver::Remote::Bridge to keep using `merged_capabilities` for Appium
+        # Copy from Selenium::WebDriver::Remote::Bridge to keep using +merged_capabilities+ for Appium
         #
-        # If `desired_capabilities` has `forceMjsonwp: true` in the capability, this bridge works with mjsonwp protocol.
-        # If `forceMjsonwp: false` or no the capability, it depends on server side whether this bridge works as w3c or mjsonwp.
+        # If +desired_capabilities+ has +forceMjsonwp: true+ in the capability, this bridge works with mjsonwp protocol.
+        # If +forceMjsonwp: false+ or no the capability, it depends on server side whether this bridge works as w3c or mjsonwp.
         #
         # @param [::Selenium::WebDriver::Remote::W3C::Capabilities, Hash] desired_capabilities A capability
         # @return [::Selenium::WebDriver::Remote::Capabilities, ::Selenium::WebDriver::Remote::W3C::Capabilities]
@@ -125,7 +125,7 @@ module Appium
           json_create(oss_status, value)
         end
 
-        # Append `appium:` prefix for Appium following W3C spec
+        # Append +appium:+ prefix for Appium following W3C spec
         # https://www.w3.org/TR/webdriver/#dfn-validate-capabilities
         #
         # @param [::Selenium::WebDriver::Remote::W3C::Capabilities, Hash] capabilities A capability

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

@@ -43,18 +43,18 @@ module Appium
           end
 
           # Perform touch actions for W3C module.
-          # Generate `touch` pointer action here and users can use this via `driver.action`
+          # Generate +touch+ pointer action here and users can use this via +driver.action+
           # - https://seleniumhq.github.io/selenium/docs/api/rb/Selenium/WebDriver/W3CActionBuilder.html
           # - https://seleniumhq.github.io/selenium/docs/api/rb/Selenium/WebDriver/PointerActions.html
           # - https://seleniumhq.github.io/selenium/docs/api/rb/Selenium/WebDriver/KeyActions.html
           #
-          # 'mouse' action is by default in the Ruby client. Appium server force the `mouse` action to `touch` once in
+          # 'mouse' action is by default in the Ruby client. Appium server force the +mouse+ action to +touch+ once in
           # the server side. So we don't consider the case.
           #
           # @example
           #
           #     element = @driver.find_element(:id, "some id")
-          #     @driver.action.click(element).perform # The `click` is a part of `PointerActions`
+          #     @driver.action.click(element).perform # The 'click' is a part of 'PointerActions'
           #
           def action(async = false)
             # Used for default duration of each touch actions
@@ -143,21 +143,21 @@ module Appium
 
           # For Appium
           # override
-          # called in `extend DriverExtensions::HasNetworkConnection`
+          # called in 'extend DriverExtensions::HasNetworkConnection'
           def network_connection
             execute :get_network_connection
           end
 
           # For Appium
           # override
-          # called in `extend DriverExtensions::HasNetworkConnection`
+          # called in 'extend DriverExtensions::HasNetworkConnection'
           def network_connection=(type)
             execute :set_network_connection, {}, { parameters: { type: type } }
           end
 
           # For Appium
           # No implementation for W3C webdriver module
-          # called in `extend DriverExtensions::HasLocation`
+          # called in 'extend DriverExtensions::HasLocation'
           def location
             obj = execute(:get_location) || {}
             ::Selenium::WebDriver::Location.new obj['latitude'], obj['longitude'], obj['altitude']
@@ -165,8 +165,17 @@ module Appium
 
           # For Appium
           # No implementation for W3C webdriver module
-          # called in `extend DriverExtensions::HasLocation`
-          def set_location(lat, lon, alt)
+          # called in +extend DriverExtensions::HasLocation+
+          # It has below code as well. We should consider the same context in Selenium 4 as backward compatibility.
+          #
+          #     def location=(loc)
+          #       # note: Location  = Struct.new(:latitude, :longitude, :altitude)
+          #       raise TypeError, "expected #{Location}, got #{loc.inspect}:#{loc.class}" unless loc.is_a?(Location)
+          #
+          #       @bridge.set_location loc.latitude, loc.longitude, loc.altitude
+          #     end
+          #
+          def set_location(lat, lon, alt = 0.0)
             loc = { latitude: lat, longitude: lon, altitude: alt }
             execute :set_location, {}, { location: loc }
           end