Deregister batch job definition by 'name:revision' (#4355)
This commit is contained in:
parent
de2e627c41
commit
30c8c3de1f
@ -718,7 +718,7 @@ class BatchBackend(BaseBackend):
|
||||
|
||||
def get_job_definition_by_name_revision(self, name, revision):
|
||||
for job_def in self._job_definitions.values():
|
||||
if job_def.name == name and job_def.revision == revision:
|
||||
if job_def.name == name and job_def.revision == int(revision):
|
||||
return job_def
|
||||
return None
|
||||
|
||||
|
@ -97,6 +97,24 @@ def test_delete_task_definition():
|
||||
len(resp["jobDefinitions"]).should.equal(0)
|
||||
|
||||
|
||||
@mock_ec2
|
||||
@mock_ecs
|
||||
@mock_iam
|
||||
@mock_batch
|
||||
def test_delete_task_definition_by_name():
|
||||
ec2_client, iam_client, ecs_client, logs_client, batch_client = _get_clients()
|
||||
_setup(ec2_client, iam_client)
|
||||
|
||||
resp = register_job_def(batch_client)
|
||||
|
||||
batch_client.deregister_job_definition(
|
||||
jobDefinition=f"{resp['jobDefinitionName']}:{resp['revision']}"
|
||||
)
|
||||
|
||||
resp = batch_client.describe_job_definitions()
|
||||
len(resp["jobDefinitions"]).should.equal(0)
|
||||
|
||||
|
||||
@mock_ec2
|
||||
@mock_ecs
|
||||
@mock_iam
|
||||
|
Loading…
Reference in New Issue
Block a user