spree_multi_vendor 2.0.1

data/spec/features/spree/admin/vendor_settings_spec.rb

@@ -0,0 +1,76 @@
+require 'spec_helper'
+
+RSpec.feature 'Admin Vendor Settings', :js do
+  let(:vendor) { create(:active_vendor) }
+  let(:blocked_vendor) { create(:blocked_vendor)}
+  let!(:user) { create(:user, vendors: [vendor]) }
+  let!(:user_with_blocked_vendor) { create(:user, vendors: [blocked_vendor]) }
+
+  context 'edit' do
+    background do
+      login_as(user, scope: :spree_user)
+      visit spree.admin_vendor_settings_path
+    end
+
+    scenario 'can update current vendor' do
+      fill_in 'vendor_name', with: 'Testing edit'
+      fill_in 'vendor_about_us', with: 'Testing about us edit'
+      fill_in 'vendor_contact_us', with: 'Testing contact us edit'
+      expect {
+        click_button 'Update'
+      }.to change { vendor.reload.slug }.from('active-vendor').to('testing-edit')
+      expect(page).to have_text 'Testing edit'
+      expect(page).to have_text 'Testing about us edit'
+      expect(page).to have_text 'Testing contact us edit'
+    end
+
+    scenario 'shows validation error with blank name' do
+      fill_in 'vendor_name', with: ''
+      click_button 'Update'
+      expect(page).to have_text 'name can\'t be blank'
+    end
+
+    scenario 'does not show validation error with blank about_us' do
+      fill_in 'vendor_about_us', with: ''
+      click_button 'Update'
+      expect(page).not_to have_text 'about_us can\'t be blank'
+    end
+
+    scenario 'does not show validation error with blank contact_us' do
+      fill_in 'vendor_contact_us', with: ''
+      click_button 'Update'
+      expect(page).not_to have_text 'contact_us can\'t be blank'
+    end
+
+    if Spree.version.to_f >= 3.6
+      scenario 'can update an existing vendor image' do
+        create(:vendor, name: 'New vendor')
+        page.attach_file("vendor_image", Spree::Core::Engine.root + 'spec/fixtures' + 'thinking-cat.jpg')
+        click_button 'Update'
+        expect(page).to have_css("img[src*='thinking-cat.jpg']")
+      end
+    end
+  end
+
+  context 'no vendor present' do
+    before do
+      vendor.destroy!
+    end
+
+    it 'renders page without errors' do
+      visit spree.admin_vendor_settings_path
+      expect(page).to have_current_path('/admin/login')
+    end
+  end
+
+  context 'blocked vendor present' do
+    background do
+      login_as(user_with_blocked_vendor, scope: :spree_user)
+    end
+
+    it 'renders forbidden page' do
+      visit spree.admin_vendor_settings_path
+      expect(current_path).to eq spree.forbidden_path
+    end
+  end
+end

data/spec/features/spree/admin/vendor_spec.rb

