solidus_core 1.0.7 → 1.1.0.beta1

data/spec/models/spree/order/tax_spec.rb

@@ -1,84 +0,0 @@
-require 'spec_helper'
-
-module Spree
-  describe Spree::Order, :type => :model do
-    let(:order) { stub_model(Spree::Order) }
-
-    context "#tax_zone" do
-      let(:bill_address) { create :address }
-      let(:ship_address) { create :address }
-      let(:order) { Spree::Order.create(:ship_address => ship_address, :bill_address => bill_address) }
-      let(:zone) { create :zone }
-
-      context "when no zones exist" do
-        it "should return nil" do
-          expect(order.tax_zone).to be_nil
-        end
-      end
-
-      context "when :tax_using_ship_address => true" do
-        before { Spree::Config.set(:tax_using_ship_address => true) }
-
-        it "should calculate using ship_address" do
-          expect(Spree::Zone).to receive(:match).at_least(:once).with(ship_address)
-          expect(Spree::Zone).not_to receive(:match).with(bill_address)
-          order.tax_zone
-        end
-      end
-
-      context "when :tax_using_ship_address => false" do
-        before { Spree::Config.set(:tax_using_ship_address => false) }
-
-        it "should calculate using bill_address" do
-          expect(Spree::Zone).to receive(:match).at_least(:once).with(bill_address)
-          expect(Spree::Zone).not_to receive(:match).with(ship_address)
-          order.tax_zone
-        end
-      end
-
-      context "when there is a default tax zone" do
-        before do
-          @default_zone = create(:zone, :name => "foo_zone")
-          allow(Spree::Zone).to receive_messages :default_tax => @default_zone
-        end
-
-        context "when there is a matching zone" do
-          before { allow(Spree::Zone).to receive_messages(:match => zone) }
-
-          it "should return the matching zone" do
-            expect(order.tax_zone).to eq(zone)
-          end
-        end
-
-        context "when there is no matching zone" do
-          before { allow(Spree::Zone).to receive_messages(:match => nil) }
-
-          it "should return the default tax zone" do
-            expect(order.tax_zone).to eq(@default_zone)
-          end
-        end
-      end
-
-      context "when no default tax zone" do
-        before { allow(Spree::Zone).to receive_messages :default_tax => nil }
-
-        context "when there is a matching zone" do
-          before { allow(Spree::Zone).to receive_messages(:match => zone) }
-
-          it "should return the matching zone" do
-            expect(order.tax_zone).to eq(zone)
-          end
-        end
-
-        context "when there is no matching zone" do
-          before { allow(Spree::Zone).to receive_messages(:match => nil) }
-
-          it "should return nil" do
-            expect(order.tax_zone).to be_nil
-          end
-        end
-      end
-    end
-
-  end
-end

data/spec/models/spree/order/totals_spec.rb

@@ -1,24 +0,0 @@
-require 'spec_helper'
-
-module Spree
-  describe Order, :type => :model do
-    let(:order) { Order.create }
-    let(:shirt) { create(:variant) }
-
-    context "adds item to cart and activates promo" do
-      let(:promotion) { Promotion.create name: 'Huhu' }
-      let(:calculator) { Calculator::FlatPercentItemTotal.new(:preferred_flat_percent => 10) }
-      let!(:action) { Promotion::Actions::CreateAdjustment.create(promotion: promotion, calculator: calculator) }
-
-      before { order.contents.add(shirt, 1) }
-
-      context "item quantity changes" do
-        it "recalculates order adjustments" do
-          expect {
-            order.contents.add(shirt, 3)
-          }.to change { order.adjustments.eligible.pluck(:amount) }
-        end
-      end
-    end
-  end
-end

data/spec/models/spree/order/updating_spec.rb

@@ -1,18 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Order, :type => :model do
-  let(:order) { stub_model(Spree::Order) }
-
-  context "#update!" do
-    let(:line_items) { [mock_model(Spree::LineItem, :amount => 5) ]}
-
-    context "when there are update hooks" do
-      before { Spree::Order.register_update_hook :foo }
-      after { Spree::Order.update_hooks.clear }
-      it "should call each of the update hooks" do
-        expect(order).to receive :foo
-        order.update!
-      end
-    end
-  end
-end

data/spec/models/spree/order/validations_spec.rb

@@ -1,15 +0,0 @@
-require 'spec_helper'
-
-module Spree
-  describe Spree::Order, :type => :model do
-    context "validations" do
-      # Regression test for #2214
-      it "does not return two error messages when email is blank" do
-        order = Spree::Order.new
-        allow(order).to receive_messages(:require_email => true)
-        order.valid?
-        expect(order.errors[:email]).to eq(["can't be blank"])
-      end
-    end
-  end
-end

