Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: tvOS 17 workaround #2178

Closed
wants to merge 2 commits into from
Closed

chore: tvOS 17 workaround #2178

wants to merge 2 commits into from

Conversation

KazuCocoa
Copy link
Member

Do not merge this yet. I'll modify the error message etc if this works for tvOS iOS 17.

This PR aims to add a workaround for appium/appium#19343

TvOS 17 changes the device paring method to use IP v6 first according to appium/appium#19343. I need more time to investigate the change, but as a workaround, we can allow a session to start by skipping device detection logic in a new session request.

On my network-connected device (iPad tho), with this change, with webDriverAgentUrl or starting WDA from xcodebuild succeeded in starting a new session request and issuing commands that worked via WDA as appium XCUITest session.

The given udid will be used for xcodebuild, so the id should be the existing udid style (not IP address), but xcuitest driver won't verify that.

@KazuCocoa KazuCocoa changed the title tvos ios17 workaround tvOS 17 workaround Nov 2, 2023
@KazuCocoa KazuCocoa changed the title tvOS 17 workaround chore: tvOS 17 workaround Nov 2, 2023
@KazuCocoa KazuCocoa closed this Nov 3, 2023
@KazuCocoa KazuCocoa deleted the tvos-ios17-workaround branch November 3, 2023 17:56
@KazuCocoa
Copy link
Member Author

This was not necessary. The ip address thing was not fully paired yet, but help Xcode to detect the device

@KazuCocoa KazuCocoa restored the tvos-ios17-workaround branch November 7, 2023 04:44
@KazuCocoa KazuCocoa reopened this Nov 7, 2023
@KazuCocoa KazuCocoa closed this Nov 7, 2023
@KazuCocoa KazuCocoa deleted the tvos-ios17-workaround branch November 7, 2023 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant