FIO-8731: Add test case to check that validation is not touching hidden fields

src/process/__tests__/fixtures/index.ts

@@ -0,0 +1,6 @@
+import skipValidForConditionallyHiddenComp from './skipValidForConditionallyHiddenComp.json';
+import data1a from './data1a.json';
+import form1 from './form1.json';
+import subs from './subs.json';
+
+export { skipValidForConditionallyHiddenComp, data1a, form1, subs };

src/process/__tests__/fixtures/skipValidForConditionallyHiddenComp.json

@@ -0,0 +1,79 @@
+{
+  "form": {
+    "title": "req",
+    "name": "req",
+    "path": "req",
+    "type": "form",
+    "display": "form",
+    "tags": [],
+    "components": [
+      {
+        "label": "hello there",
+        "applyMaskOn": "change",
+        "tableView": true,
+        "validateWhenHidden": false,
+        "key": "helloThere",
+        "type": "textfield",
+        "input": true
+      },
+      {
+        "label": "Checkbox",
+        "tableView": false,
+        "validateWhenHidden": false,
+        "key": "checkbox",
+        "type": "checkbox",
+        "input": true
+      },
+      {
+        "collapsible": false,
+        "key": "panel",
+        "conditional": {
+          "show": false,
+          "conjunction": "any",
+          "conditions": [
+            {
+              "component": "checkbox",
+              "operator": "isEqual",
+              "value": true
+            }
+          ]
+        },
+        "type": "panel",
+        "label": "Panel",
+        "input": false,
+        "tableView": false,
+        "components": [
+          {
+            "label": "Text Field",
+            "applyMaskOn": "change",
+            "tableView": true,
+            "validate": {
+              "required": true
+            },
+            "validateWhenHidden": false,
+            "key": "textField",
+            "type": "textfield",
+            "input": true
+          }
+        ]
+      },
+      {
+        "type": "button",
+        "label": "Submit",
+        "key": "submit",
+        "disableOnInvalid": true,
+        "input": true,
+        "tableView": false
+      }
+    ],
+    "created": "2024-07-29T13:38:39.681Z",
+    "modified": "2024-07-29T13:39:19.804Z"
+  },
+  "submission": {
+    "data": {
+      "helloThere": "",
+      "checkbox": true,
+      "submit": true
+    }
+  }
+}

src/process/__tests__/process.test.ts

@@ -1,10 +1,8 @@
 import { expect } from 'chai';
 import { processSync, ProcessTargets } from '../index';
 import { ValidationScope } from 'types';
+import { skipValidForConditionallyHiddenComp } from './fixtures';
 const assert = require('assert');
-const form1 = require('./fixtures/form1.json');
-const data1a = require('./fixtures/data1a.json');
-const subs = require('./fixtures/subs.json');
 /*
 describe('Process Tests', () => {
     it('Should perform the processes using the processReduced method.', async () => {
@@ -831,7 +829,7 @@ describe('Process Tests', () => {
               'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/122.0.0.0 Safari/537.36',
             pathName: '/',
             onLine: true,
-            
+
           },
           data: {
             number: 23,
@@ -947,7 +945,7 @@ describe('Process Tests', () => {
       },
       owner: '65ea3601c3792e416cabcb2a',
       access: [],
-      
+
       _vnote: '',
       state: 'submitted',
       form: '65ea368b705068f84a93c87a',
@@ -970,7 +968,7 @@ describe('Process Tests', () => {
     context.processors = ProcessTargets.evaluator;
     processSync(context);
     console.log(context.scope.errors);
-    
+
     assert.equal(context.scope.errors.length, 0);
   });
   it('should remove submission data not in a nested form definition', async function () {
@@ -2897,7 +2895,7 @@ describe('Process Tests', () => {
               "label": "Appointees",
               "hideLabel": true,
               "tableView": false,
-              
+
               "addAnother": "__add_appointee",
               "modal": true,
               "saveRow": "Close",
@@ -3016,7 +3014,7 @@ describe('Process Tests', () => {
           "state": "draft"
         }
       ],
-          
+
     };
         const submission = {
           "data": {
@@ -3056,10 +3054,11 @@ describe('Process Tests', () => {
           submit: true
 
         });
-    
+
       })
+
     it('Should not return fields from conditionally hidden containers, clearOnHide = false', async () => {
-        
+
         const TestForm45588WithClearOnHideFalse = {
           display: 'form',
           "components": [
@@ -3077,7 +3076,7 @@ describe('Process Tests', () => {
               "label": "Appointees",
               "hideLabel": true,
               "tableView": false,
-              
+
               "addAnother": "__add_appointee",
               "modal": true,
               "saveRow": "Close",
@@ -3196,9 +3195,9 @@ describe('Process Tests', () => {
           "state": "draft"
         }
       ],
-          
+
     };
-        
+
         const submission = {
           "data": {
               "candidates": [
@@ -3281,7 +3280,7 @@ describe('Process Tests', () => {
                     textField: 'test',
                     invalidField: 'bad',
                   },
-                  
+
                 },
               },
               {
@@ -3306,7 +3305,7 @@ describe('Process Tests', () => {
         context.processors = ProcessTargets.evaluator;
         processSync(context);
         console.log(JSON.stringify(context.data, null, 2));
-        
+
         expect((context.scope as ValidationScope).errors).to.have.length(0);
          expect(context.data).to.deep.equal({
           editGrid: [{ form: { data: { textField: 'test' } } }],
@@ -3343,7 +3342,27 @@ describe('Process Tests', () => {
         processSync(context);
         expect((context.scope as ValidationScope).errors).to.have.length(1);
       });
-
+
+      it('Should not validate component if it is hidden', async () => {
+        const { form, submission } = skipValidForConditionallyHiddenComp;
+        const context = {
+          form,
+          submission,
+          data: submission.data,
+          components: form.components,
+          processors: ProcessTargets.submission,
+          scope: {},
+          config: {
+            server: true,
+          },
+        };
+
+        processSync(context);
+        context.processors = ProcessTargets.evaluator;
+        processSync(context);
+        expect((context.scope as ValidationScope).errors).to.have.length(0);
+      });
+
     });
   });
   /*