From c01fd8466b633c103661652e00e90678f4d9e287 Mon Sep 17 00:00:00 2001 From: Matthew Miller Date: Sat, 19 Nov 2022 23:54:08 -0800 Subject: Fix issues in Browser due to TypeScript upgrade --- packages/browser/src/helpers/webAuthnAbortService.ts | 6 ++++-- packages/browser/src/methods/startAuthentication.test.ts | 3 +++ packages/browser/src/methods/startRegistration.test.ts | 7 +++++++ 3 files changed, 14 insertions(+), 2 deletions(-) (limited to 'packages/browser/src') diff --git a/packages/browser/src/helpers/webAuthnAbortService.ts b/packages/browser/src/helpers/webAuthnAbortService.ts index 479df55..8955526 100644 --- a/packages/browser/src/helpers/webAuthnAbortService.ts +++ b/packages/browser/src/helpers/webAuthnAbortService.ts @@ -15,8 +15,10 @@ class WebAuthnAbortService { this.controller.abort('Cancelling existing WebAuthn API call for new one'); } - this.controller = new AbortController(); - return this.controller.signal; + const newController = new AbortController(); + + this.controller = newController; + return newController.signal; } } diff --git a/packages/browser/src/methods/startAuthentication.test.ts b/packages/browser/src/methods/startAuthentication.test.ts index e112124..9ebdd77 100644 --- a/packages/browser/src/methods/startAuthentication.test.ts +++ b/packages/browser/src/methods/startAuthentication.test.ts @@ -116,6 +116,7 @@ test('should return base64url-encoded response values', async () => { }, getClientExtensionResults: () => ({}), type: 'webauthn.get', + authenticatorAttachment: '', }); }); }); @@ -163,7 +164,9 @@ test('should send extensions to authenticator if present in options', async () = const extensions: AuthenticationExtensionsClientInputs = { credProps: true, appid: 'appidHere', + // @ts-ignore uvm: true, + // @ts-ignore appidExclude: 'appidExcludeHere', }; const optsWithExts: PublicKeyCredentialRequestOptionsJSON = { diff --git a/packages/browser/src/methods/startRegistration.test.ts b/packages/browser/src/methods/startRegistration.test.ts index 2dc11be..38ea52e 100644 --- a/packages/browser/src/methods/startRegistration.test.ts +++ b/packages/browser/src/methods/startRegistration.test.ts @@ -96,9 +96,14 @@ test('should return base64url-encoded response values', async () => { response: { attestationObject: Buffer.from(mockAttestationObject, 'ascii'), clientDataJSON: Buffer.from(mockClientDataJSON, 'ascii'), + getTransports: () => [], + getAuthenticatorData: () => new Uint8Array(), + getPublicKey: () => null, + getPublicKeyAlgorithm: () => -999, }, getClientExtensionResults: () => ({}), type: 'webauthn.create', + authenticatorAttachment: '', }); }); }); @@ -132,7 +137,9 @@ test('should send extensions to authenticator if present in options', async () = const extensions: AuthenticationExtensionsClientInputs = { credProps: true, appid: 'appidHere', + // @ts-ignore uvm: true, + // @ts-ignore appidExclude: 'appidExcludeHere', }; const optsWithExts: PublicKeyCredentialCreationOptionsJSON = { -- cgit v1.2.3