-
Notifications
You must be signed in to change notification settings - Fork 35
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
Start UI testing #16 #36
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #36 +/- ##
===========================================
+ Coverage 69.85% 87.29% +17.43%
===========================================
Files 13 13
Lines 428 433 +5
===========================================
+ Hits 299 378 +79
+ Misses 129 55 -74
Continue to review full report at Codecov.
|
Thanks @ThaysPrado . I'll review as soon as possible. |
Both tests are failing here @ThaysPrado 🤔 Did you follow some guide for the UI tests? I think there should be some asserts. Maybe you can use this tutorial as a reference for the implementation: https://www.raywenderlich.com/960290-ios-unit-testing-and-ui-testing-tutorial#toc-anchor-014 This is only for a start, so even one good UI test would get this merged 😃 |
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.
Changes requested in comment above
It was my first attempt with UI testing. I will see the tutorial you mentioned. Even though it is strange, the tests worked with me, I will see this more closely. |
@ThaysPrado understood! Looking forward to the improvements. Thank you! |
Manage this branch in SquashTest this branch here: https://master-680e9.squash.io |
I started UI tests with two tests on listView and DetailsView.
hope it is useful