Skip to content

Commit

Permalink
Fix wrong expected exception in spec assertion
Browse files Browse the repository at this point in the history
It's pretty straightforward that an `ArgumentError` is raised here.
I wonder if this was changed to `RejectedExecutionError` because
somehow that was happening instead due to cross-spec interference.
  • Loading branch information
ivoanjo committed Jun 23, 2018
1 parent 0ee31d0 commit a36aa32
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions spec/concurrent/channel_spec.rb
Expand Up @@ -557,7 +557,7 @@ module Concurrent
end
end

context 'goroutines', notravis: true do
context 'goroutines' do

let(:default_executor) { Channel.const_get(:GOROUTINES) }

Expand Down Expand Up @@ -595,7 +595,7 @@ module Concurrent
it 'raises an exception when no block is given' do
expect {
Channel.go_loop
}.to raise_error(RejectedExecutionError)
}.to raise_error(ArgumentError)
end

it 'loops until the block returns false' do
Expand Down

0 comments on commit a36aa32

Please sign in to comment.