tricc-oo 1.6.12__py3-none-any.whl → 1.6.13__py3-none-any.whl

tricc_oo/converters/codesystem_to_ocl.py

@@ -31,16 +31,16 @@ def extract_properties_metadata(fhir_cs: CodeSystem) -> Dict[str, Dict]:
 
             property_types[prop.code] = {
                 "name": prop.code,
-                "datatype": ocl_type,
+                "dataType": ocl_type,
                 "description": prop.description if hasattr(prop, "description") else "",
             }
     return property_types
 
 
 def get_fhir_concept_datatype(concept):
-    datatype = extract_concept_properties(concept, ["datatype"])
+    datatype = extract_concept_properties(concept, ["dataType"])
     if datatype:
-        return datatype["datatype"]
+        return datatype["dataType"]
     else:
         return OclConstants.DATA_TYPE_NONE
 
@@ -84,7 +84,7 @@ def get_attributes_from_concept_properties(concept, property_types: Dict) -> Lis
                 "type": "Attribute",
                 "attribute_type": code,
                 "value": value,
-                "value_type": property_types[code]["datatype"],
+                "value_type": property_types[code]["dataType"],
             }
         )
     return attributes

tricc_oo/converters/datadictionnary.py

@@ -103,7 +103,7 @@ def check_and_add_concept(code_system: CodeSystem, code: str, display: str, attr
                 # TODO support other type of Codesystem Concept Property Value
                 existing_attributes
                 if p.valueString != v:
-                    logger.warning(f"conflicting value for property {k}: {p.valueString} != {v}")
+                    logger.warning(f"conflicting value for concept `{concept.code}` property ` {k}`: {p.valueString} != {v}")
         if not existing_attributes:
             new_concept.property.append(CodeSystemConceptProperty(code=k, valueString=v))
 

tricc_oo/converters/drawio_type_map.py

@@ -47,7 +47,7 @@ TYPE_MAP = {
     },
     TriccNodeType.note: {
         "objects": ["UserObject", "object"],
-        "attributes": ["relevance", "priority", "context_type"],
+        "attributes": ["relevance", "priority", "concept_type"],
         "mandatory_attributes": ["label", "name"],
         "model": TriccNodeNote,
     },
@@ -76,7 +76,7 @@ TYPE_MAP = {
             "priority",
             "trigger",
             "default",
-            "context_type",
+            "concept_type",
         ],
         "mandatory_attributes": ["label", "name", "list_name"],
         "model": TriccNodeSelectOne,
@@ -94,7 +94,7 @@ TYPE_MAP = {
             "priority",
             "trigger",
             "default",
-            "context_type",
+            "concept_type",
         ],
         "mandatory_attributes": ["label", "name", "list_name"],
         "model": TriccNodeSelectMultiple,
@@ -112,7 +112,7 @@ TYPE_MAP = {
             "priority",
             "trigger",
             "default",
-            "context_type",
+            "concept_type",
         ],
         "mandatory_attributes": ["label", "name"],
         "model": TriccNodeDecimal,
@@ -130,7 +130,7 @@ TYPE_MAP = {
             "priority",
             "trigger",
             "default",
-            "context_type",
+            "concept_type",
         ],
         "mandatory_attributes": ["label", "name"],
         "model": TriccNodeInteger,
@@ -145,7 +145,7 @@ TYPE_MAP = {
             "default",
             "constraint",
             "constraint_message",
-            "context_type",
+            "concept_type",
         ],
         "mandatory_attributes": ["label", "name"],
         "model": TriccNodeText,
@@ -160,7 +160,7 @@ TYPE_MAP = {
             "default",
             "constraint",
             "constraint_message",
-            "context_type",
+            "concept_type",
         ],
         "mandatory_attributes": ["label", "name"],
         "model": TriccNodeDate,
@@ -183,7 +183,7 @@ TYPE_MAP = {
             "save",
             "reference",
             "trigger",
-            "context_type",
+            "concept_type",
             "remote_reference",
         ],
         "mandatory_attributes": ["name", "label"],
@@ -217,7 +217,7 @@ TYPE_MAP = {
     },
     TriccNodeType.not_available: {
         "objects": ["UserObject", "object"],
-        "attributes": ["context_type"],
+        "attributes": ["concept_type"],
         "mandatory_attributes": ["label", "name", "list_name"],
         "model": TriccNodeSelectNotAvailable,
     },
@@ -233,7 +233,7 @@ TYPE_MAP = {
             "priority",
             "trigger",
             "default",
-            "context_type",
+            "concept_type",
         ],
         "mandatory_attributes": ["label", "name", "list_name"],
         "model": TriccNodeSelectYesNo,
@@ -295,7 +295,7 @@ TYPE_MAP = {
     },
     TriccNodeType.input: {
         "objects": ["UserObject", "object"],
-        "attributes": ["save", "reference", "datatype", "context_type"],
+        "attributes": ["save", "reference", "data_type", "concept_type"],
         "mandatory_attributes": ["name", "label"],
         "model": TriccNodeInput,
     },

tricc_oo/converters/xml_to_tricc.py

@@ -128,7 +128,7 @@ def create_activity(diagram, media_path, project):
         for n in nodes.values():
 
             if (
-                issubclass(n.__class__, (TriccNodeDisplayModel, TriccNodeDisplayCalculateBase))
+                issubclass(n.__class__, (TriccNodeDisplayModel, TriccNodeDisplayCalculateBase, TriccNodeInput))
                 and not isinstance(n, (TriccRhombusMixIn, TriccNodeRhombus, TriccNodeDisplayBridge))
                 and not n.name.startswith("label_")  # FIXME
             ):
@@ -139,7 +139,7 @@ def create_activity(diagram, media_path, project):
                         system,
                         n.select.name,
                         n.label,
-                        {"datatype": "Boolean", "contextType": get_context_type(n)},
+                        {"dataType": "Boolean", "conceptType": get_concept_type(n)},
                     )
                 elif not isinstance(n, TriccNodeSelectNotAvailable):
                     add_concept(
@@ -148,8 +148,20 @@ def create_activity(diagram, media_path, project):
                         n.name,
                         n.label,
                         {
-                            "datatype": get_data_type(n.tricc_type),
-                            "contextType": get_context_type(n),
+                            "dataType": get_data_type(n.tricc_type),
+                            "conceptType": get_concept_type(n),
+                        },
+                    )
+                elif not issubclass(n.__class__, TriccNodeCalculate):
+                    system = n.name.split(".")[0] if "." in n.name else "calculate"
+                    add_concept(
+                        project.code_systems,
+                        system,
+                        n.name,
+                        n.label,
+                        {
+                            "dataType": get_data_type(n.tricc_type),
+                            "conceptType": get_concept_type(n),
                         },
                     )
                 if getattr(n, "save", None):
