-
Notifications
You must be signed in to change notification settings - Fork 13
Closed
Labels
Description
I'll write more later, but this is meant to be a place to track discussions about testing.
I'm not happy with where we are right now. I don't think we have enough tests. Unit tests are easy enough, but a lot of our logic is integrated. I used to favor decomposing things into unit-testable things that take mockable onSuccess
callbacks and their stateful not-easily-tested wrappers that pass in the true values of those callbacks. But I'm not really into that anymore. I feel like there has to be a way of testing the real app logic without too much pain.
#565 is an entry in the discussion.