Default to document directory

onmyway133 · onmyway133 · commit 08ee51c899e1 · 2018-06-13T13:29:26.000+02:00
diff --git a/Source/Shared/Storage/DiskStorage.swift b/Source/Shared/Storage/DiskStorage.swift
@@ -27,7 +27,10 @@ final public class DiskStorage<T> {
       url = directory
     } else {
       url = try fileManager.url(
-        for: .cachesDirectory, in: .userDomainMask, appropriateFor: nil, create: true
+        for: .documentDirectory,
+        in: .userDomainMask,
+        appropriateFor: nil,
+        create: true
       )
     }
 
diff --git a/Tests/iOS/Tests/Storage/DiskStorageTests.swift b/Tests/iOS/Tests/Storage/DiskStorageTests.swift
@@ -30,7 +30,7 @@ final class DiskStorageTests: XCTestCase {
   /// Test that it returns the correct path
   func testDefaultPath() {
     let paths = NSSearchPathForDirectoriesInDomains(
-      .cachesDirectory, FileManager.SearchPathDomainMask.userDomainMask, true
+      .documentDirectory, FileManager.SearchPathDomainMask.userDomainMask, true
     )
     let path = "\(paths.first!)/\(config.name.capitalized)"
     XCTAssertEqual(storage.path, path)

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,10 @@ final public class DiskStorage<T> {`
`27`	`27`	`url = directory`
`28`	`28`	`} else {`
`29`	`29`	`url = try fileManager.url(`
`30`		`- for: .cachesDirectory, in: .userDomainMask, appropriateFor: nil, create: true`
	`30`	`+ for: .documentDirectory,`
	`31`	`+ in: .userDomainMask,`
	`32`	`+ appropriateFor: nil,`
	`33`	`+ create: true`
`31`	`34`	`)`
`32`	`35`	`}`
`33`	`36`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ final class DiskStorageTests: XCTestCase {`
`30`	`30`	`/// Test that it returns the correct path`
`31`	`31`	`func testDefaultPath() {`
`32`	`32`	`let paths = NSSearchPathForDirectoriesInDomains(`
`33`		`- .cachesDirectory, FileManager.SearchPathDomainMask.userDomainMask, true`
	`33`	`+ .documentDirectory, FileManager.SearchPathDomainMask.userDomainMask, true`
`34`	`34`	`)`
`35`	`35`	`let path = "\(paths.first!)/\(config.name.capitalized)"`
`36`	`36`	`XCTAssertEqual(storage.path, path)`