spree_multi_vendor 2.0.1

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/v2/storefront/cart_spec.rb

@@ -0,0 +1,67 @@
+require 'spec_helper'
+
+describe 'API V2 Storefront Cart Spec', type: :request do
+  let(:default_currency) { 'USD' }
+  let(:store) { Spree::Store.default }
+  let(:currency) { store.default_currency }
+  let(:user)  { create(:user) }
+  let(:order) { create(:order, user: user, store: store, currency: currency) }
+  let!(:vendor) { create(:vendor) }
+  let!(:vendor_2) { create(:vendor) }
+  let(:product) { create(:product_in_stock, vendor: vendor) }
+  let(:product_2) { create(:product_in_stock, vendor: vendor_2) }
+
+  include_context 'API v2 tokens'
+
+  describe 'cart#show' do
+    let(:params) { {} }
+    let!(:line_item) { create(:line_item, order: order, product: product, variant: product.default_variant) }
+
+    shared_examples 'showing the cart' do
+      before do
+        get '/api/v2/storefront/cart', headers: headers_bearer, params: params
+      end
+
+      it_behaves_like 'returns 200 HTTP status'
+      it_behaves_like 'returns valid cart JSON'
+    end
+
+    context 'includes vendor and totals data' do
+      let(:params) { { include: 'vendors,vendor_totals' } }
+
+      it_behaves_like 'showing the cart'
+
+      it 'includes vendor and vendor totals' do
+        get '/api/v2/storefront/cart', headers: headers_bearer, params: params
+
+        expect(json_response['data']).to have_relationships(:vendors, :vendor_totals)
+        expect(json_response[:included][0]).to have_id(vendor.id.to_s)
+        expect(json_response[:included][0]).to have_type('vendor')
+        expect(json_response[:included][1]).to have_id(vendor.id.to_s)
+        expect(json_response[:included][1]).to have_type('vendor_totals')
+      end
+    end
+
+    context 'includes line item vendor data' do
+      it_behaves_like 'showing the cart'
+
+      it 'via variants' do
+        get '/api/v2/storefront/cart', params: { include: 'variants.vendor' }, headers: headers_bearer
+        expect(json_response[:included][0]).to have_id(vendor.id.to_s)
+        expect(json_response[:included][0]).to have_type('vendor')
+      end
+
+      it 'via products' do
+        get '/api/v2/storefront/cart', params: { include: 'variants.product.vendor' }, headers: headers_bearer
+        expect(json_response[:included][0]).to have_id(vendor.id.to_s)
+        expect(json_response[:included][0]).to have_type('vendor')
+      end
+
+      it 'via line_items' do
+        get '/api/v2/storefront/cart', params: { include: 'line_items.vendor' }, headers: headers_bearer
+        expect(json_response[:included][0]).to have_id(vendor.id.to_s)
+        expect(json_response[:included][0]).to have_type('vendor')
+      end
+    end
+  end
+end

data/spec/requests/spree/api/v2/storefront/products_spec.rb

