solidus_backend 1.0.2 → 1.0.3

data/spec/features/admin/products/properties_spec.rb

@@ -0,0 +1,139 @@
+require 'spec_helper'
+
+describe "Properties", :type => :feature do
+  stub_authorization!
+
+  before(:each) do
+    visit spree.admin_path
+    click_link "Products"
+  end
+
+  context "Property index" do
+    before do
+      create(:property, :name => 'shirt size', :presentation => 'size')
+      create(:property, :name => 'shirt fit', :presentation => 'fit')
+      click_link "Properties"
+    end
+
+    context "listing product properties" do
+      it "should list the existing product properties" do
+        within_row(1) do
+          expect(column_text(1)).to eq("shirt size")
+          expect(column_text(2)).to eq("size")
+        end
+
+        within_row(2) do
+          expect(column_text(1)).to eq("shirt fit")
+          expect(column_text(2)).to eq("fit")
+        end
+      end
+    end
+
+    context "searching properties" do
+      it 'should list properties matching search query', :js => true do
+        fill_in "q_name_cont", :with => "size"
+        click_icon :search
+
+        expect(page).to have_content("shirt size")
+        expect(page).not_to have_content("shirt fit")
+      end
+    end
+  end
+
+  context "creating a property" do
+    it "should allow an admin to create a new product property", :js => true do
+      click_link "Properties"
+      click_link "new_property_link"
+      within('#new_property') { expect(page).to have_content("NEW PROPERTY") }
+
+      fill_in "property_name", :with => "color of band"
+      fill_in "property_presentation", :with => "color"
+      click_button "Create"
+      expect(page).to have_content("successfully created!")
+    end
+  end
+
+  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"
+      expect(page).to have_content("successfully updated!")
+      expect(page).to have_content("model 99")
+    end
+
+    it "should show validation errors" do
+      fill_in "property_name", :with => ""
+      click_button "Update"
+      expect(page).to have_content("Name can't be blank")
+    end
+  end
+
+  context "linking a property to a product", :js => true do
+    before do
+      create(:product)
+      visit spree.admin_products_path
+      click_icon :edit
+      click_link "Product Properties"
+    end
+
+    # Regression test for #2279
+    it "successfully create and then remove product property" do
+      fill_in_property
+
+      check_property_row_count(2)
+
+      delete_product_property
+
+      check_property_row_count(1)
+    end
+
+    # Regression test for #4466
+    it "successfully remove and create a product property at the same time" do
+      fill_in_property
+
+      expect(page).to have_css('tr.product_property', count: 2)
+
+      within '#spree_new_product_property' do
+        find('[id$=_property_name]').set("New Property")
+        find('[id$=_value]').set("New Value")
+      end
+
+      delete_product_property
+
+      # Give fadeOut time to complete
+      expect(page).to have_css('tr.product_property', count: 1)
+
+      click_button "Update"
+
+      expect(page).not_to have_content("Product is not found")
+
+      check_property_row_count(2)
+    end
+
+    def fill_in_property
+      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"
+    end
+
+    def delete_product_property
+      accept_alert do
+        click_icon :trash
+      end
+      expect(page).to have_content 'successfully removed'
+    end
+
+    def check_property_row_count(expected_row_count)
+      click_link "Product Properties"
+      expect(page).to have_css("tbody#product_properties tr", count: expected_row_count)
+    end
+  end
+end

data/spec/features/admin/products/prototypes_spec.rb

