Skip to content

Commit

Permalink
[EmbeddedAnsible] Remove configuration_script uniqness spec
Browse files Browse the repository at this point in the history
Doesn't seem to be requested at this point, so going to remove this
spec.  If it is something that is desired in the future, we can add it
back.

That said, if it is requested, it most likely came with a bug attached
to it, so we probably will have a few other things to solve if it turns
out this was actually a wanted/needed feature.
  • Loading branch information
NickLaMuro committed Sep 20, 2019
1 parent 8e841b9 commit bd27e0f
Showing 1 changed file with 0 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -156,18 +156,6 @@
described_class.create_in_provider(manager.id, params)
end.to raise_error(RuntimeError, error_msg)
end

# TODO: Determine if we want to have a uniqueness validation to
# replicate this functionality, otherwise delete this case.
#
# context "provider raises on create" do
# it "with a string" do
# expect(AnsibleTowerClient::Connection).to receive(:new).and_return(atc)
# expect(job_templates).to receive(:create!).and_raise(AnsibleTowerClient::Error, "Job template with this Name already exists.")

# expect { described_class.create_in_provider(manager.id, params) }.to raise_error(AnsibleTowerClient::Error, "Job template with this Name already exists.")
# end
# end
end

it ".create_in_provider_queue" do
Expand Down

0 comments on commit bd27e0f

Please sign in to comment.