@@ -0,0 +1,93 @@
+require 'spec_helper'
+
+describe 'API V2 Storefront Products Spec', type: :request do
+  let!(:products)              { create_list(:product, 5) }
+  let!(:vendor)                { create(:vendor) }
+  let!(:vendor_2)              { create(:vendor) }
+
+  before do
+    vendors = Spree::Vendor.all.to_a
+    products.each do |product|
+      product.vendor = vendors.sample
+      product.save!
+    end
+  end
+
+  describe 'products#index' do
+    context 'with no params' do
+      before { get '/api/v2/storefront/products' }
+
+      it_behaves_like 'returns 200 HTTP status'
+
+      it 'returns all products' do
+        expect(json_response['data'].count).to eq Spree::Product.available.count
+        expect(json_response['data'].first).to have_type('product')
+      end
+    end
+
+    context 'with specified vendor ids' do
+      before { get "/api/v2/storefront/products?filter[vendor_ids]=#{vendor.id}&include=vendor" }
+
+      it_behaves_like 'returns 200 HTTP status'
+
+      it 'returns products with specified ids' do
+        expect(json_response['data'].count).to eq Spree::Product.available.where(vendor_id: vendor.id).count
+        expect(json_response['included'].first).to have_id(vendor.id.to_s)
+        expect(json_response['included'].first).to have_attribute(:name).with_value(vendor.name)
+        expect(json_response['included'].first).to have_attribute(:about_us).with_value(vendor.about_us)
+      end
+    end
+
+    context 'with specified multiple filters' do
+      before { get "/api/v2/storefront/products?filter[skus]=#{products.first.sku}&filter[vendor_ids]=#{products.first.vendor_id}&include=vendor" }
+
+      it_behaves_like 'returns 200 HTTP status'
+
+      it 'returns products with specified SKU and Vendor ID' do
+        expect(json_response['data'].count).to eq 1
+        expect(json_response['data'].first).to have_id(products.first.id.to_s)
+        expect(json_response['included'].first).to have_id(products.first.vendor_id.to_s)
+        expect(json_response['included'].first).to have_attribute(:name).with_value(products.first.vendor.name)
+        expect(json_response['included'].first).to have_attribute(:about_us).with_value(products.first.vendor.about_us)
+      end
+    end
+  end
+
+  describe 'products#show' do
+    let(:product) { products.first }
+
+    context 'with existing product' do
+      before { get "/api/v2/storefront/products/#{product.slug}?include=vendor" }
+
+      it_behaves_like 'returns 200 HTTP status'
+
+      it 'returns a valid JSON response' do
+        expect(json_response['data']).to have_id(product.id.to_s)
+
+        expect(json_response['data']).to have_type('product')
+
+        expect(json_response['data']).to have_attribute(:name).with_value(product.name)
+        expect(json_response['data']).to have_attribute(:description).with_value(product.description)
+        expect(json_response['data']).to have_attribute(:price).with_value(product.price.to_s)
+        expect(json_response['data']).to have_attribute(:currency).with_value(product.currency)
+        expect(json_response['data']).to have_attribute(:display_price).with_value(product.display_price.to_s)
+        expect(json_response['data']).to have_attribute(:available_on).with_value(product.available_on.as_json)
+        expect(json_response['data']).to have_attribute(:slug).with_value(product.slug)
+        expect(json_response['data']).to have_attribute(:meta_description).with_value(product.meta_description)
+        expect(json_response['data']).to have_attribute(:meta_keywords).with_value(product.meta_keywords)
+        expect(json_response['data']).to have_attribute(:updated_at).with_value(product.updated_at.as_json)
+        expect(json_response['data']).to have_attribute(:purchasable).with_value(product.purchasable?)
+        expect(json_response['data']).to have_attribute(:in_stock).with_value(product.in_stock?)
+        expect(json_response['data']).to have_attribute(:backorderable).with_value(product.backorderable?)
+
+        expect(json_response['data']).to have_relationships(
+          :variants, :option_types, :product_properties, :default_variant, :vendor
+        )
+
+        expect(json_response['included'].first).to have_id(product.vendor_id.to_s)
+        expect(json_response['included'].first).to have_attribute(:name).with_value(product.vendor.name)
+        expect(json_response['included'].first).to have_attribute(:about_us).with_value(product.vendor.about_us)
+      end
+    end
+  end
+end

data/spec/requests/spree/api/v2/storefront/vendors_spec.rb

@@ -0,0 +1,52 @@
+require 'spec_helper'
+
+describe 'API V2 Storefront Vendor Spec', type: :request do
+
+  let!(:vendor_image) { create(:vendor_image) }
+  let!(:vendor) { create(:active_vendor, name: 'vendor', image: vendor_image) }
+  let!(:product) { create(:product, vendor: vendor) }
+
+  describe 'vendors#show' do
+    context 'with invalid slug param' do
+      before { get '/api/v2/storefront/vendors/vendor1' }
+
+      it_behaves_like 'returns 404 HTTP status'
+    end
+
+    context 'with valid slug param' do
+      before { get "/api/v2/storefront/vendors/#{vendor.slug}" }
+
+      it_behaves_like 'returns 200 HTTP status'
+
+      it 'returns one vendor' do
+        json_response = JSON.parse(response.body)
+        expect(json_response.count).to eq 1
+        expect(json_response['data']['type']).to eq('vendor')
+      end
+
+      it 'does not return included' do
+        json_response = JSON.parse(response.body)
+        expect(json_response.keys).to contain_exactly('data')
+      end
+    end
+
+    context 'with products and images included' do
+      before { get "/api/v2/storefront/vendors/#{vendor.slug}?include=products,image" }
+
+      it_behaves_like 'returns 200 HTTP status'
+
+      it 'returns product information' do
+        json_response = JSON.parse(response.body)
+        expect(json_response.keys).to contain_exactly('data', 'included')
+        expect(json_response['included'].first['id']).to eq(product.id.to_s)
+      end
+
+      it 'returns image information' do
+        json_response = JSON.parse(response.body)
+        expect(json_response.keys).to contain_exactly('data', 'included')
+
+        expect(json_response['included'].second['id']).to eq(vendor_image.id.to_s)
+      end
+    end
+  end
+end