@ledgerhq/device-transport-kit-react-native-hid 0.0.0-rn-hid-20250221115747 → 0.0.0-rnble-transport-20250422084848

package/android/src/test/kotlin/com/ledger/devicesdk/shared/androidMainInternal/transport/deviceconnection/DeviceConnectionTest.kt

@@ -0,0 +1,218 @@
+package com.ledger.devicesdk.shared.androidMainInternal.transport.deviceconnection
+
+import com.ledger.devicesdk.shared.api.apdu.SendApduResult
+import com.ledger.devicesdk.shared.internal.service.logger.LogInfo
+import com.ledger.devicesdk.shared.internal.service.logger.LoggerService
+import kotlinx.coroutines.ExperimentalCoroutinesApi
+import kotlinx.coroutines.async
+import kotlinx.coroutines.test.StandardTestDispatcher
+import kotlinx.coroutines.test.advanceTimeBy
+import kotlinx.coroutines.test.runTest
+import kotlin.time.Duration.Companion.seconds
+import kotlin.test.Test
+import org.junit.Assert.*
+import kotlin.time.Duration.Companion.milliseconds
+
+/**
+ * Those tests focus mainly on calling all public methods, and ensuring that the correct apduSender
+ * is used after reconnections.
+ * For full coverage of the possible scenarios, check the unit tests of DeviceConnectionStateMachine.
+ */
+class DeviceConnectionTest {
+    @Test
+    fun `GIVEN a device connection with an APDU sender WHEN requestSendApdu is called THEN the apduSender is used and the result is obtained`() = runTest {
+        val apduSender = MockedApduSender()
+
+        val deviceConnection = DeviceConnection(
+            sessionId = "mockId",
+            deviceApduSender = apduSender,
+            onTerminated = { },
+            isFatalSendApduFailure = { false },
+            reconnectionTimeoutDuration = 5.seconds,
+            loggerService = FakeLoggerService(),
+            coroutineDispatcher = StandardTestDispatcher(testScheduler),
+        )
+
+        // Request sending an APDU
+        deviceConnection.requestSendApdu(mockedApdu)
+
+        // Send APDU should have been called once with the correct apdu
+        assertEquals(1, apduSender.sendCalls.size)
+        assertArrayEquals(mockedApdu, apduSender.sendCalls[0])
+    }
+
+    @Test
+    fun `GIVEN a device connection with an initial APDU sender and an APDU that triggers disconnection WHEN requestSendApdu is called and a reconnection occurs THEN the second apduSender is used and the result is obtained`() =
+        runTest {
+            apdusTriggeringDisconnection.forEach { apduTriggeringDisconnection ->
+                val apduSender1 = MockedApduSender()
+                apduSender1.nextResult = mockedSuccessApduResult
+
+                val deviceConnection = DeviceConnection(
+                    sessionId = "mockId",
+                    deviceApduSender = apduSender1,
+                    onTerminated = { },
+                    isFatalSendApduFailure = { false },
+                    reconnectionTimeoutDuration = 5.seconds,
+                    loggerService = FakeLoggerService(),
+                    coroutineDispatcher = StandardTestDispatcher(testScheduler),
+                )
+
+                // Request sending an apdu
+                val result1 = deviceConnection.requestSendApdu(apduTriggeringDisconnection)
+
+                // apduSender1.sendApdu should have been called once with the correct apdu
+                assertEquals(1, apduSender1.sendCalls.size)
+                assertArrayEquals(apduTriggeringDisconnection, apduSender1.sendCalls[0])
+
+                // The result should have been obtained
+                assertEquals(mockedSuccessApduResult, result1)
+
+                // Request sending a second apdu
+                val result2 = async {
+                    deviceConnection.requestSendApdu(mockedApdu)
+                }
+
+                // apduSender1.sendApdu shouldn't have been called again
+                assertEquals(1, apduSender1.sendCalls.size)
+
+                // Simulate reconnection
+                val apduSender2 = MockedApduSender()
+                apduSender2.nextResult = mockedSuccessApduResult
+                deviceConnection.handleDeviceConnected(apduSender2)
+
+                // The result should have been obtained
+                assertEquals(mockedSuccessApduResult, result2.await())
+
+                // apduSender2.sendApdu should have been called once with the correct apdu
+                assertEquals(1, apduSender2.sendCalls.size)
+                assertArrayEquals(mockedApdu, apduSender2.sendCalls[0])
+            }
+        }
+
+    @OptIn(ExperimentalCoroutinesApi::class)
+    @Test
+    fun `GIVEN a device connection WHEN handleDeviceDisconnected is called and the timeout elapses THEN onTerminated is triggered`() =
+        runTest {
+            var terminated = false
+            val apduSender = MockedApduSender()
+
+            val deviceConnection = DeviceConnection(
+                sessionId = "mockId",
+                deviceApduSender = apduSender,
+                onTerminated = {
+                    terminated = true
+                },
+                isFatalSendApduFailure = { false },
+                reconnectionTimeoutDuration = 5.seconds,
+                loggerService = FakeLoggerService(),
+                coroutineDispatcher = StandardTestDispatcher(testScheduler),
+            )
+
+            // Simulate disconnection
+            deviceConnection.handleDeviceDisconnected()
+
+            // Timeout
+            advanceTimeBy(5.seconds)
+            assertFalse(terminated)
+            advanceTimeBy(1.milliseconds)
+            assertTrue(terminated)
+        }
+
+    @Test
+    fun `GIVEN a disconnected device connection WHEN a device gets reconnected and an APDU is requested THEN the correct apduSender is used and the result is obtained`() =
+        runTest {
+            val apduSender1 = MockedApduSender()
+
+            val deviceConnection = DeviceConnection(
+                sessionId = "mockId",
+                deviceApduSender = apduSender1,
+                onTerminated = { },
+                isFatalSendApduFailure = { false },
+                reconnectionTimeoutDuration = 5.seconds,
+                loggerService = FakeLoggerService(),
+                coroutineDispatcher = StandardTestDispatcher(testScheduler),
+            )
+
+            // Simulate disconnection
+            deviceConnection.handleDeviceDisconnected()
+
+            // Request sending an APDU
+            val result = async {
+                deviceConnection.requestSendApdu(mockedApdu)
+            }
+
+            // Simulate reconnection
+            val apduSender2 = MockedApduSender()
+            deviceConnection.handleDeviceConnected(apduSender2)
+
+            // The result should have been obtained
+            assertEquals(mockedSuccessApduResult, result.await())
+
+            // apduSender1.sendApdu should not have been called
+            assertEquals(0, apduSender1.sendCalls.size)
+
+            // apduSender2.sendApdu should have been called once with the correct apdu
+            assertEquals(1, apduSender2.sendCalls.size)
+            assertArrayEquals(mockedApdu, apduSender2.sendCalls[0])
+        }
+
+    @Test
+    fun `GIVEN a device connection WHEN requestCloseConnection is called THEN onTerminated is triggered`() = runTest {
+        var terminated = false
+
+        val deviceConnection = DeviceConnection(
+            sessionId = "mockId",
+            deviceApduSender = MockedApduSender(),
+            onTerminated = {
+                terminated = true
+            },
+            isFatalSendApduFailure = { false },
+            reconnectionTimeoutDuration = 5.seconds,
+            loggerService = FakeLoggerService(),
+            coroutineDispatcher = StandardTestDispatcher(testScheduler),
+        )
+
+        // Request closing connection
+        deviceConnection.requestCloseConnection()
+
+        // onTerminated should have been called
+        assertTrue(terminated)
+    }
+
+    // Helpers
+    companion object {
+
+        val mockedApdu: ByteArray = byteArrayOf(0x01, 0x02)
+
+        val apdusTriggeringDisconnection: List<ByteArray> = listOf(
+            byteArrayOf(0xe0.toByte(), 0xd8.toByte(), 0x01), // Open app
+            byteArrayOf(0xe0.toByte(), 0xd8.toByte(), 0x01) // Close app
+        )
+
+        val mockedSuccessApduResult =
+            SendApduResult.Success(byteArrayOf(0x05, 0x06, 0x90.toByte(), 0x00))
+
+        class MockedApduSender() : DeviceApduSender<String> {
+            // can be set from the outside for easy mocking
+            var nextResult: SendApduResult = mockedSuccessApduResult
+
+            private val _sendCalls: MutableList<ByteArray> = mutableListOf()
+            // to easily check from the outside the apdus sent
+            val sendCalls: MutableList<ByteArray>
+                get() = _sendCalls
+
+            override suspend fun send(apdu: ByteArray): SendApduResult {
+                _sendCalls += apdu
+                return nextResult
+            }
+
+            override val dependencies: String
+                get() = ""
+        }
+
+        internal class FakeLoggerService : LoggerService {
+            override fun log(info: LogInfo) {}
+        }
+    }
+}

