Fix CLDVideoPlayer duration for analytics

Author: adimiz1

Cloudinary/Classes/ios/Video/CLDVideoPlayer.swift

@@ -217,9 +217,17 @@ extension CLDVideoPlayer {
                 switch status {
                 case .loaded:
                     let duration = asset.duration
-                    let durationInSeconds = Int(CMTimeGetSeconds(duration))
+                    var durationInSeconds = 0
 
-                    if !self.loadMetadataSent && durationInSeconds > 0 && duration.isValid {
+                    if duration.isValid && duration.isNumeric && !duration.isIndefinite {
+                        let durationSeconds = CMTimeGetSeconds(duration)
+                        
+                        if durationSeconds.isFinite && !durationSeconds.isNaN {
+                            durationInSeconds = Int(durationSeconds)
+                        }
+                    }
+                    
+                    if !self.loadMetadataSent {
                         self.loadMetadataSent = true
                         self.eventsManager.sendLoadMetadataEvent(duration: durationInSeconds)
                     }

Example/Tests/UIExtensions/CLDVideoPlayerTests.swift

@@ -71,4 +71,46 @@ class CLDVideoPlayerTests: UIBaseTest {
 
         wait(for: [expectation], timeout: 5.0)
     }
+    
+    func testEventsManagerReceivesDurationValues() {
+        let eventsManager = VideoEventsManager()
+        
+        XCTAssertNoThrow({
+            eventsManager.sendLoadMetadataEvent(duration: 0)
+        }, "Sending duration 0 should not crash")
+        
+        XCTAssertNoThrow({
+            eventsManager.sendLoadMetadataEvent(duration: 10)
+        }, "Sending valid duration should not crash")
+        
+        XCTAssertNoThrow({
+            eventsManager.sendLoadMetadataEvent(duration: -1)
+        }, "Events manager should handle any integer value")
+        
+        XCTAssertNoThrow({
+            eventsManager.sendLoadMetadataEvent(duration: Int.max)
+        }, "Events manager should handle large values")
+    }
+    
+    func testDurationHandlingFlowIntegration() {
+        let expectation = XCTestExpectation(description: "Duration flow should complete")
+        
+        let player = CLDVideoPlayer(url: "data:,")
+        
+        DispatchQueue.main.asyncAfter(deadline: .now() + 1.0) {
+            XCTAssertNoThrow({
+                _ = player.currentItem
+                _ = player.status
+            }, "Accessing player properties should not crash")
+            
+            XCTAssertNoThrow({
+                player.play()
+                player.pause()
+            }, "Player methods should work after problematic URL")
+            
+            expectation.fulfill()
+        }
+        
+        wait(for: [expectation], timeout: 2.0)
+    }
 }