Verify nullify in integration test

Author: gmazoyer

infrahub_sdk/node/node.py

@@ -310,8 +310,13 @@ def _strip_unmodified_dict(data: dict, original_data: dict, variables: dict, ite
                         variables.pop(variable_key)
 
         # TODO: I do not feel _great_ about this
-        # -> I don't even know who you are -- gmazoyer (quoting Thanos)
-        if data_item in ({}, []) or (data_item is None and original_data.get(item) is None):
+        # -> I don't even know who you are (but this is not great indeed) -- gmazoyer (quoting Thanos)
+        original_data_item = original_data.get(item)
+        original_data_item_is_none = original_data_item is None or (
+            isinstance(original_data_item, dict)
+            and (("node" in original_data_item and original_data_item["node"] is None) or "id" not in original_data)
+        )
+        if item in data and (data_item in ({}, []) or (data_item is None and original_data_item_is_none)):
             data.pop(item)
 
     def _strip_unmodified(self, data: dict, variables: dict) -> tuple[dict, dict]:

tests/integration/test_infrahub_client.py

@@ -170,6 +170,12 @@ async def test_create_generic_rel_with_hfid(
         person_sophia = await client.get(kind=TESTING_PERSON, id=person_sophia.id, prefetch_relationships=True)
         assert person_sophia.favorite_animal.id == cat_luna.id
 
+        # Ensure that nullify it will remove the relationship related node
+        person_sophia.favorite_animal = None
+        await person_sophia.save()
+        person_sophia = await client.get(kind=TESTING_PERSON, id=person_sophia.id, prefetch_relationships=True)
+        assert not person_sophia.favorite_animal.id
+
     async def test_task_query(self, client: InfrahubClient, base_dataset, set_pagination_size3) -> None:
         nbr_tasks = await client.task.count()
         assert nbr_tasks

tests/unit/sdk/test_node.py

@@ -1330,7 +1330,12 @@ async def test_create_input_data(client, location_schema: NodeSchemaAPI, client_
         node = InfrahubNodeSync(client=client, schema=location_schema, data=data)
 
     assert node._generate_input_data()["data"] == {
-        "data": {"name": {"value": "JFK1"}, "description": {"value": "JFK Airport"}, "type": {"value": "SITE"}}
+        "data": {
+            "name": {"value": "JFK1"},
+            "description": {"value": "JFK Airport"},
+            "type": {"value": "SITE"},
+            "primary_tag": None,
+        }
     }
 
 
@@ -1577,7 +1582,7 @@ async def test_create_input_data_with_IPHost_attribute(client, ipaddress_schema,
         ip_address = InfrahubNodeSync(client=client, schema=ipaddress_schema, data=data)
 
     assert ip_address._generate_input_data()["data"] == {
-        "data": {"address": {"value": "1.1.1.1/24", "is_protected": True}}
+        "data": {"address": {"value": "1.1.1.1/24", "is_protected": True}, "interface": None}
     }
 
 
@@ -1591,7 +1596,7 @@ async def test_create_input_data_with_IPNetwork_attribute(client, ipnetwork_sche
         ip_network = InfrahubNodeSync(client=client, schema=ipnetwork_schema, data=data)
 
     assert ip_network._generate_input_data()["data"] == {
-        "data": {"network": {"value": "1.1.1.0/24", "is_protected": True}}
+        "data": {"network": {"value": "1.1.1.0/24", "is_protected": True}, "site": None}
     }
 
 
@@ -1789,7 +1794,7 @@ async def test_update_input_data_empty_relationship(
         "data": {
             "id": "llllllll-llll-llll-llll-llllllllllll",
             "name": {"value": "DFW"},
-            # "primary_tag": None,
+            "primary_tag": None,
             "tags": [],
             "type": {"value": "SITE"},
         },
@@ -1798,7 +1803,7 @@ async def test_update_input_data_empty_relationship(
         "data": {
             "id": "llllllll-llll-llll-llll-llllllllllll",
             "name": {"is_protected": True, "is_visible": True, "value": "DFW"},
-            # "primary_tag": None,
+            "primary_tag": None,
             "tags": [],
             "type": {"is_protected": True, "is_visible": True, "value": "SITE"},
         },