Update fast shutter and selectable source device names

src/dodal/devices/fast_shutter.py

@@ -67,10 +67,15 @@ def __init__(
     ):
         self.open_state = open_state
         self.close_state = close_state
-        with self.add_children_as_readables():
+        with self.add_children_as_readables(StandardReadableFormat.HINTED_SIGNAL):
             self.shutter_state = epics_signal_rw(type(self.open_state), pv)
         super().__init__(name)
 
+    def set_name(self, name: str, *, child_name_separator: str | None = None) -> None:
+        """Set name of the device and its children."""
+        super().set_name(name, child_name_separator=child_name_separator)
+        self.shutter_state.set_name(name)
+
 
 class DualFastShutter(StandardReadable, FastShutter[EnumTypesT], Generic[EnumTypesT]):
     """A fast shutter device that handles the positions of two other fast shutters. The
@@ -104,7 +109,7 @@ def __init__(
         self._shutter2_ref = Reference(shutter2)
         self._selected_shutter_ref = Reference(selected_source)
 
-        with self.add_children_as_readables():
+        with self.add_children_as_readables(StandardReadableFormat.HINTED_SIGNAL):
             self.shutter_state = derived_signal_rw(
                 self._read_shutter_state,
                 self._set_shutter_state,
@@ -153,3 +158,8 @@ async def _set_shutter_state(self, value: EnumTypesT):
         )
         await inactive_shutter.set(inactive_shutter.close_state)
         await active_shutter.set(value)
+
+    def set_name(self, name: str, *, child_name_separator: str | None = None) -> None:
+        """Set name of the device and its children."""
+        super().set_name(name, child_name_separator=child_name_separator)
+        self.shutter_state.set_name(name)

src/dodal/devices/selectable_source.py

@@ -1,7 +1,13 @@
 from typing import TypeVar
 
 from bluesky.protocols import Movable
-from ophyd_async.core import AsyncStatus, StandardReadable, StrictEnum, soft_signal_rw
+from ophyd_async.core import (
+    AsyncStatus,
+    StandardReadable,
+    StandardReadableFormat,
+    StrictEnum,
+    soft_signal_rw,
+)
 
 
 class SelectedSource(StrictEnum):
@@ -30,12 +36,17 @@ class SourceSelector(StandardReadable, Movable[SelectedSource]):
     """
 
     def __init__(self, name: str = ""):
-        with self.add_children_as_readables():
+        with self.add_children_as_readables(StandardReadableFormat.HINTED_SIGNAL):
             self.selected_source = soft_signal_rw(
                 SelectedSource, SelectedSource.SOURCE1
             )
         super().__init__(name)
 
+    def set_name(self, name: str, *, child_name_separator: str | None = None) -> None:
+        """Set name of the device and its children."""
+        super().set_name(name, child_name_separator=child_name_separator)
+        self.selected_source.set_name(name)
+
     @AsyncStatus.wrap
     async def set(self, value: SelectedSource):
         await self.selected_source.set(value)

tests/devices/test_fast_shutters.py

@@ -31,9 +31,7 @@ async def test_shutter_set_open_close_without_knowing_enum_values(
 
 
 async def test_shutter_read(shutter1: GenericFastShutter) -> None:
-    await assert_reading(
-        shutter1, {f"{shutter1.name}-shutter_state": partial_reading(InOut.IN)}
-    )
+    await assert_reading(shutter1, {shutter1.name: partial_reading(InOut.IN)})
 
 
 @pytest.fixture
@@ -139,7 +137,7 @@ async def test_dual_fast_shutter_read(
     )
     await assert_reading(
         dual_fast_shutter,
-        {f"{dual_fast_shutter.name}-shutter_state": partial_reading(InOut.IN)}
+        {dual_fast_shutter.name: partial_reading(InOut.IN)}
         | shutter1_read
         | shutter2_read
         | source_selector_read,

tests/devices/test_selectable_source.py

@@ -28,11 +28,7 @@ async def test_source_selector_set(
 async def test_source_selector_read(source_selector: SourceSelector) -> None:
     await assert_reading(
         source_selector,
-        {
-            f"{source_selector.name}-selected_source": partial_reading(
-                SelectedSource.SOURCE1
-            )
-        },
+        {source_selector.name: partial_reading(SelectedSource.SOURCE1)},
     )