effective_orders 1.8.1 → 2.0.0

data/lib/generators/templates/effective_orders.rb

@@ -8,6 +8,8 @@ EffectiveOrders.setup do |config|
   config.cart_items_table_name = :cart_items
   config.customers_table_name = :customers
   config.subscriptions_table_name = :subscriptions
+  config.products_table_name = :products
+
 
   # Authorization Method
   #
@@ -30,6 +32,12 @@ EffectiveOrders.setup do |config|
   # config.authorization_method = false
   config.authorization_method = Proc.new { |controller, action, resource| true }
 
+  # Skip automatically mounting the EffectiveOrders engine
+  config.skip_mount_engine = false
+
+  # Filter the @orders on admin/orders#index screen
+  # config.orders_collection_scope = Proc.new { |scope| scope.where(...) }
+
   # Register Effective::Order with ActiveAdmin if ActiveAdmin is present
   # You must have authorization to authorized?(:manage, Effective::Order)
   # For the activeadmin menu to item to show up
@@ -50,7 +58,7 @@ EffectiveOrders.setup do |config|
   # Works with effective_addresses gem
   config.use_address_full_name = true
 
-  # If set, the orders#new screen will render effective/orders/user_fields partial and capture this User Info
+  # If set, the orders#new screen will render effective/orders/_order_user_fields to capture this User Info
   # The partial can be overridden to customize the form, but the following fields are also fed into strong_paramters
   config.collect_user_fields = []
   #config.collect_user_fields = [:salutation, :first_name, :last_name] # Must be valid fields on the User object
@@ -58,8 +66,13 @@ EffectiveOrders.setup do |config|
   # Don't validate_associated :user when saving an Order
   config.skip_user_validation = false
 
+  # If set, the orders#new screen will render effective/orders/_order_note_fields to capture any Note info
+  config.collect_note = false
+  config.collect_note_required = false   # just required for the form, not a true Order model validation
+  config.collect_note_message = ''
+
   # Tax Calculation Method
-  config.tax_rate_method = Proc.new { |acts_as_purchasable| 0.05 } # Regardless of the object, charge 5% tax (GST)
+  config.order_tax_rate_method = Proc.new { |order| Effective::TaxRateCalculator.new(order: order).tax_rate }
 
   # Minimum Charge
   # Prevent orders less than this value from being purchased
@@ -99,24 +112,25 @@ EffectiveOrders.setup do |config|
 
   # config.layout = 'application'   # All EffectiveOrders controllers will use this layout
   config.layout = {
-    :carts => 'application',
-    :orders => 'application',
-    :subscriptions => 'application',
-    :admin_customers => 'application',
-    :admin_orders => 'application'
+    carts: 'application',
+    orders: 'application',
+    subscriptions: 'application',
+    admin_customers: 'application',
+    admin_orders: 'application'
   }
 
   # SimpleForm Options
-  # This Hash of options will be passed into any simple_form_for() calls
+  # This Hash of options will be passed into any client facing simple_form_for() calls
   config.simple_form_options = {}
+  config.admin_simple_form_options = {}  # For the /admin/orders/new form
 
   # config.simple_form_options = {
-  #   :html => {:class => 'form-horizontal'},
-  #   :wrapper => :horizontal_form,
-  #   :wrapper_mappings => {
-  #     :boolean => :horizontal_boolean,
-  #     :check_boxes => :horizontal_radio_and_checkboxes,
-  #     :radio_buttons => :horizontal_radio_and_checkboxes
+  #   html: {class: 'form-horizontal'},
+  #   wrapper: :horizontal_form,
+  #   wrapper_mappings: {
+  #     boolean: :horizontal_boolean,
+  #     check_boxes: :horizontal_radio_and_checkboxes,
+  #     radio_buttons: :horizontal_radio_and_checkboxes
   #   }
   # }
 
