Fixes some small mistakes

duckduckgo · Apr 15, 2024 · 58caab5 · 58caab5
1 parent c721f8f
commit 58caab5
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/LocalPackages/PixelKit/Sources/PixelKit/PixelKit.swift b/LocalPackages/PixelKit/Sources/PixelKit/PixelKit.swift
@@ -279,12 +279,14 @@ public final class PixelKit {
 
         let newError: Error?
 
-        if let event = event as? PixelKitEventV2 {
+        if let event = event as? PixelKitEventV2,
+           let error = event.error {
+
             // For v2 events we only consider the error specified in the event
             // and purposedly ignore the parameter in this call.
             // This is to encourage moving the error over to the protocol error
             // instead of still relying on the parameter of this call.
-            newError = event.error
+            newError = error
         } else {
             newError = error
         }

diff --git a/LocalPackages/PixelKit/Sources/PixelKit/PixelKitEventV2.swift b/LocalPackages/PixelKit/Sources/PixelKit/PixelKitEventV2.swift
@@ -47,7 +47,6 @@ public protocol PixelFiring {
               frequency: PixelKit.Frequency) async throws -> Bool
 }
 
-
 extension PixelKit: PixelFiring {
     @discardableResult
     public func fire(_ event: PixelKitEventV2) async throws -> Bool {

diff --git a/LocalPackages/PixelKit/Sources/PixelKitTestingUtilities/PixelKitMock.swift b/LocalPackages/PixelKit/Sources/PixelKitTestingUtilities/PixelKitMock.swift
@@ -1,5 +1,5 @@
 //
-//  PixelFireMock.swift
+//  PixelKitMock.swift
 //
 //  Copyright © 2024 DuckDuckGo. All rights reserved.
 //