Skip to content

Commit aa309c2

Browse files
committed
Use try-finally to delete resources in case of fail
1 parent 9f9610e commit aa309c2

1 file changed

Lines changed: 15 additions & 12 deletions

File tree

test/integration/models/linode/interfaces/test_interfaces.py

Lines changed: 15 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -423,19 +423,22 @@ def test_linode_interfaces_with_reserved_ips(
423423

424424
linode = create_linode_fn(client, reserved_ip, label, e2e_test_firewall)
425425

426-
linode_ips = linode.ips.ipv4.public
427-
assert len(linode_ips) == 1
428-
assert linode_ips[0].address == reserved_ip.address
429-
assert linode_ips[0].reserved == True
430-
assert linode_ips[0].linode_id == linode.id
431-
assert linode_ips[0].assigned_entity.id == linode.id
432-
assert linode_ips[0].assigned_entity.type == "linode"
433-
assert linode_ips[0].assigned_entity.label == linode.label
434-
assert (
435-
linode_ips[0].assigned_entity.url == f"/v4/linode/instances/{linode.id}"
436-
)
426+
try:
427+
linode_ips = linode.ips.ipv4.public
428+
assert len(linode_ips) == 1
429+
assert linode_ips[0].address == reserved_ip.address
430+
assert linode_ips[0].reserved == True
431+
assert linode_ips[0].linode_id == linode.id
432+
assert linode_ips[0].assigned_entity.id == linode.id
433+
assert linode_ips[0].assigned_entity.type == "linode"
434+
assert linode_ips[0].assigned_entity.label == linode.label
435+
assert (
436+
linode_ips[0].assigned_entity.url
437+
== f"/v4/linode/instances/{linode.id}"
438+
)
439+
finally:
440+
linode.delete()
437441

438-
linode.delete()
439442
reserved_ips_list = client.networking.reserved_ips(
440443
ReservedIPAddress.address == reserved_ip.address
441444
)

0 commit comments

Comments
 (0)