@@ -124,48 +138,63 @@ EffectiveOrders.setup do |config|
   # effective_orders will send out receipts to the buyer, seller and admins.
   # For all the emails, the same :subject_prefix will be prefixed.  Leave as nil / empty string if you don't want any prefix
   #
-  # The subject_for_admin_receipt, subject_for_buyer_receipt and subject_for_seller_receipt can be one of:
+  # The subject_for_admin_receipt, subject_for_buyer_receipt, subject_for_payment_request and
+  # subject_for_seller_receipt can be one of:
   # - nil / empty string to use the built in defaults
   # - A string with the full subject line for this email
   # - A Proc to create the subject line based on the email
   # In all three of these cases, the subject_prefix will still be used.
 
   # The Procs are the same for admin & buyer receipt, the seller Proc is different
-  # :subject_for_admin_receipt => Proc.new { |order| "Order #{order.to_param} has been purchased"}
-  # :subject_for_buyer_receipt => Proc.new { |order| "Order #{order.to_param} has been purchased"}
-  # :subject_for_seller_receipt => Proc.new { |order, order_items, seller| "Order #{order.to_param} has been purchased"}
+  # subject_for_order_receipt_to_admin: Proc.new { |order| "Order #{order.to_param} has been purchased"}
+  # subject_for_order_receipt_to_buyer: Proc.new { |order| "Order #{order.to_param} has been purchased"}
+  # subject_for_payment_request_to_buyer: Proc.new { |order| "Pending Order #{order.to_param}"}
+  # subject_for_order_receipt_to_seller: Proc.new { |order, order_items, seller| "Order #{order.to_param} has been purchased"}
 
   config.mailer = {
-    :send_order_receipt_to_admin => true,
-    :send_order_receipt_to_buyer => true,
-    :send_order_receipt_to_seller => true,   # Only applies to StripeConnect
-    :layout => 'effective_orders_mailer_layout',
-    :admin_email => 'admin@example.com',
-    :default_from => 'info@example.com',
-    :subject_prefix => '[example]',
-    :subject_for_admin_receipt => '',
-    :subject_for_buyer_receipt => '',
-    :subject_for_seller_receipt => '',
-    :deliver_method => nil,
-    :delayed_job_deliver => false
+    send_order_receipt_to_admin: true,
+    send_order_receipt_to_buyer: true,
+    send_order_receipt_to_seller: true,   # Only applies to StripeConnect
+    send_payment_request_to_buyer: true,
+    send_pending_order_invoice_to_buyer: true,
+    send_order_receipts_when_marked_paid_by_admin: false,
+
+    subject_prefix: '[example]',
+    subject_for_order_receipt_to_admin: '',
+    subject_for_order_receipt_to_buyer: '',
+    subject_for_order_receipt_to_seller: '',
+    subject_for_pending_order_invoice_to_buyer: '',
+    subject_for_payment_request_to_buyer: '',
+
+    layout: 'effective_orders_mailer_layout',
+
+    default_from: 'info@example.com',
+    admin_email: 'admin@example.com',
+
+    deliver_method: nil,   # :deliver (rails < 4.2), :deliver_now (rails >= 4.2) or :deliver_later
+    delayed_job_deliver: false
   }
 
+  #######################################
+  ### Payment Provider specific options
+  #######################################
+
   # Moneris configuration
   config.moneris_enabled = false
 
   if Rails.env.production?
     config.moneris = {
-      :ps_store_id => '',
-      :hpp_key => '',
-      :hpp_url => 'https://www3.moneris.com/HPPDP/index.php',
-      :verify_url => 'https://www3.moneris.com/HPPDP/verifyTxn.php'
+      ps_store_id: '',
+      hpp_key: '',
+      hpp_url: 'https://www3.moneris.com/HPPDP/index.php',
+      verify_url: 'https://www3.moneris.com/HPPDP/verifyTxn.php'
     }
   else
     config.moneris = {
-      :ps_store_id => '',
-      :hpp_key => '',
-      :hpp_url => 'https://esqa.moneris.com/HPPDP/index.php',
-      :verify_url => 'https://esqa.moneris.com/HPPDP/verifyTxn.php'
+      ps_store_id: '',
+      hpp_key: '',
+      hpp_url: 'https://esqa.moneris.com/HPPDP/index.php',
+      verify_url: 'https://esqa.moneris.com/HPPDP/verifyTxn.php'
     }
   end
 
