Use the rails logger instead of Bugsnag

app/jobs/standing_order_placement_job.rb

@@ -42,19 +42,9 @@ class StandingOrderPlacementJob
   end
 
   def move_to_completion(order)
-    until order.completed?
-      unless order.next!
-        Bugsnag.notify(RuntimeError.new("StandingOrderPlacementError"), {
-          job: "StandingOrderPlacement",
-          error: "Cannot process order due to errors",
-          data: {
-            order_number: order.number,
-            errors: order.errors.full_messages
-          }
-        })
-        break
-      end
-    end
+    until order.completed? do order.next! end
+  rescue StateMachine::InvalidTransition
+    log_completion_issue(order)
   end
 
   def unavailable_stock_lines_for(order)
@@ -73,4 +63,10 @@ class StandingOrderPlacementJob
   def send_empty_email(order, changes)
     Spree::OrderMailer.standing_order_email(order.id, 'empty', changes).deliver
   end
+
+  def log_completion_issue(order)
+    line1 = "StandingOrderPlacementError: Cannot process order #{order.number} due to errors"
+    line2 = "Errors: #{order.errors.full_messages.join(", ")}"
+    Rails.logger.info("#{line1}\n#{line2}")
+  end
 end

spec/jobs/standing_order_placement_job_spec.rb

@@ -166,6 +166,15 @@ describe StandingOrderPlacementJob do
           expect{job.send(:process, order)}.to_not enqueue_job ConfirmOrderJob
           expect(job).to have_received(:send_placement_email).with(order, anything).once
         end
+
+        context "when progression of the order fails" do
+          before { allow(order).to receive(:next) { false } }
+
+          it "logs an error" do
+            expect(Rails.logger).to receive(:info)
+            job.send(:process, order)
+          end
+        end
       end
     end
   end