atkhayar-dev-spree-multi-vendor 0.0.0

data/spec/factories/spree/vendor_image_factory.rb

@@ -0,0 +1,5 @@
+FactoryBot.define do
+  factory :vendor_image, class: Spree::VendorImage do
+    attachment { Rack::Test::UploadedFile.new(SpreeMultiVendor::Engine.root.join('spec', 'fixtures', 'thinking-cat.jpg'), 'image/jpeg') }
+  end
+end

data/spec/factories/spree/vendor_user_factory.rb

@@ -0,0 +1,6 @@
+FactoryBot.define do
+  factory :vendor_user, class: Spree::Vendor do
+    user
+    vendor
+  end
+end

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

@@ -0,0 +1,86 @@
+require 'spec_helper'
+
+RSpec.feature 'Admin Orders', :js do
+  let(:store) { Spree::Store.default }
+  let(:vendor) { create(:active_vendor, name: 'Vendor') }
+  let(:vendor_2) { create(:active_vendor, name: 'Other vendor') }
+  let(:order) { create(:order, ship_address: create(:address), bill_address: create(:address), store: store) }
+  let(:order_2) { create(:order, store: store) }
+  let!(:user) { create(:user, vendors: [vendor]) }
+  let(:product) { create(:product_in_stock, vendor: vendor, stores:[store]) }
+  let(:product_2) { create(:product_in_stock, vendor: vendor_2,stores:[store]) }
+
+  before do
+    create(:line_item, order: order, product: product)
+    create(:line_item, order: order, product: product_2)
+    order.create_proposed_shipments
+    order.update(state: :complete, completed_at: Time.current)
+    login_as(user, scope: :spree_user)
+  end
+
+  shared_examples 'shows order data' do
+    it { expect(page).to have_text(order.number) }
+    it { expect(page).to have_content(order.display_vendor_total(vendor).to_html) }
+  end
+
+  context 'index' do
+    background do
+      visit spree.admin_orders_path
+    end
+
+    it_behaves_like 'shows order data'
+
+    it 'does not show non-vendor orders' do
+      expect(page).not_to have_text(order_2.number)
+    end
+  end
+
+  context 'cart' do
+    background do
+      visit spree.cart_admin_order_path(id: order.number)
+    end
+
+    it_behaves_like 'shows order data'
+  end
+
+  context 'customer info' do
+    background do
+      visit spree.admin_order_customer_path(order_id: order.number)
+    end
+
+    it_behaves_like 'shows order data'
+
+    it 'shows customer information' do
+      expect(page).to have_text(order.ship_address.full_name)
+      expect(page).to have_text(order.ship_address.address1)
+      expect(page).to have_text(order.ship_address.city)
+
+      expect(page).to have_text(order.bill_address.full_name)
+      expect(page).to have_text(order.bill_address.address1)
+      expect(page).to have_text(order.bill_address.city)
+
+      expect(page).not_to have_selector('input[type=text]')
+    end
+  end
+
+  context 'edit' do
+    background do
+      visit spree.edit_admin_order_path(id: order.number)
+    end
+
+    it_behaves_like 'shows order data'
+
+    scenario 'vendor can add his product' do
+      select2_open label: 'Name or SKU'
+      select2_search product.name, from: 'Name or SKU'
+      select2_select product.name, from: 'Name or SKU', match: :first
+      expect(page).to have_text(product.name)
+    end
+
+    scenario 'vendor cannot add other vendors product' do
+      select2_open label: 'Name or SKU'
+      select2_search product_2.name, from: 'Name or SKU'
+      expect(page).not_to have_text(product_2.name)
+    end
+  end
+end

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