@@ -174,25 +203,25 @@ EffectiveOrders.setup do |config|
 
   if Rails.env.production?
     config.paypal = {
-      :seller_email => '',
-      :secret => '',
-      :cert_id => '',
-      :paypal_url => 'https://www.paypal.com/cgi-bin/webscr',
-      :currency => 'CAD',
-      :paypal_cert => "#{Rails.root}/config/paypalcerts/production/paypal_cert.pem",
-      :app_cert => "#{Rails.root}/config/paypalcerts/production/app_cert.pem",
-      :app_key => "#{Rails.root}/config/paypalcerts/production/app_key.pem"
+      seller_email: '',
+      secret: '',
+      cert_id: '',
+      paypal_url: 'https://www.paypal.com/cgi-bin/webscr',
+      currency: 'CAD',
+      paypal_cert: "#{Rails.root}/config/paypalcerts/production/paypal_cert.pem",
+      app_cert: "#{Rails.root}/config/paypalcerts/production/app_cert.pem",
+      app_key: "#{Rails.root}/config/paypalcerts/production/app_key.pem"
     }
   else
     config.paypal = {
-      :seller_email => '',
-      :secret => '',
-      :cert_id => '',
-      :paypal_url => 'https://www.sandbox.paypal.com/cgi-bin/webscr',
-      :currency => 'CAD',
-      :paypal_cert => "#{Rails.root}/config/paypalcerts/#{Rails.env}/paypal_cert.pem",
-      :app_cert => "#{Rails.root}/config/paypalcerts/#{Rails.env}/app_cert.pem",
-      :app_key => "#{Rails.root}/config/paypalcerts/#{Rails.env}/app_key.pem"
+      seller_email: '',
+      secret: '',
+      cert_id: '',
+      paypal_url: 'https://www.sandbox.paypal.com/cgi-bin/webscr',
+      currency: 'CAD',
+      paypal_cert: "#{Rails.root}/config/paypalcerts/#{Rails.env}/paypal_cert.pem",
+      app_cert: "#{Rails.root}/config/paypalcerts/#{Rails.env}/app_cert.pem",
+      app_key: "#{Rails.root}/config/paypalcerts/#{Rails.env}/app_key.pem"
     }
   end
 
