detekt

wireapp · Jan 22, 2025 · 7820b6e · 7820b6e
1 parent 4a4e20d
commit 7820b6e
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 13 deletions.
diff --git a/logic/src/commonMain/kotlin/com/wire/kalium/logic/feature/backup/BackupScope.kt b/logic/src/commonMain/kotlin/com/wire/kalium/logic/feature/backup/BackupScope.kt
@@ -73,10 +73,9 @@ class BackupScope internal constructor(
             restoreWeb
         )
 
-
     @DelicateKaliumApi("this is NOT a backup feature, but a feature to create an unencrypted and obfuscated copy of the database")
-    val createUnEncryptedCopy: CreateUnEncryptedCopyUseCase
-        get() = CreateUnEncryptedCopyUseCase(
+    val createUnEncryptedCopy: CreateObfuscatedCopyUseCase
+        get() = CreateObfuscatedCopyUseCase(
             userId,
             clientIdProvider,
             userRepository,

diff --git a/...re/backup/CreateUnEncryptedCopyUseCase.kt → ...ure/backup/CreateObfuscatedCopyUseCase.kt b/...re/backup/CreateUnEncryptedCopyUseCase.kt → ...ure/backup/CreateObfuscatedCopyUseCase.kt
@@ -51,8 +51,13 @@ import okio.Source
 import okio.buffer
 import okio.use
 
+/**
+ * Creates an obfuscated copy of the user's data.
+ * TO BE USED FOR DEBUGGING PURPOSES ONLY.
+ */
 @DelicateKaliumApi("This class is used for debugging purposes only")
-class CreateUnEncryptedCopyUseCase internal constructor(
+@Suppress("LongParameterList")
+class CreateObfuscatedCopyUseCase internal constructor(
     private val userId: UserId,
     private val clientIdProvider: CurrentClientIdProvider,
     private val userRepository: UserRepository,
@@ -75,13 +80,21 @@ class CreateUnEncryptedCopyUseCase internal constructor(
                 ?: return@withContext CreateBackupResult.Failure(StorageFailure.DataNotFound)
 
         try {
-            createBackupFile(userId, plainDBPath, backupFilePath).fold(
+            createBackupFile(
+                userId,
+                plainDBPath,
+                backupFilePath
+            ).fold(
                 { error -> CreateBackupResult.Failure(error) },
                 { (backupFilePath, backupSize) ->
                     if (password != null) {
                         encryptAndCompressFile(backupFilePath, password)
-                    } else CreateBackupResult.Success(backupFilePath, backupSize, backupFilePath.name)
-                })
+                    } else {
+                        CreateBackupResult.Success(backupFilePath, backupSize, backupFilePath.name)
+                    }
+                }
+
+            )
         } finally {
             obfuscatedCopyExporter.deleteCopyFile()
         }

diff --git a/persistence/src/androidMain/kotlin/com/wire/kalium/persistence/db/UserDatabase.kt b/persistence/src/androidMain/kotlin/com/wire/kalium/persistence/db/UserDatabase.kt
@@ -28,7 +28,6 @@ import com.wire.kalium.persistence.UserDatabase
 import com.wire.kalium.persistence.dao.UserIDEntity
 import com.wire.kalium.persistence.db.support.SupportOpenHelperFactory
 import com.wire.kalium.persistence.util.FileNameUtil
-import com.wire.kalium.util.FileUtil
 import kotlinx.coroutines.CoroutineDispatcher
 import net.zetetic.database.sqlcipher.SQLiteDatabase
 import java.io.File
@@ -137,4 +136,3 @@ internal actual fun createEmptyDatabaseFile(
             it.absolutePath
         }
     }
-
diff --git a/...sistence/backup/ObfiscatedCopyExporter.kt → ...sistence/backup/ObfuscatedCopyExporter.kt b/...sistence/backup/ObfiscatedCopyExporter.kt → ...sistence/backup/ObfuscatedCopyExporter.kt
@@ -83,7 +83,7 @@ class ObfuscatedCopyExporter internal constructor(
         )
     }
 
-    @Suppress("TooGenericExceptionCaught", "ReturnCount")
+    @Suppress("TooGenericExceptionCaught", "LongMethod")
     private suspend fun obfuscatePlainCopy(
         plainDBPath: String,
     ): Boolean = withContext(kaliumDispatcher.io) {
@@ -96,13 +96,15 @@ class ObfuscatedCopyExporter internal constructor(
 
         try {
             plainDBDriver.execute(
-                null, "UPDATE MessageTextContent " +
+                null,
+                "UPDATE MessageTextContent " +
                         "SET text_body = CASE " +
                         "    WHEN text_body IS NOT NULL " +
                         "    THEN substr(hex(randomblob(length(text_body))), 1, length(text_body)) " +
                         "    ELSE NULL " +
                         "END " +
-                        "WHERE text_body IS NOT NULL;", 0
+                        "WHERE text_body IS NOT NULL;",
+                0
             )
 
             plainDBDriver.execute(

diff --git a/persistence/src/jvmMain/kotlin/com/wire/kalium/persistence/db/UserDatabase.kt b/persistence/src/jvmMain/kotlin/com/wire/kalium/persistence/db/UserDatabase.kt
@@ -91,7 +91,6 @@ internal actual fun createEmptyDatabaseFile(
     userId: UserIDEntity,
 ): String? = TODO()
 
-
 /**
  * Creates an in-memory user database,
  * or returns an existing one if it already exists.