merge ios automation to develop #171
Conversation
|
Added a smoke test to verify workflow of creating an identity in the app and updated tests to reflect Jason's fix for bug 165199796. |
| name: Install appium server | ||
| command: | | ||
| npm install -g appium | ||
| npm install wd |
There was a problem hiding this comment.
Is this a global package or local? If local, why isn't it in package.json
There was a problem hiding this comment.
I’m going to check if I can add installation of appium server to package.json
As for running tests locally, you have to install appium:
npm install -g appium
npm install wd
pip install Appium-Python-Client
Then launch appium in terminal by running “appium”
And in a separate terminal run the tests from repo “python testsuites/test.py”
| @@ -198,6 +204,17 @@ jobs: | |||
| - run: | |||
| name: Fastlane | |||
| command: cd ~/uport-mobile/ios && bundle exec fastlane buildTheApp | |||
| - run: | |||
There was a problem hiding this comment.
Maybe we should run bundle exec fastlane buildTheApp only if appium tests pass?
There was a problem hiding this comment.
That's a good point.
Codecov Report
@@ Coverage Diff @@
## develop #171 +/- ##
========================================
Coverage 58.25% 58.25%
========================================
Files 233 233
Lines 6750 6750
Branches 977 977
========================================
Hits 3932 3932
Misses 2442 2442
Partials 376 376Continue to review full report at Codecov.
|
No description provided.