Display nodes that were not executed as skipped.

checkmarble · Jan 28, 2025 · a0dee00 · a0dee00
1 parent c2f5308
commit a0dee00
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 2 deletions.
diff --git a/packages/app-builder/src/components/Scenario/AstBuilder/RootAstBuilderNode/RootOrWithAnd.tsx b/packages/app-builder/src/components/Scenario/AstBuilder/RootAstBuilderNode/RootOrWithAnd.tsx
@@ -201,9 +201,19 @@ function AndOperand({
           <Tag
             border="square"
             className="w-full"
-            color={value === null ? 'orange' : value ? 'green' : 'red'}
+            color={
+              evaluation.skipped
+                ? 'grey'
+                : value === null
+                  ? 'green'
+                  : value
+                    ? 'green'
+                    : 'red'
+            }
           >
-            {t(`common:${value === null ? 'null' : value}`)}
+            {t(
+              `common:${evaluation.skipped ? 'skipped' : value === null ? 'null' : value}`,
+            )}
           </Tag>
         </div>
       );

diff --git a/packages/app-builder/src/locales/en/common.json b/packages/app-builder/src/locales/en/common.json
@@ -1,5 +1,6 @@
 {
   "null": "Null",
+  "skipped": "Skipped",
   "true": "True",
   "false": "False",
   "from_to": "From <Date>{{start_date}}</Date> to <Date>{{end_date}}</Date>",

diff --git a/packages/app-builder/src/locales/fr/common.json b/packages/app-builder/src/locales/fr/common.json
@@ -1,5 +1,6 @@
 {
   "null": "Nul",
+  "skipped": "Omis",
   "auth.logout": "Se déconnecter",
   "cancel": "Annuler",
   "error_one": "erreur",

diff --git a/packages/app-builder/src/models/node-evaluation.ts b/packages/app-builder/src/models/node-evaluation.ts
@@ -47,6 +47,7 @@ export interface NodeEvaluation {
   errors: EvaluationError[];
   children: NodeEvaluation[];
   namedChildren: Record<string, NodeEvaluation>;
+  skipped?: boolean;
 }
 
 export function NewNodeEvaluation(): NodeEvaluation {
@@ -55,6 +56,7 @@ export function NewNodeEvaluation(): NodeEvaluation {
     errors: [],
     children: [],
     namedChildren: {},
+    skipped: false,
   };
 }
 
@@ -79,5 +81,6 @@ export function adaptNodeEvaluation(dto: NodeEvaluationDto): NodeEvaluation {
     namedChildren: dto.named_children
       ? R.mapValues(dto.named_children, adaptNodeEvaluation)
       : {},
+    skipped: dto.skipped,
   };
 }
diff --git a/packages/marble-api/openapis/marblecore-api.yaml b/packages/marble-api/openapis/marblecore-api.yaml
@@ -5174,6 +5174,9 @@ components:
           type: object
           additionalProperties:
             $ref: '#/components/schemas/NodeEvaluationDto'
+        skipped:
+          type: boolean
+          default: false
     ScenarioValidationErrorDto:
       type: object
       required:

diff --git a/packages/marble-api/src/generated/marblecore-api.ts b/packages/marble-api/src/generated/marblecore-api.ts
@@ -119,6 +119,7 @@ export type NodeEvaluationDto = {
     named_children?: {
         [key: string]: NodeEvaluationDto;
     };
+    skipped?: boolean;
 };
 export type RuleExecutionDto = {
     error?: Error;