Skip to content

Commit

Permalink
fix: non-app capability handling (#1039)
Browse files Browse the repository at this point in the history
* fix: skip absolute_app_path for non-app

* update changelog

* tweak the order
  • Loading branch information
KazuCocoa authored Aug 6, 2024
1 parent 9b95da5 commit b741383
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,9 @@ Commit based release not is [release_notes.md](./release_notes.md)

Release tags are https://github.com/appium/ruby_lib/releases .

## not released
- Fix non `app` capability behavior

## 15.2.1 - 2024-08-03
- Fix client side timeout in the default http client
- Bump appium_lib_core 9.2.1+ to apply the fix
Expand Down
4 changes: 2 additions & 2 deletions lib/appium_lib/driver.rb
Original file line number Diff line number Diff line change
Expand Up @@ -269,9 +269,9 @@ def set_app_path(opts)

# return the path exists on the local
app_path = Driver.get_cap(@core.caps, 'app')
return if !app_path.nil? && File.exist?(app_path)
return if app_path.nil?
return if File.exist?(app_path)

# The app file is not exact path
@core.caps['app'] = self.class.absolute_app_path opts
end

Expand Down

0 comments on commit b741383

Please sign in to comment.