Fix media tracking calls not being dispatched on the correct queue wh…

…en tracking using AVPlayer
snowplow · Aug 19, 2024 · f4285f8 · f4285f8
1 parent 20b1fea
commit f4285f8
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/Integrationtests/TestTrackEventsToMicro.swift b/Integrationtests/TestTrackEventsToMicro.swift
@@ -13,6 +13,9 @@
 
 import XCTest
 import SnowplowTracker
+#if !os(watchOS)
+import AVKit
+#endif
 
 class TestTrackEventsToMicro: XCTestCase {
     var tracker: TrackerController?
@@ -166,6 +169,16 @@ class TestTrackEventsToMicro: XCTestCase {
         ], timeout: Micro.timeout)
     }
 
+#if !os(watchOS)
+    func testMediaTrackingUsingAVPlayer() {
+        let player = AVPlayer()
+        let mediaTracking = tracker?.media.startMediaTracking(player: player, configuration: MediaTrackingConfiguration(id: "integration-test"))
+        mediaTracking?.track(MediaReadyEvent())
+
+        wait(for: [Micro.expectCounts(good: 1)], timeout: Micro.timeout)
+    }
+#endif
+
     private func track(_ event: Event) {
         _ = tracker!.track(event)
         tracker!.emitter!.flush()

diff --git a/Sources/Core/InternalQueue/MediaControllerIQWrapper.swift b/Sources/Core/InternalQueue/MediaControllerIQWrapper.swift
@@ -45,7 +45,9 @@ class MediaControllerIQWrapper: MediaController {
 #if !os(watchOS)
     func startMediaTracking(player: AVPlayer,
                             configuration: MediaTrackingConfiguration) -> MediaTracking {
-        return InternalQueue.sync { controller.startMediaTracking(player: player, configuration: configuration) }
+        return InternalQueue.sync {
+            MediaTrackingIQWrapper(tracking: controller.startMediaTracking(player: player, configuration: configuration))
+        }
     }
 #endif