test: add new tests for OCRemoteSpacesDataSourceTest and OCSpacesRepositoryTest

Author: joragua

owncloudApp/src/main/java/com/owncloud/android/presentation/spaces/SpacesListFragment.kt

@@ -139,7 +139,7 @@ class SpacesListFragment :
         collectLatestLifecycleFlow(spacesListViewModel.spacesList) { uiState ->
             if (uiState.searchFilter != "") {
                 var spacesToListFiltered =
-                    uiState.spaces.filter { it.name.lowercase().contains(uiState.searchFilter.lowercase()) && !it.isPersonal && !it.isDisabled }
+                    uiState.spaces.filter { it.name.lowercase().contains(uiState.searchFilter.lowercase()) && !it.isPersonal }
                 val personalSpace = uiState.spaces.find { it.isPersonal }
                 personalSpace?.let {
                     spacesToListFiltered = spacesToListFiltered.toMutableList().apply {
@@ -150,7 +150,7 @@ class SpacesListFragment :
                 spacesListAdapter.setData(spacesToListFiltered, isMultiPersonal)
             } else {
                 showOrHideEmptyView(uiState.spaces)
-                spacesListAdapter.setData(uiState.spaces.filter { !it.isDisabled }, isMultiPersonal)
+                spacesListAdapter.setData(uiState.spaces, isMultiPersonal)
             }
             binding.swipeRefreshSpacesList.isRefreshing = uiState.refreshing
             uiState.error?.let { showErrorInSnackbar(R.string.spaces_sync_failed, it) }

owncloudData/src/test/java/com/owncloud/android/data/spaces/datasources/implementation/OCRemoteSpacesDataSourceTest.kt

@@ -150,4 +150,72 @@ class OCRemoteSpacesDataSourceTest {
         }
     }
 
+    @Test
+    fun `disableSpace disables a project space correctly when delete mode is false`() {
+        val disableSpaceResult = createRemoteOperationResultMock(Unit, isSuccess = true)
+
+        every {
+            ocSpaceService.disableSpace(
+                spaceId = OC_SPACE_PROJECT_WITH_IMAGE.id,
+                deleteMode = false
+            )
+        } returns disableSpaceResult
+
+        ocRemoteSpacesDataSource.disableSpace(
+            accountName = OC_ACCOUNT_NAME,
+            spaceId = OC_SPACE_PROJECT_WITH_IMAGE.id,
+            deleteMode = false
+        )
+
+        verify(exactly = 1) {
+            clientManager.getSpacesService(OC_ACCOUNT_NAME)
+            ocSpaceService.disableSpace(
+                spaceId = OC_SPACE_PROJECT_WITH_IMAGE.id,
+                deleteMode = false
+            )
+        }
+    }
+
+    @Test
+    fun `disableSpace deletes a project space correctly when delete mode is true`() {
+        val disableSpaceResult = createRemoteOperationResultMock(Unit, isSuccess = true)
+
+        every {
+            ocSpaceService.disableSpace(
+                spaceId = OC_SPACE_PROJECT_WITH_IMAGE.id,
+                deleteMode = true
+            )
+        } returns disableSpaceResult
+
+        ocRemoteSpacesDataSource.disableSpace(
+            accountName = OC_ACCOUNT_NAME,
+            spaceId = OC_SPACE_PROJECT_WITH_IMAGE.id,
+            deleteMode = true
+        )
+
+        verify(exactly = 1) {
+            clientManager.getSpacesService(OC_ACCOUNT_NAME)
+            ocSpaceService.disableSpace(
+                spaceId = OC_SPACE_PROJECT_WITH_IMAGE.id,
+                deleteMode = true
+            )
+        }
+    }
+
+    @Test
+    fun `enableSpace enables a project space correctly`() {
+        val enableSpaceResult = createRemoteOperationResultMock(SPACE_RESPONSE, isSuccess = true)
+
+        every {
+            ocSpaceService.enableSpace(OC_SPACE_PROJECT_WITH_IMAGE.id)
+        } returns enableSpaceResult
+
+        ocRemoteSpacesDataSource.enableSpace(OC_ACCOUNT_NAME, OC_SPACE_PROJECT_WITH_IMAGE.id)
+
+        verify(exactly = 1) {
+            clientManager.getSpacesService(OC_ACCOUNT_NAME)
+            ocSpaceService.enableSpace(OC_SPACE_PROJECT_WITH_IMAGE.id)
+        }
+    }
+
 }

owncloudData/src/test/java/com/owncloud/android/data/spaces/repository/OCSpacesRepositoryTest.kt

@@ -51,7 +51,7 @@ class OCSpacesRepositoryTest {
 
     private val localSpacesDataSource = mockk<LocalSpacesDataSource>(relaxUnitFun = true)
     private val localUserDataSource = mockk<LocalUserDataSource>(relaxUnitFun = true)
-    private val remoteSpacesDataSource = mockk<RemoteSpacesDataSource>()
+    private val remoteSpacesDataSource = mockk<RemoteSpacesDataSource>(relaxUnitFun = true)
     private val localCapabilitiesDataSource = mockk<LocalCapabilitiesDataSource>(relaxUnitFun = true)
     private val ocSpacesRepository = OCSpacesRepository(localSpacesDataSource, localUserDataSource, remoteSpacesDataSource,
         localCapabilitiesDataSource)
@@ -350,4 +350,31 @@ class OCSpacesRepositoryTest {
         }
     }
 
+    @Test
+    fun `disableSpace disables a space correctly when delete mode is false`() {
+        ocSpacesRepository.disableSpace(OC_ACCOUNT_NAME, OC_SPACE_PROJECT_WITH_IMAGE.id, false)
+
+        verify(exactly = 1) {
+            remoteSpacesDataSource.disableSpace(OC_ACCOUNT_NAME, OC_SPACE_PROJECT_WITH_IMAGE.id, false)
+        }
+    }
+
+    @Test
+    fun `disableSpace deletes a space correctly when delete mode is true`() {
+        ocSpacesRepository.disableSpace(OC_ACCOUNT_NAME, OC_SPACE_PROJECT_WITH_IMAGE.id, true)
+
+        verify(exactly = 1) {
+            remoteSpacesDataSource.disableSpace(OC_ACCOUNT_NAME, OC_SPACE_PROJECT_WITH_IMAGE.id, true)
+        }
+    }
+
+    @Test
+    fun `enableSpace enables a space correctly`() {
+        ocSpacesRepository.enableSpace(OC_ACCOUNT_NAME, OC_SPACE_PROJECT_WITH_IMAGE.id)
+
+        verify(exactly = 1) {
+            remoteSpacesDataSource.enableSpace(OC_ACCOUNT_NAME, OC_SPACE_PROJECT_WITH_IMAGE.id)
+        }
+    }
+
 }