Fix OnChangedDict behaviour with recent python versions (#894)

Co-authored-by: Robert Resch <robert@resch.dev>

Author: nanomad

.grit/.gritmodules/github.com/getgrit/stdlib

@@ -0,0 +1 @@
+Subproject commit dae72c105649e1a9a954bb576f210ed28fd38208

deebot_client/util/__init__.py

@@ -81,8 +81,6 @@ class OnChangedDict(dict[_KT, _VT]):
         "pop",
         "popitem",
         "update",
-        "__setitem__",
-        "__delitem__",
     )
 
     def __init__(
@@ -91,6 +89,16 @@ def __init__(
         super().__init__(iterable)
         self._on_change = on_change
 
+    # This is needed as __getattribute__ won't be invoked for implicit special method lookup
+    def __setitem__(self, key: _KT, value: _VT) -> None:
+        self._on_change()
+        super().__setitem__(key, value)
+
+    # This is needed as __getattribute__ won't be invoked for implicit special method lookup
+    def __delitem__(self, key: _KT) -> None:
+        self._on_change()
+        return super().__delitem__(key)
+
     def __getattribute__(self, name: str, /) -> Any:
         if name in OnChangedDict._MODIFYING_FUNCTIONS:
             self._on_change()

tests/util/test_init.py

@@ -3,7 +3,7 @@
 import asyncio
 from typing import Any
 
-from deebot_client.util import cancel, create_task
+from deebot_client.util import OnChangedDict, cancel, create_task
 
 
 async def test_create_task_and_cancel() -> None:
@@ -35,3 +35,39 @@ async def sleep(delay: float) -> None:
     for task in _tasks:
         assert task.cancelled()
         assert task.done()
+
+
+def test_on_changed_dict() -> None:
+    callback_count = 0
+
+    def increase_counter() -> None:
+        nonlocal callback_count
+        callback_count += 1
+
+    sut: OnChangedDict[str, int] = OnChangedDict(increase_counter)
+
+    sut["test"] = 1001
+    assert callback_count == 1
+    assert sut["test"] == 1001
+
+    sut.update({"test": 1002, "test2": 2001, "test3": 3001, "test4": 4001})
+    assert sut["test"] == 1002
+    assert callback_count == 2
+
+    del sut["test"]
+    assert "test" not in sut
+    assert callback_count == 3
+
+    assert sut.pop("test2") == 2001
+    assert "test2" not in sut
+    assert callback_count == 4
+
+    (popped_key, popped_value) = sut.popitem()
+    assert popped_key == "test4"
+    assert popped_value == 4001
+    assert "test4" not in sut
+    assert callback_count == 5
+
+    sut.clear()
+    assert sut == {}
+    assert callback_count == 6