Change to explicitly check complete state (default Spree::Order.complete scope checks only for presence of 'completed_at' timestamp)

app/mailers/producer_mailer.rb

@@ -33,7 +33,7 @@ class ProducerMailer < Spree::BaseMailer
       joins(:order => :order_cycle, :variant => :product).
       where('order_cycles.id = ?', order_cycle).
       merge(Spree::Product.in_supplier(producer)).
-      merge(Spree::Order.complete)
+      merge(Spree::Order.by_state('complete'))
   end
 
   def total_from_line_items(line_items)

spec/mailers/producer_mailer_spec.rb

@@ -39,8 +39,6 @@ describe ProducerMailer do
     order.line_items << create(:line_item, variant: p5.variants.first)
     order.finalize!
     order.cancel
-    puts order.inspect
-    order.finalize!
     order.save
     order
   end