A bit of a tidy and swift linting

foresightmobile · Sep 19, 2024 · f3b7993 · f3b7993
1 parent 56401d1
commit f3b7993
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 9 deletions.
diff --git a/FlagsmithClient/Classes/Flagsmith.swift b/FlagsmithClient/Classes/Flagsmith.swift
@@ -346,11 +346,8 @@ public final class Flagsmith: @unchecked Sendable {
     }
 
     private func handleSSEResult(_ result: Result<FlagEvent, any Error>) {
-        // Just print out the results for now
         switch result {
         case let .success(event):
-            print("handleSSEResult Received event: \(event)")
-
             // Check whether this event is anything new
             if lastUpdatedAt < event.updatedAt {
                 // Evict everything fron the cache
@@ -360,17 +357,17 @@ public final class Flagsmith: @unchecked Sendable {
                 getFeatureFlags(forIdentity: lastUsedIdentity) { result in
                     switch result {
                     case let .failure(error):
-                        print("Error getting flags in SSE stream: \(error)")
+                        print("Flagsmith - Error getting flags in SSE stream: \(error.localizedDescription)")
 
-                    case let .success(value):
+                    case .success:
                         // On success the flastream is updated automatically in the API call
-                        print("Flags updated from SSE stream: \(value)")
+                        print("Flagsmith - Flags updated from SSE stream.")
                     }
                 }
             }
 
         case let .failure(error):
-            print("handleSSEResult Error in SSE connection: \(error)")
+            print("handleSSEResult Error in SSE connection: \(error.localizedDescription)")
         }
     }
 

diff --git a/FlagsmithClient/Tests/ReconnectionDelayTests.swift b/FlagsmithClient/Tests/ReconnectionDelayTests.swift
@@ -34,4 +34,4 @@ class ReconnectionDelayTests: XCTestCase {
         reconnectionDelay.reset()
         XCTAssertEqual(reconnectionDelay.nextDelay(), 1.0, "After reset, delay should be 1.0 seconds")
     }
-}
+}
diff --git a/FlagsmithClient/Tests/SSEManagerTests.swift b/FlagsmithClient/Tests/SSEManagerTests.swift
@@ -85,7 +85,7 @@ class SSEManagerTests: FlagsmithClientTestCase {
                 requestFinished.fulfill()
             }
 
-            if case .success(_) = result {
+            if case .success = result {
                 XCTFail("Should not have succeeded")
             }
         }
-Original file line number
+Diff line change
@@ Expand Up / @@ -34,4 +34,4 @@ class ReconnectionDelayTests: XCTestCase { @@
             reconnectionDelay.reset()
             XCTAssertEqual(reconnectionDelay.nextDelay(), 1.0, "After reset, delay should be 1.0 seconds")
         }
-    }
+    }