Skip to content

Commit d506b21

Browse files
authored
correcting type error on validation (#70)
#65 - describe_clusters must be a list, not a string Signed-off-by: Joshua Root <[email protected]>
1 parent d30797b commit d506b21

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

chaosaws/ecs/actions.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,7 @@ def update_desired_count(cluster: str,
216216
###############################################################################
217217
def validate_cluster(cluster: str, client: boto3.client) -> Union[str, None]:
218218
"""Validates the provided cluster exists"""
219-
cluster = client.describe_clusters(clusters=cluster)['clusters']
219+
cluster = client.describe_clusters(clusters=[cluster])['clusters']
220220
if not cluster:
221221
return
222222
return cluster[0]['clusterArn']

tests/ecs/test_ecs_actions.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -330,5 +330,6 @@ def test_update_desired_service_count(aws_client):
330330
)
331331
update_desired_count(
332332
cluster=cluster, service=service, desired_count=1)
333+
client.describe_clusters.assert_called_with(clusters=[cluster])
333334
client.update_service.assert_called_with(
334335
cluster=cluster, service=service, desiredCount=1)

0 commit comments

Comments
 (0)