DEV: Split toggle topic close job (PR #11679)

GitHub

Looks like some existing specs need to be updated? :slightly_smiling_face:

@CvX :man_facepalming: 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