Add validation for used clientConfig arguments

src/static/helpers/passwordlessHelper.test.ts

@@ -92,10 +92,43 @@ describe('authorizePasswordless is working', () => {
       },
     };
     expect(authorizePasswordlessCustomerMock).toBeCalledWith(
-      expectedReqOptions, 
+      expectedReqOptions,
       true
     );
   });
+  test('Throw when required parameters missing', async () => {
+    const mockSlasClient = {
+      clientConfig: {
+        parameters: {
+          shortCode: 'short_code',
+          organizationId: 'organization_id',
+          clientId: 'client_id',
+        },
+      },
+      authorizePasswordlessCustomer: authorizePasswordlessCustomerMock,
+      getPasswordLessAccessToken: getPasswordLessAccessTokenMock,
+    } as unknown as ShopperLogin<{
+      shortCode: string;
+      organizationId: string;
+      clientId: string;
+      siteId: string;
+    }>;
+    const credentials = {
+      clientSecret: 'slas_private_secret',
+    };
+    const parameters = {
+      callbackURI: 'www.something.com/callback',
+      usid: 'a_usid',
+      userid: 'a_userid',
+      locale: 'a_locale',
+      mode: 'callback',
+    };
+    await expect(
+      authorizePasswordless(mockSlasClient, credentials, parameters)
+    ).rejects.toThrow(
+      'Required argument channel_id is not provided through clientConfig.parameters.siteId'
+    );
+  });
 });
 
 describe('getPasswordLessAccessToken is working', () => {
@@ -131,4 +164,33 @@ describe('getPasswordLessAccessToken is working', () => {
     };
     expect(getPasswordLessAccessTokenMock).toBeCalledWith(expectedReqOptions);
   });
+  test('Throw when required parameters missing', async () => {
+    const mockSlasClient = {
+      clientConfig: {
+        parameters: {
+          shortCode: 'short_code',
+          clientId: 'client_id',
+        },
+      },
+      authorizePasswordlessCustomer: authorizePasswordlessCustomerMock,
+      getPasswordLessAccessToken: getPasswordLessAccessTokenMock,
+    } as unknown as ShopperLogin<{
+      shortCode: string;
+      organizationId: string;
+      clientId: string;
+      siteId: string;
+    }>;
+    const credentials = {
+      clientSecret: 'slas_private_secret',
+    };
+    const parameters = {
+        pwdlessLoginToken: '123456',
+        dnt: '1',
+      };
+    await expect(
+        getPasswordLessAccessToken(mockSlasClient, credentials, parameters)
+    ).rejects.toThrow(
+      'Required argument organizationId is not provided through clientConfig.parameters.organizationId'
+    );
+  });
 });

src/static/helpers/passwordlessHelper.ts

@@ -39,6 +39,11 @@ export async function authorizePasswordless(
     mode: string;
   }
 ): Promise<Response> {
+  if (!slasClient.clientConfig.parameters.siteId) {
+    throw new Error(
+      'Required argument channel_id is not provided through clientConfig.parameters.siteId'
+    );
+  }
   const authHeaderIdSecret = `Basic ${stringToBase64(
     `${slasClient.clientConfig.parameters.clientId}:${credentials.clientSecret}`
   )}`;
@@ -92,6 +97,11 @@ export async function getPasswordLessAccessToken(
     dnt?: string;
   }
 ): Promise<TokenResponse> {
+  if (!slasClient.clientConfig.parameters.organizationId) {
+    throw new Error(
+      'Required argument organizationId is not provided through clientConfig.parameters.organizationId'
+    );
+  }
   const codeVerifier = createCodeVerifier();
   const authHeaderIdSecret = `Basic ${stringToBase64(
     `${slasClient.clientConfig.parameters.clientId}:${credentials.clientSecret}`