fix: do not ignore advisory_id while deserializing AdvisoryDataV2 from dict

Signed-off-by: Keshav Priyadarshi <git@keshav.space>

Author: keshav-space

vulnerabilities/importer.py

@@ -648,6 +648,7 @@ def to_dict(self):
     def from_dict(cls, advisory_data):
         date_published = advisory_data["date_published"]
         transformed = {
+            "advisory_id": advisory_data["advisory_id"],
             "aliases": advisory_data["aliases"],
             "summary": advisory_data["summary"],
             "affected_packages": [

vulnerabilities/tests/test_utils.py

@@ -254,3 +254,10 @@ def test_content_id_from_adv_data_and_adv_model_are_same(self):
         id_from_model = utils.compute_content_id_v2(advisory_model)
 
         self.assertEqual(id_from_data, id_from_model)
+
+    def test_content_id_from_adv_data_roundtrip_are_same(self):
+        id_from_data = utils.compute_content_id_v2(self.advisory1)
+        adv_roundtrip = AdvisoryDataV2.from_dict(self.advisory1.to_dict())
+        id_from_roundtrip = utils.compute_content_id_v2(adv_roundtrip)
+
+        self.assertEqual(id_from_data, id_from_roundtrip)