From 66ea575f4133cd73312e658e55641dbc254823da Mon Sep 17 00:00:00 2001 From: Zhiwei Liang Date: Sat, 28 Feb 2026 02:33:22 -0500 Subject: [PATCH] Add support for firewall rules version and fingerprint --- test/fixtures/networking_firewalls_123.json | 4 +++- test/fixtures/networking_firewalls_123_rules.json | 4 +++- test/integration/models/firewall/test_firewall.py | 8 ++++++++ test/unit/objects/firewall_test.py | 2 ++ 4 files changed, 16 insertions(+), 2 deletions(-) diff --git a/test/fixtures/networking_firewalls_123.json b/test/fixtures/networking_firewalls_123.json index c34a3991e..e634da39d 100644 --- a/test/fixtures/networking_firewalls_123.json +++ b/test/fixtures/networking_firewalls_123.json @@ -8,7 +8,9 @@ "outbound":[], "outbound_policy":"DROP", "inbound":[], - "inbound_policy":"DROP" + "inbound_policy":"DROP", + "version":2, + "fingerprint":"4ef67a29" }, "tags":[] } \ No newline at end of file diff --git a/test/fixtures/networking_firewalls_123_rules.json b/test/fixtures/networking_firewalls_123_rules.json index 43c8af4dc..d3f0716c4 100644 --- a/test/fixtures/networking_firewalls_123_rules.json +++ b/test/fixtures/networking_firewalls_123_rules.json @@ -2,5 +2,7 @@ "inbound": [], "inbound_policy": "DROP", "outbound": [], - "outbound_policy": "DROP" + "outbound_policy": "DROP", + "version": 2, + "fingerprint": "4ef67a29" } \ No newline at end of file diff --git a/test/integration/models/firewall/test_firewall.py b/test/integration/models/firewall/test_firewall.py index 16805f3b8..6bc5a9988 100644 --- a/test/integration/models/firewall/test_firewall.py +++ b/test/integration/models/firewall/test_firewall.py @@ -29,6 +29,10 @@ def test_get_firewall_rules(test_linode_client, test_firewall): assert rules.inbound_policy in ["ACCEPT", "DROP"] assert rules.outbound_policy in ["ACCEPT", "DROP"] + assert isinstance(rules.version, int) + assert rules.version > 0 + assert isinstance(rules.fingerprint, str) + assert len(rules.fingerprint) > 0 @pytest.mark.smoke @@ -61,6 +65,10 @@ def test_update_firewall_rules(test_linode_client, test_firewall): assert firewall.rules.inbound_policy == "ACCEPT" assert firewall.rules.outbound_policy == "DROP" + assert isinstance(firewall.rules.version, int) + assert firewall.rules.version > 0 + assert isinstance(firewall.rules.fingerprint, str) + assert len(firewall.rules.fingerprint) > 0 def test_get_devices(test_linode_client, linode_fw, test_firewall): diff --git a/test/unit/objects/firewall_test.py b/test/unit/objects/firewall_test.py index f4c6efb66..24c6c3656 100644 --- a/test/unit/objects/firewall_test.py +++ b/test/unit/objects/firewall_test.py @@ -20,6 +20,8 @@ def test_get_rules(self): self.assertEqual(rules.inbound_policy, "DROP") self.assertEqual(len(rules.outbound), 0) self.assertEqual(rules.outbound_policy, "DROP") + self.assertEqual(rules.version, 2) + self.assertEqual(rules.fingerprint, "4ef67a29") def test_update_rules(self): """