spree-point-of-sale 1.0.0

data/spec/models/spree/order_decorator_spec.rb

@@ -0,0 +1,178 @@
+require 'spec_helper'
+
+describe Spree::Order do
+  let(:user) { mock_model(Spree::User, :email => 'test-user@pos.com') }
+  let(:flat_rate_calculator) { Spree::Calculator::Shipping::FlatRate.create! }
+  let(:country) { Spree::Country.create!(:name => 'mk_country', :iso_name => "mk") }
+  let(:state) { country.states.create!(:name => 'mk_state') }
+  let(:store) { Spree::StockLocation.create!(:name => 'store', :store => true, :address1 => "home", :address2 => "town", :city => "delhi", :zipcode => "110034", :country_id => country.id, :state_id => state.id, :phone => "07777676767") }
+ 
+  [:state, :is_pos, :completed_at, :payment_state].each do |attribute|
+    it { should allow_mass_assignment_of attribute }
+  end
+
+  before do
+    @order = Spree::Order.create!(:is_pos => true)
+    @order.stub(:total).and_return 100
+    @variant = Spree::Variant.new
+    @shipment = @order.shipments.new
+    @line_item = @order.line_items.new(:quantity => 1)
+    @line_item.variant = @variant
+    @payment = mock_model(Spree::Payment)
+    @payments = [@payment]
+    @payments.stub(:delete_all).and_return(true)
+    @order.stub(:payments).and_return(@payments)
+    @content = Spree::OrderContents.new(@order)
+    @order.stub(:contents).and_return(@content)
+    @content.stub(:remove).with(@variant, 1, @shipment).and_return(true)
+  end
+
+  context 'scopes' do
+    before :each do
+      @paid_order = Spree::Order.create!(:total => 100)
+      @paid_order.update_column(:payment_state,'paid')
+      @unpaid_pos_order = Spree::Order.create!(:is_pos => true, :payment_state => 'checkout')
+      @paid_pos_order = Spree::Order.create!(:is_pos => true, :payment_state => 'paid')
+      @paid_pos_order.update_column(:payment_state,'paid')    
+      @unpaid_order = Spree::Order.create!(:payment_state => 'checkout')
+    end
+
+    it { Spree::Order.pos.should match_array([@order, @unpaid_pos_order, @paid_pos_order]) }
+    it { Spree::Order.unpaid.should match_array([@unpaid_pos_order, @unpaid_order]) }
+    it { Spree::Order.unpaid_pos_order.should eq([@unpaid_pos_order]) }
+  end
+
+  describe '#clean!' do
+    before { @order.stub(:assign_shipment_for_pos).and_return(true) }
+    it { @payments.should_receive(:delete_all).and_return(true) }
+    it { @line_item.should_receive(:variant).and_return(@variant) }
+    it { @line_item.should_receive(:quantity).and_return(1) }
+    it { @content.should_receive(:remove).with(@variant, 1, @shipment).and_return(true) }
+    it { @order.should_receive(:assign_shipment_for_pos).and_return(true) }
+    after { @order.clean! }
+  end
+
+  describe '#complete_via_pos' do
+    before do
+      @order.stub(:create_tax_charge!).and_return(true)
+      @order.stub(:pending_payments).and_return(@payments)
+      @payment.stub(:capture!).and_return(true)
+      @shipment.stub(:finalize_pos).and_return(true)
+      @order.stub(:deliver_order_confirmation_email).and_return(true)
+      @order.stub(:save!).and_return(true)
+    end
+
+    it { @order.should_receive(:touch).with(:completed_at) }
+    it { @order.should_receive(:create_tax_charge!).and_return(true) }
+    it { @order.should_receive(:pending_payments).and_return(@payments) }
+    it { @payment.should_receive(:capture!).and_return(true) }
+    it { @shipment.should_receive(:finalize_pos).and_return(true) }
+    it { @order.should_receive(:deliver_order_confirmation_email).and_return(true) }
+    it { @order.should_receive(:save!).and_return(true) }
+    
+    after { @order.complete_via_pos }
+  end
+
+  describe '#assign_shipment_for_pos' do
+    context '#is_pos?' do
+      before do
+        @order = Spree::Order.create!(:is_pos => true)
+        shipping_category = Spree::ShippingCategory.create! :name => 'test-category'
+        @shipping_method = Spree::ShippingMethod.new(:name => 'test-method') { |method| method.calculator = flat_rate_calculator }
+        @shipping_method.shipping_categories << shipping_category
+        @shipping_method.save!
+        SpreePos::Config[:pos_shipping] = @shipping_method.name
+      end
+
+      describe 'method calls' do
+        it { @order.should_receive(:is_pos?).and_return(true) }
+
+        after { @order.assign_shipment_for_pos }
+      end
+
+      it 'create a shipment for order' do
+        @order.shipments.should be_blank
+        @order.shipments.create_shipment_for_pos_order
+        @order.shipments.count.should eq(1)
+      end
+    end
+
+    context '#is_pos? false' do
+      before { @order.stub(:is_pos?).and_return(false) }
+      
+      describe 'method calls' do
+        it { @order.should_receive(:is_pos?).and_return(false) }
+        it { @order.shipments.should_not_receive(:create_shipment_for_pos_order) }        
+        
+        after { @order.assign_shipment_for_pos }
+      end      
+    end
+  end
+
+  describe '#save_payment_for_pos' do
+    before do
+      @payments.stub(:delete_all).and_return(true)
+      @payments.stub(:create).with(:amount => 100, :payment_method_id => 1, :card_name => 'MasterCard').and_return(@payment)
+    end
+    it { @payments.should_receive(:delete_all).and_return(true) }
+    it { @payments.should_receive(:create).with(:amount => 100, :payment_method_id => 1, :card_name => 'MasterCard').and_return(@payment) }
+    after { @order.save_payment_for_pos(1, 'MasterCard') }
+  end
+
+  describe '#associate_user_for_pos' do
+    context 'user with email exists' do
+      before do
+        Spree::User.stub(:where).with(:email => user.email).and_return([user])
+      end
+
+      context 'when user is valid' do
+        before { user.stub(:valid?).and_return(true) }
+
+        it { Spree::User.should_receive(:where).with(:email => user.email).and_return([user]) }
+        it { Spree::User.should_not_receive(:create_with_random_password) }
+        it { @order.should_receive(:email=).with(user.email).and_return(true) }
+        after { @order.associate_user_for_pos(user.email) }
+      end
+
+      context 'when user is not valid' do
+        before { user.stub(:valid?).and_return(false) }
+        
+        it { Spree::User.should_receive(:where).with(:email => user.email).and_return([user]) }
+        it { Spree::User.should_not_receive(:create_with_random_password) }        
+        it { @order.should_not_receive(:email=).with(user.email) }
+        after { @order.associate_user_for_pos(user.email) }
+      end
+      
+      it { @order.associate_user_for_pos(user.email).should eq(user) }
+    end
+
+    context 'user with email does not exist' do
+      before do
+        @new_user = mock_model(Spree::User)
+        Spree::User.stub(:create_with_random_password).with('new-user@pos.com').and_return(@new_user)
+      end
+
+      context 'when new user is valid' do
+        before { @new_user.stub(:valid?).and_return(true) }
+
+        it { Spree::User.should_receive(:where).with(:email => 'new-user@pos.com').and_return([]) } 
+        it { Spree::User.should_receive(:create_with_random_password).with('new-user@pos.com').and_return(@new_user) }
+        it { @order.should_receive(:email=).with('new-user@pos.com').and_return(true) }
+        
+        after { @order.associate_user_for_pos('new-user@pos.com') }
+      end
+
+      context 'when new user is not valid' do
+        before { @new_user.stub(:valid?).and_return(false) }
+        
+        it { Spree::User.should_receive(:where).with(:email => 'new-user@pos.com').and_return([]) } 
+        it { Spree::User.should_receive(:create_with_random_password).with('new-user@pos.com').and_return(@new_user) }
+        it { @order.should_not_receive(:email=).with(@new_user.email) }
+        
+        after { @order.associate_user_for_pos('new-user@pos.com') }
+      end
+
+      it { @order.associate_user_for_pos('new-user@pos.com').should eq(@new_user) }
+    end
+  end
+end

