solidus_core 1.1.0 → 1.1.1

data/spec/models/spree/stock_movement_spec.rb

@@ -0,0 +1,56 @@
+require 'spec_helper'
+
+describe Spree::StockMovement, :type => :model do
+  let(:stock_location) { create(:stock_location_with_items) }
+  let(:stock_item) { stock_location.stock_items.order(:id).first }
+  subject { build(:stock_movement, stock_item: stock_item) }
+
+  it 'should belong to a stock item' do
+    expect(subject).to respond_to(:stock_item)
+  end
+
+  it 'is readonly unless new' do
+    subject.save
+    expect {
+      subject.save
+    }.to raise_error(ActiveRecord::ReadOnlyRecord)
+  end
+
+  it 'does not update count on hand when track inventory levels is false' do
+    Spree::Config[:track_inventory_levels] = false
+    subject.quantity = 1
+    subject.save
+    stock_item.reload
+    expect(stock_item.count_on_hand).to eq(10)
+  end
+
+  it 'does not update count on hand when variant inventory tracking is off' do
+    stock_item.variant.track_inventory = false
+    subject.quantity = 1
+    subject.save
+    stock_item.reload
+    expect(stock_item.count_on_hand).to eq(10)
+  end
+
+  context "when quantity is negative" do
+    context "after save" do
+      it "should decrement the stock item count on hand" do
+        subject.quantity = -1
+        subject.save
+        stock_item.reload
+        expect(stock_item.count_on_hand).to eq(9)
+      end
+    end
+  end
+
+  context "when quantity is positive" do
+    context "after save" do
+      it "should increment the stock item count on hand" do
+        subject.quantity = 1
+        subject.save
+        stock_item.reload
+        expect(stock_item.count_on_hand).to eq(11)
+      end
+    end
+  end
+end

data/spec/models/spree/stock_transfer_spec.rb

