add no fields if no signal name (#366)

* add no fields if no signal name

* fix trailing whitespace

* add the test

* fix the imports

* add an assert line and try changing encodings

* add new line at the end

src/ophyd_async/core/standard_readable.py

@@ -254,6 +254,8 @@ def name(self) -> str:
 
  @property
  def hints(self) -> Hints:
+ if self.signal.name == "":
+ return {"fields": []}
  return {"fields": [self.signal.name]}
 
  @classmethod

tests/core/test_standard_readable.py

@@ -8,7 +8,7 @@
 from ophyd_async.core import ConfigSignal, HintedSignal, StandardReadable
 from ophyd_async.core.device import Device, DeviceVector
 from ophyd_async.core.mock_signal_backend import MockSignalBackend
-from ophyd_async.core.signal import SignalR
+from ophyd_async.core.signal import SignalR, soft_signal_r_and_setter
 from ophyd_async.protocols import AsyncConfigurable, AsyncReadable, AsyncStageable
 
 
@@ -219,3 +219,14 @@ def test_standard_readable_set_readable_signals():
  assert all(isinstance(x, ConfigSignal) for x in sr._configurables)
  assert len(sr._stageables) == 1
  assert all(isinstance(x, HintedSignal) for x in sr._stageables)
+
+
+def test_standard_readable_add_children_multi_nested():
+ inner = StandardReadable()
+ outer = StandardReadable()
+ with inner.add_children_as_readables(HintedSignal):
+ inner.a, _ = soft_signal_r_and_setter(float, initial_value=5.0)
+ inner.b, _ = soft_signal_r_and_setter(float, initial_value=6.0)
+ with outer.add_children_as_readables():
+ outer.inner = inner
+ assert outer