package/lib/cjs/api/bridge/NativeTransportModule.js

@@ -1,2 +1,2 @@
-"use strict";var p=Object.defineProperty;var T=Object.getOwnPropertyDescriptor;var i=Object.getOwnPropertyNames;var d=Object.prototype.hasOwnProperty;var l=(e,o)=>{for(var r in o)p(e,r,{get:o[r],enumerable:!0})},n=(e,o,r,a)=>{if(o&&typeof o=="object"||typeof o=="function")for(let t of i(o))!d.call(e,t)&&t!==r&&p(e,t,{get:()=>o[t],enumerable:!(a=T(o,t))||a.enumerable});return e};var u=e=>n(p({},"__esModule",{value:!0}),e);var m={};l(m,{NativeTransportModule:()=>M});module.exports=u(m);var s=require("react-native");const M=s.NativeModules.RCTTransportHIDModule;0&&(module.exports={NativeTransportModule});
+"use strict";var p=Object.defineProperty;var i=Object.getOwnPropertyDescriptor;var M=Object.getOwnPropertyNames;var T=Object.prototype.hasOwnProperty;var d=(e,o)=>{for(var r in o)p(e,r,{get:o[r],enumerable:!0})},l=(e,o,r,a)=>{if(o&&typeof o=="object"||typeof o=="function")for(let t of M(o))!T.call(e,t)&&t!==r&&p(e,t,{get:()=>o[t],enumerable:!(a=i(o,t))||a.enumerable});return e};var n=e=>l(p({},"__esModule",{value:!0}),e);var m={};d(m,{NativeTransportModule:()=>u});module.exports=n(m);var s=require("react-native");const u=s.NativeModules.LDMKTransportHIDModule;0&&(module.exports={NativeTransportModule});
 //# sourceMappingURL=NativeTransportModule.js.map

