spree_marketplace 3.1.0

data/lib/spree_marketplace/factories.rb

@@ -0,0 +1,29 @@
+require 'spree_drop_ship/factories'
+
+FactoryGirl.modify do
+  factory :supplier, :class => Spree::Supplier do
+    sequence(:name) { |i| "Big Store #{i}" }
+    email { Faker::Internet.email }
+    first_name 'First'
+    last_name 'Last'
+    url "http://example.com"
+    address
+  end
+end
+
+FactoryGirl.define do
+  # Define your Spree extensions Factories within this file to enable applications, and other extensions to use and override them.
+  #
+  # Example adding this to your spec_helper will load these Factories for use:
+  # require 'spree_marketplace/factories'
+
+  factory :supplier_bank_account, class: Spree::SupplierBankAccount do
+    supplier
+    # Details sent to Balanced.
+    name 'John Doe'
+    account_number '9900000001'
+    routing_number '121000358'
+    type 'checking'
+  end
+
+end

data/script/rails

@@ -0,0 +1,7 @@
+# This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
+
+ENGINE_ROOT = File.expand_path('../..', __FILE__)
+ENGINE_PATH = File.expand_path('../../lib/spree_marketplace/engine', __FILE__)
+
+require 'rails/all'
+require 'rails/engine/commands'

data/spec/features/admin/bank_accounts_spec.rb

@@ -0,0 +1,50 @@
+require 'spec_helper'
+
+feature 'Admin - Supplier Bank Accounts', js: true do
+
+  before do
+    country = create(:country, name: "United States")
+    create(:state, name: "Vermont", country: country)
+  end
+
+  context 'w/Stripe' do
+
+    before do
+      SpreeMarketplace::Config[:stripe_publishable_key] = 'pk_test_DIeu8D1IIjLId2xA02TEJthB'
+      SpreeMarketplace::Config[:stripe_secret_key] = 'sk_test_2K6AkjxetfMFks2xBoGKB6wy'
+    end
+
+    context 'as a Supplier' do
+
+      before do
+        login_user create(:supplier_user)
+        visit spree.account_path
+        within 'dd.supplier-info' do
+          click_link 'Edit'
+        end
+      end
+
+      scenario 'adding bank accounts should work' do
+        click_link 'Add Bank Account'
+        fill_in 'supplier_bank_account[account_number]', with: '000123456789'
+        fill_in 'supplier_bank_account[routing_number]', with: '110000000'
+        select2 'United States', from: 'Country'
+        click_button 'Save'
+        page.should have_content('xxxxxx6789')
+        page.should have_content('Supplier bank account "STRIPE TEST BANK" has been successfully created!')
+      end
+
+      scenario 'adding invalid bank accounts should display error' do
+        click_link 'Add Bank Account'
+        fill_in 'supplier_bank_account[account_number]', with: '111111111111'
+        fill_in 'supplier_bank_account[routing_number]', with: '110000000'
+        select2 'United States', from: 'Country'
+        click_button 'Save'
+        page.should have_content('Must only use a test bank account number when making transfers in test mode')
+      end
+
+    end
+
+  end
+
+end

data/spec/features/admin/products_spec.rb

