https://kotlinlang.org logo
#coroutines
Title
# coroutines
v

Vipul

07/22/2020, 5:41 PM
I am trying to test below method and assert that
remoteCloudDataStore.getFoodItems()
and
localDataStore.addFoodItems(it)
methods are called
exactly once
Copy code
fun getFoodItems(forceNetworkFetch: Boolean) = remoteCloudDataStore.getFoodItems().flatMapConcat {
        localDataStore.addFoodItems(it).flatMapConcat {
            flow { emit(FoodResult.Success(it)) }
        }
    }.flowOn(<http://dispatcher.io|dispatcher.io>)
Below is my test case
Copy code
repository.getFoodItems(true).collect {
            verify(exactly = 1) { remoteCloudDataStore.getFoodItems() }
            verify(exactly = 5) { localDataStore.addFoodItems(any()) }
        }
Surprisingly this test case pass, even if addFoodItems() method is not being called 5 times, can someone point me in correct direction what I am doing wrong here? I am passing
TestCoroutineDispatcher
during testing.
l

Luis Munoz

07/22/2020, 5:48 PM
for a test I would do
repository.getFoodItems(true).toList() verify(exactly = 1) { remoteCloudDataStore.getFoodItems() } verify(exactly = 5) { localDataStore.addFoodItems(any()) }
j

Justin Tullgren

07/22/2020, 5:57 PM
Those are both terminal in my understanding (collect/toList) so why would collect not work? Maybe coVerify is needed here and not just normal verify when using mockk
v

Vipul

07/22/2020, 6:00 PM
even after coVerify test passes, my expectation is this test should fail, because
localDataStore.addFoodItems
is not being called 5 times in method body.
If I do
toList()
I receive:
Copy code
io.mockk.MockKException: Missing calls inside verify { ... } block.
j

Justin Tullgren

07/22/2020, 6:04 PM
Sorry @Luis Munoz code looks correct to me so I guess I can't help any more. I would be interested in what you find though if you care to share in the future
l

Luis Munoz

07/22/2020, 6:14 PM
Copy code
remoteCloudDataStore.getFoodItems() returns a flow
Copy code
verify(exactly = 1) { remoteCloudDataStore.getFoodItems().toList() }
probably wrong you want to see how many items are in the list?
v

Vipul

07/22/2020, 6:16 PM
I am more curious about how to verify method interaction
how many items
test case is planned for sure 🙂
l

Luis Munoz

07/22/2020, 6:17 PM
if remoteCloudDataStore.getFoodItems() returns a flow you gotta terminate it
that is why I suggest you use toList to terminate everything
before doing assertions
v

Vipul

07/22/2020, 6:20 PM
Copy code
repository.getFoodItems(true).collect {
coVerify(exactly = 1) { remoteCloudDataStore.getFoodItems().toList() }
coVerify(exactly = 5) { localDataStore.getFoodItems().toList() }
}
Didn’t help
Test case still passes
l

Luis Munoz

07/22/2020, 6:24 PM
assertThat(remoteCloudDataStore.getFoodItems().toList().size).isEqualTo(1)
Copy code
assert localDataStore.getFoodItems().toList().size  is equal to 5
j

Justin Tullgren

07/22/2020, 6:25 PM
he wants to check the number of calls to his mock dependency. I believe you had it right the first time.
Copy code
val items = repository.getFoodItems(true).toList()
verify(exactly = 1) { remoteCloudDataStore.getFoodItems() }
l

Luis Munoz

07/22/2020, 6:25 PM
there isn't enough information
are you using runBlocking?
j

Justin Tullgren

07/22/2020, 6:26 PM
or runBlockingTest
I tried with both runBlocking & runBlockingTest
On debugging i am seeing that it doesn’t come other collect method.
Copy code
repository.getFoodItems(true).collect {
            verify(exactly = 11) { remoteCloudDataStore.getFoodItems() }
        }
l

Luis Munoz

07/22/2020, 7:42 PM
Copy code
@Test
    fun `force fetching foodItems makes network and db call`() =   {
      val latch = CountDownLatch(1)
      dispatcher.runBlockingTest {

        coEvery { restApi.getFoodItems() } returns MockData.foodItems
        every { localDataStore.addFoodItems(any()) } returns flow { MockData.foodItem }

        repository.getFoodItems(true).collect {
        
            verify(exactly = 11) { remoteCloudDataStore.getFoodItems() }
            latch.countDown()
            
        }
        
        
    }
     
      latch.await(3, TimeUnit.Seconds)
    
    }
never used TestCoroutineDispatcher so I'm not sure. Just make sure 1) that it actually calls the assertion (put a print statement before and after it). 2) that if an error gets thrown it is propagated not swallowed (do a throw RuntimeException and see that test fails and shows exception 3) that everything isn't async and just passes right through the test even when work is still being done in a background thread, you can use CountDownLatch to mark these points
doing assertion inside of collect seem wrong to me
if you only emit once why is it a flow and not a just a suspend, and if you emit more than once then how can you assert multiple times different items
j

Justin Tullgren

07/22/2020, 7:58 PM
@Vipul just to be clear, you verified in a runBlocking statement AFTER the call to collect?
Copy code
val items = repository.getFoodItems(true).toList()
verify(exactly = 1) { remoteCloudDataStore.getFoodItems() }
IE verify after the flow has terminated
yes I am verifying under collect
j

Justin Tullgren

07/22/2020, 7:59 PM
That does not look like what you Luis is saying. you are verifying in the collect lambda
Did you try this:
Copy code
coEvery { restApi.getFoodItems() } returns MockData.foodItems
every { localDataStore.addFoodItems(any()) } returns flow { MockData.foodItem }

repository.getFoodItems(true).toList()
verify(exactly = 11) { remoteCloudDataStore.getFoodItems() }
v

Vipul

07/22/2020, 8:02 PM
yup, didnt worked
Copy code
repository.getFoodItems(true).toList()
verify(exactly = 11) { remoteCloudDataStore.getFoodItems() }
l

Luis Munoz

07/22/2020, 8:02 PM
also you are doing
Copy code
remoteCloudDataStore.getFoodItems().flatMapConcat
but getFoodItems is a mock object?
so it doesn't have flatMapConcat
v

Vipul

07/22/2020, 8:03 PM
nope, I am having mock on
restApi.getFoodItems()
level
Copy code
remoteCloudDataStore = CloudCloudDataStoreImpl(restApi)
so following gets called
Copy code
restApi.getFoodItems()
l

Luis Munoz

07/22/2020, 8:04 PM
project looks interesting i'll clone it and run it in a few minutes
🙏 1
first notice that
Copy code
@Test
    fun `force fetching foodItems makes network and db call`() =

            dispatcher.runBlockingTest {

        coEvery { restApi.getFoodItems() } returns MockData.foodItems
        every { localDataStore.addFoodItems(any()) } returns flow { MockData.foodItem }

        repository.getFoodItems(true).collect {

            throw RuntimeException("test should fail")
            verify(exactly = 11) { remoteCloudDataStore.getFoodItems() }
        }
    }
does not fail
Copy code
@Test
    fun `force fetching foodItems makes network and db call`() =

            dispatcher.runBlockingTest {

        coEvery { restApi.getFoodItems() } returns MockData.foodItems
        every { localDataStore.addFoodItems(any()) } returns flow { MockData.foodItem }

        repository.getFoodItems(true).toList()

          //  throw RuntimeException("test should fail")
            coVerify(exactly = 11) { remoteCloudDataStore.getFoodItems().toList() }

    }
is probably what you want
Copy code
override fun getFoodItems(forceNetworkFetch: Boolean): Flow<FoodResult<List<FoodItem>>>
that signature is making the code overly complicated in my opinion
do you need to stream out multiple lists as a flow or can you just return a list of the collected items
v

Vipul

07/22/2020, 9:10 PM
I got your point, I can refactor it to just return list
but I am curious about this failed test 😄
did it run for you?