data/spec/models/spree/order_cancellations_spec.rb

@@ -1,120 +0,0 @@
-require 'spec_helper'
-
-describe Spree::OrderCancellations do
-  describe "#short_ship" do
-    subject { Spree::OrderCancellations.new(order).short_ship([inventory_unit]) }
-
-    let(:order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:inventory_unit) { order.inventory_units.first }
-    let(:shipment) { inventory_unit.shipment }
-
-    it "creates a UnitCancel record" do
-      expect { subject }.to change { Spree::UnitCancel.count }.by(1)
-
-      unit_cancel = Spree::UnitCancel.last
-      expect(unit_cancel.inventory_unit).to eq inventory_unit
-      expect(unit_cancel.reason).to eq Spree::UnitCancel::SHORT_SHIP
-    end
-
-    it "cancels the inventory unit" do
-      expect { subject }.to change { inventory_unit.state }.to "canceled"
-    end
-
-    it "updates the shipment.state" do
-      expect { subject }.to change { shipment.reload.state }.from('ready').to('shipped')
-    end
-
-    it "updates the order.shipment_state" do
-      expect { subject }.to change { order.shipment_state }.from('ready').to('shipped')
-    end
-
-    it "adjusts the order" do
-      expect { subject }.to change { order.total }.by(-10.0)
-    end
-
-    it "sends a cancellation email" do
-      mail_double = double
-      expect(Spree::OrderMailer).to receive(:inventory_cancellation_email).with(order, [inventory_unit]).and_return(mail_double)
-      expect(mail_double).to receive(:deliver_later)
-      subject
-    end
-
-    context "when send_cancellation_mailer is false" do
-      subject { Spree::OrderCancellations.new(order).short_ship([inventory_unit]) }
-
-      before do
-        @original_send_boolean = Spree::OrderCancellations.send_cancellation_mailer
-        Spree::OrderCancellations.send_cancellation_mailer = false
-      end
-
-      after { Spree::OrderCancellations.send_cancellation_mailer = @original_send_boolean }
-
-      it "does not send a cancellation email" do
-        expect(Spree::OrderMailer).not_to receive(:inventory_cancellation_email)
-        subject
-      end
-    end
-
-    context "with a who" do
-      subject { order.cancellations.short_ship([inventory_unit], whodunnit: 'some automated system') }
-
-      let(:user) { order.user }
-
-      it "sets the user on the UnitCancel" do
-        expect { subject }.to change { Spree::UnitCancel.count }.by(1)
-        expect(Spree::UnitCancel.last.created_by).to eq("some automated system")
-      end
-    end
-
-    context "when rounding is required" do
-      let(:order) { create(:order_ready_to_ship, line_items_count: 1, line_items_price: 0.83) }
-      let(:line_item) { order.line_items.first }
-      let(:inventory_unit_1) { line_item.inventory_units[0] }
-      let(:inventory_unit_2) { line_item.inventory_units[1] }
-
-      before do
-        order.contents.add(line_item.variant)
-        line_item.reload
-
-        # make the total $1.67 so it divides unevenly
-        Spree::Adjustment.tax.create!(
-          order: order,
-          adjustable: line_item,
-          amount: 0.01,
-          label: 'some fake tax',
-          state: 'closed',
-        )
-        order.update!
-      end
-
-      it "generates the correct total amount" do
-        order.cancellations.short_ship([inventory_unit_1])
-        order.cancellations.short_ship([inventory_unit_2])
-        expect(line_item.adjustments.non_tax.sum(:amount)).to eq -1.67
-        expect(line_item.total).to eq 0
-      end
-    end
-
-    describe 'short_ship_tax_notifier' do
-      context 'when present' do
-        let(:short_ship_tax_notifier) { double }
-
-        before do
-          @old_notifier = Spree::OrderCancellations.short_ship_tax_notifier
-          Spree::OrderCancellations.short_ship_tax_notifier = short_ship_tax_notifier
-        end
-        after do
-          Spree::OrderCancellations.short_ship_tax_notifier = @old_notifier
-        end
-
-        it 'calls the short_ship_tax_notifier' do
-          expect(short_ship_tax_notifier).to receive(:call) do |unit_cancels|
-            expect(unit_cancels.map(&:inventory_unit)).to match_array([inventory_unit])
-          end
-
-          order.cancellations.short_ship([inventory_unit])
-        end
-      end
-    end
-  end
-end

data/spec/models/spree/order_capturing_spec.rb

@@ -1,116 +0,0 @@
-require 'spec_helper'
-
-describe Spree::OrderCapturing do
-  describe '#capture_payments' do
-    subject { Spree::OrderCapturing.new(order, payment_methods).capture_payments }
-
-    context "payment methods specified" do
-      let!(:order) { create(:order, ship_address: create(:address)) }
-
-      let!(:product) { create(:product, price: 10.00) }
-      let!(:variant) do
-        create(:variant, price: 10, product: product, track_inventory: false, tax_category: tax_rate.tax_category)
-      end
-      let!(:shipping_method) { create(:free_shipping_method) }
-      let(:tax_rate) { create(:tax_rate, amount: 0.1, zone: create(:global_zone, name: "Some Tax Zone")) }
-      let(:secondary_total) { 10.0 }
-      let(:bogus_total) { order.total }
-
-      before do
-        order.contents.add(variant, 3)
-        order.update!
-        @secondary_bogus_payment = create(:payment, order: order, amount: secondary_total, payment_method: secondary_payment_method.create!(name: 'So bogus', environment: 'test'))
-        @bogus_payment = create(:payment, order: order, amount: bogus_total)
-        order.contents.advance
-        order.complete!
-        order.reload
-      end
-
-      context "payment method ordering" do
-        let(:secondary_payment_method) { SecondaryBogusPaymentMethod }
-
-        class SecondaryBogusPaymentMethod < Spree::Gateway::Bogus; end
-
-        context "SecondaryBogusPaymentMethod payments are prioritized" do
-          let(:payment_methods) { [SecondaryBogusPaymentMethod, Spree::Gateway::Bogus] }
-
-          it "captures SecondaryBogusPaymentMethod payments first" do
-            subject
-            expect(@secondary_bogus_payment.reload.capture_events.sum(:amount)).to eq(10.0)
-            expect(@bogus_payment.reload.capture_events.sum(:amount)).to eq(order.total - 10.0)
-          end
-        end
-
-        context "Bogus payments are prioritized" do
-          let(:payment_methods) { [Spree::Gateway::Bogus, SecondaryBogusPaymentMethod] }
-
-          it "captures Bogus payments first" do
-            subject
-            expect(@secondary_bogus_payment.reload.capture_events.sum(:amount)).to eq(0.0)
-            expect(@bogus_payment.reload.capture_events.sum(:amount)).to eq(order.total)
-          end
-        end
-
-        context "when the payment method ordering is configured" do
-          subject { Spree::OrderCapturing.new(order, payment_methods).capture_payments }
-
-          let(:payment_methods) { nil }
-
-          before do
-            allow(Spree::OrderCapturing).to receive(:sorted_payment_method_classes).and_return(
-              [SecondaryBogusPaymentMethod, Spree::Gateway::Bogus]
-            )
-          end
-
-          it "captures in the order specified" do
-            subject
-            expect(@secondary_bogus_payment.reload.capture_events.sum(:amount)).to eq(10.0)
-            expect(@bogus_payment.reload.capture_events.sum(:amount)).to eq(order.total - 10.0)
-          end
-        end
-      end
-
-      context "when a payment is not needed to capture the entire order" do
-        let(:bogus_total) { order.total }
-        let(:secondary_payment_method) { SecondaryBogusPaymentMethod }
-        let(:payment_methods) { [Spree::Gateway::Bogus, SecondaryBogusPaymentMethod] }
-
-        context "when void_unused_payments is true" do
-          before { allow(Spree::OrderCapturing).to receive(:void_unused_payments).and_return(true) }
-
-          it "captures for the order and voids the unused payment" do
-            subject
-            expect(order.reload.payment_state).to eq 'paid'
-            expect(@secondary_bogus_payment.reload.state).to eq 'void'
-          end
-        end
-
-        context "when void_unused_payments is false" do
-          it "captures for the order and leaves the unused payment in a pending state" do
-            subject
-            expect(order.reload.payment_state).to eq 'paid'
-            expect(@secondary_bogus_payment.reload.state).to eq 'pending'
-          end
-        end
-      end
-
-      context "when there is an error processing a payment" do
-        let(:secondary_payment_method) { ExceptionallyBogusPaymentMethod }
-        let(:bogus_total) { order.total - 1 }
-        let(:secondary_total) { 1 }
-        let(:payment_methods) { [Spree::Gateway::Bogus, ExceptionallyBogusPaymentMethod] }
-
-        class ExceptionallyBogusPaymentMethod < Spree::Gateway::Bogus
-          def capture(*args)
-            raise ActiveMerchant::ConnectionError.new("foo", nil)
-          end
-        end
-
-        it "raises an error and leaves the order in a reasonable state" do
-          expect { subject }.to raise_error(Spree::Core::GatewayError)
-          expect(order.payments.to_a.sum(&:uncaptured_amount)).to eq 1.0
-        end
-      end
-    end
-  end
-end

