[Android] Rename CustomDirectoryRepository to DirectoryRepository

Habib Kazemi git at videolan.org
Thu Aug 30 10:10:04 CEST 2018


vlc-android | branch: master | Habib Kazemi <kazemihabib1996 at gmail.com> | Mon Aug 20 09:27:11 2018 +0430| [08fe86b2abe6bb126c4e31e5982df001f53eeba3] | committer: Geoffrey Métais

Rename CustomDirectoryRepository to DirectoryRepository

Signed-off-by: Geoffrey Métais <geoffrey.metais at gmail.com>

> https://code.videolan.org/videolan/vlc-android/commit/08fe86b2abe6bb126c4e31e5982df001f53eeba3
---

 ...rectoryRepository.kt => DirectoryRepository.kt} |  5 ++---
 ...epositoryTest.kt => DirectoryRepositoryTest.kt} | 23 +++++++++++-----------
 2 files changed, 13 insertions(+), 15 deletions(-)

diff --git a/vlc-android/src/org/videolan/vlc/repository/CustomDirectoryRepository.kt b/vlc-android/src/org/videolan/vlc/repository/DirectoryRepository.kt
similarity index 76%
rename from vlc-android/src/org/videolan/vlc/repository/CustomDirectoryRepository.kt
rename to vlc-android/src/org/videolan/vlc/repository/DirectoryRepository.kt
index 0e4dd98bb..4d2517ae2 100644
--- a/vlc-android/src/org/videolan/vlc/repository/CustomDirectoryRepository.kt
+++ b/vlc-android/src/org/videolan/vlc/repository/DirectoryRepository.kt
@@ -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
diff --git a/vlc-android/test/org/videolan/vlc/repository/CustomDirectoryRepositoryTest.kt b/vlc-android/test/org/videolan/vlc/repository/DirectoryRepositoryTest.kt
similarity index 81%
rename from vlc-android/test/org/videolan/vlc/repository/CustomDirectoryRepositoryTest.kt
rename to vlc-android/test/org/videolan/vlc/repository/DirectoryRepositoryTest.kt
index a52f27c8e..063f9fcf0 100644
--- a/vlc-android/test/org/videolan/vlc/repository/CustomDirectoryRepositoryTest.kt
+++ b/vlc-android/test/org/videolan/vlc/repository/DirectoryRepositoryTest.kt
@@ -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)
     }
 



More information about the Android mailing list