fix: ARC issue for FileManager

CHANGELOG.md

@@ -22,6 +22,7 @@ This version adds a dependency on Swift.
 - Don't capture zero size screenshots ([#2459](https://github.com/getsentry/sentry-cocoa/pull/2459))
 - Use the preexisting app release version format for profiles (#2470)
 - Don't add out of date context for crashes (#2523)
+- Fix ARC issue for FileManager (#2525)
 
 ### Breaking Changes
 

Sources/Sentry/SentryFileManager.m

@@ -77,11 +77,15 @@ - (nullable instancetype)initWithOptions:(SentryOptions *)options
         self.currentFileCounter = 0;
         self.maxEnvelopes = options.maxCacheItems;
 
+        __weak SentryFileManager *weakSelf = self;
         [dispatchQueueWrapper
             dispatchAfter:10
                     block:^{
-                        SENTRY_LOG_DEBUG(@"Dispatched deletion of old envelopes from %@", self);
-                        [self deleteOldEnvelopesFromAllSentryPaths];
+                        if (weakSelf == nil) {
+                            return;
+                        }
+                        SENTRY_LOG_DEBUG(@"Dispatched deletion of old envelopes from %@", weakSelf);
+                        [weakSelf deleteOldEnvelopesFromAllSentryPaths];
                     }];
     }
     return self;

Tests/SentryTests/Helper/SentryFileManagerTests.swift

@@ -136,14 +136,7 @@ class SentryFileManagerTests: XCTestCase {
     func testDeleteOldEnvelopes() throws {
         fixture.dispatchQueueWrapper.dispatchAfterExecutesBlock = true
 
-        let envelope = TestConstants.envelope
-        let path = sut.store(envelope)
-
-        let timeIntervalSince1970 = fixture.currentDateProvider.date().timeIntervalSince1970 - (90 * 24 * 60 * 60)
-        let date = Date(timeIntervalSince1970: timeIntervalSince1970 - 1)
-        try FileManager.default.setAttributes([FileAttributeKey.creationDate: date], ofItemAtPath: path)
-
-        XCTAssertEqual(sut.getAllEnvelopes().count, 1)
+        try givenOldEnvelopes()
 
         sut = fixture.getSut()
 
@@ -181,6 +174,22 @@ class SentryFileManagerTests: XCTestCase {
 
         XCTAssertEqual(sut.getAllEnvelopes().count, 1)
     }
+
+    func testFileManagerDeallocated_OldEnvelopesNotDeleted() throws {
+        try givenOldEnvelopes()
+
+        fixture.dispatchQueueWrapper.dispatchAfterExecutesBlock = false
+
+        // Initialize sut in extra function so ARC deallocates it
+        func getSut() {
+            _ = fixture.getSut()
+        }
+        getSut()
+
+        fixture.dispatchQueueWrapper.invokeLastDispatchAfter()
+
+        XCTAssertEqual(sut.getAllEnvelopes().count, 1)
+    }
 
     func testCreateDirDoesNotThrow() throws {
         try SentryFileManager.createDirectory(atPath: "a")
@@ -634,6 +643,17 @@ class SentryFileManagerTests: XCTestCase {
             XCTFail("Failed to store garbage in Envelopes folder.")
         }
     }
+
+    private func givenOldEnvelopes() throws {
+        let envelope = TestConstants.envelope
+        let path = sut.store(envelope)
+
+        let timeIntervalSince1970 = fixture.currentDateProvider.date().timeIntervalSince1970 - (90 * 24 * 60 * 60)
+        let date = Date(timeIntervalSince1970: timeIntervalSince1970 - 1)
+        try FileManager.default.setAttributes([FileAttributeKey.creationDate: date], ofItemAtPath: path)
+
+        XCTAssertEqual(sut.getAllEnvelopes().count, 1)
+    }
 
     private func storeEvent() {
         do {