ref: Make allFilesInFolder internal (#3592)

SentryFileManager.allFilesInFolder is only used by the SentryFileManager and can be internal. Furthermore, we can safely remove the test testAllFilesInFolder as that functionality is covered with other tests.
getsentry · Jan 31, 2024 · 13cd5a8 · 13cd5a8
1 parent cff2b8c
commit 13cd5a8
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 7 deletions.
diff --git a/Sources/Sentry/include/SentryFileManager.h b/Sources/Sentry/include/SentryFileManager.h
@@ -68,8 +68,6 @@ SENTRY_NO_INIT
 
 - (void)removeFileAtPath:(NSString *)path;
 
-- (NSArray<NSString *> *)allFilesInFolder:(NSString *)path;
-
 - (void)storeAppState:(SentryAppState *)appState;
 - (void)moveAppStateToPreviousAppState;
 - (SentryAppState *_Nullable)readAppState;

diff --git a/Tests/SentryTests/Helper/SentryFileManagerTests.swift b/Tests/SentryTests/Helper/SentryFileManagerTests.swift
@@ -206,11 +206,6 @@ class SentryFileManagerTests: XCTestCase {
         try SentryFileManager.createDirectory(atPath: "a")
     }
 
-    func testAllFilesInFolder() {
-        let files = sut.allFiles(inFolder: "x")
-        XCTAssertTrue(files.isEmpty)
-    }
-
     func testDeleteFileNotExists() {
         let logOutput = TestLogOutput()
         SentryLog.setLogOutput(logOutput)