From 8e841b91c900b0367be5cf4aa598407e7d8ce691 Mon Sep 17 00:00:00 2001 From: Nick LaMuro Date: Mon, 9 Sep 2019 13:21:04 -0500 Subject: [PATCH] [EmbeddedAnsible] Remove job_status_spec TODO This probably is a case we should support, but until it comes up, going to remove this TODO as it doesn't seem like something that is needed or heavily requested. --- .../automation_manager/job/status_spec.rb | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/spec/models/manageiq/providers/embedded_ansible/automation_manager/job/status_spec.rb b/spec/models/manageiq/providers/embedded_ansible/automation_manager/job/status_spec.rb index d3c7a4b13244..d045150a6433 100644 --- a/spec/models/manageiq/providers/embedded_ansible/automation_manager/job/status_spec.rb +++ b/spec/models/manageiq/providers/embedded_ansible/automation_manager/job/status_spec.rb @@ -31,18 +31,6 @@ expect(status.normalized_status).to eq(['failed', 'Stack creation failed']) end - # TODO: remove or implement? Is canceling something we can handle? - # - # it 'parses Canceled' do - # status = described_class.new('Canceled', nil) - # expect(status.completed?).to be_truthy - # expect(status.succeeded?).to be_falsey - # expect(status.canceled?).to be_truthy - # expect(status.deleted?).to be_falsey - # expect(status.rolled_back?).to be_falsey - # expect(status.normalized_status).to eq(['create_canceled', 'Job launching was canceled']) - # end - it 'parses transient status' do miq_task.state = MiqTask::STATE_ACTIVE miq_task.status = MiqTask::STATUS_UNKNOWN