Skip to content

Feature/appium#645

Merged
tomholub merged 4 commits intomasterfrom
feature/appium
Oct 8, 2021
Merged

Feature/appium#645
tomholub merged 4 commits intomasterfrom
feature/appium

Conversation

@fcvakintos
Copy link
Contributor

@fcvakintos fcvakintos commented Oct 8, 2021

This PR introducec PR tests using appium.

close #469 // if this PR closes an issue


Tests

  • Tests added or updated

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

Copy link
Collaborator

@tomholub tomholub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The test code looks beautiful. I could not ask for more.

Also, I'll edit your PR description to match it with the issue.

platformName: 'iOS',
iosInstallPause: 5000,
deviceName: 'iPhone 11 Pro Max',
platformVersion: '14.5',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would this run iOS 15 as well? We've just bumped app minimum ios requirement to 15.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, all we need is to replace platform version or even use ENV variable for that and run what you need, I will add ENV variable for this


- iOS/Android Native Apps

## Before running tests, please create a `./apps` directory, download the app and move the app file into that directory
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could there be a command to automate this? Maybe using fastlane? To create a build and immediately test it (it will be needed for CI anyway)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, we could make comand to copy .app file

@tomholub tomholub merged commit 69be214 into master Oct 8, 2021
@tomholub tomholub deleted the feature/appium branch October 8, 2021 21:47
@fcvakintos fcvakintos restored the feature/appium branch October 11, 2021 12:47
@tomholub tomholub deleted the feature/appium branch November 9, 2021 16:27
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.

UI Tests for EKM integration

3 participants