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

iOS 12.2 webview tests fail using xcuitest-driver #12239

Closed
sadafyadegari opened this issue Mar 1, 2019 · 9 comments
Closed

iOS 12.2 webview tests fail using xcuitest-driver #12239

sadafyadegari opened this issue Mar 1, 2019 · 9 comments
Assignees
Labels
Bug a problem that needs fixing Mobile Safari related to mobile Safari driver P0 XCUITest regarding xcuitest driver

Comments

@sadafyadegari
Copy link

sadafyadegari commented Mar 1, 2019

The problem

Webview tests are failing in simulators.

Environment

  • Appium version (or git revision) that exhibits the issue: 1.11.2-rc.1 git hash: cb8dbdb
  • Last Appium version that did not exhibit the issue (if applicable): N/A
  • Desktop OS/version used to run Appium: 10.14.3 Mojave
  • Node.js version (unless using Appium.app|exe): v8.9.4
  • Npm package manager: 5.6.0
  • Mobile platform/version under test: iOS 12.2
  • Real device or emulator/simulator: N/A
  • Appium CLI: 1.11.0-beta.3
  • Xcode version: 10.2 beta 2
  • simulator version: Version 10.2 (SimulatorApp-880.5 CoreSimulator-587.32)

Details

The tests falls into an infinite loop where processes cannot locate each other over rpc.
appium-xcuitest-driver git hash: c8e4eabc17361945cc37aa8f4319594cbd7a090d
appium-WebDriverAgent revision: branch xcode 10.2 sha: 9b05749dc2054f162eb2569b22bfa9350b195aa6

Link to Appium logs

Link to the Appium logs

Code To Reproduce Issue [ Good To Have ]

Running a test with runsauce with the following capabilities

{
    "desiredCapabilities": {
        "browserName": "safari",
        "appiumVersion": "1.11.1",
        "deviceName": "iPhone 8",
        "platformVersion": "12.2",
        "launchTimeout": 35000,
        "platformName": "iOS",
        "webviewConnectRetries": 10,
        "prevent-requeue": true,
        "name": "Basic web test (default) (safari on iPhone 8 on iOS 12.2)",
        "showXcodeLog": "true"
    }
}

(related to #12101)

@dpgraham dpgraham added Bug a problem that needs fixing P0 labels Mar 1, 2019
@mykola-mokhnach
Copy link
Collaborator

The log shows that WDA is still in the build process. Is it the full log?

@mykola-mokhnach mykola-mokhnach added Needs Info typically non-actionable; needs author to respond XCUITest regarding xcuitest driver labels Mar 2, 2019
@imurchie
Copy link
Contributor

imurchie commented Mar 4, 2019

This is an issue with finding the correct Unix Domain Socket to talk with the device. I will look into it a bit.

@imurchie imurchie self-assigned this Mar 4, 2019
@imurchie imurchie added this to the Sprint 31 milestone Mar 4, 2019
@sadafyadegari
Copy link
Author

@mykola-mokhnach I had truncated the logs because it went into a fast loop looking for a app PID. I created a new git which is not truncated. https://gist.github.com/sadafyadegari/4e5020e752e8fcad2ad00275bf0298e2

@mykola-mokhnach mykola-mokhnach removed the Needs Info typically non-actionable; needs author to respond label Mar 4, 2019
@imurchie
Copy link
Contributor

imurchie commented Mar 4, 2019

Actually, it seems that launching Safari with the url as an argument does not work for these sims.

@mykola-mokhnach mykola-mokhnach added the Mobile Safari related to mobile Safari driver label Mar 5, 2019
@sadafyadegari
Copy link
Author

tested the xcode-10.2 beta 4 that came out on March 4th. Same results happen again. See logs:
https://gist.github.com/sadafyadegari/607af4c32d38de888a1f85c150a830f8

@imurchie
Copy link
Contributor

imurchie commented Mar 6, 2019

Update: In addition to the url not being correctly set, the Webkit Protocol has been compressed. I am investigating our implementation and should have some results by EOD.

@imurchie
Copy link
Contributor

imurchie commented Mar 6, 2019

Update: I have a few things to iron out but tests in general are working. Once they all work for 12.2 I will move to making sure they still work for other platforms.

@KazuCocoa
Copy link
Member

linked PR has been merged

@lock
Copy link

lock bot commented Mar 27, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug a problem that needs fixing Mobile Safari related to mobile Safari driver P0 XCUITest regarding xcuitest driver
Projects
None yet
Development

No branches or pull requests

5 participants