Add specail case when no required condition found

When conditions are added but not "required" then they will generally fail for an undefined value. This should not be the case. In other words, we need a specail exit before running conditions

Author: LMS007

src/conditions.js

@@ -13,6 +13,7 @@ function required(value) {
   return true;
 };
 required.priority = 3; // needed for ordering
+required.hasRequiredCondition = true// special case for shortcoming in validation
 
 /**
  * 

src/tests/validation.test.js

@@ -113,6 +113,7 @@ describe('shared/type-validation', () => {
   });
 
   describe('isNumeric', () => {
+
     it('does not return an error if the value is a number', () => {
       assert.equal(types.isNumeric(1), true);
     });
@@ -196,6 +197,13 @@ describe('shared/type-validation', () => {
         );
       });
 
+      it('returns true if the value is undefined and not required', async () => {
+        assert.equal(
+          await types.isNumeric.and(conditions.range(0, 100))(undefined),
+          true
+        );
+      });
+
       it('can still can be required', async () => {
         assert.equal(await types.isNumeric.and(
           conditions.range(0, 10),
@@ -278,12 +286,19 @@ describe('shared/type-validation', () => {
       assert.equal(result, 'string does not match');
     });
 
+    it('does not return an error if the value is undefined', async () => {
+        const custom = (v)=>(v === "hello world" ? true : "string does not match")
+        const result = await types.isCustom(custom)(undefined)
+        assert.equal(result, true);
+      });
+
     describe('with required', () => {
       it('runs a custom function and passes', async () => {
         const custom = (v)=>(v === "hello world" ? true : "string does not match")
         const result = await types.isCustom(custom).and(conditions.required)("hello world")
         assert.equal(result, true);
       });
+
       it('returns an error if the value is undefined', async () => {
         const custom = (v)=>(v === "hello world" ? true : "string does not match")
         const result = await types.isCustom(custom).and(conditions.required)(undefined)

src/utils.js

@@ -6,11 +6,18 @@ function attachOptions(shim /*, validOptions*/) {
   let result;
   shim.and = (...options) => {
     const validate = async (value, ...args) => {
+      
       result = await shim(value, ...args)
       if (result !== true) {
         return result
       }
 
+      let required = options.filter(v=>v && v.hasRequiredCondition == true);
+      if(required.length == 0 && value === undefined) {
+        // special case for undefined values so they avoid running further conditions.
+        return true;
+      }
+
       // ensures high priority conditions are ran first
       // so we resolve things like required before nested objects
       options.sort((v1, v2)=>{