@@ -0,0 +1,177 @@
+require 'spec_helper'
+
+RSpec.feature 'Admin Products', :js do
+  let(:vendor) { create(:active_vendor) }
+  let!(:user) { create(:user, vendors: [vendor]) }
+  let!(:admin) { create(:admin_user) }
+  let!(:option_type) { create(:option_type, name: 'Testing option') }
+  let!(:option_value) { create(:option_value, option_type: option_type) }
+  let!(:product) { create(:product, sku: 'Test1') }
+  let!(:vendor_product) { create(:product, vendor: vendor, sku: 'Test2', option_types: [option_type]) }
+
+  context 'for user with admin role' do
+    before { login_as(admin, scope: :spree_user) }
+
+    context 'index' do
+      scenario 'displays all products' do
+        visit spree.admin_products_path
+        expect(page).to have_selector('tr', count: 3)
+      end
+    end
+
+    context 'create product' do
+      scenario 'creates new product with vendor id unassigned' do
+        visit spree.admin_products_path
+        click_link 'New Product'
+        expect(current_path).to eq spree.new_admin_product_path
+
+        fill_in 'product_name', with: 'Vendor product'
+        fill_in 'product_price', with: 15
+        select2 Spree::ShippingCategory.last.name, from: 'Shipping Categories'
+
+        click_button 'Create'
+
+        expect(page).to have_text 'successfully created!'
+        expect(current_path).to eq spree.edit_admin_product_path(Spree::Product.last)
+        expect(Spree::Product.last.vendor_id).to eq nil
+      end
+      scenario 'creates new product with assigned vendor' do
+        visit spree.admin_products_path
+        click_link 'New Product'
+        expect(current_path).to eq spree.new_admin_product_path
+
+        fill_in 'product_name', with: 'Vendor product'
+        fill_in 'product_price', with: 15
+        select2 Spree::ShippingCategory.last.name, from: 'Shipping Categories'
+        select2 'Active vendor', from: 'Vendor'
+
+        click_button 'Create'
+
+        expect(page).to have_text 'successfully created!'
+        expect(current_path).to eq spree.edit_admin_product_path(Spree::Product.last)
+        expect(Spree::Product.last.vendor_id).to eq Spree::Vendor.last.id
+      end
+    end
+
+    context 'edit product' do
+      before(:each) do
+        visit spree.edit_admin_product_path(product)
+        expect(current_path).to eq spree.edit_admin_product_path(product)
+      end
+
+      scenario 'can update an existing product' do
+        fill_in 'product_name', with: 'Testing edit'
+        click_button 'Update'
+        expect(page).to have_text 'successfully updated!'
+        expect(page).to have_text 'Testing edit'
+      end
+
+      scenario 'can update product master price' do
+        fill_in 'product_price', with: 123
+        click_button 'Update'
+        expect(page).to have_text 'successfully updated!'
+        product.reload
+        expect(product.price).to eq 123
+      end
+
+      scenario 'can update product vendor' do
+        expect(product.vendor).to eq nil
+        select2 vendor.name, from: 'Vendor'
+        click_button 'Update'
+        expect(page).to have_text 'successfully updated!'
+        product.reload
+        expect(product.vendor).to eq vendor
+      end
+    end
+
+    context 'create variant' do
+      scenario 'creates new variant with vendor id assigned' do
+        visit spree.admin_product_variants_path(vendor_product)
+        click_link 'New Variant'
+        select2 'S', from: 'Size'
+        click_button 'Create'
+        expect(page).to have_text 'successfully created!'
+        expect(Spree::Variant.last.vendor_id).to eq vendor.id
+      end
+    end
+  end
+
+  context 'for user with vendor' do
+    before(:each) do
+      login_as(user, scope: :spree_user)
+      visit spree.admin_products_path
+    end
+
+    context 'index' do
+      scenario 'displays only vendor product' do
+        expect(page).to have_selector('tr', count: 2)
+      end
+    end
+
+    context 'create product' do
+      scenario 'can create a new product' do
+        click_link 'New Product'
+        expect(current_path).to eq spree.new_admin_product_path
+
+        fill_in 'product_name', with: 'Vendor product'
+        fill_in 'product_price', with: 15
+        select2 Spree::ShippingCategory.last.name, from: 'Shipping Categories'
+
+        click_button 'Create'
+
+        expect(page).to have_text 'successfully created!'
+        expect(current_path).to eq spree.edit_admin_product_path(Spree::Product.last)
+        expect(Spree::Product.last.vendor_id).to eq vendor.id
+      end
+    end
+
+    context 'edit product' do
+      before(:each) do
+        within_row(1) { click_icon :edit }
+        expect(current_path).to eq spree.edit_admin_product_path(vendor_product)
+      end
+
+      scenario 'can update an existing product' do
+        fill_in 'product_name', with: 'Testing edit'
+        click_button 'Update'
+        expect(page).to have_text 'successfully updated!'
+        expect(page).to have_text 'Testing edit'
+      end
+
+      scenario 'can update product master price' do
+        fill_in 'product_price', with: 123
+        click_button 'Update'
+        expect(page).to have_text 'successfully updated!'
+        vendor_product.reload
+        expect(vendor_product.price).to eq 123
+      end
+
+      scenario 'shows validation error with blank name' do
+        fill_in 'product_name', with: ''
+        click_button 'Update'
+        expect(page).to have_text 'Name can\'t be blank'
+      end
+    end
+
+    context 'create product property' do
+      scenario 'can create new product property' do
+        visit spree.admin_product_product_properties_path(vendor_product)
+        fill_in 'product[product_properties_attributes][0][property_name]', with: 'Testing edit'
+        click_button 'Update'
+        expect(page).to have_text 'successfully updated!'
+        # expect(Spree::ProductProperty.last.property.vendor_id).to eq vendor.id
+      end
+    end
+
+    context 'create variant' do
+      scenario 'can create new variant' do
+        visit spree.admin_product_variants_path(vendor_product)
+        click_link 'New Variant'
+        select2 'S', from: 'Size'
+        click_button 'Create'
+        expect(page).to have_text 'successfully created!'
+        expect(Spree::Variant.last.option_values).to include option_value
+      end
+    end
+  end
+end

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

