spree_drop_ship 3.1.0.1

data/spec/features/admin/suppliers_spec.rb

@@ -0,0 +1,126 @@
+require 'spec_helper'
+
+feature 'Admin - Suppliers', js: true do
+
+  before do
+    country = create(:country, name: "United States")
+    create(:state, name: "Vermont", country: country)
+    @supplier = create :supplier
+  end
+
+  context 'as an Admin' do
+
+    before do
+      login_user create(:admin_user)
+      visit spree.admin_path
+      within '[data-hook=admin_tabs]' do
+        click_link 'Suppliers'
+      end
+      page.should have_content('Listing Suppliers')
+    end
+
+    scenario 'should be able to create new supplier' do
+      click_link 'New Supplier'
+      check 'supplier_active'
+      fill_in 'supplier[name]', with: 'Test Supplier'
+      fill_in 'supplier[email]', with: 'spree@example.com'
+      fill_in 'supplier[url]', with: 'http://www.test.com'
+      fill_in 'supplier[commission_flat_rate]', with: '0'
+      fill_in 'supplier[commission_percentage]', with: '0'
+      fill_in 'supplier[address_attributes][firstname]', with: 'First'
+      fill_in 'supplier[address_attributes][lastname]', with: 'Last'
+      fill_in 'supplier[address_attributes][address1]', with: '1 Test Drive'
+      fill_in 'supplier[address_attributes][city]', with: 'Test City'
+      fill_in 'supplier[address_attributes][zipcode]', with: '55555'
+      select2 'United States', from: 'Country'
+      select2 'Vermont', from: 'State'
+      fill_in 'supplier[address_attributes][phone]', with: '555-555-5555'
+      click_button 'Create'
+      page.should have_content('Supplier "Test Supplier" has been successfully created!')
+    end
+
+    scenario 'should be able to delete supplier' do
+      click_icon 'delete'
+      page.driver.browser.switch_to.alert.accept
+      within 'table' do
+        page.should_not have_content(@supplier.name)
+      end
+    end
+
+    scenario 'should be able to edit supplier' do
+      click_icon 'edit'
+      check 'supplier_active'
+      fill_in 'supplier[name]', with: 'Test Supplier'
+      fill_in 'supplier[email]', with: 'spree@example.com'
+      fill_in 'supplier[url]', with: 'http://www.test.com'
+      fill_in 'supplier[commission_flat_rate]', with: '0'
+      fill_in 'supplier[commission_percentage]', with: '0'
+      fill_in 'supplier[address_attributes][firstname]', with: 'First'
+      fill_in 'supplier[address_attributes][lastname]', with: 'Last'
+      fill_in 'supplier[address_attributes][address1]', with: '1 Test Drive'
+      fill_in 'supplier[address_attributes][city]', with: 'Test City'
+      fill_in 'supplier[address_attributes][zipcode]', with: '55555'
+      select2 'United States', from: 'Country'
+      select2 'Vermont', from: 'State'
+      fill_in 'supplier[address_attributes][phone]', with: '555-555-5555'
+      click_button 'Update'
+      page.should have_content('Supplier "Test Supplier" has been successfully updated!')
+    end
+
+  end
+
+  context 'as a Supplier' do
+    before do
+      @user = create(:supplier_user)
+      login_user @user
+      visit spree.edit_admin_supplier_path(@user.supplier)
+    end
+
+    scenario 'should only see tabs they have access to' do
+      within '[data-hook=admin_tabs]' do
+        page.should_not have_link('Overview')
+        page.should have_link('Products')
+        page.should_not have_link('Reports')
+        page.should_not have_link('Configuration')
+        page.should_not have_link('Promotions')
+        page.should_not have_link('Suppliers')
+        page.should have_link('Shipments')
+      end
+    end
+
+    scenario 'should be able to update supplier' do
+      fill_in 'supplier[name]', with: 'Test Supplier'
+      fill_in 'supplier[email]', with: @user.email
+      fill_in 'supplier[url]', with: 'http://www.test.com'
+      fill_in 'supplier[address_attributes][firstname]', with: 'First'
+      fill_in 'supplier[address_attributes][lastname]', with: 'Last'
+      fill_in 'supplier[address_attributes][address1]', with: '1 Test Drive'
+      fill_in 'supplier[address_attributes][city]', with: 'Test City'
+      fill_in 'supplier[address_attributes][zipcode]', with: '55555'
+      select2 'United States', from: 'Country'
+      select2 'Vermont', from: 'State'
+      fill_in 'supplier[address_attributes][phone]', with: '555-555-5555'
+      page.should_not have_css('#supplier_active') # cannot edit active
+      page.should_not have_css('#supplier_featured') # cannot edit featured
+      page.should_not have_css('#s2id_supplier_user_ids') # cannot edit assigned users
+      page.should_not have_css('#supplier_commission_flat_rate') # cannot edit flat rate commission
+      page.should_not have_css('#supplier_commission_percentage') # cannot edit comission percentage
+      click_button 'Update'
+      page.should have_content('Supplier "Test Supplier" has been successfully updated!')
+      page.current_path.should eql(spree.edit_admin_supplier_path(@user.reload.supplier))
+    end
+
+  end
+
+  context 'as a User other than the suppliers' do
+
+    scenario 'should be unauthorized' do
+      supplier = create(:supplier)
+      login_user create(:user)
+      visit spree.edit_admin_supplier_path(supplier)
+      page.should have_content('Authorization Failure')
+    end
+
+  end
+
+end

