Случайный поток писем/уведомлений при попытке беззвучно переместить темы между категориями

Хорошая мысль. Честно говоря, я не уверен.

Я вижу, что это именно тот случай использования, который описан в исходном запросе на новую функцию:

Также я вижу, что несколько человек подумали то же самое, когда увидели новый флажок: Bulk editing topic categories should not trigger thousands of email notifications - #12 by mbauman

Сам PR тоже создаёт впечатление, что это было намерением:

Когда флажок «Выполнить это действие без уведомлений» установлен, задача :notify_category_change sidekiq не должна ставиться в очередь.

Но я не вижу явного покрытия этого случая в спецификациях.

Признаю, я немного утратил навык написания Ruby-спецификаций, так как последние несколько лет не пишу код ежедневно, но это действительно похоже на возможный пробел.

Теперь мне это больше кажется ошибкой. Не уверен, является ли это регрессией, но кажется, что мы упускаем покрытие этого случая.

3 лайка