Skip to content

Commit

Permalink
Don't call the .configure method as it was removed
Browse files Browse the repository at this point in the history
  • Loading branch information
carbonin committed May 25, 2017
1 parent 060a0c4 commit 0ec1271
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 23 deletions.
3 changes: 0 additions & 3 deletions app/models/embedded_ansible_worker/runner.rb
Expand Up @@ -32,9 +32,6 @@ def before_exit(*_)
def setup_ansible
raise_role_notification(:role_activate_start)

_log.info("calling EmbeddedAnsible.configure")
EmbeddedAnsible.configure unless EmbeddedAnsible.configured?

_log.info("calling EmbeddedAnsible.start")
EmbeddedAnsible.start
_log.info("calling EmbeddedAnsible.start finished")
Expand Down
20 changes: 0 additions & 20 deletions spec/models/embedded_ansible_worker/runner_spec.rb
Expand Up @@ -74,26 +74,7 @@
NotificationType.seed
end

it "configures EmbeddedAnsible if it is not configured" do
expect(EmbeddedAnsible).to receive(:start)

expect(EmbeddedAnsible).to receive(:configured?).and_return(false)
expect(EmbeddedAnsible).to receive(:configure)

runner.setup_ansible
end

it "doesn't call configure if EmbeddedAnsible is already configured" do
expect(EmbeddedAnsible).to receive(:start)

expect(EmbeddedAnsible).to receive(:configured?).and_return(true)
expect(EmbeddedAnsible).not_to receive(:configure)

runner.setup_ansible
end

it "creates a notification to inform the user that the service has started" do
expect(EmbeddedAnsible).to receive(:configured?).and_return(true)
expect(EmbeddedAnsible).to receive(:start)

runner.setup_ansible
Expand All @@ -104,7 +85,6 @@
end

it "creates a notification to inform the user that the role has been assigned" do
expect(EmbeddedAnsible).to receive(:configured?).and_return(true)
expect(EmbeddedAnsible).to receive(:start)

runner.setup_ansible
Expand Down

0 comments on commit 0ec1271

Please sign in to comment.