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
Scope SavedStateFlowHandle to ViewModel's, create assisted injection extension functions, & add CI to run unit tests and build #11
Conversation
…ateFlowHandle to the hilt ViewModelScope
sample/hilt-di/src/main/java/com/plusmobileapps/savedstateflow/Modules.kt
Show resolved
Hide resolved
Can we add tests for the injection? Would it make sense? |
@KaustubhPatange I was thinking about adding tests for this injection, although since Hilt is a compile time safety dependency injection framework and not runtime. I was thinking the compilation of the hilt sample would be sufficient enough as a test? Otherwise if we feel the need to test, is this possible to cover in a unit test or write an espresso test for the sample perhaps? |
7d276aa
to
a0a73fe
Compare
Through the unit test, it is not possible. But I get your point a sample app if compiled through CI is more than enough to see if the injection works. |
Addresses issue #10
In order to get the CI to work and build the whole project, needed to rename the packages of each sample to not conflict with the library.