TPT-4278: python-sdk: Implement support for Reserved IP for IPv4

Author: mgwoj

linode_api4/groups/networking.py

@@ -364,6 +364,19 @@ def ip_allocate(
         :returns: The new IPAddress.
         :rtype: IPAddress
         """
+        if not reserved and linode is None:
+            raise ValueError(
+                "linode is required when reserved is False."
+            )
+        if reserved and linode is None and region is None:
+            raise ValueError(
+                "Either linode or region must be provided when reserved is True."
+            )
+        if not reserved and region is not None:
+            raise ValueError(
+                "region is only valid when reserved is True."
+            )
+
         data = {
             "type": "ipv4",
             "public": public,

linode_api4/groups/tag.py

@@ -33,7 +33,7 @@ def create(
         nodebalancers=None,
         volumes=None,
         reserved_ipv4_addresses=None,
-        entities=[],
+        entities=None,
     ):
         """
         Creates a new Tag and optionally applies it to the given entities.
@@ -69,6 +69,7 @@ def create(
         :returns: The new Tag
         :rtype: Tag
         """
+        entities = entities or []
         linode_ids, nodebalancer_ids, domain_ids, volume_ids = [], [], [], []
 
         # filter input into lists of ids

test/unit/groups/networking_test.py

@@ -199,3 +199,42 @@ def test_ip_allocate_ephemeral(self):
             assert "reserved" not in body
             assert ip.linode_id == 456
             assert ip.reserved is False
+
+    def test_ip_allocate_requires_linode_when_not_reserved(self):
+        """
+        Tests that ip_allocate rejects ephemeral allocation without a linode.
+        """
+        with MethodMock("post", {}) as m:
+            with self.assertRaises(ValueError) as ctx:
+                self.client.networking.ip_allocate()
+
+            assert str(ctx.exception) == (
+                "linode is required when reserved is False."
+            )
+            assert m.called is False
+
+    def test_ip_allocate_requires_linode_or_region_when_reserved(self):
+        """
+        Tests that ip_allocate rejects reserved allocation without a linode or region.
+        """
+        with MethodMock("post", {}) as m:
+            with self.assertRaises(ValueError) as ctx:
+                self.client.networking.ip_allocate(reserved=True)
+
+            assert str(ctx.exception) == (
+                "Either linode or region must be provided when reserved is True."
+            )
+            assert m.called is False
+
+    def test_ip_allocate_rejects_region_when_not_reserved(self):
+        """
+        Tests that ip_allocate rejects region when reserved is False.
+        """
+        with MethodMock("post", {}) as m:
+            with self.assertRaises(ValueError) as ctx:
+                self.client.networking.ip_allocate(region="us-east", linode=456)
+
+            assert str(ctx.exception) == (
+                "region is only valid when reserved is True."
+            )
+            assert m.called is False