Skip to content
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

MVI test sample #284

Open
dominikmicuta opened this issue Dec 3, 2017 · 4 comments
Open

MVI test sample #284

dominikmicuta opened this issue Dec 3, 2017 · 4 comments

Comments

@dominikmicuta
Copy link

@sockeqwe please add instrumental / espresso test to MVI sample. You talked about it in Fragmented podcast and its hard to imagine and implement the intercepting part of the view without any guidance.

@sockeqwe
Copy link
Owner

sockeqwe commented Dec 3, 2017

Yes, will do so ...
Unfortunately, I wanted to keep things simple and haven't used Dagger. Now for the MVI intercepting part it may be easier to use dagger. So perhaps I have to migrate the sample app to dagger before.

In the meantime you can take a look at this presentation I gave earlier this year:

https://speakerdeck.com/sockeqwe/reactive-statemanagement-with-model-view-intent

The testing part begins at slide number 92.

@dominikmicuta
Copy link
Author

Perfect! Thanks for the link. I will go through the presentation.
Maybe sample could have also things like: navigation that don`t change the state, navigation that is emitted as part of the business logic, validation of inputs etc.
But as you already mentioned, this isn't really the scope of MVI. I just wonder how this looks like in your projects.

@lucamtudor
Copy link

I'm looking forward to it!

@sockeqwe
Copy link
Owner

Sorry, I still havent found time to do it. In the mean time you may find this useful:
https://www.youtube.com/watch?v=osM-1z5tJ4g&index=27&list=PLr5cF6ekyIDgWee0wtwMKSGcoMvt5fkn_&t=314s

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

No branches or pull requests

3 participants