@@ -0,0 +1,110 @@
+require 'spec_helper'
+
+describe "Prototypes", :type => :feature do
+  stub_authorization!
+
+  context "listing prototypes" do
+    it "should be able to list existing prototypes" do
+      create(:property, name: "model", presentation: "Model")
+      create(:property, name: "brand", presentation: "Brand")
+      create(:property, name: "shirt_fabric", presentation: "Fabric")
+      create(:property, name: "shirt_sleeve_length", presentation: "Sleeve")
+      create(:property, name: "mug_type", presentation: "Type")
+      create(:property, name: "bag_type", presentation: "Type")
+      create(:property, name: "manufacturer", presentation: "Manufacturer")
+      create(:property, name: "bag_size", presentation: "Size")
+      create(:property, name: "mug_size", presentation: "Size")
+      create(:property, name: "gender", presentation: "Gender")
+      create(:property, name: "shirt_fit", presentation: "Fit")
+      create(:property, name: "bag_material", presentation: "Material")
+      create(:property, name: "shirt_type", presentation: "Type")
+      p = create(:prototype, :name => "Shirt")
+      %w( brand gender manufacturer model shirt_fabric shirt_fit shirt_sleeve_length shirt_type ).each do |prop|
+        p.properties << Spree::Property.find_by_name(prop)
+      end
+      p = create(:prototype, name: "Mug")
+      %w( mug_size mug_type ).each do |prop|
+        p.properties << Spree::Property.find_by_name(prop)
+      end
+      p = create(:prototype, name: "Bag")
+      %w( bag_type bag_material ).each do |prop|
+        p.properties << Spree::Property.find_by_name(prop)
+      end
+
+      visit spree.admin_path
+      click_link "Products"
+      click_link "Prototypes"
+
+      within_row(1) { expect(column_text(1)).to eq "Shirt" }
+      within_row(2) { expect(column_text(1)).to eq "Mug" }
+      within_row(3) { expect(column_text(1)).to eq "Bag" }
+    end
+  end
+
+  context "creating a prototype" do
+    it "should allow an admin to create a new product prototype", :js => true do
+      visit spree.admin_path
+      click_link "Products"
+      click_link "Prototypes"
+      click_link "new_prototype_link"
+      within('#new_prototype') do
+        expect(page).to have_content("NEW PROTOTYPE")
+      end
+      fill_in "prototype_name", with: "male shirts"
+      click_button "Create"
+      expect(page).to have_content("successfully created!")
+      click_link "Prototypes"
+      within_row(1) { click_icon :edit }
+      fill_in "prototype_name", with: "Shirt 99"
+      click_button "Update"
+      expect(page).to have_content("successfully updated!")
+      expect(page).to have_content("Shirt 99")
+    end
+  end
+
+  context "editing a prototype" do
+    it "should allow to empty its properties" do
+      model_property = create(:property, name: "model", presentation: "Model")
+      brand_property = create(:property, name: "brand", presentation: "Brand")
+
+      shirt_prototype = create(:prototype, name: "Shirt", properties: [])
+      %w( brand model ).each do |prop|
+        shirt_prototype.properties << Spree::Property.find_by_name(prop)
+      end
+
+      visit spree.admin_path
+      click_link "Products"
+      click_link "Prototypes"
+
+      click_on "Edit"
+      property_ids = find_field("prototype_property_ids").value.map(&:to_i)
+      expect(property_ids).to match_array [model_property.id, brand_property.id]
+
+      unselect "Brand", from: "prototype_property_ids"
+      unselect "Model", from: "prototype_property_ids"
+
+      click_button 'Update'
+
+      click_on "Edit"
+
+      expect(find_field("prototype_property_ids").value).to be_empty
+    end
+  end
+
+  it 'should be deletable', js: true do
+    shirt_prototype = create(:prototype, name: "Shirt", properties: [])
+    shirt_prototype.taxons << create(:taxon)
+
+    visit spree.admin_path
+    click_link "Products"
+    click_link "Prototypes"
+
+    within("#spree_prototype_#{shirt_prototype.id}") do
+      page.find('.delete-resource').click
+    end
+
+    page.evaluate_script('window.confirm = function() { return true; }')
+
+    expect(page).to have_content("Prototype \"#{shirt_prototype.name}\" has been successfully removed!")
+  end
+end

data/spec/features/admin/products/stock_management_spec.rb

