test: fix ledger tests

Author: mikesposito

packages/keyring-eth-ledger-bridge/src/ledger-keyring.test.ts

@@ -968,7 +968,7 @@ describe('LedgerKeyring', function () {
           ],
         },
       };
-      const options = { version: 'V4' };
+      const options = { version: sigUtil.SignTypedDataVersion.V4 };
 
       beforeEach(async function () {
         jest
@@ -1043,7 +1043,7 @@ describe('LedgerKeyring', function () {
       it('throws an error if the signTypedData version is not v4', async function () {
         await expect(
           keyring.signTypedData(fakeAccounts[0], fixtureData, {
-            version: 'V3',
+            version: sigUtil.SignTypedDataVersion.V3,
           }),
         ).rejects.toThrow(
           'Ledger: Only version 4 of typed data signing is supported',

packages/keyring-eth-ledger-bridge/src/ledger-keyring.ts

@@ -485,15 +485,16 @@ export class LedgerKeyring implements Keyring {
   }
 
   async signTypedData<
-    Version extends SignTypedDataVersion.V4,
+    Version extends SignTypedDataVersion,
     Types extends MessageTypes,
     Options extends { version?: Version },
   >(
     withAccount: Hex,
     data: TypedMessage<Types>,
-    options: Options,
+    options?: Options,
   ): Promise<string> {
-    const isV4 = options?.version === 'V4';
+    const { version } = options ?? {};
+    const isV4 = version === 'V4';
     if (!isV4) {
       throw new Error(
         'Ledger: Only version 4 of typed data signing is supported',