@@ -160,8 +172,8 @@ def create_activity(diagram, media_path, project):
                         n.save,
                         n.label,
                         {
-                            "datatype": get_data_type(n.tricc_type),
-                            "contextType": get_context_type(n),
+                            "dataType": get_data_type(n.tricc_type),
+                            "conceptType": get_concept_type(n),
                         },
                     )
 
@@ -175,7 +187,7 @@ def create_activity(diagram, media_path, project):
         # link back the activity
         activity.root.activity = activity
         manage_dangling_calculate(activity)
-
+        # assign the process
         if activity is not None:
             if activity.root is not None:
                 project.pages[activity.id] = activity
@@ -197,6 +209,7 @@ def create_activity(diagram, media_path, project):
                         )
         if images:
             project.images += images
+        # Assign parent to NotAvailable
         for node in list(
             filter(
                 lambda p_node: isinstance(p_node, TriccNodeSelectNotAvailable),
@@ -477,10 +490,10 @@ def set_additional_attributes(attribute_names, elm, node):
             setattr(node, attributename, attribute)
 
 
-def get_context_type(node):
-    context_type = getattr(node, "context_type", None)
-    if context_type:
-        return context_type
+def get_concept_type(node):
+    concept_type = getattr(node, "concept_type", None)
+    if concept_type:
+        return concept_type
     if isinstance(node, TriccNodeSelectMultiple):
         return "Question"
     elif isinstance(node, TriccNodeSelectOption):
@@ -535,7 +548,7 @@ def get_select_options(diagram, select_node, nodes):
             activity=select_node.activity,
             group=select_node.group,
         )
-        set_additional_attributes(["save", "relevance", "context_type"], elm, option)
+        set_additional_attributes(["save", "relevance", "concept_type"], elm, option)
         load_expressions(option)
         options[i] = option
         nodes[id] = option
@@ -802,9 +815,11 @@ def set_mandatory_attribute(elm, mandatory_attributes, diagram=None):
             id = elm.attrib.get("id")
             attribute_value = _get_name(name, id, diagram_id)
         elif attributes == "list_name":
-            name = elm.attrib.get("name")
-            id = elm.attrib.get("id")
-            attribute_value = TRICC_LIST_NAME.format(clean_str(_get_name(name, id, diagram_id), replace_dots=True))
+            attribute_value = elm.attrib.get("list_name", None)
+            if not attribute_value:
+                name = elm.attrib.get("name")
+                id = elm.attrib.get("id")
+                attribute_value = TRICC_LIST_NAME.format(clean_str(_get_name(name, id, diagram_id), replace_dots=True))
         else:
             attribute_value = elm.attrib.get(attributes)
         if attribute_value is None:

tricc_oo/serializers/xls_form.py

@@ -32,6 +32,7 @@ from tricc_oo.visitors.tricc import (
     get_applicability_expression,
     get_prev_instance_skip_expression,
     get_process_skip_expression,
+    process_operation_reference,
 )
 
 logger = logging.getLogger("default")
@@ -64,7 +65,36 @@ def start_group(
         groups[name] = 0
     relevance = relevance and cur_group.relevance is not None and cur_group.relevance != ""
     past_instances = len(getattr(cur_group.base_instance, "instances", []))
+    group_calc_required = relevance is not None and (len(str(relevance)) > 100 or past_instances > 1)
     calc = None
+    if group_calc_required and getattr(cur_group.relevance, 'operator', None) != TriccOperator.ISTRUE:
+
+        calc = TriccNodeCalculate(
+            id=generate_id(get_export_group_name(name)),
+            group=cur_group.group,
+            activity=cur_group.activity,
+            name=get_export_group_name(name),
+            expression=cur_group.relevance
+        )
+
+        if calc not in cur_group.activity.calculates:
+            process_reference(
+                calc,
+                processed_nodes,
+                calculates=kwargs.get('calculates', None),
+                used_calculates=kwargs.get('used_calculates', None),
+                replace_reference=True,
+                warn=False,
+                codesystems=kwargs.get('codesystems', None)
+            )
+            cur_group.activity.calculates.append(calc)
+            cur_group.activity.nodes[calc.id] = calc
+            processed_nodes.add(calc)
+
+        cur_group.relevance = TriccOperation(
+                TriccOperator.ISTRUE,
+                [calc]
+            )
 
     relevance_expression = cur_group.relevance
     relevance_expression = get_applicability_expression(cur_group, processed_nodes, process, relevance_expression)
@@ -74,7 +104,18 @@ def start_group(
     if not relevance:
         relevance_expression_str = ""
     elif isinstance(relevance_expression, (TriccOperation, TriccStatic)):
-        relevance_expression_str = strategy.get_tricc_operation_expression(relevance_expression)
+        relevance_expression = process_operation_reference(
+            relevance_expression,
+            cur_group,
+            processed_nodes=processed_nodes,
+            calculates=kwargs.get('calculates', None),
+            used_calculates=kwargs.get('used_calculates', None),
+            replace_reference=True,
+            warn=False,
+            codesystems=kwargs.get('codesystems', None),
+        ) or relevance_expression
+        if relevance_expression:
+            relevance_expression_str = strategy.get_tricc_operation_expression(relevance_expression)
 
     # group
     values = []
@@ -95,7 +136,23 @@ def start_group(
             values.append(get_xfrom_trad(strategy, cur_group, column, SURVEY_MAP))
     df_survey.loc[len(df_survey)] = values
 
-    # Group calculates are now created during process_calculate phase
+    # calc
+    if calc and len(df_calculate[df_calculate["name"] == get_export_group_name(name)]) == 0:
+        calc_values = []
+        for column in SURVEY_MAP:
+            if column == "type":
+                calc_values.append("calculate")
+            elif column == "name":
+                value = get_export_name(calc)
+                calc_values.append(value)
+            elif column == "calculation":
+                calc_values.append(f"number({strategy.get_tricc_operation_expression(calc.expression)})")
+            elif column == "relevance":
+                calc_values.append("")
+            else:
+                calc_values.append(get_xfrom_trad(strategy, cur_group, column, SURVEY_MAP))
+
+        df_calculate.loc[len(df_calculate)] = calc_values
 
 
 # def add_background_color(input_string, color):

tricc_oo/strategies/output/dhis2_form.py

@@ -16,7 +16,7 @@ import datetime
 from tricc_oo.strategies.output.base_output_strategy import BaseOutPutStrategy
 from tricc_oo.models.base import (
     not_clean, TriccOperation,
-    TriccStatic, TriccReference, TriccOperator
+    TriccStatic, TriccReference
 )
 from tricc_oo.models.tricc import (
     TriccNodeSelectOption,
@@ -27,8 +27,6 @@ from tricc_oo.models.tricc import (
     TriccNodeActivity,
     TriccNodeSelect,
     TriccNodeSelectYesNo,
-    TriccNodeNote,
-    TriccNodeMoreInfo,
 )
 from tricc_oo.models.calculate import TriccNodeDisplayCalculateBase
 from tricc_oo.models.ordered_set import OrderedSet
@@ -46,7 +44,7 @@ class DHIS2Strategy(BaseOutPutStrategy):
 
     def __init__(self, project, output_path):
         super().__init__(project, output_path)
-        form_id = getattr(self.project.start_pages["main"].root, 'form_id', 'dhis2_program')
+        form_id = getattr(self.project.start_pages["main"], 'form_id', 'dhis2_program')
         self.program_metadata = {
             "id": self.generate_id(form_id),
             "name": form_id,
@@ -206,17 +204,13 @@ class DHIS2Strategy(BaseOutPutStrategy):
             return False
 
         if node not in processed_nodes:
-            # Skip relevance generation for TriccNodeMoreInfo as they don't create DataElements
-            if isinstance(node, TriccNodeMoreInfo):
-                return True
-
             relevance = None
             if hasattr(node, 'relevance') and node.relevance:
                 relevance = node.relevance
             if hasattr(node, 'expression') and node.expression:
                 relevance = node.expression
             if relevance:
-                relevance_str = self.convert_expression_to_string(relevance)
+                relevance_str = self.convert_expression_to_string(not_clean(relevance))
                 if relevance_str and relevance_str != 'false':
                     # Create program rule action for hiding/showing based on relevance
                     rule_id = self.generate_id(f"rule_{node.get_name()}_relevance")
@@ -231,10 +225,8 @@ class DHIS2Strategy(BaseOutPutStrategy):
                             "activity_ref": node,  # Temporary reference to be replaced with section ID
                             "programRule": {"id": rule_id},
                         }
-                        self.program_rule_actions.append(program_rule_action)
-                    elif issubclass(node.__class__, TriccNodeDisplayModel) and not issubclass(node.__class__, TriccNodeCalculateBase) and not isinstance(node, (TriccNodeNote, TriccNodeMoreInfo, TriccNodeSelectOption)):
-                        # For regular nodes that get DataElements, use HIDEFIELD action
-                        # Exclude TriccNodeNote and TriccNodeMoreInfo as they don't get DataElements
+                    else:
+                        # For regular nodes, use HIDEFIELD action
                         program_rule_action = {
                             "id": action_id,
                             "programRuleActionType": "HIDEFIELD",
@@ -243,45 +235,18 @@ class DHIS2Strategy(BaseOutPutStrategy):
                             },
                             "programRule": {"id": rule_id}
                         }
-                        self.program_rule_actions.append(program_rule_action)
+                    self.program_rule_actions.append(program_rule_action)
 
                     # Create program rule referencing the action
-                    condition = self.simplify_expression(f"({relevance_str})==false")  # Negate for hide when true
+                    condition = self.simplify_expression(f"!({relevance_str})")  # Negate for hide when true
                     condition = self.simplify_expression(condition)
                     self.program_rules.append({
                         "id": rule_id,
                         "name": f"Hide `{self.get_export_name(node)}` when condition met",
-                        "description": f"Hide `{self.get_display(node)[:128]}` based on relevance",
+                        "description": f"Hide `{self.get_display(node)}` based on relevance",
                         "condition": condition,
                         "programRuleActions": [{"id": action_id}]
                     })
-
-            # Check if field should be mandatory based on 'required' attribute
-            if bool(getattr(node, 'required', False)):
-                # Create program rule for mandatory field using relevance function
-                mandatory_rule_id = self.generate_id(f"rule_{node.get_name()}_mandatory")
-                mandatory_action_id = self.generate_id(f"action_{mandatory_rule_id}")
-
-                mandatory_program_rule_action = {
-                    "id": mandatory_action_id,
-                    "programRuleActionType": "SETMANDATORYFIELD",
-                    "dataElement": {
-                        "id": self.generate_id(self.get_export_name(node))
-                    },
-                    "programRule": {"id": mandatory_rule_id}
-                }
-                self.program_rule_actions.append(mandatory_program_rule_action)
-
-                # Create program rule for mandatory field - use relevance condition if available
-                mandatory_condition = relevance_str if relevance_str and relevance_str != 'false' else "true"
-                self.program_rules.append({
-                    "id": mandatory_rule_id,
-                    "name": f"Make `{self.get_export_name(node)}` mandatory",
-                    "description": f"Set `{self.get_display(node)[:128]}` as mandatory field",
-                    "condition": mandatory_condition,
-                    "programRuleActions": [{"id": mandatory_action_id}]
-                })
-
         return True
 
     def generate_data_element(self, node):
@@ -309,8 +274,8 @@ class DHIS2Strategy(BaseOutPutStrategy):
                 "id": de_id,
                 "name": self.get_export_name(node),
                 "shortName": node.name[:50],
-                "displayFormName": "More infromation" if isinstance(node, TriccNodeMoreInfo) else self.get_display(node),
-                "formName": "More infromation" if isinstance(node, TriccNodeMoreInfo) else self.get_display(node),
+                "displayFormName": self.get_display(node),
+                "formName": self.get_display(node),
                 "valueType": value_type,
                 "domainType": "TRACKER",
                 "aggregationType": "NONE"
@@ -547,36 +512,8 @@ class DHIS2Strategy(BaseOutPutStrategy):
             return False
 
         if node not in processed_nodes:
-            logger.debug(f"generate_export processing node: {node.get_name()} type: {type(node)}")
-            # Special handling for TriccNodeNote - transform into section description
-            if isinstance(node, TriccNodeNote):
-                logger.info(f"Found TriccNodeNote: {node.get_name()}")
-                self.handle_note_as_section_description(node, processed_nodes, **kwargs)
-            elif isinstance(node, TriccNodeMoreInfo):
-                logger.info(f"Found TriccNodeMoreInfo: {node.get_name()}")
-                # Create boolean checkbox data element for moreinfo
-                data_element = self.generate_data_element(node)
-                if data_element:
-                    # Add to program stage
-                    if self.program_metadata["programStages"]:
-                        psde_id = self.generate_id(f"psde_{node.name}")
-                        psde = {
-                            "id": psde_id,
-                            "dataElement": {"id": data_element["id"]},
-                            "compulsory": False
-                        }
-                        self.program_metadata["programStages"][-1]["programStageDataElements"].append(psde)
-
-                        # Add data element to current section
-                        if self.current_section and self.current_section in self.sections:
-                            self.sections[self.current_section]["dataElements"].append({"id": data_element["id"]})
-
-                # Handle the note section with checkbox conditioning
-                checkbox_reference = TriccReference(node.name)
-                checkbox_condition = TriccOperation(TriccOperator.ISTRUE, [checkbox_reference])
-                self.handle_note_as_section_description(node, processed_nodes, additional_condition=checkbox_condition, **kwargs)
             # Skip creating data elements for calculate nodes - they should only be program rule variables
-            elif not issubclass(node.__class__, TriccNodeCalculateBase):
+            if not issubclass(node.__class__, TriccNodeCalculateBase):
                 data_element = self.generate_data_element(node)
                 if data_element:
                     # Add to program stage
@@ -585,7 +522,7 @@ class DHIS2Strategy(BaseOutPutStrategy):
                         psde = {
                             "id": psde_id,
                             "dataElement": {"id": data_element["id"]},
-                            "compulsory": False
+                            "compulsory": bool(getattr(node, 'required', False))
                         }
                         self.program_metadata["programStages"][-1]["programStageDataElements"].append(psde)
 
@@ -595,136 +532,6 @@ class DHIS2Strategy(BaseOutPutStrategy):
 
         return True
 
-    def handle_note_as_section_description(self, node, processed_nodes, additional_condition=None, **kwargs):
-        """Transform TriccNodeNote into section description and create hide logic with section duplication
-
-        Args:
-            node: The node to process (TriccNodeNote or TriccNodeMoreInfo)
-            processed_nodes: Set of already processed nodes
-            additional_condition: Optional additional condition to combine with existing relevance
-        """
-        logger.info(f"Processing note {node.get_name()} for section description")
-        if not self.current_section or self.current_section not in self.sections:
-            logger.warning(f"No current section found for note {node.get_name()}")
-            return
-
-        # 1. Duplicate the previous section with incremented ID (like XLS form group numbering)
-        original_section = self.sections[self.current_section]
-        section_name = f"{original_section['name']}_{node.name}"
-        duplicated_section_id = self.generate_id(section_name)
-        duplicated_section = {
-            "id": duplicated_section_id,
-            "name": section_name,  # Use incremented naming pattern
-            "sortOrder": len(self.sections),
-            "programStage": {"id": self.program_metadata["programStages"][-1]["id"]},
-            "dataElements": original_section.get("dataElements", []).copy(),  # Copy data elements
-            "activity_ref": original_section.get("activity_ref")
-        }
-
-        # 2. Create a new section for the note/moreinfo with name and ID from the node
-        note_section_id = self.generate_id(self.get_export_name(node))
-        note_section_name = self.get_export_name(node)
-
-        note_section = {
-            "id": note_section_id,
-            "name": note_section_name,
-            "sortOrder": len(self.sections),
-            "programStage": {"id": self.program_metadata["programStages"][-1]["id"]},
-            "dataElements": [],
-            "activity_ref": original_section.get("activity_ref")  # Same activity reference
-        }
-
-        # Set section description to the note's label
-        if hasattr(node, 'label') and node.label:
-            note_section["description"] = node.label.replace('\u00a0', ' ').strip()
-            logger.info(f"Set note section {note_section_id} description to: {note_section['description']}")
-        elif hasattr(node, 'name') and node.name:
-            note_section["description"] = node.name.replace('\u00a0', ' ').strip()
-            logger.info(f"Set note section {note_section_id} description to: {note_section['description']}")
-        else:
-            note_section["description"] = str(node.id)
-            logger.info(f"Set note section {note_section_id} description to: {note_section['description']}")
-
-        self.sections[note_section_id] = note_section
-
-        # Add note section to program stage
-        if self.program_metadata["programStages"]:
-            self.program_metadata["programStages"][-1]["programStageSections"].append({"id": note_section_id})
-
-        logger.info(f"Created note section {note_section_id} with name '{note_section_name}'")
-
-        # 3. Inject the duplicated section as the new current section
-        self.sections[duplicated_section_id] = duplicated_section
-
-        # Add duplicated section to program stage
-        if self.program_metadata["programStages"]:
-            self.program_metadata["programStages"][-1]["programStageSections"].append({"id": duplicated_section_id})
-
-        logger.info(f"Duplicated section {self.current_section} as {duplicated_section_id}")
-
-        self.current_section = duplicated_section_id
-        logger.info(f"Set current section to duplicated section {duplicated_section_id}")
-
-        # Create hide logic for the note section based on combined relevance
-        combined_relevance = None
-
-        # Get parent activity relevance from the original section's activity_ref
-        parent_activity = original_section.get("activity_ref")
-        if parent_activity and hasattr(parent_activity, 'relevance') and parent_activity.relevance:
-            combined_relevance = parent_activity.relevance
-
-        # Combine with note relevance if it exists
-        if hasattr(node, 'relevance') and node.relevance:
-            if combined_relevance:
-                # Combine using AND operation
-                combined_relevance = TriccOperation(
-                    TriccOperator.AND,
-                    [combined_relevance, node.relevance]
-                )
-            else:
-                combined_relevance = node.relevance
-
-        # Combine with additional condition if provided
-        if additional_condition:
-            if combined_relevance:
-                # Combine using AND operation
-                combined_relevance = TriccOperation(
-                    TriccOperator.AND,
-                    [combined_relevance, additional_condition]
-                )
-            else:
-                combined_relevance = additional_condition
-
-        # Create hide logic if there's relevance - applied to the note section
-        if combined_relevance:
-            relevance_str = self.convert_expression_to_string(not_clean(combined_relevance))
-            if relevance_str and relevance_str != 'false':
-                # Create program rule action for hiding the note section
-                rule_id = self.generate_id(f"rule_{node.get_name()}_note_hide_section")
-                action_id = self.generate_id(f"action_{rule_id}")
-
-                program_rule_action = {
-                    "id": action_id,
-                    "programRuleActionType": "HIDESECTION",
-                    "activity_ref": parent_activity,  # Use activity reference like other HIDESECTION actions
-                    "programRule": {"id": rule_id},
-                }
-                self.program_rule_actions.append(program_rule_action)
-
-                # Create program rule referencing the action
-                condition = self.simplify_expression(f"({relevance_str})==false")  # Negate for hide when true
-                condition = self.simplify_expression(condition)
-                rule_name = f"Hide note section `{note_section_name}` based on relevance"
-                if additional_condition:
-                    rule_name = f"Hide moreinfo section `{note_section_name}` when checkbox false"
-                self.program_rules.append({
-                    "id": rule_id,
-                    "name": rule_name,
-                    "description": f"Hide note section `{self.get_display(node)[:128]}` based on combined relevance",
-                    "condition": condition,
-                    "programRuleActions": [{"id": action_id}]
-                })
-
     def clean_section(self, program_stages_payload):
         """Clean sections by removing empty ones and merging sections with same activity_ref"""
         sections_to_remove = set()
@@ -735,11 +542,11 @@ class DHIS2Strategy(BaseOutPutStrategy):
             section_id = section["id"]
             activity_ref = section.get("activity_ref")
             # Remove empty sections
-            if not section.get("dataElements") and not section.get("description"):
+            if not section.get("dataElements"):
                 sections_to_remove.add(section_id)
 
             # Check for sections with same activity_ref
-            elif activity_ref == prev_activity_ref and not section.get("description"):
+            elif activity_ref == prev_activity_ref:
                 # Merge this section into the existing one
                 existing_section = self.sections[prev_section_id]
 
@@ -838,8 +645,6 @@ class DHIS2Strategy(BaseOutPutStrategy):
                     # Non-activity actions (HIDEFIELD) can be added directly
                     program_rule_actions_payload.append(action)
 
-        # Filter out rules that reference non-existent actions
-        valid_action_ids = {action["id"] for action in program_rule_actions_payload}
         if self.program_rules:
             program_rules_payload = [
                 {
@@ -847,7 +652,6 @@ class DHIS2Strategy(BaseOutPutStrategy):
                     "program": {"id": self.program_metadata["id"]}
                 }
                 for rule in self.program_rules
-                if all(action_ref["id"] in valid_action_ids for action_ref in rule["programRuleActions"])
             ]
 
         if self.program_rule_variables:
@@ -906,7 +710,7 @@ class DHIS2Strategy(BaseOutPutStrategy):
             return self.get_tricc_operation_expression(r)
         elif isinstance(r, TriccReference):
             # Use variable name from concept_map
-            node_id = self.concept_map.get(r.value, self.get_export_name(r.value))
+            node_id = self.concept_map.get(r.value.name, self.get_export_name(r.value))
             return f"#{{{node_id}}}"
         elif isinstance(r, TriccStatic):
             if isinstance(r.value, bool):

tricc_oo/strategies/output/fhir_form.py

@@ -97,7 +97,7 @@ class FHIRStrategy(BaseOutPutStrategy):
             "type": self.map_tricc_type_to_fhir(node.tricc_type if hasattr(node, 'tricc_type') else 'text')
         }
         if hasattr(node, 'options') and node.options:
-            item["answerOption"] = [{"valueString": opt.name} for opt in node.options.values()]
+            item["answerOption"] = [{"valueString": opt.name} for opt in node.options]
         self.questionnaires[segment]["item"].append(item)
         return True
 

tricc_oo/strategies/output/xlsform_cht.py

@@ -829,7 +829,7 @@ class XLSFormCHTStrategy(XLSFormCDSSStrategy):
             return jar_path
 
         # Download JAR from GitHub releases
-        jar_url = "https://github.com/getodk/validate/releases/download/v1.19.2/validate.jar"
+        jar_url = "https://github.com/getodk/validate/releases/download/v1.20.0/ODK-Validate-v1.20.0.jar"
         try:
             import urllib.request
             urllib.request.urlretrieve(jar_url, jar_path)

tricc_oo/visitors/tricc.py

@@ -57,16 +57,6 @@ logger = logging.getLogger("default")
 ONE_QUESTION_AT_A_TIME = False
 
 
-def get_main_activity(activity, processed_nodes):
-    """Traverse up the activity hierarchy to find the main activity (root activity)."""
-    activities = list(set(
-        n.activity for n in  activity.relevance.get_references()
-        if hasattr(n, 'activity') and n.activity != activity and n.activity.root in processed_nodes
-    ))
-    if activities:
-        return activities[0]
-
-
 def merge_node(from_node, to_node):
     if from_node.activity != to_node.activity:
         logger.critical("Cannot merge nodes from different activities")
@@ -133,7 +123,7 @@ def get_last_version(name, processed_nodes, _list=None):
 def get_node_expressions(node, processed_nodes, process=None):
     get_overall_exp = issubclass(
         node.__class__,
-        (TriccNodeDisplayCalculateBase, TriccNodeProposedDiagnosis, TriccNodeDiagnosis, TriccNodeActivity)
+        (TriccNodeDisplayCalculateBase, TriccNodeProposedDiagnosis, TriccNodeDiagnosis)
     ) and not isinstance(node, (TriccNodeDisplayBridge))
     expression = None
     # in case of recursive call processed_nodes will be None
@@ -195,9 +185,9 @@ def get_version_inheritance(node, last_version, processed_nodes):
             path_len=node.path_len + 1,
             expression_reference=TriccOperation(
                 TriccOperator.COALESCE,
-                [TriccReference(node.save), last_version],
+                [node, last_version],
             ),
-            reference=[TriccReference(node.name)],
+            reference=[node, last_version],
             activity=node.activity,
             group=node.group,
             label=f"Save calculation for {node.label}",
@@ -255,29 +245,7 @@ def load_calculate(
             if hasattr(node, "relevance") and (node.relevance is None or isinstance(node.relevance, TriccOperation)):
                 node.relevance = get_node_expressions(node, processed_nodes=processed_nodes, process=process)
                 # manage not Available
-                if isinstance(node, TriccNodeActivity) and isinstance(node.root, TriccNodeActivityStart):
-                    past_instances = len(getattr(node.base_instance, "instances", []))
-                    group_calc_required = False  # (len(str(node.relevance)) > 50 or past_instances > 1 or len(node.calculates)>0)
-                    calc = None
-                    
-                    if group_calc_required and getattr(node.relevance, 'operator', None) != TriccOperator.ISTRUE:
-                        main_activity = get_main_activity(node, processed_nodes)
-                        if  main_activity:
-                            calc = TriccNodeCalculate(
-                                id=generate_id(f"gc.{node.id}"),
-                                group=main_activity.group,
-                                activity=main_activity,
-                                name=f"gc.{node.name}",
-                                expression_reference=node.relevance.copy()
-                            )
-                            main_activity.calculates.append(calc)
-                            main_activity.nodes[calc.id] = calc
-                            processed_nodes.add(calc)
-                            node.relevance = TriccOperation(
-                                    TriccOperator.ISTRUE,
-                                    [calc]
-                            )
-                elif isinstance(node, TriccNodeSelectNotAvailable):
+                if isinstance(node, TriccNodeSelectNotAvailable):
                     # update the checkbox
                     if node.parent:
                         if len(node.prev_nodes) == 1:
@@ -837,7 +805,7 @@ def process_operation_reference(
     operation,
     node,
     processed_nodes,
-    calculates,
+    calculates=None,
     used_calculates=None,
     replace_reference=False,
     warn=False,
@@ -1767,21 +1735,22 @@ def replace_next_node(prev_node, next_node, old_node):
 
 
 # Priority constants
-SAME_GROUP_PRIORITY = 4000
-PARENT_GROUP_PRIORITY = 3000
-ACTIVE_ACTIVITY_PRIORITY = 1400
-NON_START_ACTIVITY_PRIORITY = 1300
-ACTIVE_ACTIVITY_LOWER_PRIORITY = 1200
-FLOW_CALCULATE_NODE_PRIORITY = 2000
-RHOMBUS_PRIORITY = 1000
-DEFAULT_PRIORITY = 1100
-
-
+SAME_GROUP_PRIORITY = 7000
+PARENT_GROUP_PRIORITY = 6000
+ACTIVE_ACTIVITY_PRIORITY = 5000
+NON_START_ACTIVITY_PRIORITY = 4000
+ACTIVE_ACTIVITY_LOWER_PRIORITY = 3000
+FLOW_CALCULATE_NODE_PRIORITY_TOP_UP = 50
+RHOMBUS_PRIORITY_TO_UP = 50
+
+    
 def reorder_node_list(node_list, group, processed_nodes):
     # Cache active activities for O(1) lookup
     active_activities = {n.activity for n in processed_nodes}
-
+    MAP_PRIORITIES = {}
     def get_priority(node):
+        if node.id in MAP_PRIORITIES:
+            return MAP_PRIORITIES[node.id]
         # Cache attributes to avoid repeated getattr calls
         priority = int(getattr(node, "priority", 0) or 0)
         node_group = getattr(node, "group", None)
@@ -1789,12 +1758,7 @@ def reorder_node_list(node_list, group, processed_nodes):
 
         # Check for same group
         if group is not None and node_group and node_group.id == group.id:
-            priority += SAME_GROUP_PRIORITY
-        elif (
-            issubclass(node.__class__, TriccNodeDisplayCalculateBase) or
-            isinstance(node, TriccNodeEnd)
-        ) and not isinstance(node, TriccNodeActivityEnd) and hasattr(node, 'prev_nodes') and len(node.prev_nodes) > 0:
-            priority += FLOW_CALCULATE_NODE_PRIORITY
+            priority += SAME_GROUP_PRIORITY 
         # Check for parent group
         elif hasattr(group, "group") and group.group and node_group and node_group.id == group.group.id:
             priority += PARENT_GROUP_PRIORITY
@@ -1808,13 +1772,21 @@ def reorder_node_list(node_list, group, processed_nodes):
         elif activity and activity in active_activities:
             priority += ACTIVE_ACTIVITY_LOWER_PRIORITY
         # Check for rhombus nodes
-        elif issubclass(node.__class__, TriccRhombusMixIn):
-            priority += RHOMBUS_PRIORITY
-        else:
-            priority += DEFAULT_PRIORITY
         
-        priority = max(priority, max((get_priority(prev_node) for prev_node in node.prev_nodes), default=0))
 
+        if (
+            issubclass(node.__class__, TriccNodeDisplayCalculateBase) or
+            isinstance(node, TriccNodeEnd)
+        ) and not isinstance(node, TriccNodeActivityEnd) and hasattr(node, 'prev_nodes') and len(node.prev_nodes) > 0:
+            priority += FLOW_CALCULATE_NODE_PRIORITY_TOP_UP
+        elif issubclass(node.__class__, TriccRhombusMixIn):
+            priority += RHOMBUS_PRIORITY_TO_UP
+
+        if node.prev_nodes:
+            priority = max(priority, *[get_priority(p) for p in node.prev_nodes])
+        
+        MAP_PRIORITIES[node.id] = priority
+        
         return priority
 
     # Sort in place, highest priority first
@@ -1863,7 +1835,6 @@ def get_node_expression(in_node, processed_nodes, get_overall_exp=False, is_prev
     expression = None
     negate_expression = None
     node = in_node
- 
     if isinstance(node, (TriccNodeActivityStart, TriccNodeMainStart)):
         if is_prev and get_overall_exp:
             expression = get_node_expression(
@@ -2293,7 +2264,12 @@ def get_none_option(node):
 def get_count_terms_details(prev_node, processed_nodes, get_overall_exp, negate=False, process=None):
     opt_none = get_none_option(prev_node)
     if opt_none:
-        operation_none = TriccOperation(TriccOperator.SELECTED, [prev_node, TriccStatic(opt_none)])
+        if isinstance(opt_none, str):
+            operation_none = TriccOperation(TriccOperator.SELECTED, [prev_node, TriccStatic(opt_none)])
+        elif issubclass(opt_none.__class__, TriccBaseModel):
+            operation_none = TriccOperation(TriccOperator.SELECTED, [prev_node, opt_none])
+        else:
+            logger.critical(f"unexpected none option value {opt_none}")
     else:
         operation_none = TriccOperation(TriccOperator.SELECTED, [prev_node, TriccStatic("opt_none")])
     if isinstance(prev_node, TriccNodeSelectYesNo):

{tricc_oo-1.6.12.dist-info → tricc_oo-1.6.13.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tricc-oo
-Version: 1.6.12
+Version: 1.6.13
 Summary: Python library that converts CDSS L2 in L3
 Project-URL: Homepage, https://github.com/SwissTPH/tricc
 Project-URL: Issues, https://github.com/SwissTPH/tricc/issues

{tricc_oo-1.6.12.dist-info → tricc_oo-1.6.13.dist-info}/RECORD

@@ -1,16 +1,15 @@
 tests/build.py,sha256=Qbxvjkj_Wk2nQ-WjaMGiE1FIe3SRmJMRIgeoMoxqlfQ,6748
-tests/test_build.py,sha256=5t8iliPe_0XwoZjSGkHxUbZaNOWBfc6SpIQijh9DLUA,10037
 tests/test_cql.py,sha256=dAsLMqVaS6qxnq62fg5KqTFu6UG6pHO6Ab3NZ1c9T3Y,5248
 tests/to_ocl.py,sha256=4e-i65K3UM6wHgdVcrZcM9AyL1bahIsXJiZTXhhHgQk,2048
 tricc_oo/__init__.py,sha256=oWCE1ubmC_6iqaWOMgTei4eXVQgV202Ia-tXS1NnW_4,139
 tricc_oo/converters/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-tricc_oo/converters/codesystem_to_ocl.py,sha256=Fh7Vk73OsxljZKu1k6H9uzYwz334tpQTMZBjWWbamYE,6151
+tricc_oo/converters/codesystem_to_ocl.py,sha256=V_oZNVUVaYHgzJtDiBMSrvIJnBWmDNFBDTTHEkOfpXk,6151
 tricc_oo/converters/cql_to_operation.py,sha256=PUyV_YpUY98Ox0H_F_CN3UUf_I-BhFZVOcWWKTtwecM,14492
-tricc_oo/converters/datadictionnary.py,sha256=T2HLCBo4Am1p0kFqSH1r0PqbD8AC2IGuWkbvMvSCru0,3658
-tricc_oo/converters/drawio_type_map.py,sha256=UCPiGs7Lw0bigKScmZUnmOhACBz-FiDq92jHkI7RTSQ,9113
+tricc_oo/converters/datadictionnary.py,sha256=JasqlLKiZzKdidsA1xc2SJ_Af1Xr6A3sKfzDynto8Ho,3686
+tricc_oo/converters/drawio_type_map.py,sha256=Zp8J9iHNSJkIVrmRSM0_d4vA1X8wFPLKb8nCMPUMXKU,9114
 tricc_oo/converters/tricc_to_xls_form.py,sha256=39hwWgYNitGE-AuKtjUwNLz39tEpwc7nd9gT_gw5wjc,3842
 tricc_oo/converters/utils.py,sha256=JZrtrvvOfXwdkw49pKauzinOcauWwsy-CVcw36TjyLo,1684
-tricc_oo/converters/xml_to_tricc.py,sha256=yqnU5xg1SVoQ8vra0gOvIPwoh80pyTI332y9qJCvcRs,39654
+tricc_oo/converters/xml_to_tricc.py,sha256=ea8LNEPDe32q74AJCbEjxaLt_Po47oH45K_G8fo7TzE,40388
 tricc_oo/converters/cql/cqlLexer.py,sha256=8HArbRphcrpnAG4uogJ2rHv4tc1WLzjN0B1uFeYILAc,49141
 tricc_oo/converters/cql/cqlListener.py,sha256=fA7-8DcS2Q69ckwjdg57-OfFHBxjTZFdoSKrtw7Hffc,57538
 tricc_oo/converters/cql/cqlParser.py,sha256=x3KdrwX9nwENSEJ5Ex7_l5NMnu3kWBO0uLdYu4moTq0,414745
@@ -26,27 +25,27 @@ tricc_oo/parsers/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,
 tricc_oo/parsers/xml.py,sha256=uzkb1y18MHfqVFmZqVh0sKT4cx6u0-NcAT_lV_gHBt8,4208
 tricc_oo/serializers/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 tricc_oo/serializers/planuml.py,sha256=t57587-6L3aDncpHh58lS77Zft8yxDE9DPtXx2BeUSU,132
-tricc_oo/serializers/xls_form.py,sha256=-nT5pQG6mKIT_iJWUUjtfn8GzeGzQCR61zlTvxRmoLE,20723
+tricc_oo/serializers/xls_form.py,sha256=ydaD5l_CsMEhJlSO0XvXPdxrwsDw4HiRjTOPHogTQdw,23071
 tricc_oo/strategies/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 tricc_oo/strategies/input/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 tricc_oo/strategies/input/base_input_strategy.py,sha256=BEODXS74na1QRRcJVQ4cxiD8F7uRqaLyhE3QzKpGVvk,3891
 tricc_oo/strategies/input/drawio.py,sha256=uXAUPhXOeg0Uk_BNqlCqFBW4cWNox4VfH559bj1fhC0,12767
 tricc_oo/strategies/output/base_output_strategy.py,sha256=i9L5CVUqkEAMNyBsdHJ4xA7Nptr3myHr_fHHveDX1cU,8928
-tricc_oo/strategies/output/dhis2_form.py,sha256=O5sBcwG_i6Vx3T5-Be4MmE50CsW4obikS3p8MOOJu1o,49317
-tricc_oo/strategies/output/fhir_form.py,sha256=yDDJqdkt4PvSDEy4kmgKM8V_o757CDlE7813atyyDHM,15749
+tricc_oo/strategies/output/dhis2_form.py,sha256=jW9NW72_61ch1bHm8ShIH4xsJH-HMlZGPTT5txJxMUk,38278
+tricc_oo/strategies/output/fhir_form.py,sha256=hbL921pe1Doun4IQrJuZ_Sq2fCh98G3grYie5olC4uc,15740
 tricc_oo/strategies/output/html_form.py,sha256=qSleEZOMV_-Z04y-i-ucyd5rgAYWAyjPwMrw0IHtCRM,8604
 tricc_oo/strategies/output/openmrs_form.py,sha256=ne6TwAyhafR-WDs27QTKKFl85VD5sij_VEJtK6ZjOIE,28996
 tricc_oo/strategies/output/spice.py,sha256=QMeoismVC3PdbvwTK0PtUjWX9jl9780fbQIXn76fMXw,10761
 tricc_oo/strategies/output/xls_form.py,sha256=_pNTND7n-55EjRphJ1hSVtRYa-UkXlmwpam2OKQ8o_w,30860
 tricc_oo/strategies/output/xlsform_cdss.py,sha256=X00Lt5MzV8TX14dR4dFI1MqllI5S1e13bKbeysWM9uA,17435
-tricc_oo/strategies/output/xlsform_cht.py,sha256=gztZBUJj4LdIX1jRvrG3hnLa7qpp0IZP_GpBi7bEzvc,28511
+tricc_oo/strategies/output/xlsform_cht.py,sha256=Zy8aggR1sTBP0b33RGbfUpk8pRppI1LGQEif4E1l49A,28523
 tricc_oo/strategies/output/xlsform_cht_hf.py,sha256=xm6SKirV3nMZvM2w54_zJcXAeAgAkq-EEqGEjnOWv6c,988
 tricc_oo/visitors/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-tricc_oo/visitors/tricc.py,sha256=PpEjW4qXbtjfvvLz4iFw4Nj0Khzida1WSBh8_ZjF9sA,112142
+tricc_oo/visitors/tricc.py,sha256=WvWrL2NAsGjoo2XBLzuKU0L9uNRYE8kRVdkwo8SMwEQ,110715
 tricc_oo/visitors/utils.py,sha256=j83aAq5s5atXi3OC0jc_uJd54a8XrHHmizeeEbWZQJg,421
 tricc_oo/visitors/xform_pd.py,sha256=ryAnI3V9x3eTmJ2LNsUZfvl0_yfCqo6oBgeSu-WPqaE,9613
-tricc_oo-1.6.12.dist-info/licenses/LICENSE,sha256=Pz2eACSxkhsGfW9_iN60pgy-enjnbGTj8df8O3ebnQQ,16726
-tricc_oo-1.6.12.dist-info/METADATA,sha256=Ifx_BtNteuGhT5iysOEJdF-0NXpFWuOGuiA_KRAt6YE,8600
-tricc_oo-1.6.12.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-tricc_oo-1.6.12.dist-info/top_level.txt,sha256=NvbfMNAiy9m4b1unBsqpeOQWh4IgA1Xa33BtKA4abxk,15
-tricc_oo-1.6.12.dist-info/RECORD,,
+tricc_oo-1.6.13.dist-info/licenses/LICENSE,sha256=Pz2eACSxkhsGfW9_iN60pgy-enjnbGTj8df8O3ebnQQ,16726
+tricc_oo-1.6.13.dist-info/METADATA,sha256=QgvsAxKLVdd7mCifr9ZT-RsVQxuxIJIvgfRmfheuRDY,8600
+tricc_oo-1.6.13.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+tricc_oo-1.6.13.dist-info/top_level.txt,sha256=NvbfMNAiy9m4b1unBsqpeOQWh4IgA1Xa33BtKA4abxk,15
+tricc_oo-1.6.13.dist-info/RECORD,,

tests/test_build.py

@@ -1,260 +0,0 @@
-import unittest
-import subprocess
-import sys
-import os
-import tempfile
-import shutil
-from pathlib import Path
-import pandas as pd
-from pyxform import create_survey_from_xls
-
-
-class TestBuildScript(unittest.TestCase):
-    """Test cases for the build.py script with different argument combinations."""
-
-    def setUp(self):
-        """Set up test fixtures."""
-        self.test_data_dir = Path(__file__).parent / "data"
-        self.test_output_dir = Path(__file__).parent / "output"
-        self.demo_file = self.test_data_dir / "demo.drawio"
-
-        # Ensure test data exists
-        self.assertTrue(self.demo_file.exists(), f"Test data file {self.demo_file} does not exist")
-
-    def run_build_script(self, args):
-        """Helper method to run build.py with given arguments."""
-        cmd = [sys.executable, str(Path(__file__).parent / "build.py")] + args
-        result = subprocess.run(cmd, capture_output=True, text=True, cwd=Path(__file__).parent.parent)
-        return result
-
-    def test_basic_build_with_demo_file(self):
-        """Test basic build with demo.drawio file."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    def test_build_with_directory_input(self):
-        """Test build with directory containing drawio files."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", str(self.test_data_dir),
-                "-o", temp_dir,
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    def test_build_with_xlsform_strategy(self):
-        """Test build with XLSFormStrategy (default)."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-O", "XLSFormStrategy",
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    def test_build_with_html_strategy(self):
-        """Test build with HTMLStrategy."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-O", "HTMLStrategy",
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    def test_build_with_fhir_strategy(self):
-        """Test build with FHIRStrategy."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-O", "FHIRStrategy",
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    def test_build_with_dhis2_strategy(self):
-        """Test build with DHIS2Strategy."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-O", "DHIS2Strategy",
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    def test_build_with_openmrs_strategy(self):
-        """Test build with OpenMRSStrategy."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-O", "OpenMRSStrategy",
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    def test_build_with_cht_strategy(self):
-        """Test build with XLSFormCHTStrategy."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-O", "XLSFormCHTStrategy",
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    def test_build_with_cht_hf_strategy(self):
-        """Test build with XLSFormCHTHFStrategy."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-O", "XLSFormCHTHFStrategy",
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    def test_build_with_cdss_strategy(self):
-        """Test build with XLSFormCDSSStrategy."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-O", "XLSFormCDSSStrategy",
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    # def test_build_with_spice_strategy(self):
-    #     """Test build with SpiceStrategy."""
-    #     with tempfile.TemporaryDirectory() as temp_dir:
-    #         args = [
-    #             "-i", str(self.demo_file),
-    #             "-o", temp_dir,
-    #             "-O", "SpiceStrategy",
-    #             "-l", "i"
-    #         ]
-    #         result = self.run_build_script(args)
-    #         self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    def test_build_with_debug_level(self):
-        """Test build with debug logging level."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-l", "d"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    def test_build_with_trad_option(self):
-        """Test build with translation option."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-t",
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    def test_build_with_form_id(self):
-        """Test build with custom form ID."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-d", "test_form_123",
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-    def test_build_missing_input(self):
-        """Test build with missing input file."""
-        args = ["-o", "/tmp/test_output"]
-        result = self.run_build_script(args)
-        self.assertNotEqual(result.returncode, 0, "Build should fail with missing input")
-
-    def test_build_invalid_input_file(self):
-        """Test build with invalid input file."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            args = [
-                "-i", "/nonexistent/file.drawio",
-                "-o", temp_dir,
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertNotEqual(result.returncode, 0, "Build should fail with invalid input file")
-
-    def validate_xls_form(self, xls_path):
-        """Helper method to validate XLS form using ODK libraries."""
-        try:
-            # Convert XLS to XML using pyxform
-            survey = create_survey_from_xls(xls_path)
-            xml_output = survey.to_xml()
-
-            # Basic validation - check if XML was generated successfully
-            # In a real scenario, you might want to use odk_validate command line tool
-            if xml_output and len(xml_output.strip()) > 0:
-                return True, "Validation successful"
-            else:
-                return False, "Empty XML output"
-        except Exception as e:
-            return False, str(e)
-
-    def test_xlsform_strategy_validation(self):
-        """Test XLSFormStrategy output validation with ODK libraries."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            xls_output = Path(temp_dir) / "demo_tricc.xlsx"
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-O", "XLSFormStrategy",
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-            # Check if XLS file was created
-            self.assertTrue(xls_output.exists(), f"XLS file {xls_output} was not created")
-
-    def test_xlsform_cdss_strategy_validation(self):
-        """Test XLSFormCDSSStrategy output validation with ODK libraries."""
-        with tempfile.TemporaryDirectory() as temp_dir:
-            xls_output = Path(temp_dir) / "demo_tricc.xlsx"
-            args = [
-                "-i", str(self.demo_file),
-                "-o", temp_dir,
-                "-O", "XLSFormCDSSStrategy",
-                "-l", "i"
-            ]
-            result = self.run_build_script(args)
-            self.assertEqual(result.returncode, 0, f"Build failed: {result.stderr}")
-
-
-
-if __name__ == "__main__":
-    unittest.main()