fix(validation): correctly validate empty not required data

ui/src/util/Validator.js

@@ -249,6 +249,12 @@ class Validator {
      */
     // Validate the range of numeric field
     static NumberValidator(field, label, validator, data) {
+        // this validation should be before this function but adding it
+        // here to avoid any errors until this module is moved to ts
+        if (data === null || data === undefined) {
+            return false;
+        }
+
         const { error } = parseNumberValidator(validator.range);
         if (error) {
             return { errorField: field, errorMsg: error };

ui/src/util/tests/validators.test.ts

@@ -193,6 +193,7 @@ describe('Validator.doValidation - number case', () => {
         {
             field: 'testField',
             label: 'Test Field',
+            type: 'text',
             validators: [
                 {
                     type: 'number',
@@ -210,6 +211,24 @@ describe('Validator.doValidation - number case', () => {
         expect(result).toBe(false);
     });
 
+    it.each([undefined, null])('should return false validation for %s when optional', (value) => {
+        const validator = new Validator(entities);
+        const data = { testField: value };
+        const result = validator.doValidation(data);
+        expect(result).toBe(false);
+    });
+
+    it.each([undefined, null])('should return error for %s number when required', (value) => {
+        const requiredEntities = [{ ...entities[0], required: true }];
+        const validator = new Validator(requiredEntities);
+        const data = { testField: value };
+        const result = validator.doValidation(data);
+        expect(result).toEqual({
+            errorField: 'testField',
+            errorMsg: 'Field Test Field is required',
+        });
+    });
+
     it('should return an error for number out of range', () => {
         const validator = new Validator(entities);
         const data = { testField: 15 };