don't use inheritance with Data.define

* it introduces an unnecessary anonymous class
* mutant adds unwanted mutations in intializer

Author: porbas

ecommerce/processes/lib/processes/reservation_process.rb

@@ -8,8 +8,10 @@ def initialize(unavailable_products)
       end
     end
 
-    class ProcessState < Data.define(:order, :order_lines)
-      def initialize(order: nil, order_lines: nil) = super
+    ProcessState = Data.define(:order, :order_lines) do
+      def initialize(order: nil, order_lines: [])
+        super(order:, order_lines: order_lines.freeze)
+      end
 
       def reserved_product_ids = order_lines.keys
     end

ecommerce/processes/lib/processes/shipment_process.rb

@@ -1,6 +1,6 @@
 module Processes
   class ShipmentProcess
-    class ProcessState < Data.define(:order, :shipment)
+    ProcessState = Data.define(:order, :shipment) do
       def initialize(order: nil, shipment: nil) = super
     end