validateParams: add type validation

Author: xr0master

src/utils/validateParams/validateParams.spec.ts

@@ -1,22 +1,46 @@
-import { it, expect } from '@jest/globals';
+import { it, expect, describe } from '@jest/globals';
 import { validateParams } from './validateParams';
 
-it('should fail on the public key', () => {
-  expect(() => validateParams('', 'default_service', 'my_test_template')).toThrow(
-    'The public key is required',
-  );
+describe('should fail on the public key', () => {
+  it('no key', () => {
+    expect(() => validateParams('', 'default_service', 'my_test_template')).toThrow(
+      'The public key is required',
+    );
+  });
+
+  it('invalida type', () => {
+    expect(() => validateParams({}, 'default_service', 'my_test_template')).toThrow(
+      'The public key is required',
+    );
+  });
 });
 
-it('should fail on the service ID', () => {
-  expect(() => validateParams('d2JWGTestKeySomething', '', 'my_test_template')).toThrow(
-    'The service ID is required',
-  );
+describe('should fail on the service ID', () => {
+  it('no key', () => {
+    expect(() => validateParams('d2JWGTestKeySomething', '', 'my_test_template')).toThrow(
+      'The service ID is required',
+    );
+  });
+
+  it('invalida type', () => {
+    expect(() => validateParams('d2JWGTestKeySomething', [], 'my_test_template')).toThrow(
+      'The service ID is required',
+    );
+  });
 });
 
-it('should fail on the template ID', () => {
-  expect(() => validateParams('d2JWGTestKeySomething', 'default_service', '')).toThrow(
-    'The template ID is required',
-  );
+describe('should fail on the template ID', () => {
+  it('no key', () => {
+    expect(() => validateParams('d2JWGTestKeySomething', 'default_service', '')).toThrow(
+      'The template ID is required',
+    );
+  });
+
+  it('invalida type', () => {
+    expect(() => validateParams('d2JWGTestKeySomething', 'default_service', 3)).toThrow(
+      'The template ID is required',
+    );
+  });
 });
 
 it('should successfully pass the validation', () => {

src/utils/validateParams/validateParams.ts

@@ -1,13 +1,13 @@
-export const validateParams = (publicKey?: string, serviceID?: string, templateID?: string) => {
-  if (!publicKey) {
+export const validateParams = (publicKey?: unknown, serviceID?: unknown, templateID?: unknown) => {
+  if (!publicKey || typeof publicKey !== 'string') {
     throw 'The public key is required. Visit https://dashboard.emailjs.com/admin/account';
   }
 
-  if (!serviceID) {
+  if (!serviceID || typeof serviceID !== 'string') {
     throw 'The service ID is required. Visit https://dashboard.emailjs.com/admin';
   }
 
-  if (!templateID) {
+  if (!templateID || typeof templateID !== 'string') {
     throw 'The template ID is required. Visit https://dashboard.emailjs.com/admin/templates';
   }
 };