@@ -0,0 +1,72 @@
+require 'spec_helper'
+
+RSpec.feature 'Admin Shipping Methods', :js do
+  let(:active_vendor) { create(:active_vendor) }
+  let!(:user) { create(:user, vendors: [active_vendor]) }
+  let!(:admin) { create(:admin_user) }
+  let!(:shipping_method) { create(:shipping_method, name: 'Test') }
+  let!(:vendor_shipping_method) { create(:shipping_method, name: 'Test', vendor: active_vendor) }
+
+  context 'for user with admin role' do
+    context 'index' do
+      scenario 'displays all shipping methods' do
+        login_as(admin, scope: :spree_user)
+        visit spree.admin_shipping_methods_path
+        expect(page).to have_selector('tr', count: 3)
+      end
+    end
+  end
+
+  context 'for user with vendor' do
+    before(:each) do
+      login_as(user, scope: :spree_user)
+      visit spree.admin_shipping_methods_path
+    end
+
+    context 'index' do
+      scenario 'displays only vendor shipping method' do
+        expect(page).to have_selector('tr', count: 2)
+      end
+    end
+
+    context 'create' do
+      scenario 'can create a new shipping method' do
+        click_link 'New Shipping Method'
+        expect(current_path).to eq spree.new_admin_shipping_method_path
+
+        fill_in 'shipping_method_name', with: 'Vendor shipping method'
+        check Spree::ShippingCategory.last.name
+
+        if Spree.version.to_f >= 4.0
+          select2 'Both', from: 'Display'
+        end
+
+        click_button 'Create'
+
+        expect(page).to have_text 'successfully created!'
+        expect(current_path).to eq spree.edit_admin_shipping_method_path(Spree::ShippingMethod.last)
+        expect(Spree::ShippingMethod.last.vendor_id).to eq active_vendor.id
+      end
+    end
+
+    context 'edit' do
+      before(:each) do
+        within_row(1) { click_icon :edit }
+        expect(current_path).to eq spree.edit_admin_shipping_method_path(active_vendor.shipping_methods.first)
+      end
+
+      scenario 'can update an existing shipping method' do
+        fill_in 'shipping_method_name', with: 'Testing edit'
+        click_button 'Update'
+        expect(page).to have_text 'successfully updated!'
+        expect(page).to have_text 'Testing edit'
+      end
+
+      scenario 'shows validation error with blank name' do
+        fill_in 'shipping_method_name', with: ''
+        click_button 'Update'
+        expect(page).to_not have_text 'successfully created!'
+      end
+    end
+  end
+end

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

