Add missing callback setter to LinkedSelectionRule and TaperEdge

The '_set_callback_apply_changes' function implementation was
missing from the LinkedSelectionRule and TaperEdge classes.

To detect such issues in future, set the 'mypy-tests' pre-commit
run to use the '--strict' flag, but explicitly ignoring the
'no-untyped-def' and 'no-untyped-call' error codes.

Add an explicit test for the linked selection rule properties.

General note: The 'GenericEdgePropertyType' classes have too much
duplication for my taste, and could use some cleanup / creation
of helper functions similar to the other tree other types.

.pre-commit-config.yaml

@@ -69,4 +69,10 @@ repos:
         types: [python]
         require_serial: true
         files: "^(tests/)|^(examples/)"
-        args: ["--namespace-packages", "--explicit-package-bases"]
+        args: [
+          "--strict",
+          "--namespace-packages",
+          "--explicit-package-bases",
+          "--disable-error-code", "no-untyped-def",
+          "--disable-error-code", "no-untyped-call"
+        ]

src/ansys/acp/core/_tree_objects/linked_selection_rule.py

@@ -147,6 +147,9 @@ def parameter_2(self, value: float) -> None:
         if self._callback_apply_changes is not None:
             self._callback_apply_changes()
 
+    def _set_callback_apply_changes(self, callback_apply_changes: Callable[[], None]) -> None:
+        self._callback_apply_changes = callback_apply_changes
+
     @classmethod
     def _from_pb_object(
         cls,

src/ansys/acp/core/_tree_objects/modeling_ply.py

@@ -144,6 +144,9 @@ def offset(self, offset: float) -> None:
         if self._callback_apply_changes is not None:
             self._callback_apply_changes()
 
+    def _set_callback_apply_changes(self, callback_apply_changes: Callable[[], None]) -> None:
+        self._callback_apply_changes = callback_apply_changes
+
     @classmethod
     def _from_pb_object(
         cls,

tests/unittests/common/tree_object_tester.py

@@ -126,7 +126,7 @@ def test_collection_delitem(collection_test_data):
             object_collection[ref_id]
 
 
-class NoLockedMixin:
+class NoLockedMixin(TreeObjectTester):
     @pytest.fixture
     def collection_test_data(self, parent_object):
         object_collection = getattr(parent_object, self.COLLECTION_NAME)
@@ -147,7 +147,7 @@ def test_collection_clear(collection_test_data):
         assert len(object_collection) == 0
 
 
-class WithLockedMixin:
+class WithLockedMixin(TreeObjectTester):
     INITIAL_OBJECT_NAMES: tuple[str, ...]
 
     @pytest.fixture

tests/unittests/test_modeling_ply.py

@@ -330,3 +330,20 @@ def test_nodal_data_to_pyvista_with_component(
         ), f"Created wrong mesh type UnstructuredGrid for component '{component}'"
         assert pv_mesh.n_points == 4
         assert pv_mesh.n_cells == 1
+
+
+def test_linked_selection_rule_parameters(simple_modeling_ply, minimal_complete_model):
+    parallel_selection_rule = minimal_complete_model.create_parallel_selection_rule(
+        origin=(0.0, 0.0, 0.0), direction=(1.0, 0.0, 0.0), lower_limit=0.005, upper_limit=1.0
+    )
+    linked_parallel_rule = LinkedSelectionRule(parallel_selection_rule, template_rule=True)
+    simple_modeling_ply.selection_rules = [linked_parallel_rule]
+
+    linked_parallel_rule.parameter_1 = 1.0
+    minimal_complete_model.update()
+
+    assert linked_parallel_rule.parameter_1 == simple_modeling_ply.selection_rules[0].parameter_1
+
+    linked_rule_in_ply = simple_modeling_ply.selection_rules[0]
+    linked_rule_in_ply.parameter_1 = 1
+    assert linked_parallel_rule.parameter_1 == simple_modeling_ply.selection_rules[0].parameter_1