data/spec/models/spree/order_contents_spec.rb

@@ -1,265 +0,0 @@
-require 'spec_helper'
-
-describe Spree::OrderContents, :type => :model do
-  let(:order) { Spree::Order.create }
-  let(:variant) { create(:variant) }
-  let!(:stock_location) { variant.stock_locations.first }
-  let(:stock_location_2) { create(:stock_location) }
-
-  subject { described_class.new(order) }
-
-  context "#add" do
-    context 'given quantity is not explicitly provided' do
-      it 'should add one line item' do
-        line_item = subject.add(variant)
-        expect(line_item.quantity).to eq(1)
-        expect(order.line_items.size).to eq(1)
-      end
-    end
-
-    context 'given a shipment' do
-      it "ensure shipment calls update_amounts instead of order calling ensure_updated_shipments" do
-        shipment = create(:shipment)
-        expect(subject.order).to_not receive(:ensure_updated_shipments)
-        expect(shipment).to receive(:update_amounts)
-        subject.add(variant, 1, shipment: shipment)
-      end
-    end
-
-    context 'not given a shipment' do
-      it "ensures updated shipments" do
-        expect(subject.order).to receive(:ensure_updated_shipments)
-        subject.add(variant)
-      end
-    end
-
-    it 'should add line item if one does not exist' do
-      line_item = subject.add(variant, 1)
-      expect(line_item.quantity).to eq(1)
-      expect(order.line_items.size).to eq(1)
-    end
-
-    it 'should update line item if one exists' do
-      subject.add(variant, 1)
-      line_item = subject.add(variant, 1)
-      expect(line_item.quantity).to eq(2)
-      expect(order.line_items.size).to eq(1)
-    end
-
-    it "should update order totals" do
-      expect(order.item_total.to_f).to eq(0.00)
-      expect(order.total.to_f).to eq(0.00)
-
-      subject.add(variant, 1)
-
-      expect(order.item_total.to_f).to eq(19.99)
-      expect(order.total.to_f).to eq(19.99)
-    end
-
-    it "should create stock location associations if provided" do
-      line_item = subject.add(variant, 3, stock_location_quantities: {stock_location.id => 1, stock_location_2.id => 2})
-      order_stock_locations = line_item.order.order_stock_locations
-      expect(order_stock_locations.count).to eq(2)
-      expect(order_stock_locations.map(&:quantity)).to eq([1, 2])
-      expect(order_stock_locations.map(&:stock_location_id)).to eq([stock_location.id, stock_location_2.id])
-    end
-
-    context "running promotions" do
-      let(:promotion) { create(:promotion) }
-      let(:calculator) { Spree::Calculator::FlatRate.new(:preferred_amount => 10) }
-
-      shared_context "discount changes order total" do
-        before { subject.add(variant, 1) }
-        it { expect(subject.order.total).not_to eq variant.price }
-      end
-
-      context "one active order promotion" do
-        let!(:action) { Spree::Promotion::Actions::CreateAdjustment.create(promotion: promotion, calculator: calculator) }
-
-        it "creates valid discount on order" do
-          subject.add(variant, 1)
-          expect(subject.order.adjustments.to_a.sum(&:amount)).not_to eq 0
-        end
-
-        include_context "discount changes order total"
-      end
-
-      context "one active line item promotion" do
-        let!(:action) { Spree::Promotion::Actions::CreateItemAdjustments.create(promotion: promotion, calculator: calculator) }
-
-        it "creates valid discount on order" do
-          subject.add(variant, 1)
-          expect(subject.order.line_item_adjustments.to_a.sum(&:amount)).not_to eq 0
-        end
-
-        include_context "discount changes order total"
-      end
-    end
-  end
-
-  context "#remove" do
-    context "given an invalid variant" do
-      it "raises an exception" do
-        expect {
-          subject.remove(variant, 1)
-        }.to raise_error(ActiveRecord::RecordNotFound)
-      end
-    end
-
-    context 'given quantity is not explicitly provided' do
-      it 'should remove one line item' do
-        line_item = subject.add(variant, 3)
-        subject.remove(variant)
-
-        expect(line_item.reload.quantity).to eq(2)
-      end
-    end
-
-    context 'given a shipment' do
-      it "ensure shipment calls update_amounts instead of order calling ensure_updated_shipments" do
-        line_item = subject.add(variant, 1)
-        shipment = create(:shipment)
-        expect(subject.order).to_not receive(:ensure_updated_shipments)
-        expect(shipment).to receive(:update_amounts)
-        subject.remove(variant, 1, shipment: shipment)
-      end
-    end
-
-    context 'not given a shipment' do
-      it "ensures updated shipments" do
-        line_item = subject.add(variant, 1)
-        expect(subject.order).to receive(:ensure_updated_shipments)
-        subject.remove(variant)
-      end
-    end
-
-    it 'should reduce line_item quantity if quantity is less the line_item quantity' do
-      line_item = subject.add(variant, 3)
-      subject.remove(variant, 1)
-
-      expect(line_item.reload.quantity).to eq(2)
-    end
-
-    it 'should remove line_item if quantity matches line_item quantity' do
-      subject.add(variant, 1)
-      subject.remove(variant, 1)
-
-      expect(order.reload.find_line_item_by_variant(variant)).to be_nil
-    end
-
-    it "should update order totals" do
-      expect(order.item_total.to_f).to eq(0.00)
-      expect(order.total.to_f).to eq(0.00)
-
-      subject.add(variant,2)
-
-      expect(order.item_total.to_f).to eq(39.98)
-      expect(order.total.to_f).to eq(39.98)
-
-      subject.remove(variant,1)
-      expect(order.item_total.to_f).to eq(19.99)
-      expect(order.total.to_f).to eq(19.99)
-    end
-  end
-
-  context "update cart" do
-    let!(:shirt) { subject.add variant, 1 }
-
-    let(:params) do
-      { line_items_attributes: {
-        "0" => { id: shirt.id, quantity: 3 }
-      } }
-    end
-
-    it "changes item quantity" do
-      subject.update_cart params
-      expect(shirt.reload.quantity).to eq 3
-    end
-
-    it "updates order totals" do
-      expect {
-        subject.update_cart params
-      }.to change { subject.order.total }
-    end
-
-    context "submits item quantity 0" do
-      let(:params) do
-        { line_items_attributes: {
-          "0" => { id: shirt.id, quantity: 0 },
-        } }
-      end
-
-      it "removes item from order" do
-        expect {
-          subject.update_cart params
-        }.to change { subject.order.line_items.count }
-      end
-    end
-
-    it "ensures updated shipments" do
-      expect(subject.order).to receive(:ensure_updated_shipments)
-      subject.update_cart params
-    end
-  end
-
-  context "completed order" do
-    let(:order) { Spree::Order.create! state: 'complete', completed_at: Time.now }
-
-    before { order.shipments.create! stock_location_id: variant.stock_location_ids.first }
-
-    it "updates order payment state" do
-      expect {
-        subject.add variant
-      }.to change { order.payment_state }
-
-      expect {
-        subject.remove variant
-      }.to change { order.payment_state }
-    end
-  end
-
-  describe "#approve" do
-    context 'when a name is supplied' do
-      it 'approves the order' do
-        order.contents.approve(name: 'Jordan')
-        expect(order.approver).to be_nil
-        expect(order.approver_name).to eq('Jordan')
-        expect(order.approved_at).to be_present
-        expect(order.approved?).to be_truthy
-      end
-    end
-
-    context 'when a user is supplied' do
-      let(:user) { create(:user) }
-
-      it 'approves the order' do
-        order.contents.approve(user: user)
-        expect(order.approver).to eq(user)
-        expect(order.approver_name).to be_nil
-        expect(order.approved_at).to be_present
-        expect(order.approved?).to be_truthy
-      end
-    end
-
-    context 'when a user and a name are supplied' do
-      let(:user) { create(:user) }
-
-      it 'approves the order' do
-        order.contents.approve(user: user, name: 'Jordan')
-        expect(order.approver).to eq(user)
-        expect(order.approver_name).to eq('Jordan')
-        expect(order.approved_at).to be_present
-        expect(order.approved?).to be_truthy
-      end
-    end
-
-    context 'when neither a user nor a name are supplied' do
-      it 'raises' do
-        expect {
-          order.contents.approve
-        }.to raise_error(ArgumentError, 'user or name must be specified')
-      end
-    end
-  end
-
-end