package/lib/cjs/api/bridge/NativeTransportModule.js.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../../../../src/api/bridge/NativeTransportModule.ts"],
-  "sourcesContent": ["import { NativeModules } from \"react-native\";\n\nimport type { NativeTransportModuleType } from \"./types\";\n\n// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment\nexport const NativeTransportModule: NativeTransportModuleType =\n  NativeModules[\"RCTTransportHIDModule\"];\n"],
+  "sourcesContent": ["import { NativeModules } from \"react-native\";\n\nimport type { NativeTransportModuleType } from \"./types\";\n\n// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment\nexport const NativeTransportModule: NativeTransportModuleType =\n  NativeModules[\"LDMKTransportHIDModule\"];\n"],
   "mappings": "yaAAA,IAAAA,EAAA,GAAAC,EAAAD,EAAA,2BAAAE,IAAA,eAAAC,EAAAH,GAAA,IAAAI,EAA8B,wBAKvB,MAAMF,EACX,gBAAc",
   "names": ["NativeTransportModule_exports", "__export", "NativeTransportModule", "__toCommonJS", "import_react_native"]
 }

package/lib/cjs/package.json

@@ -19,7 +19,12 @@
   },
   "files": [
     "./android",
-    "./lib"
+    "./lib",
+    "!android/build",
+    "!android/gradle",
+    "!android/gradlew",
+    "!android/gradlew.bat",
+    "!android/local.properties"
   ],
   "scripts": {
     "prebuild": "rimraf lib",
@@ -55,7 +60,7 @@
   },
   "peerDependencies": {
     "@ledgerhq/device-management-kit": "workspace:*",
-    "react-native": ">0.72",
+    "react-native": ">0.74.1",
     "rxjs": "^7.8.1"
   }
 }

package/lib/esm/api/bridge/NativeTransportModule.js

@@ -1,2 +1,2 @@
-import{NativeModules as o}from"react-native";const t=o.RCTTransportHIDModule;export{t as NativeTransportModule};
+import{NativeModules as o}from"react-native";const t=o.LDMKTransportHIDModule;export{t as NativeTransportModule};
 //# sourceMappingURL=NativeTransportModule.js.map

package/lib/esm/api/bridge/NativeTransportModule.js.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../../../../src/api/bridge/NativeTransportModule.ts"],
-  "sourcesContent": ["import { NativeModules } from \"react-native\";\n\nimport type { NativeTransportModuleType } from \"./types\";\n\n// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment\nexport const NativeTransportModule: NativeTransportModuleType =\n  NativeModules[\"RCTTransportHIDModule\"];\n"],
+  "sourcesContent": ["import { NativeModules } from \"react-native\";\n\nimport type { NativeTransportModuleType } from \"./types\";\n\n// eslint-disable-next-line @typescript-eslint/no-unsafe-assignment\nexport const NativeTransportModule: NativeTransportModuleType =\n  NativeModules[\"LDMKTransportHIDModule\"];\n"],
   "mappings": "AAAA,OAAS,iBAAAA,MAAqB,eAKvB,MAAMC,EACXD,EAAc",
   "names": ["NativeModules", "NativeTransportModule"]
 }

package/lib/esm/package.json

@@ -19,7 +19,12 @@
   },
   "files": [
     "./android",
-    "./lib"
+    "./lib",
+    "!android/build",
+    "!android/gradle",
+    "!android/gradlew",
+    "!android/gradlew.bat",
+    "!android/local.properties"
   ],
   "scripts": {
     "prebuild": "rimraf lib",
@@ -55,7 +60,7 @@
   },
   "peerDependencies": {
     "@ledgerhq/device-management-kit": "workspace:*",
-    "react-native": ">0.72",
+    "react-native": ">0.74.1",
     "rxjs": "^7.8.1"
   }
 }

package/lib/types/api/bridge/NativeTransportModule.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"NativeTransportModule.d.ts","sourceRoot":"","sources":["../../../../src/api/bridge/NativeTransportModule.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,yBAAyB,EAAE,MAAM,SAAS,CAAC;AAGzD,eAAO,MAAM,qBAAqB,EAAE,yBACI,CAAC"}
+{"version":3,"file":"NativeTransportModule.d.ts","sourceRoot":"","sources":["../../../../src/api/bridge/NativeTransportModule.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,yBAAyB,EAAE,MAAM,SAAS,CAAC;AAGzD,eAAO,MAAM,qBAAqB,EAAE,yBACK,CAAC"}