chore: Updates IDSSepcifications to allow all checks without skipping when fail

(cherry picked from commit 078bea0cef8ceb3a31d257d8e5f65afd13128cc2)

Author: Pablo

packages/core/src/openbim/IDSSpecifications/src/facets/Facet.ts

@@ -122,11 +122,14 @@ export abstract class IDSFacet {
     }
 
     let result = modelItemResults.get(item._localId.value);
-    if (result && skipIfFails) {
-      // If there are results already and the item didn't pass
-      // return null to skip further checks
-      if (!result.pass) return null;
-    } else {
+
+    // If there are results already and the item didn't pass, skip further checks
+    if (result && skipIfFails && !result.pass) {
+      return null;
+    }
+
+    // If there are no results for this item, create them
+    if (!result) {
       const checks: IDSItemFacetCheck[] = [];
 
       result = {
@@ -142,6 +145,7 @@ export abstract class IDSFacet {
       modelItemResults.set(item._localId.value, result);
     }
 
+    // Create the checks for the current facet
     const checks: IDSCheck[] = [];
 
     const check: IDSItemFacetCheck = {
@@ -155,6 +159,7 @@ export abstract class IDSFacet {
       get: () => checks.every(({ pass }) => pass),
     });
 
+    // Add the current facet check to the item's result
     result.checks.push(check);
 
     return check.checks;