The `||` here doesn't have any effect. Assuming `Retro::STATUS_COMPLETE` isn't `nil` or `false` that will just get assigned to the status_id. Instead I might expand this:

```ruby
context 'when the retro status is complete' do
  it 'returns true' do
    retro.status_id = Retro::STATUS_COMPLETE
    retro.should be_ended
  end
end

context 'when the retro status is distributed' do
  it 'returns true' do
    retro.status_id = Retro::STATUS_DISTRIBUTED
    retro.should be_ended
  end
end
```