@@ -204,21 +233,48 @@ EffectiveOrders.setup do |config|
 
   if Rails.env.production?
     config.stripe = {
-      :secret_key => '',
-      :publishable_key => '',
-      :currency => 'usd',
-      :site_title => 'My Site',
-      :site_image => '', # A relative URL pointing to a square image of your brand or product. The recommended minimum size is 128x128px.
-      :connect_client_id => ''
+      secret_key: '',
+      publishable_key: '',
+      currency: 'usd',
+      site_title: 'My Site',
+      site_image: '', # A relative URL pointing to a square image of your brand or product. The recommended minimum size is 128x128px.
+      connect_client_id: ''
     }
   else
     config.stripe = {
-      :secret_key => '',
-      :publishable_key => '',
-      :currency => 'usd',
-      :site_title => 'My Development Site',  # Displayed on the Embedded Stripe Form
-      :site_image => '', # A relative URL pointing to a square image of your brand or product. The recommended minimum size is 128x128px.
-      :connect_client_id => ''
+      secret_key: '',
+      publishable_key: '',
+      currency: 'usd',
+      site_title: 'My Development Site',  # Displayed on the Embedded Stripe Form
+      site_image: '', # A relative URL pointing to a square image of your brand or product. The recommended minimum size is 128x128px.
+      connect_client_id: ''
+    }
+  end
+
+  # CCBill configuration
+  config.ccbill_enabled = false
+
+  # CCBill Dynamic Pricing documentation describes these variables:
+  # https://www.ccbill.com/cs/wiki/tiki-index.php?page=Dynamic+Pricing+User+Guide
+  if Rails.env.production?
+    config.ccbill = {
+      client_accnum: '',
+      client_subacc: '0000', # initial sub account
+      # Get this from CCBill Admin dashboard after setting up a form
+      form_name: '211cc', # default credit card form
+      # https://www.ccbill.com/cs/wiki/tiki-index.php?page=Webhooks+User+Guide#Appendix_A:_Currency_Codes
+      currency_code: '840', # USD
+      # You'll need to get this salt after having CCBill tech support set up dynamic pricing
+      # https://www.ccbill.com/cs/wiki/tiki-index.php?page=Dynamic+Pricing+User+Guide#Generating_the_MD5_Hash
+      dynamic_pricing_salt: ''
+    }
+  else
+    config.ccbill = {
+      client_accnum: '',
+      client_subacc: '0000',
+      form_name: '211cc',
+      currency_code: '840',
+      dynamic_pricing_salt: ''
     }
   end
 
@@ -230,4 +286,13 @@ EffectiveOrders.setup do |config|
     service: nil, # an EffectiveOrders::AppCheckout type object
     declined_flash: "Payment was unsuccessful. Please try again."
   }
+
+
+  # Pay by Cheque configuration
+  config.cheque_enabled = false
+
+  config.cheque = {
+    confirm: 'Your order will not be considered purchased until we receive your cheque. Proceed with pay by cheque?'
+  }
+
 end

data/lib/generators/templates/effective_orders_mailer_preview.rb

@@ -3,42 +3,56 @@
 # to see a preview of the following 3 emails:
 
 class EffectiveOrdersMailerPreview < ActionMailer::Preview
-  def order_receipt_to_buyer
-    Effective::OrdersMailer.order_receipt_to_buyer(build_preview_order)
-  end
-
   def order_receipt_to_admin
     Effective::OrdersMailer.order_receipt_to_admin(build_preview_order)
   end
 
+  def order_receipt_to_buyer
+    Effective::OrdersMailer.order_receipt_to_buyer(build_preview_order)
+  end
+
   # This email is only sent to sellers having sold items via StripeConnect
   def order_receipt_to_seller
     order = build_preview_order
     Effective::OrdersMailer.order_receipt_to_seller(order, preview_customer, order.order_items)
   end
 
+  def payment_request_to_buyer
+    Effective::OrdersMailer.payment_request_to_buyer(build_preview_order)
+  end
+
+  def pending_order_invoice_to_buyer
+    Effective::OrdersMailer.pending_order_invoice_to_buyer(build_preview_order)
+  end
+
+  def order_error
+    Effective::OrdersMailer.order_error(order: build_preview_order, error: "Something didn't work out")
+  end
+
   protected
 
   def build_preview_order
-    order = Effective::Order.new()
+    order = Effective::Order.new
     order.user = preview_user
     preview_order_items.each { |atts| order.order_items.build(atts) }
+    order.valid?
     order
   end
 
+  def build_address
+    Effective::Address.new(category: 'billing', full_name: 'Valued Customer', address1: '1234 Fake Street', address2: 'Suite 200', city: 'Edmonton', state_code: 'AB', country_code: 'CA', postal_code: 'T5T 2T1')
+  end
+
   private
 
   # We're building Effective::OrderItems directly rather than creating acts_as_purchasable objects
   # so that this mailer will not have to guess at your app's acts_as_purchasable objects
   def preview_order_items
