Versehentliche E-Mail-/Benachrichtigungsflut beim Versuch, Topics lautlos zwischen Kategorien zu verschieben

Guter Punkt. Ich bin ehrlich gesagt nicht sicher.

Ich sehe, dass dies genau der Anwendungsfall ist, der in der ursprünglichen Funktionsanfrage beschrieben wurde:

Ich sehe auch, dass mehrere Leute dasselbe dachten, als sie die neue Checkbox sahen: Bulk editing topic categories should not trigger thousands of email notifications - #12 by mbauman

Auch die PR selbst lässt vermuten, dass dies die Absicht war:

Wenn das Kontrollkästchen „Diese Aktion stillschweigend ausführen“ aktiviert wurde, sollte der Sidekiq-Job :notify_category_change nicht in die Warteschlange gestellt werden.

Aber ich sehe keine explizite Spezifikationsabdeckung für diesen Fall.

Zugegebenermaßen bin ich etwas aus der Übung, Ruby-Spezifikationen vorzubereiten, da ich in den letzten Jahren nicht mehr täglich Code geschrieben habe, aber dies scheint eine mögliche Lücke zu sein.

Es riecht für mich eher nach einem Fehler. Ich bin mir nicht sicher, ob es eine Regression ist oder nicht, aber es fühlt sich an, als ob uns die Abdeckung für diesen Fall fehlt.

3 „Gefällt mir“