data/spec/models/spree/payment_decorator_spec.rb

@@ -0,0 +1,115 @@
+require 'spec_helper'
+
+describe Spree::Payment do
+  it { should allow_mass_assignment_of :card_name }
+
+  let(:payment_method) { Spree::PaymentMethod.create!(:name => 'test-method') }
+  let(:card_payment_method) { Spree::PaymentMethod.create!(:name => 'card-payment')}
+  let(:order) { Spree::Order.create! }
+  context 'validations' do
+    describe 'payment_method' do
+      context 'pos_order' do
+        before do
+          order.stub(:is_pos?).and_return(true)
+          @payment = Spree::Payment.new
+          @payment.stub(:order).and_return(order)
+          @payment.save
+        end
+
+        it { @payment.errors[:payment_method].should eq(["can't be blank"])}
+      end
+
+      context 'non pos_order' do
+        before do
+          order.stub(:is_pos?).and_return(false)
+          @payment = Spree::Payment.new
+          @payment.stub(:order).and_return(order)
+          @payment.save
+        end
+
+        it { @payment.errors[:payment_method].should be_blank }
+      end
+    end
+
+    describe 'card_name presence' do
+      context 'without payment_method' do
+        before do 
+          order.stub(:is_pos?).and_return(true)
+          @payment = Spree::Payment.new
+          @payment.stub(:order).and_return(order)
+          @payment.save
+        end
+        it { @payment.errors[:card_name].should be_blank }
+      end
+
+      context 'not pos order' do
+        before do
+          order.stub(:is_pos?).and_return(false)
+          @payment = Spree::Payment.new(:payment_method_id => card_payment_method.id)
+          @payment.stub(:order).and_return(order)
+          @payment.save
+        end
+        it { @payment.errors[:card_name].should be_blank }
+      end
+
+      context 'non card payment method' do
+        before do
+          order.stub(:is_pos?).and_return(true)
+          @payment = Spree::Payment.new(:payment_method_id => payment_method.id)
+          @payment.stub(:order).and_return(order)
+          @payment.save
+        end
+        it { @payment.errors[:card_name].should be_blank }
+      end
+
+      context 'pos order with card payment' do
+        before do
+          order.stub(:is_pos?).and_return(true)
+          @payment = Spree::Payment.new(:payment_method_id => card_payment_method.id)
+          @payment.stub(:order).and_return(order)
+          @payment.save
+        end
+        it { @payment.errors[:card_name].should eq(["can't be blank"]) }
+      end
+    end
+
+    describe 'card_name absence' do
+      context 'without payment_method' do
+        before do 
+          @payment = Spree::Payment.new
+          @payment.stub(:order).and_return(order)
+          @payment.save
+        end
+        it { @payment.errors[:base].should be_blank }
+      end
+
+      context 'non card payment method' do
+        context 'with card name' do
+          before do
+            @payment = Spree::Payment.new(:payment_method_id => payment_method.id, :card_name => "MasterCard")
+            @payment.stub(:order).and_return(order)
+            @payment.save
+          end
+          it { @payment.errors[:base].should eq(['No card name to be saved with this payment']) }
+        end
+        context 'no card name' do
+          before do
+            @payment = Spree::Payment.new(:payment_method_id => payment_method.id)
+            @payment.stub(:order).and_return(order)
+            @payment.save
+          end
+          it { @payment.errors[:base].should be_blank }
+        end
+      end
+
+      context 'with card payment method' do
+        before do
+          @payment = Spree::Payment.new(:payment_method_id => card_payment_method.id)
+          @payment.stub(:order).and_return(order)
+          @payment.save
+        end
+        it { @payment.errors[:base].should be_blank }
+      end
+    end
+  end
+end

