bec-server 4.3.3__tar.gz → 4.4.0__tar.gz

{bec_server-4.3.3 → bec_server-4.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: bec-server
-Version: 4.3.3
+Version: 4.4.0
 Summary: BEC server
 Project-URL: Bug Tracker, https://github.com/bec-project/bec/issues
 Project-URL: Homepage, https://github.com/bec-project/bec

{bec_server-4.3.3 → bec_server-4.4.0}/bec_server/device_server/devices/config_update_handler.py

@@ -238,8 +238,14 @@ class ConfigUpdateHandler:
                 # apply config
                 try:
                     self.device_manager.update_config(device.obj, new_config)
+                    # Persistence flags and other signal metadata can change with config.
+                    # Publish before acknowledgement and the subsequent proxy update.
+                    error = self.device_manager.publish_device_info(device.obj)
+                    if error is not None:
+                        raise error
                 except Exception as exc:
                     self.device_manager.update_config(device.obj, old_config)
+                    self.device_manager.publish_device_info(device.obj)
                     raise DeviceConfigError(f"Error during object update. {exc}")
 
                 if "limits" in dev_config["deviceConfig"]:

{bec_server-4.3.3 → bec_server-4.4.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "bec-server"
-version = "4.3.3"
+version = "4.4.0"
 description = "BEC server"
 requires-python = ">=3.11"
 classifiers = [

bec_server-4.4.0/tests/tests_device_server/test_config_metadata_refresh.py

@@ -0,0 +1,56 @@
+"""Runtime config updates publish signal metadata before acknowledgement."""
+
+import threading
+from types import SimpleNamespace
+from unittest import mock
+
+import pytest
+
+from bec_lib.bec_errors import DeviceConfigError
+from bec_lib.messages import DeviceConfigMessage
+from bec_server.device_server.devices.config_update_handler import ConfigUpdateHandler
+
+
+def make_handler():
+    device = SimpleNamespace(save_roi_data=True)
+    manager = SimpleNamespace(
+        devices={
+            "camera": SimpleNamespace(
+                obj=device, enabled=True, _config={"deviceConfig": {"save_roi_data": True}}
+            )
+        },
+        update_config=mock.Mock(
+            side_effect=lambda obj, cfg: setattr(obj, "save_roi_data", cfg["save_roi_data"])
+        ),
+        publish_device_info=mock.Mock(return_value=None),
+    )
+    return SimpleNamespace(device_manager=manager), device
+
+
+def test_update_config_republishes_metadata_after_applying_values():
+    handler, device = make_handler()
+    observed = []
+    handler.device_manager.publish_device_info.side_effect = lambda obj: observed.append(
+        obj.save_roi_data
+    )
+    msg = DeviceConfigMessage(
+        action="update", config={"camera": {"deviceConfig": {"save_roi_data": False}}}
+    )
+    ConfigUpdateHandler._update_config(handler, msg, threading.Event())
+    assert observed == [False]
+    handler.device_manager.publish_device_info.assert_called_once_with(device)
+
+
+def test_failed_metadata_publication_restores_old_config_and_metadata():
+    handler, device = make_handler()
+    handler.device_manager.publish_device_info.side_effect = [
+        RuntimeError("Metadata unavailable"),
+        None,
+    ]
+    msg = DeviceConfigMessage(
+        action="update", config={"camera": {"deviceConfig": {"save_roi_data": False}}}
+    )
+    with pytest.raises(DeviceConfigError, match="Metadata unavailable"):
+        ConfigUpdateHandler._update_config(handler, msg, threading.Event())
+    assert device.save_roi_data is True
+    assert handler.device_manager.publish_device_info.call_count == 2