solidus_core 1.0.7 → 1.1.0.beta1

data/spec/models/spree/calculator/shipping/per_item_spec.rb

@@ -1,20 +0,0 @@
-require 'spec_helper'
-
-module Spree
-  module Calculator::Shipping
-    describe PerItem, :type => :model do
-      let(:variant1) { build(:variant) }
-      let(:variant2) { build(:variant) }
-
-      let(:package) do
-        build(:stock_package, variants_contents: { variant1 => 5, variant2 => 3 })
-      end
-
-      subject { PerItem.new(:preferred_amount => 10) }
-
-      it "correctly calculates per item shipping" do
-        expect(subject.compute(package).to_f).to eq(80) # 5 x 10 + 3 x 10
-      end
-    end
-  end
-end

data/spec/models/spree/calculator/shipping/price_sack_spec.rb

@@ -1,30 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Calculator::PriceSack, :type => :model do
-  let(:calculator) do
-    calculator = Spree::Calculator::PriceSack.new
-    calculator.preferred_minimal_amount = 5
-    calculator.preferred_normal_amount = 10
-    calculator.preferred_discount_amount = 1
-    calculator
-  end
-
-  let(:order) { stub_model(Spree::Order) }
-  let(:shipment) { stub_model(Spree::Shipment, :amount => 10) }
-
-  # Regression test for #714 and #739
-  it "computes with an order object" do
-    calculator.compute(order)
-  end
-
-  # Regression test for #1156
-  it "computes with a shipment object" do
-    calculator.compute(shipment)
-  end
-
-  # Regression test for #2055
-  it "computes the correct amount" do
-    expect(calculator.compute(2)).to eq(calculator.preferred_normal_amount)
-    expect(calculator.compute(6)).to eq(calculator.preferred_discount_amount)
-  end
-end

data/spec/models/spree/calculator/tiered_flat_rate_spec.rb

@@ -1,36 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Calculator::TieredFlatRate, :type => :model do
-  let(:calculator) { Spree::Calculator::TieredFlatRate.new }
-
-  describe "#valid?" do
-    subject { calculator.valid? }
-    context "when tiers is a hash" do
-      context "and one of the keys is not a positive number" do
-        before { calculator.preferred_tiers = { "nope" => 20 } }
-        it { is_expected.to be false }
-      end
-    end
-  end
-
-  describe "#compute" do
-    let(:line_item) { mock_model Spree::LineItem, amount: amount }
-    before do
-      calculator.preferred_base_amount = 10
-      calculator.preferred_tiers = {
-        100 => 15,
-        200 => 20
-      }
-    end
-    subject { calculator.compute(line_item) }
-    context "when amount falls within the first tier" do
-      let(:amount) { 50 }
-      it { is_expected.to eq 10 }
-    end
-    context "when amount falls within the second tier" do
-      let(:amount) { 150 }
-      it { is_expected.to eq 15 }
-    end
-  end
-end
-

data/spec/models/spree/calculator/tiered_percent_spec.rb

@@ -1,47 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Calculator::TieredPercent, :type => :model do
-  let(:calculator) { Spree::Calculator::TieredPercent.new }
-
-  describe "#valid?" do
-    subject { calculator.valid? }
-    context "when base percent is less than zero" do
-      before { calculator.preferred_base_percent = -1 }
-      it { is_expected.to be false }
-    end
-    context "when base percent is greater than 100" do
-      before { calculator.preferred_base_percent = 110 }
-      it { is_expected.to be false }
-    end
-    context "when tiers is a hash" do
-      context "and one of the keys is not a positive number" do
-        before { calculator.preferred_tiers = { "nope" => 20 } }
-        it { is_expected.to be false }
-      end
-      context "and one of the values is not a percent" do
-        before { calculator.preferred_tiers = { 10 => 110 } }
-        it { is_expected.to be false }
-      end
-    end
-  end
-
-  describe "#compute" do
-    let(:line_item) { mock_model Spree::LineItem, amount: amount }
-    before do
-      calculator.preferred_base_percent = 10
-      calculator.preferred_tiers = {
-        100 => 15,
-        200 => 20
-      }
-    end
-    subject { calculator.compute(line_item) }
-    context "when amount falls within the first tier" do
-      let(:amount) { 50 }
-      it { is_expected.to eq 5 }
-    end
-    context "when amount falls within the second tier" do
-      let(:amount) { 150 }
-      it { is_expected.to eq 22 }
-    end
-  end
-end

data/spec/models/spree/calculator_spec.rb

@@ -1,36 +0,0 @@
-require 'spec_helper'
-
-
-describe Spree::Calculator, :type => :model do
-
-  class SimpleCalculator < Spree::Calculator
-    def compute_simple_computable line_item
-      'computed'
-    end
-  end
-
-  class SimpleComputable
-  end
-
-
-  context "with computable" do
-
-    let(:calculator) { SimpleCalculator.new }
-    let(:computable) { SimpleComputable.new }
-
-    subject { SimpleCalculator.new.compute computable }
-
-    it 'calls compute method of class type' do
-      expect(subject).to eq ( 'computed' )
-    end
-
-    context 'computable does not implement right function name' do
-      let(:computable) { Spree::LineItem.new }
-
-      it 'raises an error' do
-        expect { subject }.to raise_error NotImplementedError, /Please implement \'compute_line_item\(line_item\)\' in your calculator/
-      end
-    end
-
-  end
-end

data/spec/models/spree/carton_spec.rb

@@ -1,133 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Carton do
-  let(:carton) { create(:carton) }
-
-  describe "#create" do
-    subject { carton }
-
-    it { expect { subject }.to_not raise_error }
-  end
-
-  describe "#tracking_url" do
-    subject do
-      carton.tracking_url
-    end
-
-    let(:carton) { create(:carton, shipping_method: shipping_method) }
-    let(:shipping_method) do
-      create(:shipping_method, tracking_url: "https://example.com/:tracking")
-    end
-
-    context "when tracking is not present" do
-      it { is_expected.to be_nil }
-    end
-
-    context "when tracking is present" do
-      let(:carton) do
-        create(:carton, shipping_method: shipping_method, tracking: "1Z12345")
-      end
-
-      it "uses shipping method to determine url" do
-        is_expected.to eq("https://example.com/1Z12345")
-      end
-    end
-  end
-
-  describe "#to_param" do
-    subject do
-      carton.to_param
-    end
-
-    it { is_expected.to eq carton.number }
-  end
-
-  describe "#order_numbers" do
-    subject { carton.order_numbers }
-    let(:order) { carton.orders.first }
-
-    it "returns a list of the order numbers it is associated to" do
-      expect(subject).to eq [order.number]
-    end
-  end
-
-  describe "#shipment_numbers" do
-    subject { carton.shipment_numbers }
-    let(:shipment) { carton.shipments.first }
-
-    it "returns a list of the order numbers it is associated to" do
-      expect(subject).to eq [shipment.number]
-    end
-  end
-
-  describe "#order_emails" do
-    subject { carton.order_emails }
-
-    let(:carton) { create(:carton, inventory_units: [first_order.inventory_units, second_order.inventory_units].flatten) }
-    let(:first_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:second_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:email) { 'something@something.com' }
-
-    before do
-      first_order.update_attributes!(email: email)
-      second_order.update_attributes!(email: email)
-    end
-
-    it "returns a unique list of the order emails it is associated to" do
-      expect(subject).to eq [email]
-    end
-  end
-
-  describe "#manifest" do
-    subject { carton.manifest }
-
-    let(:carton) { create(:carton, inventory_units: [first_order.inventory_units, second_order.inventory_units].flatten) }
-    let(:first_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:first_line_item) { first_order.line_items.first }
-    let(:second_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:second_line_item) { second_order.line_items.first }
-
-    it "contains only the items in both the carton and order" do
-      expect(subject.map(&:line_item)).to match_array([first_line_item, second_line_item])
-    end
-  end
-
-  describe "#manifest_for_order" do
-    subject { carton.manifest_for_order(first_order) }
-
-    let(:carton) { create(:carton, inventory_units: [first_order.inventory_units, second_order.inventory_units].flatten) }
-    let(:first_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:first_line_item) { first_order.line_items.first }
-    let(:second_order) { create(:order_ready_to_ship, line_items_count: 1) }
-
-    it "contains only the items in both the carton and order" do
-      expect(subject.map(&:line_item)).to eq [first_line_item]
-    end
-  end 
-
-  describe "#any_exchanges?" do
-    subject { carton.any_exchanges? }
-
-    let(:carton) { create(:carton, inventory_units: [first_order.inventory_units, second_order.inventory_units].flatten) }
-    let(:first_order) { create(:order_ready_to_ship, line_items_count: 1) }
-    let(:second_order) { create(:order_ready_to_ship, line_items_count: 1) }
-
-    context "when any of the inventory has an original return item" do
-      let(:return_item) { create(:return_item) }
-      before do
-        first_order.inventory_units.first.original_return_item = return_item
-        first_order.save
-      end
-
-      it "is true" do
-        expect(subject).to be_truthy
-      end
-    end
-
-    context "when none of the inventory has an original return item" do
-      it "is false" do
-        expect(subject).to be_falsey
-      end
-    end
-  end
-end

