test: add coverage for signing methods

Author: mikesposito

packages/keyring-eth-qr/jest.config.js

@@ -20,10 +20,10 @@ module.exports = merge(baseConfig, {
   // An object that configures minimum threshold enforcement for coverage results
   coverageThreshold: {
     global: {
-      branches: 46.03,
-      functions: 51.16,
-      lines: 49.51,
-      statements: 50,
+      branches: 68.25,
+      functions: 95.34,
+      lines: 80.28,
+      statements: 80.66,
     },
   },
 });

packages/keyring-eth-qr/package.json

@@ -59,6 +59,7 @@
     "uuid": "^9.0.1"
   },
   "devDependencies": {
+    "@ethereumjs/common": "^4.4.0",
     "@keystonehq/base-eth-keyring": "^0.15.1",
     "@keystonehq/metamask-airgapped-keyring": "^0.15.2",
     "@lavamoat/allow-scripts": "^3.2.1",

packages/keyring-eth-qr/src/qr-keyring-scanner-bridge.test.ts

@@ -0,0 +1,24 @@
+import {
+  type QrScanRequest,
+  QrScanRequestType,
+  QrKeyringScannerBridge,
+} from '.';
+
+describe('QrKeyringScannerBridge', () => {
+  describe('requestScan', () => {
+    it('forwards the request to the constructor hook', async () => {
+      const request: QrScanRequest = {
+        type: QrScanRequestType.PAIR,
+      };
+      const mockRequestScan = jest.fn();
+      const hooks = {
+        requestScan: mockRequestScan,
+      };
+
+      const qrKeyringScannerBridge = new QrKeyringScannerBridge(hooks);
+      await qrKeyringScannerBridge.requestScan(request);
+
+      expect(mockRequestScan).toHaveBeenCalledWith(request);
+    });
+  });
+});