data/spec/models/spree/order_decorator_spec.rb

@@ -0,0 +1,54 @@
+require 'spec_helper'
+
+describe Spree::Order do
+
+  context '#finalize_with_drop_ship!' do
+
+    after do
+      SpreeDropShip::Config[:send_supplier_email] = true
+    end
+
+    it 'should deliver drop ship orders when Spree::DropShipConfig[:send_supplier_email] == true' do
+      order = create(:order_with_totals, ship_address: create(:address))
+      order.line_items = [create(:line_item, variant: create(:variant_with_supplier)), create(:line_item, variant: create(:variant_with_supplier))]
+      order.create_proposed_shipments
+
+      order.shipments.each do |shipment|
+        Spree::DropShipOrderMailer.should_receive(:supplier_order).with(shipment.id).and_return(double(Mail, :deliver! => true))
+      end
+
+      order.finalize!
+      order.reload
+
+      # Check orders are properly split.
+      order.shipments.size.should eql(2)
+      order.shipments.each do |shipment|
+        shipment.line_items.size.should eql(1)
+        shipment.line_items.first.variant.suppliers.first.should eql(shipment.supplier)
+      end
+    end
+
+    it 'should NOT deliver drop ship orders when Spree::DropShipConfig[:send_supplier_email] == false' do
+      SpreeDropShip::Config[:send_supplier_email] = false
+      order = create(:order_with_totals, ship_address: create(:address))
+      order.line_items = [create(:line_item, variant: create(:variant_with_supplier)), create(:line_item, variant: create(:variant_with_supplier))]
+      order.create_proposed_shipments
+
+      order.shipments.each do |shipment|
+        Spree::DropShipOrderMailer.should_not_receive(:supplier_order).with(shipment.id)
+      end
+
+      order.finalize!
+      order.reload
+
+      # Check orders are properly split.
+      order.shipments.size.should eql(2)
+      order.shipments.each do |shipment|
+        shipment.line_items.size.should eql(1)
+        shipment.line_items.first.variant.suppliers.first.should eql(shipment.supplier)
+      end
+    end
+
+  end
+
+end

data/spec/models/spree/payment_decorator_spec.rb

@@ -0,0 +1,7 @@
+require 'spec_helper'
+
+describe Spree::Payment do
+
+  it { should belong_to(:payable) }
+
+end

