isBlockedValue: fix name & add type validation

Author: xr0master

src/methods/send/send.ts

@@ -8,7 +8,7 @@ import { validateParams } from '../../utils/validateParams/validateParams';
 import { validateTemplateParams } from '../../utils/validateTemplateParams/validateTemplateParams';
 import { isHeadless } from '../../utils/isHeadless/isHeadless';
 import { headlessError } from '../../errors/headlessError/headlessError';
-import { isBlockedValueInParams } from '../../utils/isBlockedValue/isBlockedValue';
+import { isBlockedValueInParams } from '../../utils/isBlockedValueInParams/isBlockedValueInParams';
 import { blockedEmailError } from '../../errors/blockedEmailError/blockedEmailError';
 import { isLimitRateHit } from '../../utils/isLimitRateHit/isLimitRateHit';
 import { limitRateError } from '../../errors/limitRateError/limitRateError';

src/methods/sendForm/sendForm.ts

@@ -8,7 +8,7 @@ import { validateForm } from '../../utils/validateForm/validateForm';
 import { validateParams } from '../../utils/validateParams/validateParams';
 import { isHeadless } from '../../utils/isHeadless/isHeadless';
 import { headlessError } from '../../errors/headlessError/headlessError';
-import { isBlockedValueInParams } from '../../utils/isBlockedValue/isBlockedValue';
+import { isBlockedValueInParams } from '../../utils/isBlockedValueInParams/isBlockedValueInParams';
 import { blockedEmailError } from '../../errors/blockedEmailError/blockedEmailError';
 import { isLimitRateHit } from '../../utils/isLimitRateHit/isLimitRateHit';
 import { limitRateError } from '../../errors/limitRateError/limitRateError';

src/utils/isBlockedValue/isBlockedValue.spec.ts renamed to src/utils/isBlockedValueInParams/isBlockedValueInParams.spec.ts

@@ -1,5 +1,5 @@
 import { it, describe, expect } from '@jest/globals';
-import { isBlockedValueInParams } from './isBlockedValue';
+import { isBlockedValueInParams } from './isBlockedValueInParams';
 import type { BlockList } from '../../types/BlockList';
 
 describe('should be disabled', () => {

src/utils/isBlockedValue/isBlockedValue.ts renamed to src/utils/isBlockedValueInParams/isBlockedValueInParams.ts

@@ -1,4 +1,5 @@
 import type { BlockList } from '../../types/BlockList';
+import { validateBlockListParams } from '../validateBlockListParams/validateBlockListParams';
 
 const isBlockListDisabled = (options: BlockList): boolean => {
   return !options.list?.length || !options.watchVariable;
@@ -14,6 +15,8 @@ export const isBlockedValueInParams = (
 ): boolean => {
   if (isBlockListDisabled(options)) return false;
 
+  validateBlockListParams(options.list, options.watchVariable);
+
   const value = getValue(params, options.watchVariable!);
 
   if (typeof value !== 'string') return false;

src/utils/validateBlockListParams/validateBlockListParams.spec.ts

@@ -0,0 +1,18 @@
+import { it, expect } from '@jest/globals';
+import { validateBlockListParams } from './validateBlockListParams';
+
+it('should fail on list is invalid type', () => {
+  expect(() => validateBlockListParams('100', 'send')).toThrow(
+    'The BlockList list has to be an array',
+  );
+});
+
+it('should fail on watchVariable is invalid type', () => {
+  expect(() => validateBlockListParams([], [])).toThrow(
+    'The BlockList watchVariable has to be a string',
+  );
+});
+
+it('should successfully pass the validation', () => {
+  expect(() => validateBlockListParams([], 'send')).not.toThrow();
+});

src/utils/validateBlockListParams/validateBlockListParams.ts

@@ -0,0 +1,9 @@
+export const validateBlockListParams = (list: unknown, watchVariable: unknown) => {
+  if (!Array.isArray(list)) {
+    throw 'The BlockList list has to be an array';
+  }
+
+  if (typeof watchVariable !== 'string') {
+    throw 'The BlockList watchVariable has to be a string';
+  }
+};