@@ -0,0 +1,225 @@
+require 'spec_helper'
+
+describe 'Admin - Products', js: true do
+
+  context 'as Admin' do
+
+    xit 'should be able to change supplier' do
+      s1 = create(:supplier)
+      s2 = create(:supplier)
+      product = create :product
+      product.add_supplier! s1
+
+      login_user create(:admin_user)
+      visit spree.admin_product_path(product)
+
+      select2 s2.name, from: 'Supplier'
+      click_button 'Update'
+
+      expect(page).to have_content("Product \"#{product.name}\" has been successfully updated!")
+      expect(product.reload.suppliers.first.id).to eql(s2.id)
+    end
+
+    it 'should display all products' do
+      product1 = create :product
+      product2 = create :product
+      product2.add_supplier! create(:supplier)
+      product2.reload
+      product3 = create :product
+      product3.add_supplier! create(:supplier)
+      product3.reload
+
+      login_user create(:admin_user)
+      visit spree.admin_products_path
+
+      page.should have_content(product1.name)
+      page.should have_content(product2.name)
+      page.should have_content(product3.name)
+    end
+
+  end
+
+  context 'as Supplier' do
+
+    before do
+      @supplier_user = create(:supplier_user)
+      @supplier = @supplier_user.supplier
+      login_user @supplier_user
+    end
+
+    context "searching products works with only suppliers in results" do
+      it "should be able to search deleted products", :js => true do
+        p1 = create(:product, :name => 'apache baseball cap', :deleted_at => "2011-01-06 18:21:13")
+        p1.add_supplier! @supplier
+        p2 = create(:product, :name => 'zomg shirt')
+        p2.add_supplier! @supplier
+        create(:product, :name => 'apache baseball bat', :deleted_at => "2011-01-06 18:21:13")
+        create(:product, :name => 'zomg bat')
+
+        visit spree.admin_products_path
+        page.should have_content("zomg shirt")
+        page.should_not have_content("apache baseball cap")
+        page.should_not have_content("apache baseball bat")
+        page.should_not have_content("zomg bat")
+        check "Show Deleted"
+        click_icon :search
+        page.should have_content("zomg shirt")
+        page.should have_content("apache baseball cap")
+        page.should_not have_content("apache baseball bat")
+        page.should_not have_content("zomg bat")
+        uncheck "Show Deleted"
+        click_icon :search
+        page.should have_content("zomg shirt")
+        page.should_not have_content("apache baseball cap")
+        page.should_not have_content("apache baseball bat")
+        page.should_not have_content("zomg bat")
+      end
+
+      it "should be able to search products by their properties with only suppliers in results" do
+        p1 = create(:product, :name => 'apache baseball cap', :sku => "A100")
+        p1.add_supplier! @supplier
+        create(:product, :name => 'apache baseball cap2', :sku => "B100")
+        create(:product, :name => 'zomg shirt')
+        p2 = create(:product, :name => 'zomg skirt')
+        p2.add_supplier! @supplier
+
+
+        visit spree.admin_products_path
+        fill_in "q_name_cont", :with => "ap"
+        click_icon :search
+        page.should have_content("apache baseball cap")
+        page.should_not have_content("apache baseball cap2")
+        page.should_not have_content("zomg shirt")
+        page.should_not have_content("zomg skirt")
+
+        fill_in "q_variants_including_master_sku_cont", :with => "A1"
+        click_icon :search
+        page.should have_content("apache baseball cap")
+        page.should_not have_content("apache baseball cap2")
+        page.should_not have_content("zomg shirt")
+        page.should_not have_content("zomg skirt")
+      end
+    end
+
+    context "creating a new product from a prototype" do
+      def build_option_type_with_values(name, values)
+        ot = FactoryGirl.create(:option_type, :name => name)
+        values.each do |val|
+          ot.option_values.create(:name => val.downcase, :presentation => val)
+        end
+        ot
+      end
+
+      let(:prototype) do
+        size = build_option_type_with_values("size", %w(Small Medium Large))
+        create(:prototype, :name => "Size", :option_types => [ size ])
+      end
+
+      before do
+        @option_type_prototype = prototype
+        @property_prototype = create(:prototype, :name => "Random")
+        @shipping_category = create(:shipping_category)
+        visit spree.admin_products_path
+        click_link "admin_new_product"
+        within('#new_product') do
+          page.should have_content("SKU")
+        end
+      end
+
+      it "should allow an supplier to create a new product and variants from a prototype" do
+        fill_in "product_name", :with => "Baseball Cap"
+        fill_in "product_sku", :with => "B100"
+        fill_in "product_price", :with => "100"
+        fill_in "product_available_on", :with => "2012/01/24"
+        select "Size", :from => "Prototype"
+        check "Large"
+        select @shipping_category.name, from: "product_shipping_category_id"
+        click_button "Create"
+        page.should have_content("successfully created!")
+        Spree::Product.last.variants.length.should == 1
+        # Check suppliers assigned properly
+        expect(Spree::Product.last.supplier_ids).to include(@supplier.id)
+      end
+
+    end
+
+    context "creating a new product" do
+
+      before do
+        @shipping_category = create(:shipping_category)
+      end
+
+      it "should allow an supplier to create a new product" do
+        visit spree.admin_products_path
+        click_link "admin_new_product"
+        within('#new_product') do
+          page.should have_content("SKU")
+        end
+        fill_in "product_name", :with => "Baseball Cap"
+        fill_in "product_sku", :with => "B100"
+        fill_in "product_price", :with => "100"
+        fill_in "product_available_on", :with => "2012/01/24"
+        select @shipping_category.name, from: "product_shipping_category_id"
+        click_button "Create"
+        page.should have_content("successfully created!")
+        click_button "Update"
+        page.should have_content("successfully updated!")
+        # Check suppliers assigned properly
+        expect(Spree::Product.last.supplier_ids).to include(@supplier.id)
+      end
+
+    end
+
+    context "cloning a product" do
+      it "should allow an supplier to clone a product" do
+        p = create(:product)
+        p.add_supplier! @supplier
+
+        visit spree.admin_products_path
+        within_row(1) do
+          click_icon :copy
+        end
+
+        page.should have_content("Product has been cloned")
+        # Check suppliers assigned properly
+        expect(Spree::Product.last.supplier_ids).to include(@supplier.id)
+      end
+
+      context "cloning a deleted product" do
+        it "should allow an supplier to clone a deleted product" do
+          p = create(:product, name: "apache baseball cap")
+          p.add_supplier! @supplier
+
+          visit spree.admin_products_path
+          check "Show Deleted"
+          click_button "Search"
+
+          page.should have_content("apache baseball cap")
+
+          within_row(1) do
+            click_icon :copy
+          end
+
+          page.should have_content("Product has been cloned")
+          # Check suppliers assigned properly
+          expect(Spree::Product.last.supplier_ids).to include(@supplier.id)
+        end
+      end
+    end
+
+    context 'updating a product' do
+      let(:product) {
+        p = create(:product)
+        p.add_supplier! @supplier
+        p.reload
+      }
+
+      it 'should not display supplier selection' do
+        visit spree.admin_product_path(product)
+        page.should_not have_css('#product_supplier_id')
+      end
+    end
+
+  end
+
+end

