Sean McQuillan
Sean McQuillan
Lots of good ideas in this thread. A use case to keep in mind while designing this is Intents used by testing, e.g. the example on this page: https://developer.android.com/reference/android/support/test/espresso/intent/Intents.html
Lgtm. Need tests for API please, also one to avoid regressing on parseresult and I'll merge it in.
Thanks for the patch btw.
This pull request is pending tests before it can be merged. Sean
Twisted logging looks good, will merge once I get a chance to make some of the new log lines grepable with prefixes.
I'm not opposed to making CouchDB be the expert on urls. Looking at the rest of the diff, it probably makes more sense to undo the partial binding that is...
Ah so this one is a bit weird - since you're not passing the exception handler from the TestCoroutineScope the exception is handled by viewModelScope (which I don't believe will...
Two options here: 1. Add test-injection of uncaught exception handler to viewModelScope for a specific viewModel 2. Add a global uncaught exception handler on CoroutineScope (CoroutineScope.addGlobalUncaughtExceptionHandler) - this mirrors the...
Agreed - this should definitely cause a test (or at least the test runner from a global uncaught exception) to fault via some mechanism.
Agreed, this does seem to violate surface expectations despite being "working as intended." Given how prevalent non-configurable test exception handlers have been in the current API designs (e.g. viewModelScope and...