Looks like some existing specs need to be updated?
@CvX sorry for that, will fix all these shenanigans, should’ve made sure the tests passed first.
@CvX this is all good now.
can we move that code to line :12 to make it more concise? Similar to combined logic from close_topic.rb
if topic.blank? || topic.open?
I don’t think that this call is needed
fab! will create object anyway
@martin-brennan beside 2 small comments, it looks good to me