Skip to content
Snippets Groups Projects
Commit 08fe86b2 authored by Habib Kazemi's avatar Habib Kazemi Committed by Geoffrey Métais
Browse files

Rename CustomDirectoryRepository to DirectoryRepository


Signed-off-by: default avatarGeoffrey Métais <geoffrey.metais@gmail.com>
parent 2be2f6a5
No related branches found
No related tags found
No related merge requests found
......@@ -4,14 +4,13 @@ import android.content.Context
import android.support.annotation.WorkerThread
import kotlinx.coroutines.experimental.Job
import kotlinx.coroutines.experimental.launch
import kotlinx.coroutines.experimental.withContext
import org.videolan.tools.SingletonHolder
import org.videolan.vlc.database.CustomDirectoryDao
import org.videolan.vlc.database.MediaDatabase
import org.videolan.vlc.database.models.CustomDirectory
import org.videolan.vlc.util.VLCIO
class CustomDirectoryRepository (private val customDirectoryDao: CustomDirectoryDao) {
class DirectoryRepository (private val customDirectoryDao: CustomDirectoryDao) {
fun addCustomDirectory(path: String): Job {
return launch(VLCIO) {
......@@ -31,5 +30,5 @@ class CustomDirectoryRepository (private val customDirectoryDao: CustomDirectory
@WorkerThread
fun customDirectoryExists(path: String) = customDirectoryDao.get(path).isNotEmpty()
companion object : SingletonHolder<CustomDirectoryRepository, Context>({ CustomDirectoryRepository(MediaDatabase.getInstance(it).customDirectoryDao()) })
companion object : SingletonHolder<DirectoryRepository, Context>({ DirectoryRepository(MediaDatabase.getInstance(it).customDirectoryDao()) })
}
\ No newline at end of file
......@@ -3,7 +3,6 @@ package org.videolan.vlc.repository
import android.arch.core.executor.testing.InstantTaskExecutorRule
import kotlinx.coroutines.experimental.runBlocking
import org.hamcrest.CoreMatchers.`is`
import org.hamcrest.CoreMatchers.hasItem
import org.junit.Assert.*
import org.junit.Before
import org.junit.Rule
......@@ -20,9 +19,9 @@ import org.videolan.vlc.util.mock
import org.videolan.vlc.util.uninitialized
@RunWith(PowerMockRunner::class)
class CustomDirectoryRepositoryTest {
class DirectoryRepositoryTest {
private val customDirectoryDao = mock<CustomDirectoryDao>()
private lateinit var customDirectoryRepository: CustomDirectoryRepository
private lateinit var directoryRepository: DirectoryRepository
@Rule
@JvmField
......@@ -32,14 +31,14 @@ class CustomDirectoryRepositoryTest {
System.setProperty("kotlinx.coroutines.blocking.checker", "disable")
val db = mock<MediaDatabase>()
`when`(db.customDirectoryDao()).thenReturn(customDirectoryDao)
customDirectoryRepository = CustomDirectoryRepository(customDirectoryDao)
directoryRepository = DirectoryRepository(customDirectoryDao)
}
@Test
fun insertTwoCustomDirectory_GetAllShouldReturnTwo() = runBlocking{
val fakeCustomDirectories = TestUtil.createCustomDirectories(2)
fakeCustomDirectories.forEach {
customDirectoryRepository.addCustomDirectory(it.path).join()
directoryRepository.addCustomDirectory(it.path).join()
}
val inserted = argumentCaptor<CustomDirectory>()
......@@ -51,7 +50,7 @@ class CustomDirectoryRepositoryTest {
`when`(customDirectoryDao.getAll()).thenReturn(fakeCustomDirectories)
val customDirectories = customDirectoryRepository.getCustomDirectories()
val customDirectories = directoryRepository.getCustomDirectories()
verify(customDirectoryDao).getAll()
assertThat(customDirectories.size, `is`(2))
}
......@@ -60,7 +59,7 @@ class CustomDirectoryRepositoryTest {
fun insertTwoCustomDirectory_DeleteOneShouldDeleteOne() = runBlocking{
val fakeCustomDirectories = TestUtil.createCustomDirectories(2)
fakeCustomDirectories.forEach {
customDirectoryRepository.addCustomDirectory(it.path).join()
directoryRepository.addCustomDirectory(it.path).join()
}
val inserted = argumentCaptor<CustomDirectory>()
......@@ -69,7 +68,7 @@ class CustomDirectoryRepositoryTest {
assertThat(inserted.allValues[0], `is`(fakeCustomDirectories[0]))
assertThat(inserted.allValues[1], `is`(fakeCustomDirectories[1]))
customDirectoryRepository.deleteCustomDirectory(fakeCustomDirectories[0].path)
directoryRepository.deleteCustomDirectory(fakeCustomDirectories[0].path)
val deleted = argumentCaptor<CustomDirectory>()
verify(customDirectoryDao).delete(deleted.capture() ?: uninitialized())
......@@ -80,7 +79,7 @@ class CustomDirectoryRepositoryTest {
fun insertOneCustomDirectory_CheckExistenceShouldBeTrue() = runBlocking{
val fakeCustomDirectories = TestUtil.createCustomDirectories(1)
fakeCustomDirectories.forEach {
customDirectoryRepository.addCustomDirectory(it.path).join()
directoryRepository.addCustomDirectory(it.path).join()
}
val inserted = argumentCaptor<CustomDirectory>()
......@@ -90,7 +89,7 @@ class CustomDirectoryRepositoryTest {
`when`(customDirectoryDao.get(fakeCustomDirectories[0].path)).thenReturn(fakeCustomDirectories)
val bool = customDirectoryRepository.customDirectoryExists(fakeCustomDirectories[0].path)
val bool = directoryRepository.customDirectoryExists(fakeCustomDirectories[0].path)
assertTrue(bool)
}
......@@ -98,7 +97,7 @@ class CustomDirectoryRepositoryTest {
fun insertOneCustomDirectory_CheckExistenceForWrongPathShouldBeFalse() = runBlocking{
val fakeCustomDirectories = TestUtil.createCustomDirectories(1)
fakeCustomDirectories.forEach {
customDirectoryRepository.addCustomDirectory(it.path).join()
directoryRepository.addCustomDirectory(it.path).join()
}
val inserted = argumentCaptor<CustomDirectory>()
......@@ -108,7 +107,7 @@ class CustomDirectoryRepositoryTest {
`when`(customDirectoryDao.get(fakeCustomDirectories[0].path)).thenReturn(fakeCustomDirectories)
val bool = customDirectoryRepository.customDirectoryExists(fakeCustomDirectories[0].path+"foo")
val bool = directoryRepository.customDirectoryExists(fakeCustomDirectories[0].path+"foo")
assertFalse(bool)
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment