feat: cover VersionHistoryViewModel with unit test (WPB-21661)

app/src/main/kotlin/com/wire/android/di/accountScoped/CellsModule.kt

@@ -210,13 +210,6 @@ class CellsModule {
     fun provideGetNodeVersionsUseCase(cellsScope: CellsScope): GetNodeVersionsUseCase =
         cellsScope.getNodeVersions
 
-    @ViewModelScoped
-    @Provides
-    fun provideRefreshHelper(cellsScope: CellsScope, kaliumConfigs: KaliumConfigs): CellAssetRefreshHelper = CellAssetRefreshHelper(
-        refreshAsset = cellsScope.refreshAsset,
-        featureFlags = kaliumConfigs
-    )
-
     @ViewModelScoped
     @Provides
     fun provideRestoreNodeVersionUseCase(cellsScope: CellsScope): RestoreNodeVersionUseCase =
@@ -226,4 +219,11 @@ class CellsModule {
     @Provides
     fun provideDownloadCellVersionUseCase(cellsScope: CellsScope): DownloadCellVersionUseCase =
         cellsScope.downloadCellVersion
+
+    @ViewModelScoped
+    @Provides
+    fun provideRefreshHelper(cellsScope: CellsScope, kaliumConfigs: KaliumConfigs): CellAssetRefreshHelper = CellAssetRefreshHelper(
+        refreshAsset = cellsScope.refreshAsset,
+        featureFlags = kaliumConfigs
+    )
 }

app/src/test/kotlin/com/wire/android/util/StringUtilTest.kt

@@ -46,6 +46,36 @@ class StringUtilTest {
         assert(expected == actual)
     }
 
+    @Test
+    fun givenFilenameWithExtension_whenCalled_thenShouldInsertStringBeforeExtension() {
+        val originalFilename = "document.pdf"
+        val textToInsert = "new"
+
+        val result = originalFilename.addBeforeExtension(textToInsert)
+
+        assertEquals("document_new.pdf", result)
+    }
+
+    @Test
+    fun givenFilenameWithoutExtension_whenCalled_thenShouldAppendString() {
+        val originalFilename = "myfile"
+        val textToInsert = "_version2"
+
+        val result = originalFilename.addBeforeExtension(textToInsert)
+
+        assertEquals("myfile_version2", result)
+    }
+
+    @Test
+    fun givenFilenameWithMultipleDots_whenCalled_thenShouldInsertBeforeLastDot() {
+        val originalFilename = "archive.2024.tar.gz"
+        val textToInsert = "backup"
+
+        val result = originalFilename.addBeforeExtension(textToInsert)
+
+        assertEquals("archive.2024_backup.tar.gz", result)
+    }
+
     @Suppress("LongMethod")
     @Test
     fun givenDifferentMarkdownsWithOnlyWhitespaces_whenCheckingIfNotBlank_thenReturnProperValues() {

core/ui-common/src/main/kotlin/com/wire/android/util/StringUtils.kt

@@ -49,12 +49,17 @@ fun String.capitalizeFirstLetter(): String = lowercase().replaceFirstChar(Char::
 fun String.normalizeFileName(): String = this.replace("/", "")
 
 fun String.addBeforeExtension(insert: String): String {
-    val dotIndex = this.lastIndexOf('.')
-    return if (dotIndex != -1) {
-        val name = this.take(dotIndex)
-        val ext = this.substring(dotIndex)
-        "${name}_$insert$ext"
-    } else {
-        this + insert
+    val lastDotIndex = this.lastIndexOf('.')
+    if (lastDotIndex <= 0) {
+        return this + insert
     }
+
+    val extensionBlockIndex = this.lastIndexOf('.', lastDotIndex - 1).let {
+        if (it == -1) lastDotIndex else it
+    }
+
+    val name = this.take(extensionBlockIndex)
+    val ext = this.substring(extensionBlockIndex)
+
+    return "${name}_$insert$ext"
 }

features/cells/build.gradle.kts

@@ -53,6 +53,7 @@ dependencies {
     testRuntimeOnly(libs.junit5.engine)
     androidTestImplementation(libs.androidx.test.extJunit)
     androidTestImplementation(libs.androidx.espresso.core)
+    testImplementation(testFixtures(project(":core:ui-common")))
 }
 
 android {

features/cells/src/main/java/com/wire/android/feature/cells/ui/versioning/VersionHistoryScreen.kt

@@ -145,9 +145,9 @@ fun VersionHistoryScreen(
 private fun VersionHistoryScreenContent(
     versionsGroupedByTime: List<VersionGroup>,
     versionHistoryState: State<VersionHistoryState>,
-    onRefresh: () -> Unit,
     optionsBottomSheetState: WireModalSheetState<Pair<String, CellVersion>>,
     restoreDialogState: RestoreDialogState,
+    onRefresh: () -> Unit,
     modifier: Modifier = Modifier,
     fileName: String? = null,
     restoreVersion: () -> Unit = {},

features/cells/src/main/java/com/wire/android/feature/cells/ui/versioning/VersionHistoryViewModel.kt

@@ -156,7 +156,6 @@ class VersionHistoryViewModel @Inject constructor(
             }
     }
 
-    // TODO: Unit test coming in another PR
     fun showRestoreConfirmationDialog(versionId: String) {
         restoreDialogState.value = restoreDialogState.value.copy(
             visible = true,
@@ -166,7 +165,6 @@ class VersionHistoryViewModel @Inject constructor(
         )
     }
 
-    // TODO: Unit test coming in another PR
     fun hideRestoreConfirmationDialog() {
         restoreDialogState.value = restoreDialogState.value.copy(
             restoreVersionState = RestoreVersionState.Idle,
@@ -175,7 +173,6 @@ class VersionHistoryViewModel @Inject constructor(
         )
     }
 
-    // TODO: Unit test coming in another PR
     fun restoreVersion() {
         with(restoreDialogState) {
             restoreDialogState.value = value.copy(
@@ -205,7 +202,6 @@ class VersionHistoryViewModel @Inject constructor(
         }
     }
 
-    // TODO: Unit test coming in another PR
     fun downloadVersion(versionId: String, versionDate: String) {
         viewModelScope.launch {
             downloadState.value = DownloadState.Downloading(0, 0)

features/cells/src/main/java/com/wire/android/feature/cells/util/FileHelper.kt

@@ -22,9 +22,9 @@ import android.content.ContentValues
 import android.content.Context
 import android.content.Intent
 import android.net.Uri
-import android.os.Build
 import android.os.Environment
 import android.provider.MediaStore
+import android.os.Build
 import androidx.core.content.FileProvider
 import dagger.hilt.android.qualifiers.ApplicationContext
 import okio.Path