atkhayar-dev-spree-multi-vendor 0.0.0

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,193 @@
+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
+
+    it 'sends mails only once per vendor' do
+      mail_double_1 = double(:mail_double_1)
+      mail_double_2 = double(:mail_double_1)
+
+      expect(Spree::VendorMailer).to receive(:vendor_notification_email)
+        .with(be_an(Integer), vendor.id) { mail_double_1 }
+      expect(mail_double_1).to receive(:deliver_later)
+      expect(Spree::VendorMailer).to receive(:vendor_notification_email)
+        .with(be_an(Integer), vendor_2.id) { mail_double_2 }
+      expect(mail_double_2).to receive(:deliver_later)
+
+      order.next!
+    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(320) }
+    end
+
+    describe '#display_subtotal' do
+      it { expect(order.display_vendor_subtotal(vendor).to_s).to eq('$320.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

data/spec/models/spree/product_spec.rb

@@ -0,0 +1,19 @@
+require 'timecop'
+
+describe Spree::Product do
+  let(:vendor) { create(:vendor) }
+  let(:product) { create(:product, vendor: vendor) }
+
+  context '.whitelisted_ransackable_associations' do
+    it { expect(described_class.whitelisted_ransackable_associations).to include('vendor') }
+    it { expect(described_class.whitelisted_ransackable_associations.count).to be > 1 }
+  end
+
+  it 'touches vendor after update' do
+    time = Time.current + 1.hour
+    product
+    Timecop.freeze(time) do
+      expect { product.touch }.to change { vendor.reload.updated_at }
+    end
+  end
+end

data/spec/models/spree/variant_spec.rb

@@ -0,0 +1,104 @@
+require 'spec_helper'
+
+describe Spree::Variant do
+  let(:vendor) { create :vendor }
+  let(:other_vendor) { create :vendor, name: 'Other vendor' }
+  let(:product) { create(:product, vendor: vendor) }
+  subject { build(:variant, product: product) }
+
+  context 'after create' do
+    it "propagate to stock items only for Vendor's Stock Locations" do
+      expect { create(:variant, product: product) }.to change {
+        Spree::StockItem.where(stock_location: vendor.stock_locations.first).count
+      }
+    end
+
+    it "doesn't propagate to stock items for other vendors Stock Locations" do
+      expect { create(:variant, product: product) }.to_not change {
+        Spree::StockItem.where(stock_location: other_vendor.stock_locations.first).count
+      }
+    end
+  end
+
+  describe '#assign_vendor' do
+    context 'product vendorized, variant vendorized' do
+      it 'copies from product' do
+        expect(subject.attributes['vendor_id']).to be_nil
+        subject.save!
+        expect(subject.reload.attributes['vendor_id']).to eq(product.vendor_id)
+      end
+    end
+
+    context 'product not vendorized, variant vendorized' do
+      before do
+        SpreeMultiVendor::Config[:vendorized_models] = ['variant']
+      end
+
+      after do
+        SpreeMultiVendor::Config[:vendorized_models] = SpreeMultiVendor::Configuration::DEFAULT_VENDORIZED_MODELS
+      end
+
+      context 'no vendor' do
+        let(:product) { create(:product) }
+
+        it { expect { subject.save! }.not_to change { subject.vendor }.from(nil) }
+      end
+
+      context 'vendor assigned' do
+        subject { build(:variant, product: product, vendor: vendor) }
+
+        it { expect { subject.save! }.not_to change { subject.vendor }.from(vendor) }
+      end
+    end
+
+    context 'product vendorized, variant not vendorized' do
+      before do
+        SpreeMultiVendor::Config[:vendorized_models] = ['product']
+      end
+
+      after do
+        SpreeMultiVendor::Config[:vendorized_models] = SpreeMultiVendor::Configuration::DEFAULT_VENDORIZED_MODELS
+      end
+
+      it { expect { subject.save! }.not_to raise_error }
+    end
+  end
+
+  describe '#vendor' do
+    context 'product vendorized, variant vendorized' do
+      before { subject.save! }
+
+      it { expect(subject.vendor).to eq(product.vendor) }
+    end
+
+    context 'product not vendorized, variant vendorized' do
+      before do
+        SpreeMultiVendor::Config[:vendorized_models] = ['variant']
+      end
+
+      after do
+        SpreeMultiVendor::Config[:vendorized_models] = SpreeMultiVendor::Configuration::DEFAULT_VENDORIZED_MODELS
+      end
+
+      let(:product) { create(:product) }
+      subject { build(:variant, product: product, vendor: vendor) }
+
+      it { expect(subject.vendor).to eq(vendor) }
+    end
+
+    context 'product vendorized, variant not vendorized' do
+      before do
+        SpreeMultiVendor::Config[:vendorized_models] = ['product']
+        subject.save!
+      end
+
+      after do
+        SpreeMultiVendor::Config[:vendorized_models] = SpreeMultiVendor::Configuration::DEFAULT_VENDORIZED_MODELS
+      end
+
+      it { expect(subject.vendor).to eq(product.vendor) }
+    end
+  end
+
+  it { expect(described_class.whitelisted_ransackable_associations).to include('vendor') }
+end

data/spec/models/spree/vendor_order_totals_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+
+describe Spree::VendorOrderTotals do
+  let(:vendor) { create(:vendor) }
+  let(:order) { create(:completed_order_with_totals, line_items_count: 6, line_items_price: 100, shipment_cost: 50) }
+  let!(:shipment) { create(:shipment, order: order, cost: 20, stock_location: vendor.stock_locations.first) }
+  let!(:promotion) { create(:promotion_with_item_adjustment, adjustment_rate: 10, code: '10off') }
+
+  subject { described_class.new(order: order, vendor: vendor) }
+
+  before do
+    order.products.update_all(vendor_id: vendor.id)
+    order.vendor_line_items(vendor).update_all(additional_tax_total: 10)
+    order.vendor_line_items(vendor).update_all(included_tax_total: 5)
+    order.coupon_code = '10off'
+    Spree::PromotionHandler::Coupon.new(order).apply
+    order.reload
+  end
+
+  Spree::VendorOrderTotals::METHOD_NAMES.each do |method_name|
+    describe "##{method_name}" do
+      it { expect(subject.send(method_name)).to eq(order.send("vendor_#{method_name}", vendor)) }
+    end
+
+    describe "#display_#{method_name}" do
+      it { expect(subject.send("display_#{method_name}")).to eq(order.send("display_vendor_#{method_name}", vendor)) }
+    end
+  end
+
+  describe '#item_count' do
+    it { expect(subject.item_count).to eq(order.vendor_item_count(vendor)) }
+  end
+end

data/spec/models/spree/vendor_spec.rb

@@ -0,0 +1,57 @@
+require 'spec_helper'
+
+describe Spree::Vendor do
+  describe 'associations' do
+    it { is_expected.to have_many(:commissions) }
+    it { is_expected.to have_many(:products) }
+    it { is_expected.to have_many(:shipping_methods) }
+    it { is_expected.to have_many(:stock_locations) }
+    it { is_expected.to have_many(:variants) }
+    it { is_expected.to have_many(:vendor_users) }
+    it { is_expected.to have_many(:users).through(:vendor_users) }
+    it { is_expected.to have_one(:image) }
+  end
+
+  describe 'validations' do
+    it { is_expected.to validate_presence_of(:name) }
+    it { is_expected.to allow_value('flower').for(:name) }
+    it { is_expected.to allow_value('flower455').for(:name) }
+    it { is_expected.to allow_value('flower 455').for(:name) }
+    it { is_expected.to allow_value('flower abc').for(:name) }
+    it { is_expected.to allow_value('flower455 abc').for(:name) }
+    it { is_expected.to allow_value('flower 455 abc').for(:name) }
+    it { is_expected.to allow_value('flower & bucket').for(:name) }
+  end
+
+  describe 'initial state' do
+    it 'initial state should be pending' do
+      should be_pending
+    end
+  end
+
+  describe 'after_create' do
+    let!(:vendor) { build(:vendor) }
+
+    it 'creates a stock location with default country' do
+      expect { vendor.save! }.to change(Spree::StockLocation, :count).by(1)
+      stock_location = Spree::StockLocation.last
+      expect(vendor.stock_locations.first).to eq stock_location
+      expect(stock_location.country).to eq Spree::Country.default
+    end
+
+    it 'should act as list' do
+      expect(vendor).to respond_to(:set_list_position)
+    end
+  end
+
+  describe 'after_update' do
+    let!(:vendor) { create(:vendor) }
+
+    it 'updates stock_location names when vendor name changed' do
+      old_name = vendor.name
+      new_name = 'new vendor name'
+      vendor.name = new_name
+      expect { vendor.save! }.to change { vendor.stock_locations.pluck(:name).uniq }.from([old_name]).to([new_name])
+    end
+  end
+end

data/spec/models/spree/vendor_user_spec.rb

@@ -0,0 +1,12 @@
+require 'spec_helper'
+
+describe Spree::VendorUser do
+  describe 'associations' do
+    it { is_expected.to belong_to(:vendor) }
+    it { is_expected.to belong_to(:user) }
+  end
+
+  describe 'validations' do
+    it { is_expected.to validate_uniqueness_of(:vendor_id).scoped_to(:user_id) }
+  end
+end

data/spec/requests/spree/api/v1/products_spec.rb

@@ -0,0 +1,221 @@
+require 'spec_helper'
+
+module Spree
+  describe 'API V1 Products Spec', type: :request do
+    let(:vendor) { create(:vendor, state: :active) }
+    let(:other_vendor) { create(:vendor) }
+    let(:shipping_category) { create(:shipping_category) }
+    let(:selected_product) { product_without_vendor }
+    let(:params) { {} }
+
+    let!(:product_without_vendor) { create(:product) }
+    let!(:product_with_vendor_vendor) { create(:product, vendor: vendor) }
+    let!(:product_with_other_vendor) { create(:product, vendor: other_vendor) }
+
+    before { stub_authentication! }
+
+    describe 'products#index' do
+      before { get '/api/v1/products', params: params }
+
+      context 'as a regular user' do
+        it 'gets a list of all products' do
+          expect(response.status).to eq(200)
+          expect(json_response[:products].size).to eq(3)
+        end
+      end
+
+      context 'as an admin' do
+        sign_in_as_admin!
+
+        it 'gets a list of all products' do
+          expect(response.status).to eq(200)
+          expect(json_response[:products].size).to eq(3)
+        end
+      end
+
+      context 'as a vendor' do
+        let!(:current_api_user) do
+          user = create(:user, vendors: [vendor])
+          user
+        end
+
+        it 'gets a list of products assigned to that vendor' do
+          expect(response.status).to eq(200)
+          expect(json_response[:products].size).to eq(1)
+          expect(json_response[:products][0][:id]).to eq(vendor.products.first.id)
+        end
+      end
+    end
+
+    describe 'products#show' do
+      before { get "/api/v1/products/#{selected_product.id}" }
+
+      it 'gets a product' do
+        expect(response.status).to eq(200)
+      end
+
+      context 'as a vendor' do
+        let!(:current_api_user) do
+          user = create(:user, vendors: [vendor])
+          user
+        end
+
+        describe 'cannot view product of another vendor' do
+          let(:selected_product) { product_with_other_vendor }
+
+          it do
+            expect(response.status).to eq(404)
+          end
+        end
+
+        describe 'can view product assigned to that vendor' do
+          let(:selected_product) { product_with_vendor_vendor }
+
+          it do
+            expect(response.status).to eq(200)
+          end
+        end
+      end
+    end
+
+    describe 'products#create' do
+      let(:params) do
+        {
+          product: {
+            name: 'Product',
+            shipping_category_id: shipping_category.id,
+            price: 12.34
+          }
+        }
+      end
+
+      before { post '/api/v1/products', params: params }
+
+      context 'as regular user' do
+        it 'cannot create a new product' do
+          expect(response.status).to eq(401)
+        end
+      end
+
+      context 'as an admin' do
+        sign_in_as_admin!
+
+        it 'can create a new product' do
+          expect(response.status).to eq(201)
+          expect(json_response['name']).to eq(params[:product][:name])
+          expect(json_response['price']).to eq(params[:product][:price].to_s)
+          expect(json_response['shipping_category_id']).to eq(params[:product][:shipping_category_id])
+        end
+      end
+
+      context 'as a vendor' do
+        let!(:current_api_user) do
+          user = create(:user, vendors: [vendor])
+          user
+        end
+
+        it 'can create a new product assigned to vendor' do
+          expect(response.status).to eq(201)
+          product = Spree::Product.find(json_response[:id])
+          expect(product.vendor).to eq(vendor)
+        end
+      end
+    end
+
+    describe 'products#update' do
+      let(:params) do
+        {
+          product: {
+            name: 'Updated Prodouct',
+            shipping_category_id: shipping_category.id,
+            price: 12.34
+          }
+        }
+      end
+
+      before { put "/api/v1/products/#{selected_product.id}", params: params }
+
+      context 'as regular user' do
+        it 'cannot update product' do
+          expect(response.status).to eq(401)
+        end
+      end
+
+      context 'as an admin' do
+        sign_in_as_admin!
+
+        it 'can update product' do
+          expect(response.status).to eq(200)
+          expect(json_response['name']).to eq(params[:product][:name])
+          expect(json_response['price']).to eq(params[:product][:price].to_s)
+          expect(json_response['shipping_category_id']).to eq(params[:product][:shipping_category_id])
+        end
+      end
+
+      context 'as a vendor' do
+        let!(:current_api_user) do
+          user = create(:user, vendors: [vendor])
+          user
+        end
+
+        describe 'cannot update product of another vendor' do
+          let(:selected_product) { product_with_other_vendor }
+
+          it do
+            expect(response.status).to eq(404)
+          end
+        end
+
+        describe 'can update product assigned to that vendor' do
+          let(:selected_product) { product_with_vendor_vendor }
+
+          it do
+            expect(response.status).to eq(200)
+          end
+        end
+      end
+    end
+
+    describe 'products#destroy' do
+      before { delete "/api/v1/products/#{selected_product.id}" }
+
+      context 'as regular user' do
+        it 'cannot delete product' do
+          expect(response.status).to eq(401)
+        end
+      end
+
+      context 'as an admin' do
+        sign_in_as_admin!
+
+        it 'can delete product' do
+          expect(response.status).to eq(204)
+          expect(product_without_vendor.reload.deleted_at).to_not eq(nil)
+        end
+      end
+
+      context 'as a vendor' do
+        let!(:current_api_user) do
+          user = create(:user, vendors: [vendor])
+          user
+        end
+
+        describe 'cannot delete product of another vendor' do
+          let(:selected_product) { product_with_other_vendor }
+
+          it do
+            expect(response.status).to eq(404)
+          end
+        end
+
+        describe 'can delete product assigned to that vendor' do
+          let(:selected_product) { product_with_vendor_vendor }
+
+          it do
+            expect(response.status).to eq(204)
+          end
+        end
+      end
+    end
+  end
+end