@@ -0,0 +1,90 @@
+require 'spec_helper'
+
+RSpec.feature 'Admin Stock Locations', :js do
+  let(:active_vendor) { create(:active_vendor) }
+  let!(:product) { create(:product, vendor_id: active_vendor.id, name: 'Test') }
+  let!(:user) { create(:user, vendors: [active_vendor]) }
+  let!(:admin) { create(:admin_user) }
+  let!(:stock_location) { create(:stock_location, name: 'Test') }
+
+  context 'for user with admin role' do
+    context 'index' do
+      scenario 'displays all stock locations' do
+        login_as(admin, scope: :spree_user)
+        visit spree.admin_stock_locations_path
+        expect(page).to have_selector('tr', count: 3)
+      end
+    end
+  end
+
+  context 'for user with vendor' do
+    before(:each) do
+      login_as(user, scope: :spree_user)
+      visit spree.admin_stock_locations_path
+    end
+
+    context 'index' do
+      scenario 'displays only vendor stock location' do
+        expect(page).to have_selector('tr', count: 2)
+      end
+    end
+
+    context 'stock movements' do
+      scenario 'displays stock movements for vendor stock location' do
+        click_on 'Stock Movements'
+        expect(page).to have_text 'Stock Movements for Active vendor'
+      end
+
+      scenario 'can create a new stock movement for vendor stock location' do
+        click_on 'Stock Movements'
+        click_on 'New Stock Movement'
+        expect(current_path).to eq spree.new_admin_stock_location_stock_movement_path(active_vendor.stock_locations.first)
+
+        fill_in 'stock_movement_quantity', with: 5
+        select2_open label: 'Stock Item'
+        select2_search product.name, from: 'Stock Item'
+        select2_select product.name, from: 'Stock Item', match: :first
+
+        click_button 'Create'
+
+        expect(page).to have_text 'successfully created!'
+        expect(Spree::StockMovement.count).to eq 1
+      end
+    end
+
+    context 'create' do
+      scenario 'can create a new stock location' do
+        click_link 'New Stock Location'
+        expect(current_path).to eq spree.new_admin_stock_location_path
+
+        fill_in 'stock_location_name', with: 'Vendor stock location'
+
+        click_button 'Create'
+
+        expect(page).to have_text 'successfully created!'
+        expect(current_path).to eq spree.admin_stock_locations_path
+        expect(Spree::StockLocation.last.vendor_id).to eq active_vendor.id
+      end
+    end
+
+    context 'edit' do
+      before(:each) do
+        within_row(1) { click_icon :edit }
+        expect(current_path).to eq spree.edit_admin_stock_location_path(active_vendor.stock_locations.first)
+      end
+
+      scenario 'can update an existing stock location' do
+        fill_in 'stock_location_name', with: 'Testing edit'
+        click_button 'Update'
+        expect(page).to have_text 'successfully updated!'
+        expect(page).to have_text 'Testing edit'
+      end
+
+      scenario 'shows validation error with blank name' do
+        fill_in 'stock_location_name', with: ''
+        click_button 'Update'
+        expect(page).to_not have_text 'successfully created!'
+      end
+    end
+  end
+end

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

@@ -0,0 +1,74 @@
+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
+
+    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
+
+  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.admin_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