@@ -0,0 +1,290 @@
+require 'spec_helper'
+
+module Spree
+  describe StockTransfer, :type => :model do
+    let(:destination_location) { create(:stock_location_with_items) }
+    let(:source_location) { create(:stock_location_with_items) }
+    let(:stock_item) { source_location.stock_items.order(:id).first }
+    let(:variant) { stock_item.variant }
+    let(:stock_transfer) do
+      StockTransfer.create(description: 'PO123', source_location: source_location, destination_location: destination_location)
+    end
+
+    subject { stock_transfer }
+
+    describe '#description' do
+      subject { super().description }
+      it { is_expected.to eq 'PO123' }
+    end
+
+    describe '#to_param' do
+      subject { super().to_param }
+      it { is_expected.to match /T\d+/ }
+    end
+
+    describe "transfer item building" do
+      let(:stock_transfer) do
+        variant = source_location.stock_items.first.variant
+        stock_transfer = Spree::StockTransfer.new(
+          number: "T123",
+          source_location: source_location,
+          destination_location: destination_location
+        )
+        stock_transfer.transfer_items.build(variant: variant, expected_quantity: 5)
+        stock_transfer
+      end
+
+      subject { stock_transfer.save }
+
+      it { is_expected.to eq true }
+
+      it "creates the associated transfer item" do
+        expect { subject }.to change { Spree::TransferItem.count }.by(1)
+      end
+    end
+
+    describe "#receivable?" do
+      subject { stock_transfer.receivable? }
+
+      context "finalized" do
+        before do
+          stock_transfer.update_attributes(finalized_at: Time.now)
+        end
+
+        it { is_expected.to eq false }
+      end
+
+      context "shipped" do
+        before do
+          stock_transfer.update_attributes(shipped_at: Time.now)
+        end
+
+        it { is_expected.to eq false }
+      end
+
+      context "closed" do
+        before do
+          stock_transfer.update_attributes(closed_at: Time.now)
+        end
+
+        it { is_expected.to eq false }
+      end
+
+      context "finalized and closed" do
+        before do
+          stock_transfer.update_attributes(finalized_at: Time.now, closed_at: Time.now)
+        end
+
+        it { is_expected.to eq false }
+      end
+
+      context "shipped and closed" do
+        before do
+          stock_transfer.update_attributes(shipped_at: Time.now, closed_at: Time.now)
+        end
+
+        it { is_expected.to eq false }
+      end
+
+      context "finalized and shipped" do
+        before do
+          stock_transfer.update_attributes(finalized_at: Time.now, shipped_at: Time.now)
+        end
+
+        it { is_expected.to eq true }
+      end
+    end
+
+    describe "#finalizable?" do
+      subject { stock_transfer.finalizable? }
+
+      context "finalized" do
+        before do
+          stock_transfer.update_attributes(finalized_at: Time.now)
+        end
+
+        it { is_expected.to eq false }
+      end
+
+      context "shipped" do
+        before do
+          stock_transfer.update_attributes(shipped_at: Time.now)
+        end
+
+        it { is_expected.to eq false }
+      end
+
+      context "closed" do
+        before do
+          stock_transfer.update_attributes(closed_at: Time.now)
+        end
+
+        it { is_expected.to eq false }
+      end
+
+      context "finalized and closed" do
+        before do
+          stock_transfer.update_attributes(finalized_at: Time.now, closed_at: Time.now)
+        end
+
+        it { is_expected.to eq false }
+      end
+
+      context "shipped and closed" do
+        before do
+          stock_transfer.update_attributes(shipped_at: Time.now, closed_at: Time.now)
+        end
+
+        it { is_expected.to eq false }
+      end
+
+      context "no action taken on stock transfer" do
+        before do
+          stock_transfer.update_attributes(finalized_at: nil, shipped_at: nil, closed_at: nil)
+        end
+
+        it { is_expected.to eq true }
+      end
+    end
+
+    describe "#finalize" do
+      let(:user) { create(:user) }
+
+      subject { stock_transfer.finalize(user) }
+
+      context "can be finalized" do
+        it "sets a finalized_at date" do
+          expect { subject }.to change { stock_transfer.finalized_at }
+        end
+
+        it "sets the finalized_by to the supplied user" do
+          subject
+          expect(stock_transfer.finalized_by).to eq user
+        end
+      end
+
+      context "can't be finalized" do
+        before do
+          stock_transfer.update_attributes(finalized_at: Time.now)
+        end
+
+        it "doesn't set a finalized_at date" do
+          expect { subject }.to_not change { stock_transfer.finalized_at }
+        end
+
+        it "doesn't set a finalized_by user" do
+          expect { subject }.to_not change { stock_transfer.finalized_by }
+        end
+
+        it "adds an error message" do
+          subject
+          expect(stock_transfer.errors.full_messages).to include Spree.t(:stock_transfer_cannot_be_finalized)
+        end
+      end
+    end
+
+    describe "#close" do
+      let(:user) { create(:user) }
+      let(:stock_transfer) { create(:receivable_stock_transfer_with_items) }
+
+      subject { stock_transfer.close(user) }
+
+      context "can be closed" do
+        it "sets a closed_at date" do
+          expect { subject }.to change { stock_transfer.closed_at }
+        end
+
+        it "sets the closed_by to the supplied user" do
+          subject
+          expect(stock_transfer.closed_by).to eq user
+        end
+      end
+
+      context "can't be closed" do
+        before do
+          stock_transfer.update_attributes(finalized_at: nil)
+        end
+
+        it "doesn't set a closed_at date" do
+          expect { subject }.to_not change { stock_transfer.closed_at }
+        end
+
+        it "doesn't set a closed_by user" do
+          expect { subject }.to_not change { stock_transfer.closed_by }
+        end
+
+        it "adds an error message" do
+          subject
+          expect(stock_transfer.errors.full_messages).to include Spree.t(:stock_transfer_must_be_receivable)
+        end
+      end
+    end
+
+    describe "destroying" do
+      subject { stock_transfer.destroy }
+
+      context "stock transfer is finalized" do
+        before do
+          stock_transfer.update_attributes!(finalized_at: Time.now)
+        end
+
+        it "doesn't destroy the stock transfer" do
+          expect { subject }.to_not change { Spree::StockTransfer.count }
+        end
+
+        it "adds an error message to the model" do
+          subject
+          expect(stock_transfer.errors.full_messages).to include Spree.t('errors.messages.cannot_delete_finalized_stock_transfer')
+        end
+      end
+
+      context "stock transfer is not finalized" do
+        before do
+          stock_transfer.update_attributes!(finalized_at: nil)
+        end
+
+        it "destroys the stock transfer" do
+          expect { subject }.to change { Spree::StockTransfer.count }.by(-1)
+        end
+      end
+    end
+
+    describe '#transfer' do
+      let(:stock_transfer) { create(:stock_transfer_with_items) }
+
+      before do
+        stock_transfer.transfer_items.each { |item| item.update_attributes(expected_quantity: 1) }
+      end
+
+      subject { stock_transfer.transfer }
+
+      context 'with enough stock' do
+
+        it 'creates stock movements for transfer items' do
+          expect{ subject }.to change{ Spree::StockMovement.count }.by(stock_transfer.transfer_items.count)
+        end
+      end
+
+      context 'without enough stock' do
+        before do
+          stockless_variant = stock_transfer.transfer_items.last.variant
+          stock_transfer.source_location.stock_item(stockless_variant).set_count_on_hand(0)
+        end
+
+        it 'rollsback the transaction' do
+          expect{ subject }.to_not change{ Spree::StockMovement.count }
+        end
+
+        it 'adds errors' do
+          subject
+          expect(stock_transfer.errors.full_messages.join(', ')).to match /not enough inventory/
+        end
+
+        it 'returns false' do
+          expect(subject).to eq false
+        end
+
+      end
+    end
+  end
+end

