PM-14805: Ensure results cannot be double wrapped from 'asSuccess' (#…

…4283)

app/src/main/java/com/x8bit/bitwarden/data/platform/util/ResultExtensions.kt

@@ -14,9 +14,16 @@ inline fun <T, R> Result<T>.flatMap(transform: (T) -> Result<R>): Result<R> =
 
 /**
  * Returns the given receiver of type [T] as a "success" [Result].
+ *
+ * Note that this will never double wrap the `Result` and we return the original value if [T] is
+ * already an instance of `Result`
  */
-fun <T> T.asSuccess(): Result<T> =
+fun <T> T.asSuccess(): Result<T> = if (this is Result<*>) {
+    @Suppress("UNCHECKED_CAST")
+    this as Result<T>
+} else {
     Result.success(this)
+}
 
 /**
  * Returns the given [Throwable] as a "failure" [Result].

app/src/test/java/com/x8bit/bitwarden/data/platform/util/ResultTest.kt

@@ -59,6 +59,14 @@ class ResultTest {
         )
     }
 
+    @Test
+    fun `asSuccess returns a success Result with the correct content that is not double-wrapped`() {
+        assertEquals(
+            Result.success("Test"),
+            "Test".asSuccess().asSuccess(),
+        )
+    }
+
     @Test
     fun `asFailure returns a failure Result with the correct content`() {
         val throwable = IllegalStateException("Test")