data/spec/models/spree/product_decorator_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+
+describe Spree::Product do
+
+  let(:product) { create :product }
+
+  it '#supplier?' do
+    product.supplier?.should eq false
+    product.add_supplier! create(:supplier)
+    product.reload.supplier?.should eq true
+  end
+
+end

data/spec/models/spree/shipment_decorator_spec.rb

@@ -0,0 +1,47 @@
+require 'spec_helper'
+
+describe Spree::Shipment do
+
+  describe 'Scopes' do
+
+    it '#by_supplier' do
+      supplier = create(:supplier)
+      stock_location_1 = supplier.stock_locations.first
+      stock_location_2 = create(:stock_location, supplier: supplier)
+      shipment_1 = create(:shipment)
+      shipment_2 = create(:shipment, stock_location: stock_location_1)
+      shipment_3 = create(:shipment)
+      shipment_4 = create(:shipment, stock_location: stock_location_2)
+      shipment_5 = create(:shipment)
+      shipment_6 = create(:shipment, stock_location: stock_location_1)
+
+      expect(subject.class.by_supplier(supplier.id)).to match_array([shipment_2, shipment_4, shipment_6])
+    end
+
+  end
+
+  describe '#after_ship' do
+
+    it 'should capture payment if balance due' do
+      skip 'TODO make it so!'
+    end
+
+    it 'should track commission for shipment' do
+      supplier = create(:supplier_with_commission)
+      shipment = create(:shipment, stock_location: supplier.stock_locations.first)
+
+      expect(shipment.supplier_commission.to_f).to eql(0.0)
+      shipment.stub final_price_with_items: 10.0
+      shipment.send(:after_ship)
+      expect(shipment.reload.supplier_commission.to_f).to eql(1.5)
+    end
+
+  end
+
+  it '#final_price_with_items' do
+    shipment = build :shipment
+    shipment.stub item_cost: 50.0, final_price: 5.5
+    expect(shipment.final_price_with_items.to_f).to eql(55.5)
+  end
+
+end

data/spec/models/spree/stock/splitter/drop_ship_spec.rb

@@ -0,0 +1,62 @@
+require 'spec_helper'
+
+module Spree
+  module Stock
+    module Splitter
+      describe DropShip do
+
+        let(:supplier_1) { create(:supplier) }
+        let(:supplier_2) { create(:supplier) }
+
+        let(:variant_1) {
+          v = create(:variant)
+          v.product.add_supplier! supplier_1
+          v.reload.supplier_variants.find_by_supplier_id(supplier_1.id).update_column(:cost, 5)
+          v.product.add_supplier! supplier_2
+          v.reload.supplier_variants.find_by_supplier_id(supplier_2.id).update_column(:cost, 6)
+          v
+        }
+        let(:variant_2) {
+          v = create(:variant)
+          v.product.add_supplier! supplier_1
+          v.reload.supplier_variants.find_by_supplier_id(supplier_1.id).update_column(:cost, 5)
+          v.product.add_supplier! supplier_2
+          v.reload.supplier_variants.find_by_supplier_id(supplier_2.id).update_column(:cost, 4)
+          v
+        }
+        let(:variant_3) {
+          v = create(:variant)
+          v.product.add_supplier! supplier_1
+          v.product.add_supplier! supplier_2
+          v.reload
+        }
+        let(:variant_4) { create(:variant) }
+
+        let(:variants){
+          [variant_1, variant_2, variant_3, variant_4]
+        }
+
+        let(:packer) { build(:stock_packer) }
+
+        subject { DropShip.new(packer) }
+
+        it 'splits packages for drop ship' do
+          package = Package.new(packer.stock_location)
+          package = Package.new(packer.stock_location)
+          4.times { |i| package.add build(:inventory_unit, variant: variants[i]) }
+
+          packages = subject.split([package])
+          packages.count.should eq 3
+
+          expect(packages[0].stock_location).to eq(packer.stock_location)
+          expect(packages[0].contents.count).to eq(1)
+          expect(packages[1].stock_location).to eq(supplier_1.stock_locations.first)
+          expect(packages[1].contents.count).to eq(2)
+          expect(packages[2].stock_location).to eq(supplier_2.stock_locations.first)
+          expect(packages[2].contents.count).to eq(1)
+        end
+
+      end
+    end
+  end
+end

