-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[UI Test] Update Appium to 5.0.0-rc.8 #22117
Conversation
#pragma warning disable CS0618 // Type or member is obsolete | ||
_app.Driver.LaunchApp(); | ||
#pragma warning restore CS0618 // Type or member is obsolete | ||
} | ||
else | ||
{ | ||
_app.Driver.ActivateApp(_app.GetAppId()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the new code path now work with packaged and unpakcaged apps?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Take a look to #22117 (comment)
else if (_app.GetTestDevice() == TestDevice.Windows) | ||
{ | ||
#pragma warning disable CS0618 // Type or member is obsolete | ||
_app.Driver.LaunchApp(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LaunchApp
have been removed.
Cannot execute the launchApp script because not exists.
_app.Driver.ExecuteScript("windows: launchApp", new Dictionary<string, object>());
And ActivateApp
is not implemented.
if(_app.Driver is WindowsDriver windowsDriver)
windowsDriver.ActivateApp(_app.GetAppId());
The alternative could be to quit the session and create a new one, but it is a bigger change. The Windows tests are going to fail until fix this.
Description of Change
Update Appium to 5.0.0-rc.8.
https://dev.azure.com/dnceng/internal/_build/results?buildId=2440349&view=results