diff --git a/app/jobs/subscription_placement_job.rb b/app/jobs/subscription_placement_job.rb index 842884efe7..fe270c3a45 100644 --- a/app/jobs/subscription_placement_job.rb +++ b/app/jobs/subscription_placement_job.rb @@ -26,6 +26,6 @@ class SubscriptionPlacementJob < ActiveJob::Base end def place_order_for(proxy_order) - PlaceOrder.new(proxy_order, summarizer, JobLogger.logger, CapQuantity.new(proxy_order.order)).call + PlaceProxyOrder.new(proxy_order, summarizer, JobLogger.logger, CapQuantity.new(proxy_order.order)).call end end diff --git a/app/services/place_order.rb b/app/services/place_proxy_order.rb similarity index 98% rename from app/services/place_order.rb rename to app/services/place_proxy_order.rb index 535a9028dc..667f63690c 100644 --- a/app/services/place_order.rb +++ b/app/services/place_proxy_order.rb @@ -1,6 +1,6 @@ # frozen_string_literal: true -class PlaceOrder +class PlaceProxyOrder def initialize(proxy_order, summarizer, logger, stock_changes_loader) @proxy_order = proxy_order @subscription = proxy_order.subscription diff --git a/spec/jobs/subscription_placement_job_spec.rb b/spec/jobs/subscription_placement_job_spec.rb index 5a69a7f8fc..bc8d7a9a97 100644 --- a/spec/jobs/subscription_placement_job_spec.rb +++ b/spec/jobs/subscription_placement_job_spec.rb @@ -59,14 +59,14 @@ describe SubscriptionPlacementJob do it "processes placeable proxy_orders" do summarizer = instance_double(OrderManagement::Subscriptions::Summarizer) - service = PlaceOrder.new( + service = PlaceProxyOrder.new( proxy_order, summarizer, JobLogger.logger, CapQuantity.new(proxy_order.order) ) - allow(PlaceOrder).to receive(:new) { service } + allow(PlaceProxyOrder).to receive(:new) { service } allow(service).to receive(:call) job.perform @@ -92,8 +92,8 @@ describe SubscriptionPlacementJob do before do expect_any_instance_of(Spree::Payment).to_not receive(:process!) - allow_any_instance_of(PlaceOrder).to receive(:send_placement_email) - allow_any_instance_of(PlaceOrder).to receive(:send_empty_email) + allow_any_instance_of(PlaceProxyOrder).to receive(:send_placement_email) + allow_any_instance_of(PlaceProxyOrder).to receive(:send_empty_email) end context "when the order is already complete" do @@ -101,7 +101,7 @@ describe SubscriptionPlacementJob do it "records an issue and ignores it" do summarizer = instance_double(OrderManagement::Subscriptions::Summarizer, record_order: true) - service = PlaceOrder.new( + service = PlaceProxyOrder.new( proxy_order, summarizer, JobLogger.logger, @@ -140,7 +140,7 @@ describe SubscriptionPlacementJob do it "does not place the order, clears all adjustments, and sends an empty_order email" do summarizer = instance_double(OrderManagement::Subscriptions::Summarizer, record_order: true, record_issue: true) - service = PlaceOrder.new( + service = PlaceProxyOrder.new( proxy_order, summarizer, JobLogger.logger, @@ -164,7 +164,7 @@ describe SubscriptionPlacementJob do instance_double(OrderManagement::Subscriptions::Summarizer, record_order: true, record_success: true) end let(:service) do - PlaceOrder.new( + PlaceProxyOrder.new( proxy_order, summarizer, JobLogger.logger, diff --git a/spec/services/place_order_spec.rb b/spec/services/place_proxy_order_spec.rb similarity index 99% rename from spec/services/place_order_spec.rb rename to spec/services/place_proxy_order_spec.rb index ec1337137b..12858692d0 100644 --- a/spec/services/place_order_spec.rb +++ b/spec/services/place_proxy_order_spec.rb @@ -2,7 +2,7 @@ require 'spec_helper' -describe PlaceOrder do +describe PlaceProxyOrder do include ActiveSupport::Testing::TimeHelpers subject { described_class.new(proxy_order, summarizer, logger, stock_changes_loader) }