data/spec/models/spree/stock_location_decorator_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+
+describe Spree::StockLocation do
+
+  it { should belong_to(:supplier) }
+
+  subject { create(:stock_location, backorderable_default: true) }
+
+  context "propagate variants" do
+
+    let(:variant) { build(:variant) }
+    let(:stock_item) { subject.propagate_variant(variant) }
+
+    context "passes backorderable default config" do
+      context "true" do
+        before { subject.backorderable_default = true }
+        it { stock_item.backorderable.should be true }
+      end
+
+      context "false" do
+        before { subject.backorderable_default = false }
+        it { stock_item.backorderable.should be false }
+      end
+    end
+
+    context 'does not propagate for non supplier variants' do
+      before { subject.supplier_id = create(:supplier).id }
+      it { stock_item.should be_nil }
+    end
+
+  end
+
+end

data/spec/models/spree/supplier_ability_spec.rb

@@ -0,0 +1,172 @@
+require 'spec_helper'
+require 'cancan/matchers'
+require 'spree/testing_support/ability_helpers'
+
+describe Spree::SupplierAbility do
+
+  let(:user) { create(:user, supplier: create(:supplier)) }
+  let(:ability) { Spree::SupplierAbility.new(user) }
+  let(:token) { nil }
+
+  context 'for Dash' do
+    let(:resource) { Spree::Admin::RootController }
+
+    context 'requested by supplier' do
+      it_should_behave_like 'access denied'
+      it_should_behave_like 'no index allowed'
+      it_should_behave_like 'admin denied'
+    end
+  end
+
+  context 'for Product' do
+    let(:resource) { create(:product) }
+
+    it_should_behave_like 'index allowed'
+    it_should_behave_like 'admin granted'
+
+    context 'requested by another suppliers user' do
+      let(:resource) {
+        product = create(:product)
+        product.add_supplier!(create(:supplier))
+        product
+      }
+      it_should_behave_like 'access denied'
+    end
+
+    context 'requested by suppliers user' do
+      let(:resource) {
+        product = create(:product)
+        product.add_supplier!(user.supplier)
+        product.reload
+      }
+      # it_should_behave_like 'access granted'
+      it { ability.should be_able_to :read, resource }
+      it { ability.should be_able_to :stock, resource }
+    end
+  end
+
+  context 'for Shipment' do
+    context 'requested by another suppliers user' do
+      let(:resource) { Spree::Shipment.new({stock_location: create(:stock_location, supplier: create(:supplier))}, without_protection: true) }
+      it_should_behave_like 'access denied'
+      it_should_behave_like 'no index allowed'
+      it_should_behave_like 'admin denied'
+      it { ability.should_not be_able_to :ready, resource }
+      it { ability.should_not be_able_to :ship, resource }
+    end
+
+    context 'requested by suppliers user' do
+      context 'when order is complete' do
+        let(:resource) {
+          order = create(:completed_order_for_drop_ship_with_totals)
+          order.stock_locations.first.update_attribute :supplier, user.supplier
+          Spree::Shipment.new({order: order, stock_location: order.stock_locations.first }, without_protection: true)
+        }
+        it_should_behave_like 'access granted'
+        it_should_behave_like 'index allowed'
+        it_should_behave_like 'admin granted'
+        it { ability.should be_able_to :ready, resource }
+        it { ability.should be_able_to :ship, resource }
+      end
+
+      context 'when order is incomplete' do
+        let(:resource) { Spree::Shipment.new({stock_location: create(:stock_location, supplier: user.supplier)}, without_protection: true) }
+        it_should_behave_like 'access denied'
+        it { ability.should_not be_able_to :ready, resource }
+        it { ability.should_not be_able_to :ship, resource }
+      end
+    end
+  end
+
+  context 'for StockItem' do
+    let(:resource) { Spree::StockItem }
+
+    it_should_behave_like 'index allowed'
+    it_should_behave_like 'admin granted'
+
+    context 'requested by another suppliers user' do
+      let(:resource) {
+        supplier = create(:supplier)
+        variant = create(:product).master
+        variant.product.add_supplier! supplier
+        supplier.stock_locations.first.stock_items.first
+      }
+      it_should_behave_like 'access denied'
+    end
+
+    context 'requested by suppliers user' do
+      let(:resource) {
+        variant = create(:product).master
+        variant.product.add_supplier! user.supplier
+        user.supplier.stock_locations.first.stock_items.first
+      }
+      it_should_behave_like 'access granted'
+    end
+  end
+
+  context 'for StockLocation' do
+    context 'requsted by another suppliers user' do
+      let(:resource) {
+        supplier = create(:supplier)
+        variant = create(:product).master
+        variant.product.add_supplier! supplier
+        supplier.stock_locations.first
+      }
+      it_should_behave_like 'create only'
+    end
+
+    context 'requested by suppliers user' do
+      let(:resource) {
+        variant = create(:product).master
+        variant.product.add_supplier! user.supplier
+        user.supplier.stock_locations.first
+      }
+      it_should_behave_like 'access granted'
+      it_should_behave_like 'admin granted'
+      it_should_behave_like 'index allowed'
+    end
+  end
+
+  context 'for StockMovement' do
+    let(:resource) { Spree::StockMovement }
+
+    it_should_behave_like 'index allowed'
+    it_should_behave_like 'admin granted'
+
+    context 'requested by another suppliers user' do
+      let(:resource) {
+        supplier = create(:supplier)
+        variant = create(:product).master
+        variant.product.add_supplier! supplier
+        Spree::StockMovement.new({ stock_item: supplier.stock_locations.first.stock_items.first }, without_protection: true)
+      }
+      it_should_behave_like 'create only'
+    end
+
+    context 'requested by suppliers user' do
+      let(:resource) {
+        variant = create(:product).master
+        variant.product.add_supplier! user.supplier
+        Spree::StockMovement.new({ stock_item: user.supplier.stock_locations.first.stock_items.first }, without_protection: true)
+      }
+      it_should_behave_like 'access granted'
+    end
+  end
+
+  context 'for Supplier' do
+    context 'requested by any user' do
+      let(:ability) { Spree::SupplierAbility.new(create(:user)) }
+      let(:resource) { Spree::Supplier }
+
+      it_should_behave_like 'admin denied'
+      it_should_behave_like 'access denied'
+    end
+
+    context 'requested by suppliers user' do
+      let(:resource) { user.supplier }
+      it_should_behave_like 'admin granted'
+      it_should_behave_like 'update only'
+    end
+  end
+
+end

