Skip to content

Commit

Permalink
Merge pull request #1940 from isaacfreeman/fix/order_resend_error
Browse files Browse the repository at this point in the history
Fix order resend
  • Loading branch information
gmacdougall committed May 23, 2017
2 parents 5fa5c37 + 6158088 commit d3eb81c
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 1 deletion.
2 changes: 1 addition & 1 deletion backend/app/controllers/spree/admin/orders_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ def approve
end

def resend
OrderMailer.confirm_email(@order.id, true).deliver_later
OrderMailer.confirm_email(@order, true).deliver_later
flash[:success] = Spree.t(:order_email_resent)

redirect_to(spree.edit_admin_order_path(@order))
Expand Down
11 changes: 11 additions & 0 deletions backend/spec/controllers/spree/admin/orders_controller_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,17 @@
end
end

context "#resend" do
let(:order) { create(:completed_order_with_totals) }
it "resends order email" do
mail_message = double "Mail::Message"
expect(Spree::OrderMailer).to receive(:confirm_email).with(order, true).and_return mail_message
expect(mail_message).to receive :deliver_later
post :resend, params: { id: order.number }
expect(flash[:success]).to eq Spree.t(:order_email_resent)
end
end

context "pagination" do
it "can page through the orders" do
get :index, params: { page: 2, per_page: 10 }
Expand Down

0 comments on commit d3eb81c

Please sign in to comment.