fix: Workaround FileNotFoundException on Android when recovering sessions

src/Sentry/GlobalSessionManager.cs

@@ -151,6 +151,12 @@
         }
 
         var filePath = Path.Combine(_persistenceDirectoryPath, PersistedSessionFileName);
+        if (!_options.FileSystem.FileExists(filePath))
+        {
+            _options.LogDebug("A persisted session file was not found at '{0}'.", filePath);
+            return null;
+        }
+
         try
         {
             var recoveredUpdate = _persistedSessionProvider(filePath);
@@ -195,15 +201,9 @@
 
             return sessionUpdate;
         }
-        catch (Exception ex) when (ex is FileNotFoundException or DirectoryNotFoundException)
-        {
-            // Not a notable error
-            _options.LogDebug("A persisted session does not exist ({0}) at {1}.", ex.GetType().Name, filePath);
-            return null;
-        }
         catch (Exception ex)
         {
             _options.LogError(ex, "Failed to recover persisted session from the file system '{0}'.", filePath);
 
             return null;
         }

test/Sentry.Tests/GlobalSessionManagerTests.cs

@@ -261,45 +261,28 @@
     }
 
     [Fact]
-    public void TryRecoverPersistedSession_FileNotFoundException_LogDebug()
+    public void TryRecoverPersistedSession_NoSessionFile_LogDebug()
     {
         // Arrange
+        _fixture.PersistedSessionProvider = _ => throw new FileNotFoundException();
         var sut = _fixture.GetSut();
-        sut = new GlobalSessionManager(
-            _fixture.Options,
-            persistedSessionProvider: _ => throw new FileNotFoundException());
 
         // Act
         sut.TryRecoverPersistedSession();
 
         // Assert
-        _fixture.Logger.Entries.Should().Contain(e => e.Level == SentryLevel.Debug);
-    }
-
-    [Fact]
-    public void TryRecoverPersistedSession_DirectoryNotFoundException_LogDebug()
-    {
-        // Arrange
-        var sut = _fixture.GetSut();
-        sut = new GlobalSessionManager(
-            _fixture.Options,
-            persistedSessionProvider: _ => throw new DirectoryNotFoundException());
-
-        // Act
-        sut.TryRecoverPersistedSession();
-
-        // Assert
-        _fixture.Logger.Entries.Should().Contain(e => e.Level == SentryLevel.Debug);
+        _fixture.Logger.Entries.Should().Contain(e =>
+            e.Level == SentryLevel.Debug
+            && e.Message.Contains("A persisted session file was not found"));
     }
 
     [Fact]
     public void TryRecoverPersistedSession_EndOfStreamException_LogError()
     {
         // Arrange
+        _fixture.PersistedSessionProvider = _ => throw new EndOfStreamException();
         var sut = _fixture.GetSut();
-        sut = new GlobalSessionManager(
-            _fixture.Options,
-            persistedSessionProvider: _ => throw new EndOfStreamException());
+        sut.StartSession();
 
         // Act
         sut.TryRecoverPersistedSession();
@@ -383,14 +366,14 @@
     public void TryRecoverPersistedSession_CrashDelegateReturnsTrueWithPauseTimestamp_EndsAsCrashed()
     {
         // Arrange
-        _fixture.Options.CrashedLastRun = () => true;
-        // Session was paused before persisted:
-        var pausedTimestamp = DateTimeOffset.Now;
+        var pausedTimestamp = DateTimeOffset.Now; // Session was paused before persisted:
         _fixture.PersistedSessionProvider = _ => new PersistedSessionUpdate(
             AnySessionUpdate(),
             pausedTimestamp);
+        _fixture.Options.CrashedLastRun = () => true;
 
         var sut = _fixture.GetSut();
+        sut.StartSession();
 
         // Act
         var persistedSessionUpdate = sut.TryRecoverPersistedSession();
@@ -412,6 +395,7 @@
             pausedTimestamp);
 
         var sut = _fixture.GetSut();
+        sut.StartSession();
 
         // Act
         var persistedSessionUpdate = sut.TryRecoverPersistedSession();
@@ -573,8 +557,8 @@
         var persistedSessionUpdate = sut.TryRecoverPersistedSession();
 
         // Assert
         persistedSessionUpdate.Should().NotBeNull();
         persistedSessionUpdate!.EndStatus.Should().Be(SessionEndStatus.Unhandled);
     }
 
     [Fact]
@@ -593,7 +577,7 @@
         var persistedSessionUpdate = sut.TryRecoverPersistedSession();
 
         // Assert
         persistedSessionUpdate.Should().NotBeNull();
         persistedSessionUpdate!.EndStatus.Should().Be(SessionEndStatus.Crashed);
     }
 
@@ -615,7 +599,7 @@
 
         // Assert
         // Crash takes priority over all other end statuses
         persistedSessionUpdate.Should().NotBeNull();
         persistedSessionUpdate!.EndStatus.Should().Be(SessionEndStatus.Crashed);
     }