-
Notifications
You must be signed in to change notification settings - Fork 28
added wrapper methods for getContextHandles and setContext #149
added wrapper methods for getContextHandles and setContext #149
Conversation
You have successfully added a new SonarCloud configuration ``. As part of the setup process, we have scanned this repository and found no existing alerts. In the future, you will see all code scanning alerts on the repository Security tab. |
src/main/java/com/codeborne/selenide/appium/SelenideAppiumTargetLocator.java
Show resolved
Hide resolved
@Test | ||
void contexts() { | ||
SelenideAppium.openAndroidDeepLink("mydemoapprn://webview", "com.saucelabs.mydemoapp.rn"); | ||
SelenideAppium.setContext("NATIVE_APP"); |
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.
It would be great to test switching between different contexts.
Does this app has some other contexts in addition to NATIVE_APP
?
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.
@asolntsev - I have altered the test now to validate switching context. I have also added another method to get the current context
Kudos, SonarCloud Quality Gate passed!
|
Proposed changes
Solves the issue
Checklist
gradlew check
command