BaseProxyObject: construct child proxy objects with correct paths

BaseProxyObject.get_children() was constructing child proxy objects
using the parent object's path. This commit corrects that.

Author: whitslack

src/dbus_fast/proxy_object.py

@@ -351,7 +351,8 @@ def get_children(self) -> list[BaseProxyObject]:
         """Get the child nodes of this proxy object according to the introspection data."""
         if self._children is None:
             self._children = [
-                self.__class__(self.bus_name, self.path, child, self.bus)
+                self.__class__(self.bus_name, f"{self.path}/{child.name}",
+                        child, self.bus)
                 for child in self.introspection.nodes
             ]
 

tests/test_introspection.py

@@ -10,7 +10,7 @@
 )
 from dbus_fast import introspection as intr
 from dbus_fast.constants import ErrorType
-from dbus_fast.errors import DBusError
+from dbus_fast.errors import DBusError, InterfaceNotFoundError
 from dbus_fast.message_bus import BaseMessageBus
 from dbus_fast.proxy_object import BaseProxyObject, BaseProxyInterface
 
@@ -206,7 +206,6 @@ def __init__(self,
         super().__init__(bus_name, path, introspection, bus, MockProxyInterface)
 
 
-@pytest.mark.xfail(reason='buggy BaseProxyObject.get_children', raises=AssertionError, strict=True)
 def test_inline_child():
     bus = MockMessageBus({
             'com.example': {
@@ -255,7 +254,7 @@ def test_inline_child():
     assert bus.introspect_count == 1
 
 
-@pytest.mark.xfail(reason='buggy BaseProxyObject.get_children', raises=AssertionError, strict=True)
+@pytest.mark.xfail(reason='buggy BaseProxyObject.get_children', raises=InterfaceNotFoundError, strict=True)
 def test_noninline_child():
     obj0_node = intr.Node.parse(strict_data)
     bus = MockMessageBus({