chore(pre-commit.ci): auto fixes

Author: pre-commit-ci[bot]

src/dbus_fast/aio/message_bus.py

@@ -318,12 +318,7 @@ async def introspect(
         finally:
             timer_handle.cancel()
 
-    def introspect_sync(
-        self,
-        bus_name: str,
-        path: str,
-        *args, **kwargs
-    ) -> intr.Node:
+    def introspect_sync(self, bus_name: str, path: str, *args, **kwargs) -> intr.Node:
         return self._loop.run_until_complete(
             self.introspect(bus_name, path, *args, **kwargs)
         )

src/dbus_fast/glib/proxy_object.py

@@ -1,5 +1,4 @@
 import xml.etree.ElementTree as ET
-from typing import Union
 
 from .. import introspection as intr
 from ..constants import ErrorType

src/dbus_fast/proxy_object.py

@@ -311,7 +311,9 @@ def child_paths(self) -> List[str]:
             - :class:`InvalidIntrospectionError <dbus_fast.InvalidIntrospectionError>` - If the introspection data for the node is not valid.
         """
         if self._child_paths is None:
-            self._child_paths = [f"{self.path}/{n.name}" for n in self.introspection.nodes]
+            self._child_paths = [
+                f"{self.path}/{n.name}" for n in self.introspection.nodes
+            ]
         return self._child_paths
 
     def get_interface(self, name: str) -> BaseProxyInterface:
@@ -379,9 +381,12 @@ 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, f"{self.path}/{child.name}",
-                        child if child.interfaces or child.nodes else None,
-                        self.bus)
+                self.__class__(
+                    self.bus_name,
+                    f"{self.path}/{child.name}",
+                    child if child.interfaces or child.nodes else None,
+                    self.bus,
+                )
                 for child in self.introspection.nodes
             ]
 

tests/test_introspection.py

@@ -1,5 +1,4 @@
 import os
-import pytest
 import xml.etree.ElementTree as ET
 
 from dbus_fast import (
@@ -12,7 +11,7 @@
 from dbus_fast.constants import ErrorType
 from dbus_fast.errors import DBusError
 from dbus_fast.message_bus import BaseMessageBus
-from dbus_fast.proxy_object import BaseProxyObject, BaseProxyInterface
+from dbus_fast.proxy_object import BaseProxyInterface, BaseProxyObject
 
 with open(f"{os.path.dirname(__file__)}/data/strict-introspection.xml") as f:
     strict_data = f.read()
@@ -154,30 +153,27 @@ def test_default_interfaces():
 
 
 class MockMessageBus(BaseMessageBus):
-
-    def __init__(self,
-        nodes: dict[str, dict[str, intr.Node]]
-    ) -> None:
+    def __init__(self, nodes: dict[str, dict[str, intr.Node]]) -> None:
         super().__init__(ProxyObject=MockProxyObject)
         self.nodes = nodes
         self.introspect_count = 0
+
         # disable bus name tracking for testing purposes
         class MockNameOwners:
             @staticmethod
             def get(key, default):
-                return ':'
+                return ":"
+
         self._name_owners = MockNameOwners
 
     def introspect_sync(self, bus_name: str, path: str) -> intr.Node:
         self.introspect_count = self.introspect_count + 1
         service = self.nodes.get(bus_name)
         if service is None:
-            raise DBusError(ErrorType.NAME_HAS_NO_OWNER,
-                    f"unknown service: {bus_name}")
+            raise DBusError(ErrorType.NAME_HAS_NO_OWNER, f"unknown service: {bus_name}")
         node = service.get(path)
         if node is None:
-            raise DBusError(ErrorType.UNKNOWN_OBJECT,
-                    f"unknown object: {path}")
+            raise DBusError(ErrorType.UNKNOWN_OBJECT, f"unknown object: {path}")
         return node
 
     def _setup_socket(self) -> None:
@@ -188,7 +184,6 @@ def _init_high_level_client(self) -> None:
 
 
 class MockProxyInterface(BaseProxyInterface):
-
     def _add_method(self, intr_method: intr.Method) -> None:
         pass
 
@@ -197,21 +192,22 @@ def _add_property(self, intr_property: intr.Property) -> None:
 
 
 class MockProxyObject(BaseProxyObject):
-
-    def __init__(self,
+    def __init__(
+        self,
         bus_name: str,
         path: str,
         introspection: intr.Node | str | ET.Element | None,
-        bus: BaseMessageBus
+        bus: BaseMessageBus,
     ) -> None:
         super().__init__(bus_name, path, introspection, bus, MockProxyInterface)
 
 
 def test_inline_child():
-    bus = MockMessageBus({
-            'com.example': {
-                '/com/example/parent_object': intr.Node.parse(
-"""<!DOCTYPE node PUBLIC "-//freedesktop//DTD D-BUS Object Introspection 1.0//EN"
+    bus = MockMessageBus(
+        {
+            "com.example": {
+                "/com/example/parent_object": intr.Node.parse(
+                    """<!DOCTYPE node PUBLIC "-//freedesktop//DTD D-BUS Object Introspection 1.0//EN"
     "http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd">
 <node name="/com/example/parent_object">
     <interface name="com.example.ParentInterface">
@@ -222,78 +218,84 @@ def test_inline_child():
             <property name="ChildProperty" type="i"/>
         </interface>
     </node>
-</node>""")
+</node>"""
+                )
             }
-        })
-    introspection = bus.introspect_sync('com.example',
-            '/com/example/parent_object')
+        }
+    )
+    introspection = bus.introspect_sync("com.example", "/com/example/parent_object")
     assert bus.introspect_count == 1
 
-    parent = bus.get_proxy_object('com.example', '/com/example/parent_object',
-            introspection)
-    assert parent.bus_name == 'com.example'
-    assert parent.path == '/com/example/parent_object'
-    assert parent.child_paths == [
-            '/com/example/parent_object/child_object'
-        ]
-    interface = parent.get_interface('com.example.ParentInterface')
-    assert interface.bus_name == 'com.example'
-    assert interface.path == '/com/example/parent_object'
+    parent = bus.get_proxy_object(
+        "com.example", "/com/example/parent_object", introspection
+    )
+    assert parent.bus_name == "com.example"
+    assert parent.path == "/com/example/parent_object"
+    assert parent.child_paths == ["/com/example/parent_object/child_object"]
+    interface = parent.get_interface("com.example.ParentInterface")
+    assert interface.bus_name == "com.example"
+    assert interface.path == "/com/example/parent_object"
     assert [method.name for method in interface.introspection.methods] == [
-            'ParentMethod'
-        ]
+        "ParentMethod"
+    ]
 
     child = next(iter(parent.get_children()))
-    assert child.bus_name == 'com.example'
-    assert child.path == '/com/example/parent_object/child_object'
-    interface = child.get_interface('com.example.ChildInterface')
+    assert child.bus_name == "com.example"
+    assert child.path == "/com/example/parent_object/child_object"
+    interface = child.get_interface("com.example.ChildInterface")
     assert [prop.name for prop in interface.introspection.properties] == [
-            'ChildProperty'
-        ]
+        "ChildProperty"
+    ]
 
     # getting the inline child should not have required another introspection
     assert bus.introspect_count == 1
 
 
 def test_noninline_child():
     obj0_node = intr.Node.parse(strict_data)
-    bus = MockMessageBus({
-            'com.example': {
+    bus = MockMessageBus(
+        {
+            "com.example": {
                 obj0_node.name: obj0_node,
-                f'{obj0_node.name}/child_of_sample_object': intr.Node.parse(
-f"""<!DOCTYPE node PUBLIC "-//freedesktop//DTD D-BUS Object Introspection 1.0//EN"
+                f"{obj0_node.name}/child_of_sample_object": intr.Node.parse(
+                    f"""<!DOCTYPE node PUBLIC "-//freedesktop//DTD D-BUS Object Introspection 1.0//EN"
     "http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd">
 <node name="{obj0_node.name}/child_of_sample_object">
     <interface name="com.example.ChildInterface">
         <property name="ChildProperty" type="i"/>
     </interface>
-</node>"""),
+</node>"""
+                ),
             }
-        })
-    introspection = bus.introspect_sync('com.example',
-            '/com/example/sample_object0')
+        }
+    )
+    introspection = bus.introspect_sync("com.example", "/com/example/sample_object0")
     assert bus.introspect_count == 1
-    parent = bus.get_proxy_object('com.example', '/com/example/sample_object0',
-            introspection)
+    parent = bus.get_proxy_object(
+        "com.example", "/com/example/sample_object0", introspection
+    )
 
-    assert parent.path == '/com/example/sample_object0'
+    assert parent.path == "/com/example/sample_object0"
     assert parent.child_paths == [
-            '/com/example/sample_object0/child_of_sample_object',
-            '/com/example/sample_object0/another_child_of_sample_object',
-        ]
+        "/com/example/sample_object0/child_of_sample_object",
+        "/com/example/sample_object0/another_child_of_sample_object",
+    ]
     children = parent.get_children()
     assert [child.path for child in children] == parent.child_paths
 
     # merely listing the children should not have required another introspection
     assert bus.introspect_count == 1
 
-    child = next(child for child in children if child.path == \
-            '/com/example/sample_object0/child_of_sample_object')
-    interface = child.get_interface('com.example.ChildInterface')
+    child = next(
+        child
+        for child in children
+        if child.path == "/com/example/sample_object0/child_of_sample_object"
+    )
+    interface = child.get_interface("com.example.ChildInterface")
 
     # obtaining the child interface should have required a second introspection
     assert bus.introspect_count == 2
 
     assert [prop.name for prop in interface.introspection.properties] == [
-            'ChildProperty'
-        ]
+        "ChildProperty"
+    ]