data/spec/models/spree/supplier_spec.rb

@@ -0,0 +1,123 @@
+require 'spec_helper'
+
+describe Spree::Supplier do
+
+  it { should belong_to(:address) }
+
+  it { should have_many(:products).through(:variants) }
+  it { should have_many(:stock_locations) }
+  it { should have_many(:users) }
+  it { should have_many(:variants).through(:supplier_variants) }
+
+  it { should validate_presence_of(:email) }
+  it { should validate_presence_of(:name) }
+
+  it '#deleted?' do
+    subject.deleted_at = nil
+    subject.deleted_at?.should eql(false)
+    subject.deleted_at = Time.now
+    subject.deleted_at?.should eql(true)
+  end
+
+  context '#assign_user' do
+
+    before do
+      @instance = build(:supplier)
+    end
+
+    it 'with user' do
+      Spree.user_class.should_not_receive :find_by_email
+      @instance.email = 'test@test.com'
+      @instance.users << create(:user)
+      @instance.save
+    end
+
+    it 'with existing user email' do
+      user = create(:user, email: 'test@test.com')
+      Spree.user_class.should_receive(:find_by_email).with(user.email).and_return(user)
+      @instance.email = user.email
+      @instance.save
+      @instance.reload.users.first.should eql(user)
+    end
+
+  end
+
+  it '#create_stock_location' do
+    Spree::StockLocation.count.should eql(0)
+    supplier = create :supplier
+    Spree::StockLocation.first.active.should be true
+    Spree::StockLocation.first.country.should eql(supplier.address.country)
+    Spree::StockLocation.first.supplier.should eql(supplier)
+  end
+
+  context '#send_welcome' do
+
+    after do
+      SpreeDropShip::Config[:send_supplier_email] = true
+    end
+
+    before do
+      @instance = build(:supplier)
+      @mail_message = double('Mail::Message')
+    end
+
+    context 'with Spree::DropShipConfig[:send_supplier_email] == false' do
+
+      it 'should not send' do
+        SpreeDropShip::Config[:send_supplier_email] = false
+        expect {
+          Spree::SupplierMailer.should_not_receive(:welcome).with(an_instance_of(Integer))
+        }
+        @instance.save
+      end
+
+    end
+
+    context 'with Spree::DropShipConfig[:send_supplier_email] == true' do
+
+      it 'should send welcome email' do
+        expect {
+          Spree::SupplierMailer.should_receive(:welcome).with(an_instance_of(Integer))
+        }
+        @instance.save
+      end
+
+    end
+
+  end
+
+  it '#set_commission' do
+    SpreeDropShip::Config.set default_commission_flat_rate: 1
+    SpreeDropShip::Config.set default_commission_percentage: 1
+    supplier = create :supplier
+    SpreeDropShip::Config.set default_commission_flat_rate: 0
+    SpreeDropShip::Config.set default_commission_percentage: 0
+    # Default configuration is 0.0 for each.
+    supplier.commission_flat_rate.to_f.should eql(1.0)
+    supplier.commission_percentage.to_f.should eql(1.0)
+    # With custom commission applied.
+    supplier = create :supplier, commission_flat_rate: 123, commission_percentage: 25
+    supplier.commission_flat_rate.should eql(123.0)
+    supplier.commission_percentage.should eql(25.0)
+  end
+
+  describe '#shipments' do
+
+    let!(:supplier) { create(:supplier) }
+
+    it 'should return shipments for suppliers stock locations' do
+      stock_location_1 = supplier.stock_locations.first
+      stock_location_2 = create(:stock_location, supplier: supplier)
+      shipment_1 = create(:shipment)
+      shipment_2 = create(:shipment, stock_location: stock_location_1)
+      shipment_3 = create(:shipment)
+      shipment_4 = create(:shipment, stock_location: stock_location_2)
+      shipment_5 = create(:shipment)
+      shipment_6 = create(:shipment, stock_location: stock_location_1)
+
+      expect(supplier.shipments).to match_array([shipment_2, shipment_4, shipment_6])
+    end
+
+  end
+
+end

data/spec/models/spree/supplier_variant_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe Spree::SupplierVariant do
+  skip "add some examples to (or delete) #{__FILE__}"
+end

data/spec/models/spree/user_decorator_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+describe Spree.user_class do
+
+  it { should belong_to(:supplier) }
+
+  it { should have_many(:variants).through(:supplier) }
+
+  let(:user) { build :user }
+
+  it '#supplier?' do
+    user.supplier?.should be false
+    user.supplier = build :supplier
+    user.supplier?.should be true
+  end
+
+end

data/spec/models/spree/variant_decorator_spec.rb

@@ -0,0 +1,7 @@
+require 'spec_helper'
+
+describe Spree::Variant do
+  it 'should populate stock item for each of the master variant suppliers' do
+    skip 'todo'
+  end
+end