Allow comparison with a plain Mapping

Since the comparison only compares the underlying `dict` and doesn't
have into account the receiver, but just the current cache state, we can
safely allow comparing to other `Mapping`s too.

Signed-off-by: Leandro Lucarella <[email protected]>

Author: llucax

src/frequenz/channels/experimental/_grouping_latest_value_cache.py

@@ -200,9 +200,13 @@ def __eq__(self, other: object, /) -> bool:
         Returns:
             `True` if the caches are equal, `False` otherwise.
         """
-        if not isinstance(other, GroupingLatestValueCache):
-            return NotImplemented
-        return self._latest_value_by_key == other._latest_value_by_key
+        match other:
+            case GroupingLatestValueCache():
+                return self._latest_value_by_key == other._latest_value_by_key
+            case Mapping():
+                return self._latest_value_by_key == other
+            case _:
+                return NotImplemented
 
     @override
     def __ne__(self, value: object, /) -> bool:

tests/test_grouping_latest_value_cache_integration.py

@@ -88,7 +88,7 @@ async def test_latest_value_cache_key() -> None:  # pylint: disable=too-many-sta
     assert cache.keys() == expected.keys()
     assert list(cache.values()) == list(expected.values())
     assert list(cache.items()) == list(expected.items())
-    # assert cache == expected
+    assert cache == expected
     assert list(cache) == list(expected)
 
     cache.clear()