data/spec/models/spree/store_credit_category_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+describe Spree::StoreCreditCategory, :type => :model do
+  describe "#non_expiring?" do
+    let(:store_credit_category) { build(:store_credit_category, name: category_name) }
+
+    context "non-expiring type store credit" do
+      let(:category_name) { "Non-expiring" }
+      it { expect(store_credit_category).to be_non_expiring }
+    end
+
+    context "expiring type store credit" do
+      let(:category_name) { "Expiring" }
+      it { expect(store_credit_category).not_to be_non_expiring }
+    end
+  end
+end

data/spec/models/spree/store_credit_event_spec.rb

@@ -0,0 +1,314 @@
+require 'spec_helper'
+
+describe Spree::StoreCreditEvent do
+
+  describe ".exposed_events" do
+
+    [
+      Spree::StoreCredit::ELIGIBLE_ACTION,
+      Spree::StoreCredit::AUTHORIZE_ACTION,
+    ].each do |action|
+      let(:action) { action }
+      it "excludes #{action} actions" do
+        event = create(:store_credit_event, action: action)
+        expect(described_class.exposed_events).not_to include event
+      end
+    end
+
+    [
+      Spree::StoreCredit::VOID_ACTION,
+      Spree::StoreCredit::CREDIT_ACTION,
+      Spree::StoreCredit::CAPTURE_ACTION,
+      Spree::StoreCredit::ALLOCATION_ACTION,
+    ].each do |action|
+      it "includes #{action} actions" do
+        event = create(:store_credit_event, action: action)
+        expect(described_class.exposed_events).to include event
+      end
+    end
+
+    it "excludes invalidated store credit events" do
+      invalidated_store_credit = create(:store_credit, invalidated_at: Time.now)
+      event = create(:store_credit_event, action: Spree::StoreCredit::VOID_ACTION, store_credit: invalidated_store_credit)
+      expect(described_class.exposed_events).not_to include event
+    end
+  end
+
+  describe "update reason validation" do
+    subject { event.valid? }
+
+    context "adjustment event" do
+      context "has an update reason" do
+        let(:event) { build(:store_credit_adjustment_event) }
+
+        it "returns true" do
+          expect(subject).to eq true
+        end
+      end
+
+      context "doesn't have an update reason" do
+        let(:event) { build(:store_credit_adjustment_event, update_reason: nil) }
+
+        it "returns false" do
+          expect(subject).to eq false
+        end
+
+        it "adds an error message indicating the update reason is missing" do
+          subject
+          expect(event.errors.full_messages).to match ["Update reason can't be blank"]
+        end
+      end
+    end
+
+    context "invalidate event" do
+      context "has an update reason" do
+        let(:event) { build(:store_credit_invalidate_event) }
+
+        it "returns true" do
+          expect(subject).to eq true
+        end
+      end
+
+      context "doesn't have an update reason" do
+        let(:event) { build(:store_credit_invalidate_event, update_reason: nil) }
+
+        it "returns false" do
+          expect(subject).to eq false
+        end
+
+        it "adds an error message indicating the update reason is missing" do
+          subject
+          expect(event.errors.full_messages).to match ["Update reason can't be blank"]
+        end
+      end
+    end
+
+    context "event doesn't require an update reason" do
+      let(:event) { build(:store_credit_auth_event) }
+
+      it "returns true" do
+        expect(subject).to eq true
+      end
+    end
+  end
+
+  describe "#capture_action?" do
+    subject { event.capture_action? }
+
+    context "for capture events" do
+      let(:event) { create(:store_credit_capture_event) }
+
+      it "returns true" do
+        expect(subject).to eq true
+      end
+    end
+
+    context "for non-capture events" do
+      let(:event) { create(:store_credit_auth_event) }
+
+      it "returns false" do
+        expect(subject).to eq false
+      end
+    end
+  end
+
+  describe "#authorization_action?" do
+    subject { event.authorization_action? }
+
+    context "for auth events" do
+      let(:event) { create(:store_credit_auth_event) }
+
+      it "returns true" do
+        expect(subject).to eq true
+      end
+    end
+
+    context "for non-auth events" do
+      let(:event) { create(:store_credit_capture_event) }
+
+      it "returns false" do
+        expect(subject).to eq false
+      end
+    end
+  end
+
+  describe "#action_requires_reason?" do
+    subject { event.action_requires_reason? }
+
+    context "for adjustment events" do
+      let(:event) { create(:store_credit_adjustment_event) }
+
+      it "returns true" do
+        expect(subject).to eq true
+      end
+    end
+
+    context "for invalidate events" do
+      let(:event) { create(:store_credit_invalidate_event) }
+
+      it "returns true" do
+        expect(subject).to eq true
+      end
+    end
+
+    context "for capture events" do
+      let(:event) { create(:store_credit_capture_event) }
+
+      it "returns false" do
+        expect(subject).to eq false
+      end
+    end
+
+    context "for authorize events" do
+      let(:event) { create(:store_credit_auth_event) }
+
+      it "returns false" do
+        expect(subject).to eq false
+      end
+    end
+
+    context "for allocation events" do
+      let(:event) { create(:store_credit_event, action: Spree::StoreCredit::ALLOCATION_ACTION) }
+
+      it "returns false" do
+        expect(subject).to eq false
+      end
+    end
+
+    context "for void events" do
+      let(:event) { create(:store_credit_event, action: Spree::StoreCredit::VOID_ACTION) }
+
+      it "returns false" do
+        expect(subject).to eq false
+      end
+    end
+
+    context "for credit events" do
+      let(:event) { create(:store_credit_event, action: Spree::StoreCredit::CREDIT_ACTION) }
+
+      it "returns false" do
+        expect(subject).to eq false
+      end
+    end
+  end
+
+  describe "#display_amount" do
+    let(:event_amount) { 120.0 }
+
+    subject { create(:store_credit_auth_event, amount: event_amount) }
+
+    it "returns a Spree::Money instance" do
+      expect(subject.display_amount).to be_instance_of(Spree::Money)
+    end
+
+    it "uses the events amount attribute" do
+      expect(subject.display_amount).to eq Spree::Money.new(event_amount, { currency: subject.currency })
+    end
+  end
+
+  describe "#display_user_total_amount" do
+    let(:user_total_amount) { 300.0 }
+
+    subject { create(:store_credit_auth_event, user_total_amount: user_total_amount) }
+
+    it "returns a Spree::Money instance" do
+      expect(subject.display_user_total_amount).to be_instance_of(Spree::Money)
+    end
+
+    it "uses the events user_total_amount attribute" do
+      expect(subject.display_user_total_amount).to eq Spree::Money.new(user_total_amount, { currency: subject.currency })
+    end
+  end
+
+  describe "#display_event_date" do
+    let(:date) { DateTime.new(2014, 06, 1) }
+
+    subject { create(:store_credit_auth_event, created_at: date) }
+
+    it "returns the date the event was created with the format month/date/year" do
+      expect(subject.display_event_date).to eq "June 01, 2014"
+    end
+  end
+
+  describe "#display_action" do
+    subject { event.display_action  }
+
+    context "capture event" do
+      let(:event) { create(:store_credit_capture_event) }
+
+      it "returns the action's display text" do
+        expect(subject).to eq "Used"
+      end
+    end
+
+    context "allocation event" do
+      let(:event) { create(:store_credit_event, action: Spree::StoreCredit::ALLOCATION_ACTION) }
+
+      it "returns the action's display text" do
+        expect(subject).to eq "Added"
+      end
+    end
+
+    context "void event" do
+      let(:event) { create(:store_credit_event, action: Spree::StoreCredit::VOID_ACTION) }
+
+      it "returns the action's display text" do
+        expect(subject).to eq "Credit"
+      end
+    end
+
+    context "credit event" do
+      let(:event) { create(:store_credit_event, action: Spree::StoreCredit::CREDIT_ACTION) }
+
+      it "returns the action's display text" do
+        expect(subject).to eq "Credit"
+      end
+    end
+
+    context "adjustment event" do
+      let(:event) { create(:store_credit_adjustment_event) }
+
+      it "returns the action's display text" do
+        expect(subject).to eq "Adjustment"
+      end
+    end
+
+    context "authorize event" do
+      let(:event) { create(:store_credit_auth_event) }
+
+      it "returns nil" do
+        expect(subject).to be_nil
+      end
+    end
+
+    context "eligible event" do
+      let(:event) { create(:store_credit_event, action: Spree::StoreCredit::ELIGIBLE_ACTION) }
+
+      it "returns nil" do
+        expect(subject).to be_nil
+      end
+    end
+  end
+
+  describe "#order" do
+    context "there is no associated payment with the event" do
+      subject { create(:store_credit_auth_event) }
+
+      it "returns nil" do
+        expect(subject.order).to be_nil
+      end
+    end
+
+    context "there is an associated payment with the event" do
+      let(:authorization_code) { "1-SC-TEST" }
+      let(:order)              { create(:order) }
+      let!(:payment)           { create(:store_credit_payment, order: order, response_code: authorization_code) }
+
+      subject { create(:store_credit_auth_event, action: Spree::StoreCredit::CAPTURE_ACTION, authorization_code: authorization_code) }
+
+      it "returns the order associated with the payment" do
+        expect(subject.order).to eq order
+      end
+    end
+  end
+end