add a test for optional attribute with nameType=partial and concept name in template

Author: lukaspie

.github/workflows/plugin_test.yaml

@@ -33,7 +33,7 @@ jobs:
             branch: main
             tests_to_run: tests/.    
           - plugin: pynxtools-mpes
-            branch: entry-identifier
+            branch: main
             tests_to_run: tests/.
           - plugin: pynxtools-raman
             branch: sibling-inheritance

src/pynxtools/dataconverter/helpers.py

@@ -151,7 +151,6 @@ def _log(self, path: str, log_type: ValidationProblem, value: Optional[Any], *ar
             )
         elif log_type == ValidationProblem.KeyToBeRemoved:
             logger.warning(f"The attribute {path} will not be written.")
-
         elif log_type == ValidationProblem.InvalidConceptForNonVariadic:
             value = cast(Any, value)
             log_text = f"Given {value.type} name '{path}' conflicts with the non-variadic name '{value}'"

src/pynxtools/dataconverter/nexus_tree.py

@@ -942,7 +942,6 @@ def convert_to_hashable(item):
                         pass
                     else:
                         # TODO: should we be this strict here? Or can appdefs define additional terms?
-                        print(self, set_items, set_elem_enum_items)
                         pass
             return True
 

tests/dataconverter/test_validation.py

@@ -762,6 +762,15 @@ def listify_template(data_dict: Template):
             [],
             id="concept-name-given-for-nonvariadic-field",
         ),
+        pytest.param(
+            alter_dict(
+                TEMPLATE,
+                "/ENTRY[my_entry]/optional_parent/@AXISNAME_indices[@required_child_indices]",
+                0,
+            ),
+            [],
+            id="concept-name-given-for-optional-attribute",
+        ),
         pytest.param(
             alter_dict(
                 remove_from_dict(