AmpelAlert: restore id-based hash

ampel/alert/AmpelAlert.py

@@ -41,6 +41,9 @@ class AmpelAlert:
 	tag: None | Tag | list[Tag] = None #: Optional tag associated with this alert
 	extra: None | JDict = None #: Optional information associated with this alert
 
+	def __hash__(self) -> int:
+		return id(self)
+
 	def get_values(self,
 		key: str,
 		filters: None | Sequence[JDict] = None

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "ampel-interface"
-version = "0.10.4.post0"
+version = "0.10.4.post1"
 description = "Base classes for the Ampel analysis platform"
 authors = ["Valery Brinnel"]
 maintainers = ["Jakob van Santen <[email protected]>"]

tests/test_AmpelAlert.py

@@ -1,9 +1,37 @@
+from copy import deepcopy
+
+import pytest
+from pydantic import ValidationError
+
 from ampel.alert.AmpelAlert import AmpelAlert
+from ampel.base.AmpelBaseModel import AmpelBaseModel
 from ampel.protocol.AmpelAlertProtocol import AmpelAlertProtocol
 
 
 def test_protocol():
     alert = AmpelAlert(0, 0, [])
-    assert isinstance(alert, AmpelAlertProtocol), "dataclass fields recognized as protocol"
+    assert isinstance(
+        alert, AmpelAlertProtocol
+    ), "dataclass fields recognized as protocol"
 
     assert alert.tag is None, "defaults are populated"
+
+    class Model(AmpelBaseModel):
+        alerts: list[AmpelAlertProtocol]
+
+    # this works: AmpelAlert is a structural sub-type of AmpelAlertProtocol
+    m = Model(
+        alerts=[AmpelAlert(id=0, stock="stockystock", datapoints=[{"id": 0}])]
+    )
+    assert isinstance(
+        m.model_dump(mode="json")["alerts"][0], dict
+    ), "dataclass serialized as dict"
+    # this does not: pydantic does not know how to deserialize a dict to AmpelAlertProtocol
+    with pytest.raises(ValidationError):
+        Model(**m.model_dump(mode="json"))
+
+
+def test_hash():
+    """AmpelAlert should be hashable"""
+    alert = AmpelAlert(id=0, stock="stockystock", datapoints=[{"id": 0}])
+    assert hash(alert) != hash(deepcopy(alert)), "hash is not the same after deepcopy"