data/spec/models/spree/classification_spec.rb

@@ -1,15 +0,0 @@
-require 'spec_helper'
-
-module Spree
-  describe Classification, :type => :model do
-    # Regression test for #3494
-    it "cannot link the same taxon to the same product more than once" do
-      product = create(:product)
-      taxon = create(:taxon)
-      add_taxon = lambda { product.taxons << taxon }
-      expect(add_taxon).not_to raise_error
-      expect(add_taxon).to raise_error(ActiveRecord::RecordInvalid)
-    end
-
-  end
-end

data/spec/models/spree/concerns/display_money_spec.rb

@@ -1,43 +0,0 @@
-require 'spec_helper'
-
-module Spree
-
-  describe DisplayMoney do
-    let(:test_class) do
-      Class.new do
-        extend DisplayMoney
-        def total; 10.0; end
-      end
-    end
-
-    describe ".money_methods" do
-      before { test_class.money_methods :total }
-
-      context "currency is not defined" do
-        it "generates a display_method that builds a Spree::Money without options" do
-          expect(test_class.new.display_total).to eq Spree::Money.new(10.0)
-        end
-      end
-
-      context "currency is defined" do
-        before { test_class.class_eval { def currency; "USD"; end } }
-
-        it "generates a display_* method that builds a Spree::Money with currency" do
-          expect(test_class.new.display_total).to eq Spree::Money.new(10.0, currency: "USD")
-        end
-      end
-
-      context "with multiple + options" do
-        before do
-          test_class.class_eval { def amount; 20.0; end }
-          test_class.money_methods :total, amount: { no_cents: true }
-        end
-
-        it "generates a display_* method that builds a Spree::Money with the specified options" do
-          expect(test_class.new.display_total).to eq Spree::Money.new(10.0)
-          expect(test_class.new.display_amount).to eq Spree::Money.new(20.0, no_cents: true)
-        end
-      end
-    end
-  end
-end

data/spec/models/spree/concerns/user_methods_spec.rb

@@ -1,41 +0,0 @@
-require 'spec_helper'
-
-describe Spree::UserMethods do
-  let(:test_user)        { create :user }
-
-  describe '#has_spree_role?' do
-    subject { test_user.has_spree_role? name }
-
-    let(:role) { Spree::Role.create(name: name) }
-    let(:name) { 'test' }
-
-    context 'with a role' do
-      before { test_user.spree_roles << role }
-      it     { is_expected.to be_truthy }
-    end
-
-    context 'without a role' do
-      it { is_expected.to be_falsy }
-    end
-  end
-
-  describe '#last_incomplete_spree_order' do
-    subject { test_user.last_incomplete_spree_order }
-
-    context 'with an incomplete order' do
-      let(:last_incomplete_order) { create :order, user: test_user }
-
-      before do
-        create(:order, user: test_user, created_at: 1.day.ago)
-        create(:order, user: create(:user))
-        last_incomplete_order
-      end
-
-      it { is_expected.to eq last_incomplete_order }
-    end
-
-    context 'without an incomplete order' do
-      it { is_expected.to be_nil }
-    end
-  end
-end