@@ -0,0 +1,125 @@
+require 'spec_helper'
+
+RSpec.feature 'Admin Vendors', :js do
+  let!(:admin) { create(:admin_user) }
+  let!(:vendor_user) { create(:user) }
+
+  background do
+    login_as(admin, scope: :spree_user)
+    create(:active_vendor, users: [vendor_user]) if Spree::Vendor.find_by(name: 'Active vendor').nil?
+    create(:pending_vendor)
+    create(:blocked_vendor)
+    visit spree.admin_vendors_path
+  end
+
+  context 'index' do
+    scenario 'displays existing vendors' do
+      within_row(1) do
+        expect(column_text(1)).to eq vendor_user.email
+        expect(column_text(2)).to eq 'Active vendor'
+        expect(column_text(3)).to eq 'active'
+      end
+    end
+
+    scenario 'filter by multiple states' do
+      click_button 'Filter'
+      select2 'Active', from: 'State'
+      select2 'Pending', from: 'State'
+
+      click_button 'Filter Results'
+
+      expect(page).to have_content('active')
+      expect(page).to have_content('pending')
+      expect(page).not_to have_content('blocked')
+    end
+  end
+
+  context 'create' do
+    scenario 'can create a new vendor' do
+      click_link 'New Vendor'
+      expect(current_path).to eq spree.new_admin_vendor_path
+
+      fill_in 'vendor_name', with: 'Test'
+      fill_in 'vendor_about_us', with: 'About...'
+      fill_in 'vendor_contact_us', with: 'Contact...'
+      expect(find_field('vendor_commission_rate').value).to eq '5.0'
+      select 'Blocked'
+
+      click_button 'Create'
+
+      expect(page).to have_text 'successfully created!'
+      expect(current_path).to eq spree.admin_vendors_path
+    end
+
+    scenario 'shows validation error with blank name' do
+      click_link 'New Vendor'
+      expect(current_path).to eq spree.new_admin_vendor_path
+
+      fill_in 'vendor_name', with: ''
+      click_button 'Create'
+
+      expect(page).to have_text 'name can\'t be blank'
+    end
+
+    scenario 'shows validation error with repeated name' do
+      click_link 'New Vendor'
+      expect(current_path).to eq spree.new_admin_vendor_path
+
+      fill_in 'vendor_name', with: 'Active vendor'
+      click_button 'Create'
+
+      expect(page).to have_text 'name has already been taken'
+    end
+  end
+
+  context 'edit' do
+    background do
+      within_row(1) { click_icon :edit }
+      expect(current_path).to eq spree.edit_admin_vendor_path('active-vendor')
+    end
+
+    scenario 'can update an existing vendor' do
+      fill_in 'vendor_name', with: 'Testing edit'
+      fill_in 'vendor_about_us', with: 'Testing about us'
+      fill_in 'vendor_contact_us', with: 'Testing contact us'
+      fill_in 'vendor_commission_rate', with: '5.5'
+      click_button 'Update'
+      expect(page).to have_text 'successfully updated!'
+      expect(page).to have_text 'Testing edit'
+    end
+
+    if Spree.version.to_f >= 3.6
+      scenario 'can update an existing vendor image' do
+        page.attach_file("vendor_image", Spree::Core::Engine.root + 'spec/fixtures' + 'thinking-cat.jpg')
+        expect { click_button 'Update' }.to change(Spree::VendorImage, :count).by(1)
+        expect(page).to have_text 'successfully updated!'
+      end
+    end
+
+    scenario 'shows validation error with blank name' do
+      fill_in 'vendor_name', with: ''
+      click_button 'Update'
+      expect(page).to have_text 'name can\'t be blank'
+    end
+
+    scenario 'does not show validation error with blank about_us' do
+      fill_in 'vendor_about_us', with: ''
+      click_button 'Update'
+      expect(page).not_to have_text 'about_us can\'t be blank'
+    end
+
+    scenario 'does not show validation error with blank contact_us' do
+      fill_in 'vendor_contact_us', with: ''
+      click_button 'Update'
+      expect(page).not_to have_text 'contact_us can\'t be blank'
+    end
+
+    scenario 'shows validation error with repeated name' do
+      create(:vendor, name: 'New vendor')
+
+      fill_in 'vendor_name', with: 'New vendor'
+      click_button 'Update'
+      expect(page).to have_text 'name has already been taken'
+    end
+  end
+end

data/spec/mailers/vendor_mailer_spec.rb

@@ -0,0 +1,45 @@
+require 'spec_helper'
+
+describe Spree::VendorMailer, type: :mailer do
+  let(:store) { create(:store)}
+  let(:vendor) { Spree::Vendor.new(id: 1, name: 'Test vendor', state: 'active', notification_email: 'test-vendor@example.com') }
+  let(:other_vendor) { Spree::Vendor.new(id: 2, name: 'Other vendor', state: 'active', notification_email: 'other-vendor@example.com') }
+  let(:order) { create(:completed_order_with_totals, line_items_count: 5) }
+
+  context 'vendor notification email' do
+    before do
+      order.line_items.first.product.update(vendor: vendor)
+      order.line_items.last.product.update(vendor: other_vendor)
+    end
+
+    it 'is sent from current store email address' do
+      notification_email = Spree::VendorMailer.vendor_notification_email(order.id, vendor.id)
+      expect(notification_email.from).to have_content(Spree::Store.current.mail_from_address)
+    end
+
+    it 'is sent to vendors email address' do
+      notification_email = Spree::VendorMailer.vendor_notification_email(order.id, vendor.id)
+      expect(notification_email.to).to have_content(vendor.contact_us)
+    end
+
+    it 'subject contains order number' do
+      notification_email = Spree::VendorMailer.vendor_notification_email(order.id, vendor.id)
+      expect(notification_email.subject).to have_content(order.number)
+    end
+
+    it 'body contains vendors items' do
+      notification_email = Spree::VendorMailer.vendor_notification_email(order.id, vendor.id)
+      vendors_item = order.line_items.select { |line_item| line_item.product.vendor == vendor }.first
+      expect(notification_email.body.parts.first).to have_text(vendors_item.product.name)
+      expect(notification_email.body.parts.last).to have_text(vendors_item.product.name)
+    end
+
+    it 'body does not contain other vendors items' do
+      notification_email = Spree::VendorMailer.vendor_notification_email(order.id, vendor.id)
+      other_vendors_item = order.line_items.select { |line_item| line_item.product.vendor != vendor }.first
+      expect(notification_email.body.parts.first).not_to have_text(other_vendors_item.product.name)
+      expect(notification_email.body.parts.last).not_to have_text(other_vendors_item.product.name)
+    end
+  end
+
+end

data/spec/models/spree/line_item_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+describe Spree::LineItem do
+  let(:vendor) { create(:vendor) }
+
+  describe '#vendor' do
+    context 'vendorized product' do
+      let(:product) { create(:product, vendor: vendor) }
+      let(:line_item) { build(:line_item, product: product, variant: product.default_variant) }
+
+      it { expect(line_item.vendor).not_to be_nil }
+      it { expect(line_item.vendor).to eq(product.vendor) }
+    end
+
+    context 'vendorized variant' do
+      let(:product) { create(:product) }
+      let(:variant) { create(:variant, vendor: vendor, product: product) }
+      let(:line_item) { build(:line_item, product: product, variant: variant) }
+
+      before do
+        SpreeMultiVendor::Config[:vendorized_models] = ['variant']
+      end
+
+      after do
+        SpreeMultiVendor::Config[:vendorized_models] = SpreeMultiVendor::Configuration::DEFAULT_VENDORIZED_MODELS
+      end
+
+      it { expect(line_item.vendor).not_to be_nil }
+      it { expect(line_item.vendor).to eq(variant.vendor) }
+    end
+  end
+end

data/spec/models/spree/order_commission_spec.rb

@@ -0,0 +1,8 @@
+require 'spec_helper'
+
+describe Spree::OrderCommission do
+  describe 'associations' do
+    it { is_expected.to belong_to(:order) }
+    it { is_expected.to belong_to(:vendor) }
+  end
+end

data/spec/models/spree/order_spec.rb

