@capgo/native-audio 8.2.12 → 8.2.14

package/ios/Tests/NativeAudioPluginTests/PluginTests.swift

@@ -3,16 +3,18 @@ import Capacitor
 import AVFoundation
 @testable import NativeAudioPlugin
 
+// swiftlint:disable file_length
 class PluginTests: XCTestCase {
 
-    var plugin: NativeAudio!
-    var tempFileURL: URL!
+    var plugin = NativeAudio()
+    var tempFileURL = URL(fileURLWithPath: NSTemporaryDirectory().appending("testAudio.wav"))
     var testAssetId = "testAssetId"
     var testRemoteAssetId = "testRemoteAssetId"
 
     override func setUp() {
         super.setUp()
         plugin = NativeAudio()
+        plugin.isRunningTests = true
 
         // Create a temporary audio file for testing
         let audioFilePath = NSTemporaryDirectory().appending("testAudio.wav")
@@ -51,6 +53,18 @@ class PluginTests: XCTestCase {
         FileManager.default.createFile(atPath: path, contents: Data(), attributes: nil)
     }
 
+    private func makeCall(callbackId: String, options: [String: Any], onErrorMessage: String) -> CAPPluginCall {
+        guard let call = CAPPluginCall(callbackId: callbackId, options: options, success: { _, _ in }, error: { _ in }) else {
+            XCTFail("Failed to create CAPPluginCall: \(onErrorMessage)")
+            fatalError("Failed to create CAPPluginCall")
+        }
+        return call
+    }
+
+}
+
+extension PluginTests {
+
     func testAudioAssetInitialization() {
         let expectation = XCTestExpectation(description: "Initialize AudioAsset")
 
@@ -61,8 +75,7 @@ class PluginTests: XCTestCase {
                 withAssetId: self.testAssetId,
                 withPath: self.tempFileURL.path,
                 withChannels: 1,
-                withVolume: 0.5,
-                withFadeDelay: 0.5
+                withVolume: 0.5
             )
 
             // Add it to the plugin's audio list
@@ -71,7 +84,6 @@ class PluginTests: XCTestCase {
             // Verify initial values
             XCTAssertEqual(asset.assetId, self.testAssetId)
             XCTAssertEqual(asset.initialVolume, 0.5)
-            XCTAssertEqual(asset.fadeDelay, 0.5)
 
             expectation.fulfill()
         }
@@ -89,8 +101,7 @@ class PluginTests: XCTestCase {
                 withAssetId: self.testAssetId,
                 withPath: self.tempFileURL.path,
                 withChannels: 1,
-                withVolume: 1.0,
-                withFadeDelay: 0.5
+                withVolume: 1.0
             )
 
             // Add it to the plugin's audio list
@@ -98,7 +109,7 @@ class PluginTests: XCTestCase {
 
             // Test setting volume
             let testVolume: Float = 0.7
-            asset.setVolume(volume: NSNumber(value: testVolume))
+            asset.setVolume(volume: NSNumber(value: testVolume), fadeDuration: 0)
 
             // We can't directly check player.volume as it may take time to set
             // So we'll just verify the method doesn't crash
@@ -123,7 +134,6 @@ class PluginTests: XCTestCase {
                 withPath: testURL,
                 withChannels: 1,
                 withVolume: 0.6,
-                withFadeDelay: 0.3,
                 withHeaders: nil
             )
 
@@ -133,7 +143,6 @@ class PluginTests: XCTestCase {
             // Verify initial values
             XCTAssertEqual(asset.assetId, self.testRemoteAssetId)
             XCTAssertEqual(asset.initialVolume, 0.6)
-            XCTAssertEqual(asset.fadeDelay, 0.3)
             XCTAssertNotNil(asset.asset, "AVURLAsset should be created")
 
             expectation.fulfill()
@@ -144,16 +153,12 @@ class PluginTests: XCTestCase {
 
     func testPluginPreloadMethod() {
         // Create a plugin call to test the preload method
-        let call = CAPPluginCall(callbackId: "test", options: [
+        let call = makeCall(callbackId: "test", options: [
             "assetId": testAssetId,
             "assetPath": tempFileURL.path,
             "volume": 0.8,
             "audioChannelNum": 2
-        ], success: { (_, _) in
-            // Success case
-        }, error: { (_) in
-            XCTFail("Preload shouldn't fail")
-        })!
+        ], onErrorMessage: "Preload call creation")
 
         // Call the plugin method
         plugin.preload(call)
@@ -180,8 +185,7 @@ class PluginTests: XCTestCase {
                 withAssetId: self.testAssetId,
                 withPath: self.tempFileURL.path,
                 withChannels: 1,
-                withVolume: 1.0,
-                withFadeDelay: 0.2
+                withVolume: 1.0
             )
 
             // Test fade functionality (just make sure it doesn't crash)
@@ -227,7 +231,6 @@ class PluginTests: XCTestCase {
                 withPath: testURL,
                 withChannels: 1,
                 withVolume: 0.6,
-                withFadeDelay: 0.3,
                 withHeaders: nil
             )
 
@@ -264,8 +267,7 @@ class PluginTests: XCTestCase {
                 withAssetId: self.testAssetId,
                 withPath: self.tempFileURL.path,
                 withChannels: 1,
-                withVolume: 1.0,
-                withFadeDelay: 0.5
+                withVolume: 1.0
             )
 
             // Ensure the fade timer is nil initially
@@ -315,17 +317,13 @@ class PluginTests: XCTestCase {
         let expectation = XCTestExpectation(description: "PlayOnce with auto-play")
         var returnedAssetId: String?
 
-        let call = CAPPluginCall(callbackId: "test", options: [
+        let call = makeCall(callbackId: "test", options: [
             "assetPath": tempFileURL.path,
             "volume": 1.0,
             "isUrl": true,
             "autoPlay": true
-        ], success: { (result, _) in
-            // Capture the returned assetId
-            returnedAssetId = result?.data?["assetId"] as? String
-        }, error: { (_) in
-            XCTFail("PlayOnce shouldn't fail")
-        })!
+        ], onErrorMessage: "playOnce auto-play call")
+        _ = returnedAssetId
 
         plugin.playOnce(call)
 
@@ -357,16 +355,12 @@ class PluginTests: XCTestCase {
     func testPlayOnceWithoutAutoPlay() {
         let expectation = XCTestExpectation(description: "PlayOnce without auto-play")
 
-        let call = CAPPluginCall(callbackId: "test", options: [
+        let call = makeCall(callbackId: "test", options: [
             "assetPath": tempFileURL.path,
             "volume": 0.8,
             "isUrl": true,
             "autoPlay": false
-        ], success: { (_, _) in
-            // Success case
-        }, error: { (_) in
-            XCTFail("PlayOnce shouldn't fail")
-        })!
+        ], onErrorMessage: "playOnce no-autoplay call")
 
         plugin.playOnce(call)
 
@@ -401,16 +395,12 @@ class PluginTests: XCTestCase {
     func testPlayOnceCleanupAfterCompletion() {
         let expectation = XCTestExpectation(description: "PlayOnce cleanup after completion")
 
-        let call = CAPPluginCall(callbackId: "test", options: [
+        let call = makeCall(callbackId: "test", options: [
             "assetPath": tempFileURL.path,
             "volume": 1.0,
             "isUrl": true,
             "autoPlay": true
-        ], success: { (_, _) in
-            // Success case
-        }, error: { (_) in
-            XCTFail("PlayOnce shouldn't fail")
-        })!
+        ], onErrorMessage: "playOnce cleanup call")
 
         plugin.playOnce(call)
 
@@ -470,17 +460,13 @@ class PluginTests: XCTestCase {
             return
         }
 
-        let call = CAPPluginCall(callbackId: "test", options: [
+        let call = makeCall(callbackId: "test", options: [
             "assetPath": deletableURL.absoluteString,
             "volume": 1.0,
             "isUrl": true,
             "autoPlay": true,
             "deleteAfterPlay": true
-        ], success: { (_, _) in
-            // Success case
-        }, error: { (_) in
-            XCTFail("PlayOnce shouldn't fail")
-        })!
+        ], onErrorMessage: "playOnce delete-after-play call")
 
         plugin.playOnce(call)
 
@@ -518,10 +504,14 @@ class PluginTests: XCTestCase {
         wait(for: [expectation], timeout: 5.0)
     }
 
+}
+
+extension PluginTests {
+
     func testPlayOnceWithNotificationMetadata() {
         let expectation = XCTestExpectation(description: "PlayOnce with notification metadata")
 
-        let call = CAPPluginCall(callbackId: "test", options: [
+        let call = makeCall(callbackId: "test", options: [
             "assetPath": tempFileURL.path,
             "volume": 1.0,
             "isUrl": true,
@@ -532,11 +522,7 @@ class PluginTests: XCTestCase {
                 "album": "Test Album",
                 "artworkUrl": "https://example.com/artwork.jpg"
             ]
-        ], success: { (_, _) in
-            // Success case
-        }, error: { (_) in
-            XCTFail("PlayOnce shouldn't fail")
-        })!
+        ], onErrorMessage: "playOnce metadata call")
 
         plugin.playOnce(call)
 
@@ -573,16 +559,12 @@ class PluginTests: XCTestCase {
         let expectation = XCTestExpectation(description: "PlayOnce error handling and cleanup")
 
         // Use an invalid file path to trigger error
-        let call = CAPPluginCall(callbackId: "test", options: [
+        let call = makeCall(callbackId: "test", options: [
             "assetPath": "/invalid/path/to/nonexistent.wav",
             "volume": 1.0,
             "isUrl": true,
             "autoPlay": true
-        ], success: { (_, _) in
-            XCTFail("Should not succeed with invalid path")
-        }, error: { (_) in
-            // Expected error case
-        })!
+        ], onErrorMessage: "playOnce error handling call")
 
         plugin.playOnce(call)
 
@@ -616,32 +598,24 @@ class PluginTests: XCTestCase {
         var firstAssetId: String?
         var secondAssetId: String?
 
-        let call1 = CAPPluginCall(callbackId: "test1", options: [
+        let call1 = makeCall(callbackId: "test1", options: [
             "assetPath": tempFileURL.path,
             "volume": 1.0,
             "isUrl": true,
             "autoPlay": false
-        ], success: { (result, _) in
-            // Capture returned assetId from public API
-            firstAssetId = result?.data?["assetId"] as? String
-        }, error: { (_) in
-            XCTFail("PlayOnce shouldn't fail")
-        })!
+        ], onErrorMessage: "playOnce unique call1")
+        _ = firstAssetId
 
         plugin.playOnce(call1)
 
         // Create second playOnce
-        let call2 = CAPPluginCall(callbackId: "test2", options: [
+        let call2 = makeCall(callbackId: "test2", options: [
             "assetPath": tempFileURL.path,
             "volume": 1.0,
             "isUrl": true,
             "autoPlay": false
-        ], success: { (result, _) in
-            // Capture returned assetId from public API
-            secondAssetId = result?.data?["assetId"] as? String
-        }, error: { (_) in
-            XCTFail("PlayOnce shouldn't fail")
-        })!
+        ], onErrorMessage: "playOnce unique call2")
+        _ = secondAssetId
 
         plugin.playOnce(call2)
 
@@ -671,3 +645,4 @@ class PluginTests: XCTestCase {
         wait(for: [expectation], timeout: 5.0)
     }
 }
+// swiftlint:enable file_length

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@capgo/native-audio",
-  "version": "8.2.12",
+  "version": "8.2.14",
   "description": "A native plugin for native audio engine",
   "license": "MPL-2.0",
   "main": "dist/plugin.cjs.js",