Fix FileNotFoundException when storing segment values

Author: romtsn

sentry-android-replay/src/main/java/io/sentry/android/replay/ReplayCache.kt

@@ -55,7 +55,7 @@ public class ReplayCache(
     internal val frames = mutableListOf<ReplayFrame>()
 
     private val ongoingSegment = LinkedHashMap<String, String>()
-    private val ongoingSegmentFile: File? by lazy {
+    internal val ongoingSegmentFile: File? by lazy {
         if (replayCacheDir == null) {
             return@lazy null
         }
@@ -275,6 +275,9 @@ public class ReplayCache(
             if (isClosed.get()) {
                 return
             }
+            if (ongoingSegmentFile?.exists() != true) {
+                ongoingSegmentFile?.createNewFile()
+            }
             if (ongoingSegment.isEmpty()) {
                 ongoingSegmentFile?.useLines { lines ->
                     lines.associateTo(ongoingSegment) {

sentry-android-replay/src/main/java/io/sentry/android/replay/capture/BaseCaptureStrategy.kt

@@ -5,6 +5,7 @@ import android.view.MotionEvent
 import io.sentry.Breadcrumb
 import io.sentry.DateUtils
 import io.sentry.IScopes
+import io.sentry.SentryLevel.ERROR
 import io.sentry.SentryOptions
 import io.sentry.SentryReplayEvent.ReplayType
 import io.sentry.SentryReplayEvent.ReplayType.BUFFER
@@ -183,7 +184,11 @@ internal abstract class BaseCaptureStrategy(
                         task()
                     }
                 } else {
-                    task()
+                    try {
+                        task()
+                    } catch (e: Throwable) {
+                        options.logger.log(ERROR, "Failed to execute task $TAG.runInBackground", e)
+                    }
                 }
             }
 

sentry-android-replay/src/test/java/io/sentry/android/replay/ReplayCacheTest.kt

@@ -285,6 +285,21 @@ class ReplayCacheTest {
         assertFalse(File(replayCache.replayCacheDir, ONGOING_SEGMENT).exists())
     }
 
+    @Test
+    fun `when file does not exist upon persisting creates it`() {
+        val replayId = SentryId()
+        val replayCache = fixture.getSut(
+            tmpDir,
+            replayId
+        )
+
+        replayCache.ongoingSegmentFile?.delete()
+
+        replayCache.persistSegmentValues("key", "value")
+        val segmentValues = File(replayCache.replayCacheDir, ONGOING_SEGMENT).readLines()
+        assertEquals("key=value", segmentValues[0])
+    }
+
     @Test
     fun `stores segment key value pairs`() {
         val replayId = SentryId()