@@ -0,0 +1,82 @@
+require 'spec_helper'
+
+describe "Stock Management", :type => :feature do
+  stub_authorization!
+
+  before(:each) do
+    visit spree.admin_path
+  end
+
+  context "given a product with a variant and a stock location" do
+    let!(:stock_location) { create(:stock_location, name: 'Default') }
+    let!(:product) { create(:product, name: 'apache baseball cap', price: 10) }
+    let!(:variant) { create(:variant, product: product) }
+    let(:stock_item) { variant.stock_items.find_by(stock_location: stock_location) }
+
+    before do
+      stock_location.stock_item(variant).update_column(:count_on_hand, 10)
+
+      click_link "Products"
+      within_row(1) { click_icon :edit }
+      click_link "Stock Management"
+    end
+
+    # Regression test for #3304
+    # It is OK to still render the stock page, ensure no errors in this case
+    context "with no stock location" do
+      before do
+        @product = create(:product, name: 'apache baseball cap', price: 10)
+        v = @product.variants.create!(sku: 'FOOBAR')
+        Spree::StockLocation.destroy_all
+        click_link "Products"
+        within_row(1) do
+          click_icon :edit
+        end
+        click_link "Stock Management"
+      end
+
+      it "renders" do
+        expect(page).to have_content(Spree.t(:editing_product))
+        expect(page.current_url).to match("admin/products/apache-baseball-cap/stock")
+      end
+    end
+
+    it "can create a positive stock adjustment", js: true do
+      adjust_count_on_hand('14')
+      stock_item.reload
+      expect(stock_item.count_on_hand).to eq 14
+      expect(stock_item.stock_movements.count).to eq 1
+      expect(stock_item.stock_movements.first.quantity).to eq 4
+    end
+
+    it "can create a negative stock adjustment", js: true do
+      adjust_count_on_hand('4')
+      stock_item.reload
+      expect(stock_item.count_on_hand).to eq 4
+      expect(stock_item.stock_movements.count).to eq 1
+      expect(stock_item.stock_movements.first.quantity).to eq -6
+    end
+
+    def adjust_count_on_hand(count_on_hand)
+      find(:css, ".fa-edit[data-id='#{stock_item.id}']").click
+      find(:css, "[data-variant-id='#{variant.id}'] input[type='number']").set(count_on_hand)
+      find(:css, ".fa-check[data-id='#{stock_item.id}']").click
+      expect(page).to have_content('Updated successfully')
+    end
+
+    context "with multiple stock locations" do
+      before do
+        create(:stock_location, name: 'Other location', propagate_all_variants: false)
+      end
+
+      it "can add stock items to other stock locations", js: true do
+        visit current_url
+        fill_in "variant-count-on-hand-#{variant.id}", with: '3'
+        targetted_select2_search "Other location", from: "#s2id_variant-stock-location-#{variant.id}"
+        find(:css, ".fa-plus[data-variant-id='#{variant.id}']").click
+        wait_for_ajax
+        expect(page).to have_content('Created successfully')
+      end
+    end
+  end
+end

data/spec/features/admin/products/variant_spec.rb

@@ -0,0 +1,51 @@
+# encoding: utf-8
+require 'spec_helper'
+
+describe "Variants", :type => :feature do
+  stub_authorization!
+
+  let(:product) { create(:product_with_option_types, :price => "1.99", :cost_price => "1.00", :weight => "2.5", :height => "3.0", :width => "1.0", :depth => "1.5") }
+
+  context "creating a new variant" do
+    it "should allow an admin to create a new variant" do
+      product.options.each do |option|
+        create(:option_value, :option_type => option.option_type)
+      end
+
+      visit spree.admin_path
+      click_link "Products"
+      within_row(1) { click_icon :edit }
+      click_link "Variants"
+      click_on "New Variant"
+      expect(find('input#variant_price').value).to eq("1.99")
+      expect(find('input#variant_cost_price').value).to eq("1.00")
+      expect(find('input#variant_weight').value).to eq("2.50")
+      expect(find('input#variant_height').value).to eq("3.00")
+      expect(find('input#variant_width').value).to eq("1.00")
+      expect(find('input#variant_depth').value).to eq("1.50")
+      expect(page).to have_select('variant[tax_category_id]')
+    end
+  end
+
+  context "listing variants" do
+    context "currency displaying" do
+      context "using Russian Rubles" do
+        before do
+          Spree::Config[:currency] = "RUB"
+        end
+
+        let!(:variant) do
+          create(:variant, :product => product, :price => 19.99)
+        end
+
+        # Regression test for #2737
+        context "uses руб as the currency symbol" do
+          it "on the products listing page" do
+            visit spree.admin_product_variants_path(product)
+            within_row(1) { expect(page).to have_content("19.99 ₽") }
+          end
+        end
+      end
+    end
+  end
+end

data/spec/features/admin/promotion_adjustments_spec.rb

