unit testing progress

Author: lgarber-akamai

linode_api4/objects/linode.py

@@ -1920,7 +1920,7 @@ def interfaces(self):
                 "_interfaces",
                 [
                     LinodeInterface(
-                        self._client, self.id, iface["id"], json=iface
+                        self._client, iface["id"], self.id, json=iface
                     )
                     for iface in result["interfaces"]
                 ],

test/fixtures/linode_instances_124_interfaces.json

@@ -12,7 +12,7 @@
         "ipv4": {
           "addresses": [
             {
-              "address": "172.30.0.5O",
+              "address": "172.30.0.50",
               "primary": true
             }
           ],

test/fixtures/linode_instances_124_interfaces_123.json

@@ -10,7 +10,7 @@
     "ipv4": {
       "addresses": [
         {
-          "address": "172.30.0.5O",
+          "address": "172.30.0.50",
           "primary": true
         }
       ],

test/unit/objects/linode_interface_test.py

@@ -0,0 +1,54 @@
+from datetime import datetime
+from test.unit.base import ClientBaseCase
+
+from linode_api4 import (
+    LinodeInterface,
+)
+
+
+class LinodeInterfaceTest(ClientBaseCase):
+    """
+    Tests methods of the LinodeInterface class
+    """
+
+    @staticmethod
+    def assert_linode_124_interface_123(iface: LinodeInterface):
+        assert iface.id == 123
+
+        assert isinstance(iface.created, datetime)
+        assert isinstance(iface.updated, datetime)
+
+        assert iface.default_route.ipv4
+        assert iface.default_route.ipv6
+
+        assert iface.mac_address == "22:00:AB:CD:EF:01"
+        assert iface.version == 1
+
+        assert iface.vlan is None
+        assert iface.vpc is None
+
+        # public.ipv4 assertions
+        assert iface.public.ipv4.addresses[0].address == "172.30.0.50"
+        assert iface.public.ipv4.addresses[0].primary
+
+        assert iface.public.ipv4.shared[0].address == "172.30.0.51"
+        assert iface.public.ipv4.shared[0].linode_id == 125
+
+        # public.ipv6 assertions
+        assert iface.public.ipv6.ranges[0].range == "2600:3cO9:e001:59::/64"
+        assert (
+            iface.public.ipv6.ranges[0].route_target
+            == "2600:3cO9::ff:feab:cdef"
+        )
+
+        assert iface.public.ipv6.ranges[1].range == "2600:3cO9:e001:5a::/64"
+        assert (
+            iface.public.ipv6.ranges[1].route_target
+            == "2600:3cO9::ff:feab:cdef"
+        )
+
+        assert iface.public.ipv6.shared[0].range == "2600:3cO9:e001:2a::/64"
+        assert iface.public.ipv6.shared[0].route_target is None
+
+        assert iface.public.ipv6.slaac[0].address == "2600:3cO9::ff:feab:cdef"
+        assert iface.public.ipv6.slaac[0].prefix == 64

test/unit/objects/linode_test.py

@@ -463,6 +463,20 @@ def test_get_placement_group(self):
         assert pg.label == "test"
         assert pg.placement_group_type == "anti_affinity:local"
 
+    def test_get_interfaces(self):
+        # Local import to avoid circular dependency
+        from linode_interface_test import (  # pylint: disable=import-outside-toplevel
+            LinodeInterfaceTest,
+        )
+
+        instance = Instance(self.client, 124)
+
+        interfaces = instance.interfaces
+
+        LinodeInterfaceTest.assert_linode_124_interface_123(
+            next(iface for iface in interfaces if iface.id == 123)
+        )
+
 
 class DiskTest(ClientBaseCase):
     """