-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Tests for welcome activity #2721
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2721 +/- ##
======================================
Coverage 2.71% 2.71%
======================================
Files 267 267
Lines 12838 12838
Branches 1141 1141
======================================
Hits 349 349
Misses 12463 12463
Partials 26 26 Continue to review full report at Codecov.
|
|
||
@Test | ||
fun swipeBeyondBounds(){ | ||
if (!ConfigUtils.isBetaFlavour()){ |
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.
This test should probably still run on beta
|
||
@Test | ||
fun swipeTillLastAndFinish(){ | ||
if (!ConfigUtils.isBetaFlavour()){ |
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.
This test should probably still run on beta
Thanks for approving the PR @maskaravivek @domdomegg :) |
@vanshikaarora Can you create another PR so the two tests also run on beta (i.e. remove the if condition wrapping them) |
And tag me in to review it :) |
Description (required)
Fixes #2644 Extend WelcomeActivityTest
Changes made
Added the above two tests
Tests performed (required)
Tested ProdDebug on vivo 1713 with API level 24.