solidus_marketplace 0.1.0

data/spec/features/admin/suppliers_spec.rb

@@ -0,0 +1,130 @@
+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 MarketMaker (aka admin)' do
+
+    before do
+      login_user create(:admin_user)
+      visit spree.admin_path
+      within '[data-hook=admin_tabs]' do
+        click_link 'Suppliers'
+      end
+      expect(page).to have_content('Suppliers')
+    end
+
+    xscenario '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 of America', from: 'Country'
+      select2 'Vermont', from: 'State'
+      fill_in 'supplier[address_attributes][phone]', with: '555-555-5555'
+      click_button 'Create'
+      expect(page).to have_content('Supplier "Test Supplier" has been successfully created!')
+    end
+
+    xscenario 'should be able to delete supplier' do
+      click_icon 'delete'
+      page.driver.browser.switch_to.alert.accept
+      within 'table' do
+        expect(page).to_not have_content(@supplier.name)
+      end
+    end
+
+    xscenario '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'
+      expect(page).to have_content('Supplier "Test Supplier" has been successfully updated!')
+    end
+
+  end
+
+  context 'as a Supplier' do
+    before do
+      @user = create(:supplier_admin)
+      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
+        expect(page).to have_link('Products')
+        expect(page).to have_link('Stock')
+        expect(page).to have_link('Stock Locations')
+        expect(page).to have_link('Profile')
+        expect(page).to have_link('Orders')
+        expect(page).to have_link('Suppliers')
+
+        expect(page).to_not have_link('Overview')
+        expect(page).to_not have_link('Reports')
+        expect(page).to_not have_link('Configuration')
+        expect(page).to_not have_link('Promotions')
+      end
+    end
+
+    xscenario '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'
+      expect(page).to_not have_css('#supplier_active') # cannot edit active
+      expect(page).to_not have_css('#supplier_featured') # cannot edit featured
+      expect(page).to_not have_css('#s2id_supplier_user_ids') # cannot edit assigned users
+      expect(page).to_not have_css('#supplier_commission_flat_rate') # cannot edit flat rate commission
+      expect(page).to_not have_css('#supplier_commission_percentage') # cannot edit comission percentage
+      click_button 'Update'
+      expect(page).to have_content('Supplier "Test Supplier" has been successfully updated!')
+      expect(page.current_path).to 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)
+      expect(page).to have_content('Authorization Failure')
+    end
+
+  end
+
+end

data/spec/models/spree/order_decorator_spec.rb

@@ -0,0 +1,80 @@
+require 'spec_helper'
+
+describe Spree::Order do
+
+  context '#finalize_with_supplier!' do
+
+    after do
+      SolidusMarketplace::Config[:send_supplier_email] = true
+    end
+
+    xit 'should deliver marketplace orders when SolidusMarketplace::Config[: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|
+        expect(Spree::MarketplaceOrderMailer).to receive(:supplier_order).with(shipment.id).and_return(double(Mail, :deliver! => true))
+      end
+
+      order.finalize!
+      order.reload
+
+      # Check orders are properly split.
+      expect(order.shipments.size).to eql(2)
+      order.shipments.each do |shipment|
+        expect(shipment.line_items.size).to eql(1)
+        expect(shipment.line_items.first.variant.suppliers.first).to eql(shipment.supplier)
+      end
+    end
+
+    it 'should NOT deliver marketplace orders when SolidusMarketplace::Config[:send_supplier_email] == false' do
+      SolidusMarketplace::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|
+        expect(Spree::MarketplaceOrderMailer).not_to receive(:supplier_order).with(shipment.id)
+      end
+
+      order.finalize!
+      order.reload
+
+      # Check orders are properly split.
+      expect(order.shipments.size).to eql(2)
+      order.shipments.each do |shipment|
+        expect(shipment.line_items.size).to eql(1)
+        expect(shipment.line_items.first.variant.suppliers.first).to eql(shipment.supplier)
+      end
+    end
+
+  end
+
+  describe "#supplier_total" do
+    context "when passed a supplier" do
+      it "returns the total commission earned for the order for a given supplier" do
+        order = create(:completed_order_from_supplier_with_totals, ship_address: create(:address))
+        supplier = order.suppliers.first
+        expected_supplier_total = Spree::Money.new(15.00)
+        expect(order.total).to eq(150.0)
+        expect(order.suppliers.count).to eq(1)
+        expect(order.supplier_total(supplier)).to eq(expected_supplier_total)
+      end
+    end
+
+    context "when passed a user associated with a supplier" do
+      it "returns the total commission earned for the order for a given supplier" do
+        order = create(:completed_order_from_supplier_with_totals, ship_address: create(:address))
+        supplier = order.suppliers.first
+        supplier_user = create(:supplier_user, supplier: supplier)
+
+        expected_supplier_total = Spree::Money.new(15.00)
+        expect(order.total).to eq(150.0)
+        expect(order.suppliers.count).to eq(1)
+        expect(order.supplier_total(supplier_user)).to eq(expected_supplier_total)
+      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,56 @@
+require 'spec_helper'
+
+describe Spree::Product do
+
+  let(:product) { create :product }
+  let(:supplier1) { create(:supplier) }
+  let(:supplier2) { create(:supplier) }
+
+  describe "#add_supplier!" do
+    context "when passed a supplier" do
+      it "adds the supplier to product's list of supppliers" do
+        expect(product.suppliers).to be_empty
+        product.add_supplier!(supplier1)
+        expect(product.reload.suppliers).to include(supplier1)
+      end
+    end
+
+    context "when passed a supplier_id" do
+      it "adds the supplier to product's list of supppliers" do
+        expect(product.suppliers).to be_empty
+        product.add_supplier!(supplier2.id)
+        expect(product.reload.suppliers).to include(supplier2)
+      end
+    end
+  end
+
+  describe "#add_suppliers!" do
+    it "adds multiple suppliers to the product's list of suppliers" do
+      expect(product.suppliers).to be_empty
+      product.add_suppliers!([supplier1.id, supplier2.id])
+      expect(product.reload.suppliers).to include(supplier1)
+      expect(product.reload.suppliers).to include(supplier2)
+    end
+  end
+
+  describe "#remove_suppliers!" do
+    it "removes multiple suppliers from the product's list of suppliers" do
+      product.add_suppliers!([supplier1.id, supplier2.id])
+      expect(product.reload.suppliers).to include(supplier1)
+      expect(product.reload.suppliers).to include(supplier2)
+
+      product.remove_suppliers!([supplier1.id, supplier2.id])
+      expect(product.suppliers).to be_empty
+    end
+  end
+
+  describe '#supplier?' do
+    it "returns true if one or more suppliers are present" do
+      expect(product.supplier?).to eq false
+      product.add_supplier! create(:supplier)
+      expect(product.reload.supplier?).to eq true
+    end
+  end
+
+
+end

data/spec/models/spree/shipment_decorator_spec.rb

@@ -0,0 +1,48 @@
+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
+
+    xit '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)
+      allow(shipment).to receive(:final_price_with_items).and_return(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
+    allow(shipment).to receive(:item_cost).and_return(50.0)
+    allow(shipment).to receive(:final_price).and_return(5.5)
+    expect(shipment.final_price_with_items.to_f).to eql(55.5)
+  end
+
+end

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

@@ -0,0 +1,62 @@
+require 'spec_helper'
+
+module Spree
+  module Stock
+    module Splitter
+      describe Marketplace 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 { Marketplace.new(packer.stock_location) }
+
+        it 'splits packages for suppliers to 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])
+          expect(packages.count).to 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 }
+        xit { expect(stock_item.backorderable).to eq true }
+      end
+
+      context "false" do
+        before { subject.backorderable_default = false }
+        xit { expect(stock_item.backorderable).to eq false }
+      end
+    end
+
+    context 'does not propagate for non supplier variants' do
+      before { subject.supplier_id = create(:supplier).id }
+      it { expect(stock_item).to be_nil }
+    end
+
+  end
+
+end

data/spec/models/spree/supplier_ability_spec.rb

@@ -0,0 +1,182 @@
+require 'spec_helper'
+require 'cancan/matchers'
+require 'spree/testing_support/ability_helpers'
+
+describe Spree::SupplierAbility do
+
+  let(:supplier) { create(:supplier) }
+  let(:supplier_admin_role) { build :role, name: "supplier_admin" }
+  let(:user) { create(:user, supplier: supplier) }
+  let(:ability) { Spree::SupplierAbility.new(user) }
+  let(:token) { nil }
+
+  before(:each) do
+    user.spree_roles << supplier_admin_role
+  end
+
+  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) }
+
+    before(:each) do
+      resource.add_supplier!(user.supplier)
+      resource.reload
+    end
+
+    it_should_behave_like 'index allowed'
+    it_should_behave_like 'read and update access granted'
+    it_should_behave_like 'admin granted'
+
+    context 'requested by another suppliers user' do
+      let(:other_resource) {
+        product = create(:product)
+        product.add_supplier!(create(:supplier))
+        product
+      }
+      it { expect(ability).to_not be_able_to :read, other_resource }
+    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 { expect(ability).to be_able_to :read, resource }
+      it { expect(ability).to 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))}) }
+      it_should_behave_like 'access denied'
+      it_should_behave_like 'no index allowed'
+      it_should_behave_like 'admin denied'
+      it { expect(ability).to_not be_able_to :ready, resource }
+      it { expect(ability).to_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_from_supplier_with_totals)
+          order.stock_locations.first.update_attribute :supplier, user.supplier
+          Spree::Shipment.new({order: order, stock_location: order.stock_locations.first })
+        }
+        it_should_behave_like 'read only'
+        it_should_behave_like 'index allowed'
+        it_should_behave_like 'admin granted'
+      end
+
+      context 'when order is incomplete' do
+        let(:resource) { Spree::Shipment.new({stock_location: create(:stock_location, supplier: user.supplier)}) }
+        it_should_behave_like 'access denied'
+        it { expect(ability).to_not be_able_to :ready, resource }
+        it { expect(ability).to_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 '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
+      }
+      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 })
+      }
+      it_should_behave_like 'admin denied'
+    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 })
+      }
+      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 'read and update access granted'
+    end
+  end
+
+end