data/spec/models/spree/payment_method/point_of_sale_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe Spree::PaymentMethod::PointOfSale do
+  let(:pending_payment) { mock_model(Spree::Payment, :state => 'pending') }
+  let(:complete_payment) { mock_model(Spree::Payment, :state => 'complete') }
+  let(:void_payment) { mock_model(Spree::Payment, :state => 'void') }
+  before { @point_of_sale_payment = Spree::PaymentMethod::PointOfSale.new }
+  it { @point_of_sale_payment.actions.should eq(["capture", "void"]) }
+  it { @point_of_sale_payment.can_capture?(pending_payment).should be_true }
+  it { @point_of_sale_payment.can_capture?(complete_payment).should be_false }
+  it { @point_of_sale_payment.can_void?(pending_payment).should be_true }
+  it { @point_of_sale_payment.can_void?(void_payment).should be_false }
+  it { @point_of_sale_payment.source_required?.should be_false }
+  it { @point_of_sale_payment.payment_profiles_supported?.should be_false }
+
+  it 'voids a payment' do
+    ActiveMerchant::Billing::Response.should_receive(:new).with(true, "", {}, {}).and_return(true)
+    @point_of_sale_payment.void
+  end
+
+  it 'captures a payment' do
+    ActiveMerchant::Billing::Response.should_receive(:new).with(true, "", {}, {}).and_return(true)
+    @point_of_sale_payment.capture
+  end
+end

