test: cover spaces search filter with SpacesListFragmentTest

Extract the personal-space filtering into a testable filterSpaces() extension
and add unit coverage for the multi-personal name match, single-personal
pin-at-front, and single-personal regression paths.

Author: mvanhorn

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

@@ -181,21 +181,7 @@ class SpacesListFragment :
         collectLatestLifecycleFlow(spacesListViewModel.spacesList) { uiState ->
             var spacesToListFiltered: List<OCSpace>
             if (uiState.searchFilter != "") {
-                val searchFilter = uiState.searchFilter.lowercase()
-                spacesToListFiltered = if (isMultiPersonal) {
-                    uiState.spaces.filter { it.name.lowercase().contains(searchFilter) && shouldShowDisabledSpace(it) }
-                } else {
-                    uiState.spaces.filter { it.name.lowercase().contains(searchFilter) && !it.isPersonal &&
-                            shouldShowDisabledSpace(it) }
-                }
-                if (!isMultiPersonal) {
-                    val personalSpace = uiState.spaces.find { it.isPersonal }
-                    personalSpace?.let {
-                        spacesToListFiltered = spacesToListFiltered.toMutableList().apply {
-                            add(0, personalSpace)
-                        }
-                    }
-                }
+                spacesToListFiltered = uiState.spaces.filterSpaces(uiState.searchFilter, isMultiPersonal, ::shouldShowDisabledSpace)
                 showOrHideEmptyView(spacesToListFiltered)
                 spacesListAdapter.setData(spacesToListFiltered, isMultiPersonal)
             } else {
@@ -524,3 +510,16 @@ class SpacesListFragment :
     }
 
 }
+
+internal fun List<OCSpace>.filterSpaces(
+    query: String,
+    multi: Boolean,
+    visible: (OCSpace) -> Boolean,
+): List<OCSpace> {
+    val queryLowercase = query.lowercase()
+    val spacesMatchingName = filter { it.name.lowercase().contains(queryLowercase) }
+    val matchingSpaces = spacesMatchingName.filter { (multi || !it.isPersonal) && visible(it) }
+    if (multi) return matchingSpaces
+    val personalSpace = find { it.isPersonal } ?: return matchingSpaces
+    return listOf(personalSpace) + matchingSpaces
+}

owncloudApp/src/test/java/com/owncloud/android/presentation/spaces/SpacesListFragmentTest.kt

@@ -0,0 +1,35 @@
+/**
+ * ownCloud Android client application
+ *
+ * @author Matt Van Horn
+ * Copyright (C) 2026 ownCloud GmbH.
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2,
+ * as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+package com.owncloud.android.presentation.spaces
+
+import com.owncloud.android.testutil.OC_SPACE_PERSONAL
+import org.junit.Assert.assertEquals
+import org.junit.Test
+
+class SpacesListFragmentTest {
+    @Test
+    fun `filterSpaces filters multi-personal names and preserves single-personal pinning`() {
+        val personalSpaces = listOf(OC_SPACE_PERSONAL.copy(name = "Alice Personal"), OC_SPACE_PERSONAL.copy(name = "Bob Personal"))
+        val singlePersonalSpaces = listOf(OC_SPACE_PERSONAL.copy(name = "Project Personal"))
+        assertEquals(personalSpaces, personalSpaces.filterSpaces("personal", true) { true })
+        assertEquals(listOf(personalSpaces[0]), personalSpaces.filterSpaces("alice", true) { true })
+        assertEquals(singlePersonalSpaces, singlePersonalSpaces.filterSpaces("project", false) { true })
+    }
+}