use keychain accessibility

ski-u · Dec 14, 2024 · 58e0db5 · 58e0db5
1 parent bbb0f88
commit 58e0db5
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 13 deletions.
diff --git a/Features/Sources/APIKeyClient/APIKeyClient.swift b/Features/Sources/APIKeyClient/APIKeyClient.swift
@@ -4,8 +4,8 @@ import Models
 
 @DependencyClient
 public struct APIKeyClient: Sendable {
-    public var getKey: @Sendable () throws -> APIKey?
-    public var setKey: @Sendable (APIKey?) throws -> Void
+    public var getKey: @Sendable () -> APIKey?
+    public var setKey: @Sendable (APIKey?) -> Void
 }
 
 extension APIKeyClient: TestDependencyKey {

diff --git a/Features/Sources/APIKeyClientLive/LiveKey.swift b/Features/Sources/APIKeyClientLive/LiveKey.swift
@@ -11,14 +11,10 @@ extension APIKeyClient: DependencyKey {
 
         return .init(
             getKey: {
-                try keychain.get("APIKey").map(APIKey.init(rawValue:))
+                keychain["APIKey"].map(APIKey.init(rawValue:))
             },
             setKey: {
-                if let key = $0 {
-                    try keychain.set(key.rawValue, key: "APIKey")
-                } else {
-                    try keychain.remove("APIKey")
-                }
+                keychain.accessibility(.whenUnlockedThisDeviceOnly)["APIKey"] = $0?.rawValue
             }
         )
     }

diff --git a/NASAClientTests/APIKeyClientLive/APIKeyClientLiveTests.swift b/NASAClientTests/APIKeyClientLive/APIKeyClientLiveTests.swift
@@ -12,14 +12,14 @@ struct APIKeyClientLiveTests {
     }
 
     @Test
-    func liveValue() throws {
+    func liveValue() {
         let client = APIKeyClient.liveValue(service: "test")
         let key = APIKey(rawValue: "key")
 
-        try client.setKey(key)
-        #expect(try client.getKey() == key)
+        client.setKey(key)
+        #expect(client.getKey() == key)
 
-        try client.setKey(nil)
-        #expect(try client.getKey() == nil)
+        client.setKey(nil)
+        #expect(client.getKey() == nil)
     }
 }