Commit 9371cf3a authored by Tiago Botelho's avatar Tiago Botelho

Fix positive raise_error expectation without specifying the error type...

Fix positive raise_error expectation without specifying the error type expected in Project::DestroyService specs
parent 86f5a4aa
...@@ -142,13 +142,13 @@ describe Projects::DestroyService do ...@@ -142,13 +142,13 @@ describe Projects::DestroyService do
context 'when `execute` raises unexpected error' do context 'when `execute` raises unexpected error' do
before do before do
expect_any_instance_of(Project) expect_any_instance_of(Project)
.to receive(:destroy!).and_raise(Exception.new("Other error message")) .to receive(:destroy!).and_raise(Exception.new('Other error message'))
end end
it 'allows error to bubble up and rolls back project deletion' do it 'allows error to bubble up and rolls back project deletion' do
expect do expect do
Sidekiq::Testing.inline! { destroy_project(project, user, {}) } Sidekiq::Testing.inline! { destroy_project(project, user, {}) }
end.to raise_error end.to raise_error(Exception, 'Other error message')
expect(project.reload.pending_delete).to be(false) expect(project.reload.pending_delete).to be(false)
expect(project.delete_error).to include("Other error message") expect(project.delete_error).to include("Other error message")
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment