api change

AmmarHalees · Dec 22, 2023 · 09b60c6 · 09b60c6
1 parent 7d522f4
commit 09b60c6
Show file tree

Hide file tree

Showing 4 changed files with 38 additions and 11 deletions.
diff --git a/src/core/validateField.ts b/src/core/validateField.ts
@@ -1,4 +1,11 @@
-import { RulesObject, FieldError, CustomFunction } from "../types";
+import {
+  RulesObject,
+  FieldError,
+  CustomFunction,
+  ConfigMap,
+  ValidationFunction,
+  Criterion,
+} from "../types";
 import _utils from "../utils";
 
 export function validateField(
@@ -8,9 +15,7 @@ export function validateField(
 ): Array<FieldError> {
   const errors: Array<FieldError> = [];
 
-  const configMap: {
-    [key: string]: (value: string, criterion: any, message: string) => void;
-  } = {
+  const configMap: ConfigMap = {
     minLength: (value: string, criterion: number, message: string) => {
       if (!_utils.isString(value))
         throw new Error(
@@ -44,11 +49,24 @@ export function validateField(
     },
   };
 
+  function isKeyOfConfigMap(key: string): key is keyof ConfigMap {
+    return key in configMap;
+  }
+
   try {
     Object.entries(rulesObject).forEach(([key, { criterion, message }]) => {
-      if (configMap[key] && configMap[key] !== undefined) {
-        const defaultFunction = () => {};
-        (configMap[key] || defaultFunction)(value, criterion, message);
+      if (isKeyOfConfigMap(key)) {
+        const validationFunction = configMap[
+          key as keyof ConfigMap
+        ] as ValidationFunction<Criterion>;
+
+        if (validationFunction) {
+          validationFunction(value, criterion, message);
+        } else {
+          throw new Error(
+            `Validation function for ${key} does not exist in configMap`
+          );
+        }
       }
     });
   } catch (e) {

diff --git a/src/core/validateForm.ts b/src/core/validateForm.ts
@@ -1,9 +1,16 @@
 import { doKeysMatch } from "../internal/do-keys-match";
-import { FieldError, NameRuleMap, KeyValuePair, RulesObject } from "../types";
+import {
+  FieldError,
+  NameRuleMap,
+  KeyValuePair,
+  RulesObject,
+  FormDataShape,
+} from "../types";
 import _utils from "../utils";
 import { validateField } from "./validateField";
 
-export function validateForm(data: KeyValuePair | FormData, NameRuleMap: NameRuleMap) {
+
+export function validateForm(data: FormDataShape, NameRuleMap: NameRuleMap) {
   const errors: Array<FieldError> = [];
 
   /*-------- Error Guards --------*/

diff --git a/src/internal/do-keys-match.ts b/src/internal/do-keys-match.ts
@@ -1,6 +1,6 @@
-import { KeyValuePair, NameRuleMap } from "../types";
+import { FormDataShape, NameRuleMap } from "../types";
 
-export function doKeysMatch(data: KeyValuePair | FormData, NameRuleMap: NameRuleMap) {
+export function doKeysMatch(data: FormDataShape, NameRuleMap: NameRuleMap) {
   const dataKeys = Object.keys(data);
   const NameRuleMapKeys = Object.keys(NameRuleMap);
 

diff --git a/src/types/index.ts b/src/types/index.ts
@@ -5,6 +5,8 @@ export interface Rule {
   message: string;
 }
 
+export type FormDataShape = KeyValuePair | { [k: string]: FormDataEntryValue };
+
 export interface RulesObject {
   required?: Rule;
   minLength?: Rule;