Add POST-time firewall devices integration test

Author: lgarber-akamai

linode_api4/groups/networking.py

@@ -43,7 +43,9 @@ def firewall_create(
         self,
         label: str,
         rules: Dict[str, Any],
-        devices: Optional[Union[FirewallCreateDevicesOptions, Dict[str, Any]]],
+        devices: Optional[
+            Union[FirewallCreateDevicesOptions, Dict[str, Any]]
+        ] = None,
         **kwargs,
     ):
         """

test/integration/groups/__init__.py

@@ -0,0 +1,56 @@
+from test.integration.conftest import get_region
+from test.integration.helpers import get_test_label
+
+import pytest
+
+from linode_api4 import FirewallCreateDevicesOptions
+
+
+@pytest.fixture
+def create_linode_without_firewall(test_linode_client):
+    """
+    WARNING: This is specifically reserved for Firewall testing.
+             Don't use this if the Linode will not be assigned to a firewall.
+    """
+
+    client = test_linode_client
+    region = get_region(client, {"Cloud Firewall"}, "core").id
+
+    label = get_test_label()
+
+    instance = client.linode.instance_create(
+        "g6-nanode-1",
+        region,
+        label=label,
+    )
+
+    yield client, instance
+
+    instance.delete()
+
+
+@pytest.fixture
+def create_firewall_with_device(create_linode_without_firewall):
+    client, target_instance = create_linode_without_firewall
+
+    firewall = client.networking.firewall_create(
+        get_test_label(),
+        rules={
+            "inbound_policy": "DROP",
+            "outbound_policy": "DROP",
+        },
+        devices=FirewallCreateDevicesOptions(linodes=[target_instance.id]),
+    )
+
+    yield firewall, target_instance
+
+    firewall.delete()
+
+
+def test_create_firewall_with_devices(create_firewall_with_device):
+    firewall, target_instance = create_firewall_with_device
+
+    devices = firewall.devices
+
+    assert len(devices) == 1
+    assert devices[0].entity.id == target_instance.id

test/integration/groups/test_networking.py

@@ -6,7 +6,11 @@
 import pytest
 
 from linode_api4 import ApiError
-from linode_api4.objects import ConfigInterface, ObjectStorageKeys, Region
+from linode_api4.objects import (
+    ConfigInterface,
+    ObjectStorageKeys,
+    Region,
+)
 
 
 @pytest.fixture(scope="session")