Functions refactored for defensive coding

jest.config.js

@@ -24,6 +24,7 @@ const config = {
     'react-markdown': '<rootDir>/__mocks__/react-markdown.tsx',
     '^dexie$': require.resolve('dexie'),
   },
+  testEnvironment: 'jsdom',
 };
 
 module.exports = config;

packages/esm-form-render-app/src/form-validator.test.ts

@@ -0,0 +1,11 @@
+/** @jest-environment jsdom */
+
+import { handleFormValidation } from './form-validator';
+
+describe('handleFormValidation', () => {
+  it('should fail when no schema is passed to the function', async () => {
+    await expect(() => handleFormValidation(null, null)).rejects.toThrow(
+      'Invalid argument: "schema" cannot be null, undefined or an empty object. Please provide a valid object.',
+    );
+  });
+});

packages/esm-form-render-app/src/form-validator.ts

@@ -1,39 +1,62 @@
 import { openmrsFetch } from '@openmrs/esm-framework';
+import { OHRIFormSchema } from '@openmrs/openmrs-form-engine-lib';
 import { async } from 'rxjs';
 
 export const handleFormValidation = async (schema, configObject) => {
   const errors = [];
   const warnings = [];
 
-  if (schema) {
-    const parsedForm = typeof schema == 'string' ? JSON.parse(schema) : schema;
+  if (!schema || !Object.keys(schema)?.length) {
+    throw new Error(
+      'Invalid argument: "schema" cannot be null, undefined or an empty object. Please provide a valid object.',
+    );
+  }
 
-    const asyncTasks = [];
+  if (!configObject || !Object.keys(configObject)?.length) {
+    throw new Error(
+      'Invalid argument: "configObject" cannot be null, undefined or an empty object. Please provide a valid object.',
+    );
+  }
 
-    parsedForm.pages?.forEach((page) =>
-      page.sections?.forEach((section) =>
-        section.questions?.forEach((question) => {
-          asyncTasks.push(
-            handleQuestionValidation(question, errors, configObject, warnings),
-            handleAnswerValidation(question, errors, warnings),
+  const parsedForm = typeof schema == 'string' ? JSON.parse(schema) : schema;
+
+  const asyncTasks = [];
+
+  parsedForm.pages?.forEach((page) =>
+    page.sections?.forEach((section) =>
+      section.questions?.forEach((question) => {
+        asyncTasks.push(
+          handleQuestionValidation(question, errors, configObject, warnings),
+          handleAnswerValidation(question, errors, warnings),
+        );
+        question.type === 'obsGroup' &&
+          question.questions?.forEach((obsGrpQuestion) =>
+            asyncTasks.push(
+              handleQuestionValidation(obsGrpQuestion, errors, configObject, warnings),
+              handleAnswerValidation(question, errors, warnings),
+            ),
           );
-          question.type === 'obsGroup' &&
-            question.questions?.forEach((obsGrpQuestion) =>
-              asyncTasks.push(
-                handleQuestionValidation(obsGrpQuestion, errors, configObject, warnings),
-                handleAnswerValidation(question, errors, warnings),
-              ),
-            );
-        }),
-      ),
-    );
-    await Promise.all(asyncTasks);
+      }),
+    ),
+  );
+  await Promise.all(asyncTasks);
 
-    return [errors, warnings];
-  }
+  return [errors, warnings];
 };
 
 const handleQuestionValidation = async (conceptObject, errorsArray, configObject, warningsArray) => {
+  if (!configObject || !Object.keys(configObject)?.length) {
+    throw new Error(
+      'Invalid argument: "configObject" cannot be null, undefined or an empty object. Please provide a valid object.',
+    );
+  }
+
+  if (!conceptObject || !Object.keys(conceptObject)?.length) {
+    throw new Error(
+      'Invalid argument: "conceptObject" cannot be null, undefined or an empty object. Please provide a valid object.',
+    );
+  }
+
   const conceptRepresentation =
     'custom:(uuid,display,datatype,answers,conceptMappings:(conceptReferenceTerm:(conceptSource:(name),code)))';