Update and rename ordering tests after renaming refund to return.

Author: dysk

…ordering/test/add_item_to_refund_test.rb …ordering/test/add_item_to_return_test.rbecommerce/ordering/test/add_item_to_refund_test.rb renamed to ecommerce/ordering/test/add_item_to_return_test.rb ecommerce/ordering/test/add_item_to_refund_test.rb renamed to ecommerce/ordering/test/add_item_to_return_test.rb

@@ -1,16 +1,16 @@
 require_relative "test_helper"
 
 module Ordering
-  class AddItemToRefundTest < Test
-    cover "Ordering::OnAddItemToRefund*"
+  class AddItemToReturnTest < Test
+    cover "Ordering::OnAddItemToReturn*"
 
-    def test_add_item_to_refund
+    def test_add_item_to_return
       order_id = SecureRandom.uuid
       aggregate_id = SecureRandom.uuid
       product_1_id = SecureRandom.uuid
       product_2_id = SecureRandom.uuid
       product_3_id = SecureRandom.uuid
-      stream = "Ordering::Refund$#{aggregate_id}"
+      stream = "Ordering::Return$#{aggregate_id}"
 
       arrange(
         Pricing::SetPrice.new(product_id: product_1_id, price: 11),
@@ -22,18 +22,18 @@ def test_add_item_to_refund
         Pricing::AddPriceItem.new(order_id: order_id, product_id: product_3_id, price: 33),
         Pricing::AcceptOffer.new(order_id: order_id),
         Fulfillment::RegisterOrder.new(order_id: order_id),
-        CreateDraftRefund.new(refund_id: aggregate_id, order_id: order_id),
-        AddItemToRefund.new(
-            refund_id: aggregate_id,
+        CreateDraftReturn.new(return_id: aggregate_id, order_id: order_id),
+        AddItemToReturn.new(
+            return_id: aggregate_id,
             order_id: order_id,
             product_id: product_2_id
           )
         )
 
       expected_events = [
-        ItemAddedToRefund.new(
+        ItemAddedToReturn.new(
           data: {
-            refund_id: aggregate_id,
+            return_id: aggregate_id,
             order_id: order_id,
             product_id: product_2_id
           }
@@ -42,8 +42,8 @@ def test_add_item_to_refund
 
       assert_events(stream, *expected_events) do
         act(
-          AddItemToRefund.new(
-            refund_id: aggregate_id,
+          AddItemToReturn.new(
+            return_id: aggregate_id,
             order_id: order_id,
             product_id: product_2_id
           )
@@ -61,16 +61,16 @@ def test_add_item_raises_exceeds_order_quantity_error
         Pricing::AddPriceItem.new(order_id: order_id, product_id: product_id, price: 11),
         Pricing::AcceptOffer.new(order_id: order_id),
         Fulfillment::RegisterOrder.new(order_id: order_id),
-        CreateDraftRefund.new(refund_id: aggregate_id, order_id: order_id),
-        AddItemToRefund.new(
-          refund_id: aggregate_id,
+        CreateDraftReturn.new(return_id: aggregate_id, order_id: order_id),
+        AddItemToReturn.new(
+          return_id: aggregate_id,
           order_id: order_id,
           product_id: product_id
         )
       )
 
-      assert_raises(Ordering::Refund::ExceedsOrderQuantityError) do
-        act(AddItemToRefund.new(refund_id: aggregate_id, order_id: order_id, product_id: product_id))
+      assert_raises(Ordering::Return::ExceedsOrderQuantityError) do
+        act(AddItemToReturn.new(return_id: aggregate_id, order_id: order_id, product_id: product_id))
       end
     end
   end

…rdering/test/create_draft_refund_test.rb …rdering/test/create_draft_return_test.rbecommerce/ordering/test/create_draft_refund_test.rb renamed to ecommerce/ordering/test/create_draft_return_test.rb ecommerce/ordering/test/create_draft_refund_test.rb renamed to ecommerce/ordering/test/create_draft_return_test.rb

@@ -1,14 +1,14 @@
 require_relative "test_helper"
 
 module Ordering
-  class CreateDraftRefundTest < Test
-    cover "Ordering::OnCreateDraftRefund*"
+  class CreateDraftReturnTest < Test
+    cover "Ordering::OnCreateDraftReturn*"
 
-    def test_draft_refund_created
+    def test_draft_return_created
       order_id = SecureRandom.uuid
       aggregate_id = SecureRandom.uuid
       product_id = SecureRandom.uuid
-      stream = "Ordering::Refund$#{aggregate_id}"
+      stream = "Ordering::Return$#{aggregate_id}"
 
       arrange(
         Pricing::SetPrice.new(product_id: product_id, price: 11),
@@ -19,19 +19,19 @@ def test_draft_refund_created
       )
 
       expected_events = [
-        DraftRefundCreated.new(
+        DraftReturnCreated.new(
           data: {
-            refund_id: aggregate_id,
+            return_id: aggregate_id,
             order_id: order_id,
-            refundable_products: [{ product_id:, quantity: 2 }]
+            returnable_products: [{ product_id:, quantity: 2 }]
           }
         )
       ]
 
       assert_events(stream, *expected_events) do
         act(
-          CreateDraftRefund.new(
-            refund_id: aggregate_id,
+          CreateDraftReturn.new(
+            return_id: aggregate_id,
             order_id: order_id
           )
         )

…ing/test/remove_item_from_refund_test.rb …ing/test/remove_item_from_return_test.rbecommerce/ordering/test/remove_item_from_refund_test.rb renamed to ecommerce/ordering/test/remove_item_from_return_test.rb ecommerce/ordering/test/remove_item_from_refund_test.rb renamed to ecommerce/ordering/test/remove_item_from_return_test.rb

@@ -1,16 +1,16 @@
 require_relative "test_helper"
 
 module Ordering
-  class RemoveItemFromRefundTest < Test
-    cover "Ordering::OnRemoveItemFromRefund*"
+  class RemoveItemFromReturnTest < Test
+    cover "Ordering::OnRemoveItemFromReturn*"
 
-    def test_removing_items_from_refund
+    def test_removing_items_from_return
       order_id = SecureRandom.uuid
       aggregate_id = SecureRandom.uuid
       product_1_id = SecureRandom.uuid
       product_2_id = SecureRandom.uuid
       product_3_id = SecureRandom.uuid
-      stream = "Ordering::Refund$#{aggregate_id}"
+      stream = "Ordering::Return$#{aggregate_id}"
 
       arrange(
         Pricing::SetPrice.new(product_id: product_1_id, price: 11),
@@ -22,21 +22,21 @@ def test_removing_items_from_refund
         Pricing::AddPriceItem.new(order_id: order_id, product_id: product_3_id, price: 33),
         Pricing::AcceptOffer.new(order_id: order_id),
         Fulfillment::RegisterOrder.new(order_id: order_id),
-        CreateDraftRefund.new(
-          refund_id: aggregate_id,
+        CreateDraftReturn.new(
+          return_id: aggregate_id,
           order_id: order_id
         ),
-        AddItemToRefund.new(
-          refund_id: aggregate_id,
+        AddItemToReturn.new(
+          return_id: aggregate_id,
           order_id: order_id,
           product_id: product_1_id
         )
       )
 
       expected_events = [
-        ItemRemovedFromRefund.new(
+        ItemRemovedFromReturn.new(
           data: {
-            refund_id: aggregate_id,
+            return_id: aggregate_id,
             order_id: order_id,
             product_id: product_1_id
           }
@@ -45,8 +45,8 @@ def test_removing_items_from_refund
 
       assert_events(stream, *expected_events) do
         act(
-          RemoveItemFromRefund.new(
-            refund_id: aggregate_id,
+          RemoveItemFromReturn.new(
+            return_id: aggregate_id,
             order_id: order_id,
             product_id: product_1_id
           )
@@ -64,26 +64,26 @@ def test_cant_remove_item_with_0_quantity
         Pricing::AddPriceItem.new(order_id: order_id, product_id: product_id, price: 11),
         Pricing::AcceptOffer.new(order_id: order_id),
         Fulfillment::RegisterOrder.new(order_id: order_id),
-        CreateDraftRefund.new(
-          refund_id: aggregate_id,
+        CreateDraftReturn.new(
+          return_id: aggregate_id,
           order_id: order_id
         ),
-        AddItemToRefund.new(
-          refund_id: aggregate_id,
+        AddItemToReturn.new(
+          return_id: aggregate_id,
           order_id: order_id,
           product_id: product_id
         ),
-        RemoveItemFromRefund.new(
-          refund_id: aggregate_id,
+        RemoveItemFromReturn.new(
+          return_id: aggregate_id,
           order_id: order_id,
           product_id: product_id
         )
       )
 
-      assert_raises(Refund::RefundHaveNotBeenRequestedForThisProductError) do
+      assert_raises(Return::ReturnHaveNotBeenRequestedForThisProductError) do
         act(
-          RemoveItemFromRefund.new(
-            refund_id: aggregate_id,
+          RemoveItemFromReturn.new(
+            return_id: aggregate_id,
             order_id: order_id,
             product_id: product_id
           )

…/ordering/test/refund_items_list_test.rb …/ordering/test/return_items_list_test.rbecommerce/ordering/test/refund_items_list_test.rb renamed to ecommerce/ordering/test/return_items_list_test.rb ecommerce/ordering/test/refund_items_list_test.rb renamed to ecommerce/ordering/test/return_items_list_test.rb

@@ -1,12 +1,12 @@
 require_relative "test_helper"
 
 module Ordering
-  class RefundItemsListTest < Test
+  class ReturnItemsListTest < Test
 
     def test_initialize
       list = ItemsList.new
 
-      assert_equal 0, list.refund_items.size
+      assert_equal 0, list.return_items.size
     end
 
     def test_increase_item_quantity
@@ -16,12 +16,12 @@ def test_increase_item_quantity
 
       list.increase_quantity(product_one_id)
 
-      assert_equal 1, list.refund_items.size
+      assert_equal 1, list.return_items.size
       assert_equal 1, list.quantity(product_one_id)
 
       list.increase_quantity(product_two_id)
 
-      assert_equal 2, list.refund_items.size
+      assert_equal 2, list.return_items.size
       assert_equal 1, list.quantity(product_two_id)
     end
 
@@ -32,17 +32,17 @@ def test_decrease_item_quantity
       list.increase_quantity(product_id)
       list.increase_quantity(product_id)
 
-      assert_equal 1, list.refund_items.size
+      assert_equal 1, list.return_items.size
       assert_equal 2, list.quantity(product_id)
 
       list.decrease_quantity(product_id)
 
-      assert_equal 1, list.refund_items.size
+      assert_equal 1, list.return_items.size
       assert_equal 1, list.quantity(product_id)
 
       list.decrease_quantity(product_id)
 
-      assert_equal 0, list.refund_items.size
+      assert_equal 0, list.return_items.size
     end
   end
 end

…rdering/test/refundable_products_test.rb …rdering/test/returnable_products_test.rbecommerce/ordering/test/refundable_products_test.rb renamed to ecommerce/ordering/test/returnable_products_test.rb ecommerce/ordering/test/refundable_products_test.rb renamed to ecommerce/ordering/test/returnable_products_test.rb

@@ -1,10 +1,10 @@
 require_relative "test_helper"
 
 module Ordering
-  class RefundableProductsTest < Test
-    cover "Ordering::RefundableProducts"
+  class ReturnableProductsTest < Test
+    cover "Ordering::ReturnableProducts"
 
-    def test_product_quantity_available_to_refund
+    def test_product_quantity_available_to_return
       order_id = SecureRandom.uuid
       order_number = Fulfillment::FakeNumberGenerator::FAKE_NUMBER
       product_1_id = SecureRandom.uuid
@@ -42,9 +42,9 @@ def test_product_quantity_available_to_refund
         stream_name: "Fulfillment::Order$#{order_id}"
       )
 
-      refundable_products = RefundableProducts.new.call(event_store, order_id)
+      returnable_products = ReturnableProducts.new.call(event_store, order_id)
 
-      assert_equal([{ product_id: product_1_id, quantity: 1 }, { product_id: product_2_id, quantity: 1 }], refundable_products)
+      assert_equal([{ product_id: product_1_id, quantity: 1 }, { product_id: product_2_id, quantity: 1 }], returnable_products)
     end
 
     def test_order_have_to_be_placed
@@ -66,7 +66,7 @@ def test_order_have_to_be_placed
         ],
         stream_name: stream_name)
 
-      assert_nil(RefundableProducts.new.call(event_store, order_id))
+      assert_nil(ReturnableProducts.new.call(event_store, order_id))
     end
   end
 end