data/spec/models/spree/credit_card_spec.rb

@@ -1,334 +0,0 @@
-require 'spec_helper'
-
-describe Spree::CreditCard, type: :model do
-  let(:valid_credit_card_attributes) do
-    {
-      number: '4111111111111111',
-      verification_value: '123',
-      expiry: "12 / #{(Time.now.year + 1).to_s.last(2)}",
-      name: 'Spree Commerce'
-    }
-  end
-
-  def self.payment_states
-    Spree::Payment.state_machine.states.keys
-  end
-
-  def stub_rails_env(environment)
-    allow(Rails).to receive_messages(env: ActiveSupport::StringInquirer.new(environment))
-  end
-
-  let(:credit_card) { Spree::CreditCard.new }
-
-  before(:each) do
-
-    @order = create(:order)
-    @payment = Spree::Payment.create(:amount => 100, :order => @order)
-
-    @success_response = double('gateway_response', success?: true, authorization: '123', avs_result: { 'code' => 'avs-code' })
-    @fail_response = double('gateway_response', success?: false)
-
-    @payment_gateway = mock_model(Spree::PaymentMethod,
-      payment_profiles_supported?: true,
-      authorize: @success_response,
-      purchase: @success_response,
-      capture: @success_response,
-      void: @success_response,
-      credit: @success_response,
-      environment: 'test'
-    )
-
-    allow(@payment).to receive_messages payment_method: @payment_gateway
-  end
-
-  context "#can_capture?" do
-    it "should be true if payment is pending" do
-      payment = mock_model(Spree::Payment, pending?: true, created_at: Time.now)
-      expect(credit_card.can_capture?(payment)).to be true
-    end
-
-    it "should be true if payment is checkout" do
-      payment = mock_model(Spree::Payment, pending?: false, checkout?: true, created_at: Time.now)
-      expect(credit_card.can_capture?(payment)).to be true
-    end
-  end
-
-  context "#can_void?" do
-    it "should be true if payment is not void" do
-      payment = mock_model(Spree::Payment, failed?: false, void?: false)
-      expect(credit_card.can_void?(payment)).to be true
-    end
-  end
-
-  context "#can_credit?" do
-    it "should be false if payment is not completed" do
-      payment = mock_model(Spree::Payment, completed?: false)
-      expect(credit_card.can_credit?(payment)).to be false
-    end
-
-    it "should be false when credit_allowed is zero" do
-      payment = mock_model(Spree::Payment, completed?: true, credit_allowed: 0, order: mock_model(Spree::Order, payment_state: 'credit_owed'))
-      expect(credit_card.can_credit?(payment)).to be false
-    end
-  end
-
-  context "#valid?" do
-    it "should validate presence of number" do
-      credit_card.attributes = valid_credit_card_attributes.except(:number)
-      expect(credit_card).not_to be_valid
-      expect(credit_card.errors[:number]).to eq(["can't be blank"])
-    end
-
-    it "should validate presence of security code" do
-      credit_card.attributes = valid_credit_card_attributes.except(:verification_value)
-      expect(credit_card).not_to be_valid
-      expect(credit_card.errors[:verification_value]).to eq(["can't be blank"])
-    end
-
-    it "validates name presence" do
-      credit_card.valid?
-      expect(credit_card.error_on(:name).size).to eq(1)
-    end
-
-    it "should only validate on create" do
-      credit_card.attributes = valid_credit_card_attributes
-      credit_card.save
-      expect(credit_card).to be_valid
-    end
-
-    context "encrypted data is present" do
-      it "does not validate presence of number or cvv" do
-        credit_card.encrypted_data = "$fdgsfgdgfgfdg&gfdgfdgsf-"
-        credit_card.valid?
-        expect(credit_card.errors[:number]).to be_empty
-        expect(credit_card.errors[:verification_value]).to be_empty
-      end
-    end
-
-    context "imported is true" do
-      it "does not validate presence of number or cvv" do
-        credit_card.imported = true
-        credit_card.valid?
-        expect(credit_card.errors[:number]).to be_empty
-        expect(credit_card.errors[:verification_value]).to be_empty
-      end
-    end
-  end
-
-  context "#save" do
-    before do
-      credit_card.attributes = valid_credit_card_attributes
-      credit_card.save!
-    end
-
-    let!(:persisted_card) { Spree::CreditCard.find(credit_card.id) }
-
-    it "should not actually store the number" do
-      expect(persisted_card.number).to be_blank
-    end
-
-    it "should not actually store the security code" do
-      expect(persisted_card.verification_value).to be_blank
-    end
-  end
-
-  context "#number=" do
-    it "should strip non-numeric characters from card input" do
-      credit_card.number = "6011000990139424"
-      expect(credit_card.number).to eq("6011000990139424")
-
-      credit_card.number = "  6011-0009-9013-9424  "
-      expect(credit_card.number).to eq("6011000990139424")
-    end
-
-    it "should not raise an exception on non-string input" do
-      credit_card.number = Hash.new
-      expect(credit_card.number).to be_nil
-    end
-  end
-
-  # Regression test for #3847 & #3896
-  context "#expiry=" do
-    it "can set with a 2-digit month and year" do
-      credit_card.expiry = '04 / 15'
-      expect(credit_card.month).to eq('4')
-      expect(credit_card.year).to eq('2015')
-    end
-
-    it "can set with a 2-digit month and 4-digit year" do
-      credit_card.expiry = '04 / 2015'
-      expect(credit_card.month).to eq('4')
-      expect(credit_card.year).to eq('2015')
-    end
-
-    it "can set with a 2-digit month and 4-digit year without whitespace" do
-      credit_card.expiry = '04/15'
-      expect(credit_card.month).to eq('4')
-      expect(credit_card.year).to eq('2015')
-    end
-
-    it "can set with a 2-digit month and 4-digit year without whitespace" do
-      credit_card.expiry = '04/2015'
-      expect(credit_card.month).to eq('4')
-      expect(credit_card.year).to eq('2015')
-    end
-
-    it "can set with a 2-digit month and 4-digit year without whitespace and slash" do
-      credit_card.expiry = '042015'
-      expect(credit_card.month).to eq('4')
-      expect(credit_card.year).to eq('2015')
-    end
-
-    it "can set with a 2-digit month and 2-digit year without whitespace and slash" do
-      credit_card.expiry = '0415'
-      expect(credit_card.month).to eq('4')
-      expect(credit_card.year).to eq('2015')
-    end
-
-    it "does not blow up when passed an empty string" do
-      expect { credit_card.expiry = '' }.not_to raise_error
-    end
-
-    # Regression test for #4725
-    it "does not blow up when passed one number" do
-      expect { credit_card.expiry = '12' }.not_to raise_error
-    end
-
-  end
-
-  context "#cc_type=" do
-    it "converts between the different types" do
-      credit_card.cc_type = 'mastercard'
-      expect(credit_card.cc_type).to eq('master')
-
-      credit_card.cc_type = 'maestro'
-      expect(credit_card.cc_type).to eq('master')
-
-      credit_card.cc_type = 'amex'
-      expect(credit_card.cc_type).to eq('american_express')
-
-      credit_card.cc_type = 'dinersclub'
-      expect(credit_card.cc_type).to eq('diners_club')
-
-      credit_card.cc_type = 'some_outlandish_cc_type'
-      expect(credit_card.cc_type).to eq('some_outlandish_cc_type')
-    end
-
-    it "assigns the type based on card number in the event of js failure" do
-      credit_card.number = '4242424242424242'
-      credit_card.cc_type = ''
-      expect(credit_card.cc_type).to eq('visa')
-
-      credit_card.number = '5555555555554444'
-      credit_card.cc_type = ''
-      expect(credit_card.cc_type).to eq('master')
-
-      credit_card.number = '378282246310005'
-      credit_card.cc_type = ''
-      expect(credit_card.cc_type).to eq('american_express')
-
-      credit_card.number = '30569309025904'
-      credit_card.cc_type = ''
-      expect(credit_card.cc_type).to eq('diners_club')
-
-      credit_card.number = '3530111333300000'
-      credit_card.cc_type = ''
-      expect(credit_card.cc_type).to eq('jcb')
-
-      credit_card.number = ''
-      credit_card.cc_type = ''
-      expect(credit_card.cc_type).to eq('')
-
-      credit_card.number = nil
-      credit_card.cc_type = ''
-      expect(credit_card.cc_type).to eq('')
-    end
-  end
-
-  context "#associations" do
-    it "should be able to access its payments" do
-      expect { credit_card.payments.to_a }.not_to raise_error
-    end
-  end
-
-  context "#first_name" do
-    before do
-      credit_card.name = "Ludwig van Beethoven"
-    end
-
-    it "extracts the first name" do
-      expect(credit_card.first_name).to eq "Ludwig"
-    end
-  end
-
-  context "#last_name" do
-    before do
-      credit_card.name = "Ludwig van Beethoven"
-    end
-
-    it "extracts the last name" do
-      expect(credit_card.last_name).to eq "van Beethoven"
-    end
-  end
-
-  context "#to_active_merchant" do
-    before do
-      credit_card.number = "4111111111111111"
-      credit_card.year = Time.now.year
-      credit_card.month = Time.now.month
-      credit_card.name = "Ludwig van Beethoven"
-      credit_card.verification_value = 123
-    end
-
-    it "converts to an ActiveMerchant::Billing::CreditCard object" do
-      am_card = credit_card.to_active_merchant
-      expect(am_card.number).to eq("4111111111111111")
-      expect(am_card.year).to eq(Time.now.year)
-      expect(am_card.month).to eq(Time.now.month)
-      expect(am_card.first_name).to eq("Ludwig")
-      expect(am_card.last_name).to eq("van Beethoven")
-      expect(am_card.verification_value).to eq(123)
-    end
-  end
-
-  it 'ensures only one credit card per user is default at a time' do
-    user = FactoryGirl.create(:user)
-    first = FactoryGirl.create(:credit_card, user: user, default: true)
-    second = FactoryGirl.create(:credit_card, user: user, default: true)
-
-    expect(first.reload.default).to eq false
-    expect(second.reload.default).to eq true
-
-    first.default = true
-    first.save!
-
-    expect(first.reload.default).to eq true
-    expect(second.reload.default).to eq false
-  end
-
-  it 'allows default credit cards for different users' do
-    first = FactoryGirl.create(:credit_card, user: FactoryGirl.create(:user), default: true)
-    second = FactoryGirl.create(:credit_card, user: FactoryGirl.create(:user), default: true)
-
-    expect(first.reload.default).to eq true
-    expect(second.reload.default).to eq true
-  end
-
-  it 'allows this card to save even if the previously default card has expired' do
-    user = FactoryGirl.create(:user)
-    first = FactoryGirl.create(:credit_card, user: user, default: true)
-    second = FactoryGirl.create(:credit_card, user: user, default: false)
-    first.update_columns(year: DateTime.now.year, month: 1.month.ago.month)
-
-    expect { second.update_attributes!(default: true) }.not_to raise_error
-  end
-
-  it 'allows this card to save even if the previously default card has expired' do
-    user = FactoryGirl.create(:user)
-    first = FactoryGirl.create(:credit_card, user: user, default: true)
-    second = FactoryGirl.create(:credit_card, user: user, default: false)
-    first.update_columns(year: DateTime.now.year, month: 1.month.ago.month)
-
-    expect { second.update_attributes!(default: true) }.not_to raise_error
-  end
-end