progress

Author: lgarber-akamai

linode_api4/objects/linode.py

@@ -1900,9 +1900,16 @@ def upgrade_interfaces(
         dry_run: bool = False,
     ) -> UpgradeInterfacesResult:
         """
-        Automatically upgrades all legacy config interfaces of a single configuration profile to Linode interfaces.
+        Automatically upgrades all legacy config interfaces of a
+        single configuration profile to Linode interfaces.
+
+        NOTE: If dry_run is True, interfaces in the result will be
+              of type MappedObject rather than LinodeInterface.
 
         API Documentation: Not yet available.
+
+        :returns: Information about the newly upgraded interfaces.
+        :rtype: UpgradeInterfacesResult
         """
         params = {"config_id": config, dry_run: dry_run}
 
@@ -1912,7 +1919,18 @@ def upgrade_interfaces(
             data=_flatten_request_body_recursive(drop_null_keys(params)),
         )
 
-        # TODO: Inject LinodeInterface objects here
+        # We don't convert interface dicts to LinodeInterface objects on dry runs
+        # actual API entities aren't created
+        if dry_run:
+            result["interfaces"] = [
+                MappedObject(**iface) for iface in result["interfaces"]
+            ]
+        else:
+            result["interfaces"] = [
+                LinodeInterface(self._client, self.id, iface["id"], iface)
+                for iface in result["interfaces"]
+            ]
+
         return UpgradeInterfacesResult.from_json(result)