-
Notifications
You must be signed in to change notification settings - Fork 662
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
Add Bacs confirmation activity tests #10014
Add Bacs confirmation activity tests #10014
Conversation
Diffuse output:
APK
|
ecc6b0f
to
18f7805
Compare
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.
Love this!
import java.util.concurrent.TimeUnit | ||
|
||
@RunWith(RobolectricTestRunner::class) | ||
internal class BacsConfirmationActivityTest { |
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.
Did you run these with ShampooRule to make sure they're not flakey?
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.
Just validated against ShampooRule
, no flakes with 100 iterations.
Summary
Add Bacs confirmation activity tests
Motivation
Adds full flow activity testing for Bacs.
Testing