Cover ClientOrders handlers with mutation testing

Existing tests already kill all ProductHandler mutants — remove the
three wildcard ignores. Separate persist from broadcast logic in
RemoveItemFromOrder so its data path is testable, replacing the
whole-class wildcard with targeted broadcast-only ignores.

Author: andrzejkrzywda

apps/rails_application/.mutant.yml

@@ -40,16 +40,17 @@ matcher:
     - ClientOrders::Configuration#subscribe_and_link_to_stream
     - ClientOrders::Configuration#link_to_stream
     - ClientOrders::Configuration#with_order
-    - ClientOrders::Rendering::OrdersList*
-    - ClientOrders::RemoveItemFromOrder*
-    - ClientOrders::AddItemToOrder#show_remove_item_button
-    - ClientOrders::AddItemToOrder#remove_button_html
+    - ClientOrders::AddItemToOrder#call
     - ClientOrders::AddItemToOrder#broadcast_to_ui
     - ClientOrders::AddItemToOrder#broadcast_update
-    - ClientOrders::AddItemToOrder#call
-    - ClientOrders::ProductHandlers::ChangeProductName*
-    - ClientOrders::ProductHandlers::ChangeProductPrice*
-    - ClientOrders::ProductHandlers::RegisterProduct*
+    - ClientOrders::AddItemToOrder#show_remove_item_button
+    - ClientOrders::AddItemToOrder#remove_button_html
+    - ClientOrders::RemoveItemFromOrder#call
+    - ClientOrders::RemoveItemFromOrder#broadcast_to_ui
+    - ClientOrders::RemoveItemFromOrder#broadcast_update
+    - ClientOrders::RemoveItemFromOrder#zero_quantity?
+    - ClientOrders::RemoveItemFromOrder#event_store
+    - ClientOrders::Rendering::OrdersList*
     - ClientOrders::Rendering::ShowOrder*
     - ClientOrders::Rendering::EditOrder*
     - Processes::Configuration*

apps/rails_application/app/read_models/client_orders/remove_item_from_order.rb

@@ -1,16 +1,23 @@
 module ClientOrders
   class RemoveItemFromOrder
     def call(event)
-      product_id = event.data.fetch(:product_id)
-      order_id = event.data.fetch(:order_id)
-      item = find(order_id , product_id)
+      item = persist_item(event)
+      broadcast_to_ui(item, event)
+    end
+
+    def persist_item(event)
+      item = find(event.data.fetch(:order_id), event.data.fetch(:product_id))
       item.product_quantity -= 1
       item.product_quantity > 0 ? item.save! : item.destroy!
+      item
+    end
 
+    def broadcast_to_ui(item, event)
+      order_id = event.data.fetch(:order_id)
+      product_id = event.data.fetch(:product_id)
       broadcast_update(order_id, product_id, "product_quantity", item.product_quantity)
       broadcast_update(order_id, product_id, "value", ActiveSupport::NumberHelper.number_to_currency(item.value))
       broadcast_update(order_id, product_id, "remove_item_button", "") if zero_quantity?(item)
-
       event_store.link_event_to_stream(event, "ClientOrders$all")
     end