-    tax_rate = (EffectiveOrders.tax_rate_method.call(Object.new()) rescue -1)
-    tax_rate = 0.05 if tax_rate < 0.0
-
     [
-      {title: 'Item One', quantity: 2, price: 999, tax_exempt: false, tax_rate: tax_rate},
-      {title: 'Item Two', quantity: 1, price: 25000, tax_exempt: false, tax_rate: tax_rate},
-      {title: 'Item Three', quantity: 1, price: 8999, tax_exempt: false, tax_rate: tax_rate},
-      {title: 'Item Four', quantity: 1, price: 100000, tax_exempt: false, tax_rate: tax_rate}
+      {title: 'Item One', quantity: 2, price: 999, tax_exempt: false},
+      {title: 'Item Two', quantity: 1, price: 25000, tax_exempt: false},
+      {title: 'Item Three', quantity: 1, price: 8999, tax_exempt: false},
+      {title: 'Item Four', quantity: 1, price: 100000, tax_exempt: false}
     ]
   end
 
@@ -51,11 +65,12 @@ class EffectiveOrdersMailerPreview < ActionMailer::Preview
         user.first_name = 'Valued'
         user.last_name = 'Customer'
       end
+
+      user.billing_address = build_address if user.respond_to?(:billing_address=)
     end
   end
 
   def preview_customer
     Effective::Customer.new(user: preview_user)
   end
-
 end

data/spec/controllers/admin/orders_controller_spec.rb

