Skip to content

Commit 54255ab

Browse files
committed
destroy pipeline
1 parent ac75062 commit 54255ab

File tree

3 files changed

+7
-8
lines changed

3 files changed

+7
-8
lines changed

azure/azure-pr-teardown-pipeline.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -59,9 +59,8 @@ jobs:
5959
6060
echo "unsubscribing SQS to MNS for notifications..."
6161
poetry run python src/unsubscribe_mns.py
62-
displayName: "UnSubcribe MNS"
62+
displayName: "Unsubscribe MNS"
6363
workingDirectory: "$(Pipeline.Workspace)/s/$(SERVICE_NAME)/mns_subscription"
64-
condition: eq(1,2)
6564
6665
- bash: |
6766
export AWS_PROFILE=apim-dev
@@ -76,6 +75,6 @@ jobs:
7675
export SUB_ENVIRONMENT="$WORKSPACE"
7776
7877
terraform workspace select "$SUB_ENVIRONMENT"
79-
make plan aws_account_no=${account_id} environment="$ENVIRONMENT" sub_environment="$SUB_ENVIRONMENT"
78+
make destroy aws_account_no=${account_id} environment="$ENVIRONMENT" sub_environment="$SUB_ENVIRONMENT"
8079
displayName: Destroy terraform PR workspace and linked resources
8180
retryCountOnTaskFailure: 2

mns_subscription/src/mns_service.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ def delete_subscription(self, subscription_id: str) -> bool:
100100
else:
101101
MnsService.handle_response(response)
102102

103-
def check_delete_subcription(self):
103+
def check_delete_subscription(self):
104104
try:
105105
resource = self.get_subscription()
106106
if not resource:

mns_subscription/tests/test_mns_service.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,7 @@ def test_check_delete_subscription_success(self, mock_get_subscription, mock_del
226226
mock_delete_subscription.return_value = True
227227

228228
service = MnsService(self.authenticator)
229-
result = service.check_delete_subcription()
229+
result = service.check_delete_subscription()
230230
self.assertEqual(result, "Subscription successfully deleted")
231231
mock_get_subscription.assert_called_once()
232232
mock_delete_subscription.assert_called_once_with("sub-123")
@@ -236,15 +236,15 @@ def test_check_delete_subscription_no_resource(self, mock_get_subscription):
236236
# No subscription found
237237
mock_get_subscription.return_value = None
238238
service = MnsService(self.authenticator)
239-
result = service.check_delete_subcription()
239+
result = service.check_delete_subscription()
240240
self.assertEqual(result, "No matching subscription found to delete.")
241241

242242
@patch.object(MnsService, "get_subscription")
243243
def test_check_delete_subscription_missing_id(self, mock_get_subscription):
244244
# Resource with no id field
245245
mock_get_subscription.return_value = {"not_id": "not-id"}
246246
service = MnsService(self.authenticator)
247-
result = service.check_delete_subcription()
247+
result = service.check_delete_subscription()
248248
self.assertEqual(result, "Subscription resource missing 'id' field.")
249249

250250
@patch.object(MnsService, "delete_subscription")
@@ -253,7 +253,7 @@ def test_check_delete_subscription_raises(self, mock_get_subscription, mock_dele
253253
mock_get_subscription.return_value = {"id": "sub-123"}
254254
mock_delete_subscription.side_effect = Exception("Error!")
255255
service = MnsService(self.authenticator)
256-
result = service.check_delete_subcription()
256+
result = service.check_delete_subscription()
257257
self.assertTrue(result.startswith("Error deleting subscription: Error!"))
258258

259259
def mock_response(self, status_code, json_data=None):

0 commit comments

Comments
 (0)