Skip to content

Commit

Permalink
Reverting previous changes to add notification to finish retirement.
Browse files Browse the repository at this point in the history
Since not all objects have user/tenant information, we're going
to create a content PR to add notifications to Automate methods where
that information is available.

https://bugzilla.redhat.com/show_bug.cgi?id=1446466
  • Loading branch information
tinaafitz committed May 1, 2017
1 parent 75d8148 commit 93af270
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 4 deletions.
1 change: 0 additions & 1 deletion app/models/mixins/retirement_mixin.rb
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,6 @@ def finish_retirement
$log.info("Calling audit event for: #{message} ")
raise_audit_event(retired_event_name, message)
$log.info("Called audit event for: #{message} ")
Notification.create(:type => retired_event_name, :subject => self)
end

def start_retirement
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,6 @@
end

it "#finish_retirement" do
expect(Notification).to receive(:create)
expect(@stack.retirement_state).to be_nil
@stack.finish_retirement
@stack.reload
Expand Down
1 change: 0 additions & 1 deletion spec/models/service/retirement_management_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,6 @@

it "#finish_retirement" do
expect(@service.retirement_state).to be_nil
expect(Notification).to receive(:create)
@service.finish_retirement
@service.reload
expect(@service.retired).to be_truthy
Expand Down
1 change: 0 additions & 1 deletion spec/models/vm/retirement_management_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,6 @@

it "#finish_retirement" do
expect(@vm.retirement_state).to be_nil
expect(Notification).to receive(:create)
@vm.finish_retirement
@vm.reload

Expand Down

0 comments on commit 93af270

Please sign in to comment.