Add VPC update test

Author: lgarber-akamai

linode_api4/objects/linode_interfaces.py

@@ -40,8 +40,8 @@ class LinodeInterfacesSettings(Base):
 
 @dataclass
 class LinodeInterfaceDefaultRouteOptions(JSONObject):
-    ipv4: bool = False
-    ipv6: bool = False
+    ipv4: Optional[bool] = None
+    ipv6: Optional[bool] = None
 
 
 @dataclass
@@ -102,12 +102,14 @@ class LinodeInterfaceVLANOptions(JSONObject):
 
 
 # Interface PUT Options
+# NOTE: These classes currently match their POST equivalents
+# but are implemented separately for future-proofing.
 
 
 @dataclass
 class LinodeInterfaceDefaultRouteUpdateOptions(JSONObject):
-    ipv4: bool = False
-    ipv6: bool = False
+    ipv4: Optional[bool] = None
+    ipv6: Optional[bool] = None
 
 
 @dataclass
@@ -206,7 +208,7 @@ class LinodeInterfaceVPCIPv4(JSONObject):
 
 @dataclass
 class LinodeInterfaceVPC(JSONObject):
-    put_class = LinodeInterfaceVPCIPv4Options
+    put_class = LinodeInterfaceVPCOptions
 
     vpc_id: int = 0
     subnet_id: int = 0

test/fixtures/linode_instances_124_interfaces_456.json

@@ -12,8 +12,10 @@
         "subnet_id": 789,
         "ipv4" : {
             "addresses": [
-                { "address": "192.168.22.3",
-                  "primary": true }
+                {
+                    "address": "192.168.22.3",
+                    "primary": true
+                }
             ],
             "ranges": [
                 { "range": "192.168.22.16/28"},

test/unit/objects/linode_interface_test.py

@@ -1,10 +1,13 @@
 from datetime import datetime
 from test.unit.base import ClientBaseCase
 
+
 from linode_api4 import (
     LinodeInterface,
     LinodeInterfacePublicIPv4AddressUpdateOptions,
     LinodeInterfacePublicIPv6RangeUpdateOptions,
+    LinodeInterfaceVPCIPv4AddressUpdateOptions,
+    LinodeInterfaceVPCIPv4RangeUpdateOptions,
 )
 
 
@@ -172,4 +175,48 @@ def test_update_public(self):
                 },
             }
 
-    # TODO (Enhanced Interfaces): iface update tests
+    def test_update_vpc(self):
+        iface = LinodeInterface(self.client, 456, 124)
+
+        self.assert_linode_124_interface_456(iface)
+
+        iface.default_route.ipv4 = False
+
+        iface.vpc.subnet_id = 456
+
+        iface.vpc.ipv4.addresses = [
+            LinodeInterfaceVPCIPv4AddressUpdateOptions(
+                address="192.168.22.4", primary=False, nat_1_1_address="auto"
+            )
+        ]
+
+        iface.vpc.ipv4.ranges = [
+            LinodeInterfaceVPCIPv4RangeUpdateOptions(
+                range="192.168.22.17/28",
+            )
+        ]
+
+        with self.mock_put("/linode/instances/124/interfaces/456") as m:
+            iface.save()
+
+            assert m.called
+
+            print(m.call_data)
+            assert m.call_data == {
+                "default_route": {
+                    "ipv4": False,
+                },
+                "vpc": {
+                    "subnet_id": 456,
+                    "ipv4": {
+                        "addresses": [
+                            {
+                                "address": "192.168.22.4",
+                                "primary": False,
+                                "nat_1_1_address": "auto",
+                            },
+                        ],
+                        "ranges": [{"range": "192.168.22.17/28"}],
+                    },
+                },
+            }