Enable reportUnsafeMultipleInheritance

pyproject.toml

@@ -108,7 +108,6 @@ reportMissingTypeStubs = "information"      # consider bumping to warning/error
 reportUninitializedInstanceVariable = false # until https://github.com/DetachHead/basedpyright/issues/491
 reportUnnecessaryTypeIgnoreComment = false  # temporary
 reportMissingParameterType = false          # likely will be enabled (although ruff does cover this)
-reportUnsafeMultipleInheritance = false     # likely will be enabled
 reportMissingSuperCall = false              # likely will be enabled
 reportIgnoreCommentWithoutRule = false      # likely will be enabled (but only if enableTypeIgnoreComments remains disabled)
 

tests/mcproto/protocol/helpers.py

@@ -33,7 +33,7 @@ def assert_has_data(self, data: bytearray, ensure_called: bool = True) -> None:
             raise AssertionError(f"Write function mock expected data {data!r}, but was {self.call_data!r}")
 
 
-class WriteFunctionAsyncMock(WriteFunctionMock, AsyncMock):
+class WriteFunctionAsyncMock(WriteFunctionMock, AsyncMock):  # pyright: ignore[reportUnsafeMultipleInheritance]
     """Asynchronous mock write function, storing the written data."""
 
 
@@ -71,5 +71,5 @@ def assert_read_everything(self, ensure_called: bool = True) -> None:
             )
 
 
-class ReadFunctionAsyncMock(ReadFunctionMock, AsyncMock):
+class ReadFunctionAsyncMock(ReadFunctionMock, AsyncMock):  # pyright: ignore[reportUnsafeMultipleInheritance]
     """Asynchronous mock read function, giving pre-defined data."""

tests/mcproto/test_connection.py

@@ -13,7 +13,7 @@
 from tests.mcproto.protocol.helpers import ReadFunctionAsyncMock, ReadFunctionMock, WriteFunctionMock
 
 
-class MockSocket(CustomMockMixin[MagicMock], MagicMock):
+class MockSocket(CustomMockMixin[MagicMock], MagicMock):  # pyright: ignore[reportUnsafeMultipleInheritance]
     """Mock version of a socket (synchronous), using our mocked writer and reader methods.
 
     See :class:`tests.mcproto.protocol.helpers.ReadFunctionMock` and
@@ -53,7 +53,7 @@ def shutdown(self, __how: int, /) -> None:
         """Mock version of shutdown, without any real implementation."""
 
 
-class MockStreamWriter(CustomMockMixin[MagicMock], MagicMock):
+class MockStreamWriter(CustomMockMixin[MagicMock], MagicMock):  # pyright: ignore[reportUnsafeMultipleInheritance]]
     """Mock version of :class:`asyncio.StreamWriter` using our mocked writer method."""
 
     spec_set = asyncio.StreamWriter
@@ -77,7 +77,7 @@ def close(self) -> None:
         self._closed = True
 
 
-class MockStreamReader(CustomMockMixin[MagicMock], MagicMock):
+class MockStreamReader(CustomMockMixin[MagicMock], MagicMock):  # pyright: ignore[reportUnsafeMultipleInheritance]]
     """Mock version of :class:`asyncio.StreamReader` using our mocked reader method."""
 
     spec_set = asyncio.StreamReader