data/spec/features/admin/properties_spec.rb

@@ -0,0 +1,91 @@
+require 'spec_helper'
+
+describe "Admin - Properties" do
+
+  context 'as Supplier' do
+
+    before do
+      @supplier_user = create(:supplier_user)
+      @supplier = @supplier_user.supplier
+      login_user @supplier_user
+      visit spree.admin_products_path
+    end
+
+    context "listing product properties" do
+      it "should list the existing product properties" do
+        create(:property, name: 'shirt size', presentation: 'size')
+        create(:property, name: 'shirt fit', presentation: 'fit')
+
+        click_link "Properties"
+        within_row(1) do
+          column_text(1).should == "shirt size"
+          column_text(2).should == "size"
+        end
+        skip 'used to work not sure why not anymore'
+        within_row(2) do
+          column_text(1).should == "shirt fit"
+          column_text(2).should == "fit"
+        end
+      end
+    end
+
+    context "creating a property" do
+      it "should allow an admin to create a new product property", :js => true do
+        skip 'Not sure if we want to allow this yet.'
+        # click_link "Properties"
+        # click_link "new_property_link"
+        # within('#new_property') { page.should have_content("New Property") }
+        #
+        # fill_in "property_name", :with => "color of band"
+        # fill_in "property_presentation", :with => "color"
+        # click_button "Create"
+        # page.should have_content("successfully created!")
+      end
+    end
+
+    # TODO 'Not sure if we want to allow this yet.'
+    # context "editing a property" do
+    #   before(:each) do
+    #     create(:property)
+    #     click_link "Properties"
+    #     within_row(1) { click_icon :edit }
+    #   end
+    #
+    #   it "should allow an admin to edit an existing product property" do
+    #     fill_in "property_name", :with => "model 99"
+    #     click_button "Update"
+    #     page.should have_content("successfully updated!")
+    #     page.should have_content("model 99")
+    #   end
+    #
+    #   it "should show validation errors" do
+    #     fill_in "property_name", :with => ""
+    #     click_button "Update"
+    #     page.should have_content("Name can't be blank")
+    #   end
+    # end
+
+    context "linking a property to a product", js: true do
+      before do
+        skip 'Still need to add proper support for managing products'
+        p = create(:product)
+        p.add_supplier! @supplier
+
+        visit spree.admin_products_path
+        click_icon :edit
+        click_link "Product Properties"
+      end
+
+      # Regression test for #2279
+      it 'should be successful' do
+        expect(page).to have_content('Editing Product')
+        fill_in "product_product_properties_attributes_0_property_name", with: "A Property"
+        fill_in "product_product_properties_attributes_0_value", with: "A Value"
+        click_button "Update"
+        click_link "Product Properties"
+        expect(page).to have_css("tbody#product_properties")
+        expect(all("tbody#product_properties tr").count).to eq 2
+      end
+    end
+  end
+end

data/spec/features/admin/relations_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+feature 'Admin - Product Relation Management', js: true do
+
+  before do
+    Spree::RelationType.create(:name => "Related Products", :applies_to => "Spree::Product")
+    @user = create(:supplier_user)
+    @supplier1 = @user.supplier
+    @supplier2 = create(:supplier)
+    @product1 = create :product
+    @product1.add_supplier! @supplier1
+    # TODO shoudl we allow them to relate to anyones product or only their own?
+    @product2 = create :product
+  end
+
+  context 'as Supplier' do
+    scenario 'should be able to add relations' do
+      skip 'for some reason targetted_select2_search is not working properly'
+      login_user @user
+      visit spree.related_admin_product_path(@product1)
+      targetted_select2_search @product2.name, from: 'Name or SKU'
+      select2 @product2.name, from: 'Name or SKU'
+      click_link 'Add'
+      wait_for_ajax
+      within '#products-table-wrapper' do
+        page.should have_content(@product2.name)
+      end
+    end
+  end
+
+end

data/spec/features/admin/return_authorizations_spec.rb

@@ -0,0 +1,7 @@
+require 'spec_helper'
+
+describe "Admin - Return Authorizations" do
+
+  skip 'TODO: need to write, but should likely wait until after https://github.com/spree/spree/issues/4026'
+
+end