Skip to content

Commit 3606ea7

Browse files
committed
address review comments
Signed-off-by: Swati Mukund Bagal <sbagal@redhat.com>
1 parent 1eaae51 commit 3606ea7

1 file changed

Lines changed: 12 additions & 8 deletions

File tree

tests/model_serving/maas_billing/maas_subscription/test_cascade_deletion.py

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ def test_delete_last_subscription_denies_access(
119119
f"got {response.status_code}: {(response.text or '')[:200]}"
120120
)
121121
finally:
122-
with MaaSSubscription(
122+
restored_subscription = MaaSSubscription(
123123
client=admin_client,
124124
name=original_name,
125125
namespace=maas_subscription_tinyllama_free.namespace,
@@ -136,10 +136,14 @@ def test_delete_last_subscription_denies_access(
136136
priority=original_priority,
137137
teardown=False,
138138
wait_for_resource=True,
139-
) as restored_subscription:
140-
restored_subscription.wait_for_condition(
141-
condition="Ready",
142-
status="True",
143-
timeout=300,
144-
)
145-
LOGGER.info("Restored original subscription %s", restored_subscription.name)
139+
)
140+
141+
restored_subscription.deploy(wait=True)
142+
143+
restored_subscription.wait_for_condition(
144+
condition="Ready",
145+
status="True",
146+
timeout=300,
147+
)
148+
149+
LOGGER.info("Restored original subscription %s", restored_subscription.name)

0 commit comments

Comments
 (0)