@@ -0,0 +1,179 @@
+require 'spec_helper'
+
+describe Spree::Order do
+  let(:vendor) { create(:vendor, name: 'Vendor 1') }
+  let(:vendor_2) { create(:vendor, name: 'Vendor 2') }
+
+  let(:order) { create(:order_with_line_items, state: 'payment', line_items_count: 6, line_items_price: 100, shipment_cost: 50) }
+
+  before do
+    order.line_items.each_with_index do |li, idx|
+      product = li.product
+      product.vendor = idx.odd? ? vendor : vendor_2
+      product.save
+    end
+  end
+
+  context 'associations' do
+    it { is_expected.to have_many(:commissions) }
+  end
+
+  context 'complete with spree_multi_vendor' do
+    before { allow(order).to receive_messages payment_required?: false }
+
+    it 'generates commission records' do
+      expect do
+        order.next!
+        order.reload
+      end.to change { Spree::OrderCommission.count }.by(2).and change { vendor.commissions.count }.by(1)
+                                                          .and change { vendor_2.commissions.count }.by(1)
+
+      order.line_items.includes(product: :vendor).group_by { |li| li.product.vendor }.each do |v, line_items|
+        commission_ammount = v.commission_rate * line_items.pluck(:pre_tax_amount).sum.to_f / 100
+
+        expect(v.commissions.sum(:amount)).to eq(commission_ammount)
+      end
+    end
+  end
+
+  context 'vendor methods' do
+    let!(:shipment) { create(:shipment, order: order, cost: 20, pre_tax_amount: 20, stock_location: vendor.stock_locations.first) }
+    let!(:promotion) { create(:promotion_with_item_adjustment, adjustment_rate: 10, code: '10off') }
+
+    before do
+      order.coupon_code = '10off'
+      Spree::PromotionHandler::Coupon.new(order).apply
+      order.reload
+    end
+
+    describe '#vendor_line_items' do
+      it { expect(order.vendor_line_items(vendor)).to eq(order.line_items.for_vendor(vendor)) }
+      it { expect(order.vendor_line_items(vendor).count).to eq(3) }
+    end
+
+    describe '#vendor_shipments' do
+      it { expect(order.vendor_shipments(vendor).to_a).to eq([shipment]) }
+      it { expect(order.vendor_shipments(vendor).count).to eq(1) }
+    end
+
+    describe '#vendor_subtotal' do
+      it { expect(order.vendor_subtotal(vendor)).to eq(270) }
+    end
+
+    describe '#display_subtotal' do
+      it { expect(order.display_vendor_subtotal(vendor).to_s).to eq('$270.00') }
+    end
+
+    describe '#vendor_pre_tax_item_amount' do
+      it { expect(order.vendor_pre_tax_item_amount(vendor)).to eq(300) }
+    end
+
+    describe '#displayvendor_pre_tax_item_amount' do
+      it { expect(order.display_vendor_pre_tax_item_amount(vendor).to_s).to eq('$300.00') }
+    end
+
+    describe '#vendor_pre_tax_total' do
+      it { expect(order.vendor_pre_tax_total(vendor)).to eq(320) }
+    end
+
+    describe '#displayvendor_pre_tax_total' do
+      it { expect(order.display_vendor_pre_tax_total(vendor).to_s).to eq('$320.00') }
+    end
+
+    context 'ship total' do
+      before do
+        order.vendor_shipments(vendor).update_all(adjustment_total: 5)
+      end
+
+      describe '#vendor_ship_total' do
+        it { expect(order.vendor_ship_total(vendor)).to eq(25) }
+      end
+
+      describe '#display_vendor_ship_total' do
+        it { expect(order.display_vendor_ship_total(vendor).to_s).to eq('$25.00') }
+      end
+
+      describe '#vendor_pre_tax_ship_amount' do
+        it { expect(order.vendor_pre_tax_ship_amount(vendor)).to eq(20) }
+      end
+
+      describe '#display_vendor_pre_tax_ship_amount' do
+        it { expect(order.display_vendor_pre_tax_ship_amount(vendor).to_s).to eq('$20.00') }
+      end
+    end
+
+    describe '#vendor_promo_total' do
+      it { expect(order.vendor_promo_total(vendor)).to eq(-30) }
+    end
+
+    describe '#display_vendor_promo_total' do
+      it { expect(order.display_vendor_promo_total(vendor).to_s).to eq('-$30.00') }
+    end
+
+    context 'additional tax' do
+      before do
+        order.vendor_line_items(vendor).update_all(additional_tax_total: 10)
+        order.vendor_shipments(vendor).update_all(additional_tax_total: 1)
+      end
+
+      describe '#vendor_additional_tax_total' do
+        it { expect(order.vendor_additional_tax_total(vendor)).to eq(31) }
+      end
+
+      describe '#display_vendor_additional_tax_total' do
+        it { expect(order.display_vendor_additional_tax_total(vendor).to_s).to eq('$31.00') }
+      end
+    end
+
+    context 'included tax' do
+      before do
+        order.vendor_line_items(vendor).update_all(included_tax_total: 5)
+        order.vendor_shipments(vendor).update_all(included_tax_total: 1)
+      end
+
+      describe '#vendor_included_tax_total' do
+        it { expect(order.vendor_included_tax_total(vendor)).to eq(16) }
+      end
+
+      describe '#display_vendor_included_tax_total' do
+        it { expect(order.display_vendor_included_tax_total(vendor).to_s).to eq('$16.00') }
+      end
+    end
+
+    describe '#vendor_total' do
+      it { expect(order.vendor_total(vendor)).to eq(290) }
+    end
+
+    describe '#display_vendor_total' do
+      it { expect(order.display_vendor_total(vendor).to_s).to eq('$290.00') }
+
+      context 'respects currency' do
+        before { order.currency = 'GBP' }
+
+        it { expect(order.display_vendor_total(vendor).to_s).to eq('£290.00') }
+      end
+    end
+
+    describe '#vendor_item_count' do
+      before { order.vendor_line_items(vendor).last.update(quantity: 2) }
+
+      it { expect(order.vendor_item_count(vendor)).to eq(4) }
+    end
+
+    describe '#vendor_ids' do
+      it { expect(order.vendor_ids).to match_array([vendor.id, vendor_2.id]) }
+
+      context 'when a line item does not belong to any vendor' do
+        before { order.vendor_line_items(vendor_2).each { |li| li.product.update_columns(vendor_id: nil) } }
+        it { expect(order.vendor_ids).to match_array([vendor.id]) }
+      end
+    end
+
+    describe '#vendor_totals' do
+      it { expect(order.vendor_totals).to be_kind_of(Array) }
+      it { expect(order.vendor_totals).not_to be_empty }
+      it { expect(order.vendor_totals.first.order).to eq(order) }
+      it { expect(order.vendor_totals.map(&:vendor)).to match_array([vendor, vendor_2]) }
+    end
+  end
+end