@@ -0,0 +1,220 @@
+require 'spec_helper'
+
+describe "Promotion Adjustments", :type => :feature do
+  stub_authorization!
+
+  context "coupon promotions", :js => true do
+    before(:each) do
+      visit spree.admin_path
+      click_link "Promotions"
+      click_link "New Promotion"
+    end
+
+    it "should allow an admin to create a flat rate discount coupon promo" do
+      fill_in "Name", :with => "Promotion"
+      fill_in "Base code", :with => "order"
+      fill_in "Number of codes", :with => "1"
+      click_button "Create"
+      expect(page).to have_content("Editing Promotion")
+
+      select2 "Item total", :from => "Add rule of type"
+      within('#rule_fields') { click_button "Add" }
+
+      find('[id$=_preferred_amount]').set(30)
+      within('#rule_fields') { click_button "Update" }
+
+      select2 "Create whole-order adjustment", :from => "Add action of type"
+      within('#action_fields') { click_button "Add" }
+      select2 "Flat Rate", :from => "Calculator"
+      within('#actions_container') { click_button "Update" }
+
+      within('.calculator-fields') { fill_in "Amount", :with => 5 }
+      within('#actions_container') { click_button "Update" }
+
+      promotion = Spree::Promotion.find_by_name("Promotion")
+      expect(promotion.codes.first.value).to eq("order")
+
+      first_rule = promotion.rules.first
+      expect(first_rule.class).to eq(Spree::Promotion::Rules::ItemTotal)
+      expect(first_rule.preferred_amount).to eq(30)
+
+      first_action = promotion.actions.first
+      expect(first_action.class).to eq(Spree::Promotion::Actions::CreateAdjustment)
+      first_action_calculator = first_action.calculator
+      expect(first_action_calculator.class).to eq(Spree::Calculator::FlatRate)
+      expect(first_action_calculator.preferred_amount).to eq(5)
+    end
+
+    it "should allow an admin to create a single user coupon promo with flat rate discount" do
+      fill_in "Name", :with => "Promotion"
+      fill_in "promotion[usage_limit]", :with => "1"
+      fill_in "Base code", :with => "single_use"
+      fill_in "Number of codes", :with => "1"
+      click_button "Create"
+      expect(page).to have_content("Editing Promotion")
+
+      select2 "Create whole-order adjustment", :from => "Add action of type"
+      within('#action_fields') { click_button "Add" }
+      select2 "Flat Rate", :from => "Calculator"
+      within('#actions_container') { click_button "Update" }
+      within('#action_fields') { fill_in "Amount", :with => "5" }
+      within('#actions_container') { click_button "Update" }
+
+      promotion = Spree::Promotion.find_by_name("Promotion")
+      expect(promotion.usage_limit).to eq(1)
+      expect(promotion.codes.first.value).to eq("single_use")
+
+      first_action = promotion.actions.first
+      expect(first_action.class).to eq(Spree::Promotion::Actions::CreateAdjustment)
+      first_action_calculator = first_action.calculator
+      expect(first_action_calculator.class).to eq(Spree::Calculator::FlatRate)
+      expect(first_action_calculator.preferred_amount).to eq(5)
+    end
+
+    it "should allow an admin to create an automatic promo with flat percent discount" do
+      fill_in "Name", :with => "Promotion"
+      click_button "Create"
+      expect(page).to have_content("Editing Promotion")
+
+      select2 "Item total", :from => "Add rule of type"
+      within('#rule_fields') { click_button "Add" }
+
+      find('[id$=_preferred_amount]').set(30)
+      within('#rule_fields') { click_button "Update" }
+
+      select2 "Create whole-order adjustment", :from => "Add action of type"
+      within('#action_fields') { click_button "Add" }
+      select2 "Flat Percent", :from => "Calculator"
+      within('#actions_container') { click_button "Update" }
+      within('.calculator-fields') { fill_in "Flat Percent", :with => "10" }
+      within('#actions_container') { click_button "Update" }
+
+      promotion = Spree::Promotion.find_by_name("Promotion")
+      expect(promotion.codes.first).to be_nil
+
+      first_rule = promotion.rules.first
+      expect(first_rule.class).to eq(Spree::Promotion::Rules::ItemTotal)
+      expect(first_rule.preferred_amount).to eq(30)
+
+      first_action = promotion.actions.first
+      expect(first_action.class).to eq(Spree::Promotion::Actions::CreateAdjustment)
+      first_action_calculator = first_action.calculator
+      expect(first_action_calculator.class).to eq(Spree::Calculator::FlatPercentItemTotal)
+      expect(first_action_calculator.preferred_flat_percent).to eq(10)
+    end
+
+    it "should allow an admin to create an product promo with percent per item discount" do
+      create(:product, :name => "RoR Mug")
+
+      fill_in "Name", :with => "Promotion"
+      click_button "Create"
+      expect(page).to have_content("Editing Promotion")
+
+      select2 "Product(s)", :from => "Add rule of type"
+      within("#rule_fields") { click_button "Add" }
+      select2_search "RoR Mug", :from => "Choose products"
+      within('#rule_fields') { click_button "Update" }
+
+      select2 "Create per-line-item adjustment", :from => "Add action of type"
+      within('#action_fields') { click_button "Add" }
+      select2 "Percent Per Item", :from => "Calculator"
+      within('#actions_container') { click_button "Update" }
+      within('.calculator-fields') { fill_in "Percent", :with => "10" }
+      within('#actions_container') { click_button "Update" }
+
+      promotion = Spree::Promotion.find_by_name("Promotion")
+      expect(promotion.codes.first).to be_nil
+
+      first_rule = promotion.rules.first
+      expect(first_rule.class).to eq(Spree::Promotion::Rules::Product)
+      expect(first_rule.products.map(&:name)).to include("RoR Mug")
+
+      first_action = promotion.actions.first
+      expect(first_action.class).to eq(Spree::Promotion::Actions::CreateItemAdjustments)
+      first_action_calculator = first_action.calculator
+      expect(first_action_calculator.class).to eq(Spree::Calculator::PercentOnLineItem)
+      expect(first_action_calculator.preferred_percent).to eq(10)
+    end
+
+    xit "should allow an admin to create an automatic promotion with free shipping (no code)" do
+      fill_in "Name", :with => "Promotion"
+      click_button "Create"
+      expect(page).to have_content("Editing Promotion")
+
+      select2 "Item total", :from => "Add rule of type"
+      within('#rule_fields') { click_button "Add" }
+      find('[id$=_preferred_amount]').set(30)
+      within('#rule_fields') { click_button "Update" }
+
+      select2 "Create whole-order adjustment", :from => "Add action of type"
+      within('#action_fields') { click_button "Add" }
+      select2 "Free Shipping", :from => "Calculator"
+      within('#actions_container') { click_button "Update" }
+
+      promotion = Spree::Promotion.find_by_name("Promotion")
+      expect(promotion.code.first.value).to be_blank
+
+      first_rule = promotion.rules.first
+      expect(first_rule.class).to eq(Spree::Promotion::Rules::ItemTotal)
+
+      first_action = promotion.actions.first
+      expect(first_action.class).to eq(Spree::Promotion::Actions::CreateAdjustment)
+      first_action_calculator = first_action.calculator
+      expect(first_action_calculator.class).to eq(Spree::Calculator::FreeShipping)
+    end
+
+    it "should allow an admin to create an automatic promo requiring a landing page to be visited" do
+      fill_in "Name", :with => "Promotion"
+      fill_in "Path", :with => "content/cvv"
+      click_button "Create"
+      expect(page).to have_content("Editing Promotion")
+
+      select2 "Create whole-order adjustment", :from => "Add action of type"
+      within('#action_fields') { click_button "Add" }
+      select2 "Flat Rate", :from => "Calculator"
+      within('#actions_container') { click_button "Update" }
+      within('.calculator-fields') { fill_in "Amount", :with => "4" }
+      within('#actions_container') { click_button "Update" }
+
+      promotion = Spree::Promotion.find_by_name("Promotion")
+      expect(promotion.path).to eq("content/cvv")
+      expect(promotion.codes.first).to be_nil
+      expect(promotion.rules).to be_blank
+
+      first_action = promotion.actions.first
+      expect(first_action.class).to eq(Spree::Promotion::Actions::CreateAdjustment)
+      first_action_calculator = first_action.calculator
+      expect(first_action_calculator.class).to eq(Spree::Calculator::FlatRate)
+      expect(first_action_calculator.preferred_amount).to eq(4)
+    end
+
+    it "ceasing to be eligible for a promotion with item total rule then becoming eligible again" do
+      fill_in "Name", :with => "Promotion"
+      click_button "Create"
+      expect(page).to have_content("Editing Promotion")
+
+      select2 "Item total", :from => "Add rule of type"
+      within('#rule_fields') { click_button "Add" }
+      find('[id$=_preferred_amount]').set(50)
+      within('#rule_fields') { click_button "Update" }
+
+      select2 "Create whole-order adjustment", :from => "Add action of type"
+      within('#action_fields') { click_button "Add" }
+      select2 "Flat Rate", :from => "Calculator"
+      within('#actions_container') { click_button "Update" }
+      within('.calculator-fields') { fill_in "Amount", :with => "5" }
+      within('#actions_container') { click_button "Update" }
+
+      promotion = Spree::Promotion.find_by_name("Promotion")
+
+      first_rule = promotion.rules.first
+      expect(first_rule.class).to eq(Spree::Promotion::Rules::ItemTotal)
+      expect(first_rule.preferred_amount).to eq(50)
+
+      first_action = promotion.actions.first
+      expect(first_action.class).to eq(Spree::Promotion::Actions::CreateAdjustment)
+      expect(first_action.calculator.class).to eq(Spree::Calculator::FlatRate)
+      expect(first_action.calculator.preferred_amount).to eq(5)
+    end
+  end
+end