data/spec/models/spree/shipment_decorator_spec.rb

@@ -0,0 +1,172 @@
+require 'spec_helper'
+
+describe Spree::Shipment do
+  let(:order) { Spree::Order.create! }
+  let(:flat_rate_calculator) { Spree::Calculator::Shipping::FlatRate.create! }
+  let(:country) { Spree::Country.create!(:name => 'mk_country', :iso_name => "mk") }
+  let(:state) { country.states.create!(:name => 'mk_state') }
+  let(:store) { Spree::StockLocation.create!(:name => 'store', :store => true, :address1 => "home", :address2 => "town", :city => "delhi", :zipcode => "110034", :country_id => country.id, :state_id => state.id, :phone => "07777676767") }
+  
+  before do
+    @shipment = Spree::Shipment.new
+  end
+
+  describe '#finalize_pos' do
+    before do
+      @inventory_unit = mock_model(Spree::InventoryUnit)
+      @inventory_unit.stub(:ship!).and_return(true)
+      @inventory_units = [@inventory_unit]
+      @shipment.stub(:inventory_units).and_return(@inventory_units)
+      @shipment.stub(:state=).with('shipped').and_return(true)
+      @shipment.stub(:touch).with(:delivered_at).and_return(true)
+      @shipment.stub(:save).and_return(true)
+    end
+
+    it { @shipment.should_receive(:state=).with('shipped').and_return(true) }
+    it { @shipment.should_receive(:inventory_units).and_return(@inventory_units) }
+    it { @inventory_unit.should_receive(:ship!).and_return(true) }
+    it { @shipment.should_receive(:touch).with(:delivered_at).and_return(true) }
+    it { @shipment.should_receive(:save).and_return(true) }
+    after { @shipment.finalize_pos }
+  end
+
+  describe 'create_shipment_for_pos_order' do
+    before do
+      shipping_category = Spree::ShippingCategory.create! :name => 'test-category'
+      @shipping_method = Spree::ShippingMethod.new(:name => 'test-method') { |method| method.calculator = flat_rate_calculator }
+      @shipping_method.shipping_categories << shipping_category
+      @shipping_method.save!
+      SpreePos::Config[:pos_shipping] = @shipping_method.name
+      store.save
+    end
+    
+    it 'looks for the first active store' do
+      Spree::StockLocation.should_receive(:stores).and_return(Spree::StockLocation)
+      Spree::StockLocation.should_receive(:active).and_return([store])
+      order.shipments.create_shipment_for_pos_order
+    end
+
+    it 'looks for the config shipping method' do
+      Spree::ShippingMethod.should_receive(:where).with(:name => SpreePos::Config[:pos_shipping]).and_return([@shipping_method])
+      order.shipments.create_shipment_for_pos_order
+    end
+
+    describe 'creates a new shipment for the order' do
+      before { order.shipments.create_shipment_for_pos_order }
+      it { order.shipments.count.should eq(1) }
+      it { order.shipments.should_not be_blank }
+      it { order.shipments.first.shipping_method.should eq(@shipping_method) }
+      it { order.shipments.first.stock_location.should eq(store) }
+    end
+  end
+
+  describe 'validate empty inventory' do
+    before do
+      @order = Spree::Order.create!
+      @shipment = @order.shipments.create!
+      @shipment.stock_location = store
+      @shipment.save!
+      @inventory_unit = @shipment.inventory_units.create!
+    end
+
+    context 'not a pos order' do
+      it 'no error on shipment' do
+        @shipment.reload
+        @shipment.save
+        @shipment.errors.should be_blank
+      end
+    end
+
+    context 'pos order' do
+      before { @order.update_column(:is_pos, true) }
+
+      context 'stock location present and not changed' do
+        before do
+          @shipment.reload
+          @shipment.stock_location = store
+          @shipment.save
+        end
+
+        it { @shipment.errors[:base].should be_blank }
+      end
+
+      context 'stock location present and changed' do
+        before do
+          @shipment.stock_location = nil
+          @shipment.save!
+          @shipment.reload
+          @shipment.stock_location = store
+          @shipment.save
+        end
+
+        it { @shipment.errors[:base].should eq(["Inventory Units assigned for the order. Please remove them to change stock location"]) }
+      end
+
+      context 'stock_location not present' do
+        before do
+          @shipment.stock_location = nil
+          @shipment.save
+        end
+
+        it { @shipment.errors[:base].should be_blank }
+      end
+    end
+  end
+
+  describe 'udpate_order_addresses_from_stock_location' do
+    before do
+      @order = Spree::Order.create!
+      @shipment = @order.shipments.create!
+      @shipment.stock_location = store
+      @shipment.save!
+    end
+
+    context 'not a pos order' do
+      before { @order.reload }
+
+      it { @order.ship_address.should be_nil }
+      it { @order.bill_address.should be_nil }
+    end
+
+    context 'pos order' do
+      before { @order.update_column(:is_pos, true) }
+
+      context 'stock location present and not changed' do
+        before do
+          @shipment.reload
+          @shipment.stock_location = store
+          @shipment.save!
+          @order.reload
+        end
+
+        it { @order.ship_address.should be_nil }
+        it { @order.bill_address.should be_nil }
+      end
+
+      context 'stock location present and changed' do
+        before do
+          @shipment.stock_location = nil
+          @shipment.save!
+          @shipment.reload
+          @shipment.stock_location = store
+          @shipment.save!
+          @order.reload
+        end
+
+        it { @order.ship_address.should eq(store.address) }
+        it { @order.bill_address.should eq(store.address) }
+      end
+
+      context 'stock_location not present' do
+        before do
+          @shipment.stock_location = nil
+          @shipment.save!
+          @order.reload
+        end
+
+        it { @order.ship_address.should be_nil }
+        it { @order.bill_address.should be_nil }
+      end
+    end
+  end
+end