Added suspend to upsertAll (it was missed in a previous PR)
Move the use of withContext to only wrap the mapping functions
Remove MainCoroutineRule from both TaskDaoTest and DefaultTaskRepositoryTest. In the latter, a testScope and testDispatcher are used to guarantee deterministic behaviour
Remove pluralization tasksRepository to taskRepository in line with other recent changes
Here's what I've done:
suspendtoupsertAll(it was missed in a previous PR)withContextto only wrap the mapping functionsMainCoroutineRulefrom bothTaskDaoTestandDefaultTaskRepositoryTest. In the latter, atestScopeandtestDispatcherare used to guarantee deterministic behaviourtasksRepositorytotaskRepositoryin line with other recent changes