@@ -0,0 +1,242 @@
+require 'spec_helper'
+
+describe Admin::OrdersController, type: :controller do
+  routes { EffectiveOrders::Engine.routes }
+
+  let!(:user1) { FactoryGirl.create(:user, email: 'bbb@example.com') }
+  let!(:user2) { FactoryGirl.create(:user, email: 'ccc@example.com') }
+  let!(:user3) { FactoryGirl.create(:user, email: 'aaa@example.com') }
+  let(:cart) { FactoryGirl.create(:cart, user: user1) }
+
+  before { sign_in cart.user }
+
+  describe 'GET #new' do
+    it 'should render admin new order page successfully' do
+      get :new
+
+      expect(response).to be_successful
+      expect(response).to render_template :new
+      expect(assigns(:order)).to be_an Effective::Order
+      expect(assigns(:order)).to be_new_record
+      expect(assigns(:page_title)).to eq 'New Order'
+    end
+  end
+
+  describe 'POST #create' do
+    let(:user1) { FactoryGirl.create(:user, email: 'bbb@example.com', billing_address: FactoryGirl.create(:address), shipping_address: FactoryGirl.create(:address)) }
+
+    context 'when success' do
+      let(:order_params) { { effective_order: { user_id: user1.id, order_items_attributes: { '0' => { purchasable_attributes: { title: 'test product 1', price: '10000', tax_exempt: '1' }, quantity: '2', '_destroy' => 'false' }, '1' => { purchasable_attributes: { title: 'test product 2', price: '30000', tax_exempt: '0' }, quantity: '3', '_destroy' => 'false' } }, send_payment_request_to_buyer: '1' } } }
+
+      shared_context 'creates objects in db correctly' do
+        it 'should create new custom order with pending state' do
+          expect { post :create, order_params.merge(commit: button_pressed) }.to change { Effective::Order.count }.from(0).to(1)
+
+          expect(assigns(:order)).to be_persisted
+          expect(assigns(:order).pending?).to be_truthy
+          expect(assigns(:order).user).to eq user1
+          expect(assigns(:order).billing_address).to eq user1.billing_address
+          expect(assigns(:order).shipping_address).to eq user1.shipping_address
+
+          expect(assigns(:order).order_items.count).to eq 2
+
+          first_item = assigns(:order).order_items.sort.first
+          expect(first_item).to be_persisted
+          expect(first_item.title).to eq 'test product 1'
+          expect(first_item.quantity).to eq 2
+          expect(first_item.price).to eq 10000
+          expect(first_item.tax_exempt).to be_truthy
+
+          second_item = assigns(:order).order_items.sort.last
+          expect(second_item).to be_persisted
+          expect(second_item.title).to eq 'test product 2'
+          expect(second_item.quantity).to eq 3
+          expect(second_item.price).to eq 30000
+          expect(second_item.tax_exempt).to be_falsey
+        end
+
+        it 'should create new effective products' do
+          expect { post :create, order_params.merge(commit: button_pressed) }.to change { Effective::Product.count }.from(0).to(2)
+
+          first_product = Effective::Product.all.sort.first
+          expect(first_product.title).to eq 'test product 1'
+          expect(first_product.price).to eq 10000
+
+          second_product = Effective::Product.all.sort.last
+          expect(second_product.title).to eq 'test product 2'
+          expect(second_product.price).to eq 30000
+        end
+      end
+
+      context 'when "Save" button is pressed' do
+        let(:button_pressed) { 'Save' }
+
+        it_should_behave_like 'creates objects in db correctly'
+
+        it 'should redirect to admin orders index page with success message' do
+          post :create, order_params.merge(commit: button_pressed)
+
+          expect(response).to be_redirect
+          expect(response).to redirect_to EffectiveOrders::Engine.routes.url_helpers.admin_order_path(assigns(:order))
+          expect(flash[:success]).to eq "Successfully created order. #{assigns(:order).user.email} has been sent a request for payment."
+        end
+      end
+
+      context 'when "Save and Add New" button is pressed' do
+        let(:button_pressed) { 'Save and Add New' }
+
+        it_should_behave_like 'creates objects in db correctly'
+
+        it 'should redirect to admin new order page with success message' do
+          post :create, order_params.merge(commit: button_pressed)
+
+          expect(response).to be_redirect
+          expect(response).to redirect_to EffectiveOrders::Engine.routes.url_helpers.new_admin_order_path
+          expect(flash[:success]).to eq "Successfully created order. #{assigns(:order).user.email} has been sent a request for payment."
+        end
+      end
+    end
+
+    context 'when failed' do
+      let(:order_params) { { effective_order: { user_id: user1.id, order_items_attributes: { '0' => { purchasable_attributes: { title: 'test product 1', price: '0', tax_exempt: '1' }, quantity: '2', '_destroy' => 'false' } } } } }
+
+      shared_context 'does not create objects in db and redirects to admin new order page with danger message' do
+        it 'should not create order' do
+          expect { post :create, order_params.merge(commit: button_pressed) }.not_to change { Effective::Order.count }
+
+          expect(assigns(:order)).to be_new_record
+          expect(assigns(:order).valid?).to be_falsey
+          expect(assigns(:order).pending?).to be_truthy
+          expect(assigns(:order).user).to eq user1
+          expect(assigns(:order).billing_address).to eq user1.billing_address
+          expect(assigns(:order).shipping_address).to eq user1.shipping_address
+
+          expect(assigns(:order).order_items.to_a.count).to eq 1
+
+          item = assigns(:order).order_items.first
+          expect(item).to be_new_record
+          expect(item.valid?).to be_falsey
+          expect(item.title).to eq 'test product 1'
+          expect(item.quantity).to eq 2
+          expect(item.price).to eq 0
+          expect(item.tax_exempt).to be_truthy
+        end
+
+        it 'should not create product' do
+          expect { post :create, order_params.merge(commit: button_pressed) }.not_to change { Effective::Product.count }
+        end
+
+        it 'should render admin new order page with danger message' do
+          post :create, order_params.merge(commit: button_pressed)
+
+          expect(response).to be_successful
+          expect(response).to render_template :new
+          expect(assigns(:page_title)).to eq 'New Order'
+          expect(flash[:danger]).to eq 'Unable to create order'
+        end
+      end
+
+      context 'when "Save" button is pressed' do
+        let(:button_pressed) { 'Save' }
+
+        it_should_behave_like 'does not create objects in db and redirects to admin new order page with danger message'
+      end
+
+      context 'when "Save and Add New" button is pressed' do
+        let(:button_pressed) { 'Save and Add New' }
+
+        it_should_behave_like 'does not create objects in db and redirects to admin new order page with danger message'
+      end
+    end
+  end
+
+  describe 'POST #mark_as_paid' do
+    let(:order) { FactoryGirl.create(:pending_order) }
+
+    before { request.env['HTTP_REFERER'] = 'where_i_came_from' }
+
+    context 'when success' do
+      it 'should update order state and redirect to orders admin index page with success message' do
+        post :mark_as_paid, id: order.to_param
+
+        expect(response).to be_redirect
+        expect(response).to redirect_to EffectiveOrders::Engine.routes.url_helpers.admin_order_path(assigns(:order))
+        expect(assigns(:order)).to eq order
+        expect(assigns(:order).purchased?).to be_truthy
+        expect(assigns(:order).payment).to eq(details: 'Marked as paid by admin')
+        expect(flash[:success]).to eq 'Order marked as paid successfully'
+      end
+    end
+
+    context 'when failed' do
+      before { Effective::Order.any_instance.stub(:purchase!).and_return(false) }
+
+      it 'should redirect back with danger message' do
+        post :mark_as_paid, id: order.to_param
+
+        expect(response).to be_redirect
+        expect(response).to redirect_to 'where_i_came_from'
+        expect(assigns(:order)).to eq order
+        expect(assigns(:order).purchased?).to be_falsey
+        expect(flash[:danger]).to eq 'Unable to mark order as paid'
+      end
+    end
+  end
+
+  describe 'POST #send_payment_request' do
+    let(:user) { FactoryGirl.create(:user, email: 'user@example.com') }
+    let(:order) { FactoryGirl.create(:order, user: user) }
+
+    context 'when success' do
+      before { Effective::Order.any_instance.should_receive(:send_payment_request_to_buyer!).once.and_return(true) }
+
+      context 'when referrer page is present' do
+        before { request.env['HTTP_REFERER'] = 'where_i_came_from' }
+
+        it 'should redirect to previous page with success message' do
+          post :send_payment_request, id: order.to_param
+
+          expect(response).to be_redirect
+          expect(response).to redirect_to 'where_i_came_from'
+          expect(flash[:success]).to eq 'Successfully sent payment request to user@example.com'
+        end
+      end
+
+      context 'when referrer page is not present' do
+        it 'should redirect to admin order show page with success message' do
+          post :send_payment_request, id: order.to_param
+
+          expect(response).to be_redirect
+          expect(response).to redirect_to EffectiveOrders::Engine.routes.url_helpers.admin_order_path(order)
+          expect(flash[:success]).to eq 'Successfully sent payment request to user@example.com'
+        end
+      end
+    end
+
+    context 'when failed' do
+      before { Effective::Order.any_instance.should_receive(:send_payment_request_to_buyer!).once.and_return(false) }
+
+      context 'when referrer page is present' do
+        before { request.env['HTTP_REFERER'] = 'where_i_came_from' }
+
+        it 'should redirect to previous page with danger message' do
+          post :send_payment_request, id: order.to_param
+
+          expect(response).to be_redirect
+          expect(response).to redirect_to 'where_i_came_from'
+          expect(flash[:danger]).to eq 'Unable to send payment request'
+        end
+      end
+
+      context 'when referrer page is not present' do
+        it 'should redirect to admin order show page with danger message' do
+          post :send_payment_request, id: order.to_param
+
+          expect(response).to be_redirect
+          expect(response).to redirect_to EffectiveOrders::Engine.routes.url_helpers.admin_order_path(order)
+          expect(flash[:danger]).to eq 'Unable to send payment request'
+        end
+      end
+    end
+  end
+end