[Test Coverage] Add Unit Test for CaptureLocationTaskViewModel about ground-android HOT 6 OPEN

Sounds good to refactor the code first. I thought you were referring to adding tests for the view model explicitly, which we shouldn't do if possible.

from ground-android.

Also, as the current usage of AbstractTaskViewModel is a bit big, I think we should create a separate issue for fixing the AbstractTaskViewModel.

WDYT?

from ground-android.

For the other tasks, we added tests for the task fragment which takes care of hilt dependencies automatically including the view model. This also provides coverage for what the user sees rather than testing the VM separately. Can you please check if we can do the same for CaptureLocationTask as well?

from ground-android.

Let me rephrase,

Not sure I get it, but what I meant earlier is that we are passing the Resources object in the ViewModel, which is not a good practice and making our ViewModel dependent on the Android library.

So, we should separate the resource out of ViewModel and then go for writing correct test cases.

from ground-android.

Sounds good to refactor the code first. I thought you were referring to adding tests for the view model explicitly, which we shouldn't do if possible.

ok, great, for refactoring, I will create a separate issue.

Regarding testing viewModel, aren't we testing it separately? So, we are doing only UI tests for fragments and based on that we are confirming the viewModel is also tested, right? or did I missed anything?

from ground-android.

Regarding testing viewModel, aren't we testing it separately? So, we are doing only UI tests for fragments and based on that we are confirming the viewModel is also tested, right? or did I missed anything?

That's correct. The rationale is that this ensures that we have enough coverage for the fragment as well instead of having duplicate tests for UI and ViewModel

from ground-android.