solidus_backend 2.10.2 → 2.11.3

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

@@ -1,141 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe "Properties", type: :feature do
-  stub_authorization!
-
-  before(:each) do
-    visit spree.admin_path
-    click_nav "Products"
-  end
-
-  context "Property index" do
-    before do
-      create(:property, name: 'shirt size', presentation: 'size')
-      create(:property, name: 'shirt fit', presentation: 'fit')
-      click_link "Property Types"
-    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_button '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 "Property Types"
-      click_link "new_property_link"
-      within('#new_property') { expect(page).to have_content("New Property Type") }
-
-      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 type" do
-    before(:each) do
-      create(:property)
-      click_link "Property Types"
-      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 https://github.com/spree/spree/issues/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 https://github.com/spree/spree/issues/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('Products')
-      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/stock_management_spec.rb

@@ -1,101 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe "Product Stock", 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_nav "Products"
-      within_row(1) { click_icon :edit }
-      click_link "Product Stock"
-    end
-
-    # Regression test for https://github.com/spree/spree/issues/3304
-    # It is OK to still render the stock page, ensure no errors in this case
-    context "with no stock location" do
-      before do
-        Spree::StockLocation.destroy_all
-        find_by_id('content-header').click_link('Products')
-        within_row(1) do
-          click_icon :edit
-        end
-        click_link "Product Stock"
-      end
-
-      it "renders" do
-        expect(page).to have_content('Productsapache baseball cap')
-        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 24
-      expect(stock_item.stock_movements.count).to eq 1
-      expect(stock_item.stock_movements.first.quantity).to eq 14
-    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 6
-      expect(stock_item.stock_movements.count).to eq 1
-      expect(stock_item.stock_movements.first.quantity).to eq(-4)
-    end
-
-    it "can toggle backorderable", js: true do
-      toggle_backorderable(value: false)
-
-      click_link "Product Stock"
-      within("tr#spree_variant_#{variant.id}") do
-        expect(find(:css, "input[type='checkbox']")).not_to be_checked
-      end
-    end
-
-    def adjust_count_on_hand(count_on_hand)
-      within("tr#spree_variant_#{variant.id}") do
-        find(:css, "input[type='number']").set(count_on_hand)
-        click_icon :check
-      end
-      expect(page).to have_content('Updated Successfully')
-    end
-
-    def toggle_backorderable(value: true)
-      within("tr#spree_variant_#{variant.id}") do
-        find(:css, "input[type='checkbox']").set(value)
-        click_icon :check
-      end
-      expect(page).to have_content('Updated Successfully')
-    end
-
-    context "with stock locations that don't have stock items for variant yet" 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
-        within('.variant-stock-items', text: variant.sku) do
-          fill_in "variant-count-on-hand-#{variant.id}", with: '3'
-          select "Other location", from: "stock_location_id"
-          click_icon(:plus)
-        end
-        expect(page).to have_content('Created successfully')
-      end
-    end
-  end
-end

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

@@ -1,89 +0,0 @@
-# frozen_string_literal: true
-
-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_nav "Products"
-      within_row(1) { click_icon :edit }
-      click_link "Variants"
-      click_on "New Variant"
-      expect(page).to have_field('variant_price', with: "1.99")
-      expect(page).to have_field('variant_cost_price', with: "1.00")
-      expect(page).to have_field('variant_weight', with: "2.50")
-      expect(page).to have_field('variant_height', with: "3.00")
-      expect(page).to have_field('variant_width', with: "1.00")
-      expect(page).to have_field('variant_depth', with: "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
-          stub_spree_preferences(currency: "RUB")
-        end
-
-        let!(:variant) do
-          create(:variant, product: product, price: 19.99)
-        end
-
-        # Regression test for https://github.com/spree/spree/issues/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
-
-  context "editing existent variant" do
-    let!(:variant) { create(:variant, product: product) }
-
-    context "if product has an option type" do
-      let!(:option_type) { create(:option_type) }
-      let!(:option_value) { create(:option_value, option_type: option_type) }
-
-      before do
-        product.option_types << option_type
-        variant.option_values << option_value
-      end
-
-      it "page has a field for editing the option value", js: true do
-        visit spree.edit_admin_product_variant_path(product, variant)
-        expect(page).to have_css("label", text: option_type.presentation)
-        expect(page).to have_select('Size', selected: 'S')
-      end
-    end
-  end
-
-  context "deleting a variant", js: true do
-    let!(:variant) { create(:variant, product: product) }
-    let!(:option_type) { create(:option_type) }
-    let!(:option_value) { create(:option_value, option_type: option_type) }
-
-    it "can delete a variant" do
-      visit spree.admin_product_variants_path(product)
-      within 'tr', text: 'Size: S' do
-        accept_alert do
-          click_icon :trash
-        end
-      end
-
-      expect(page).to have_no_text('Size: S')
-    end
-  end
-end

data/spec/features/admin/promotion_adjustments_spec.rb

@@ -1,286 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe "Promotion Adjustments", type: :feature, js: true do
-  stub_authorization!
-
-  context "creating a new promotion", js: true do
-    before(:each) do
-      visit spree.new_admin_promotion_path
-      expect(page).to have_title("New Promotion - Promotions")
-    end
-
-    it "should allow an admin to create a flat rate discount coupon promo" do
-      fill_in "Name", with: "SAVE SAVE SAVE"
-      fill_in "Promotion Code", with: "order"
-
-      click_button "Create"
-      expect(page).to have_title("SAVE SAVE SAVE - Promotions")
-
-      select "Item Total", from: "Discount Rules"
-      within('#rule_fields') { click_button "Add" }
-
-      find('[id$=_preferred_amount]').set(30)
-      within('#rule_fields') { click_button "Update" }
-
-      select "Create whole-order adjustment", from: "Adjustment type"
-      within('#action_fields') do
-        click_button "Add"
-        select "Flat Rate", from: I18n.t('spree.admin.promotions.actions.calculator_label')
-        fill_in "Amount", with: 5
-      end
-      within('#actions_container') { click_button "Update" }
-      expect(page).to have_text 'successfully updated'
-
-      promotion = Spree::Promotion.find_by(name: "SAVE SAVE SAVE")
-      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: "SAVE SAVE SAVE"
-      fill_in "promotion[usage_limit]", with: "1"
-      fill_in "Promotion Code", with: "single_use"
-
-      click_button "Create"
-      expect(page).to have_title("SAVE SAVE SAVE - Promotions")
-
-      select "Create whole-order adjustment", from: "Adjustment type"
-      within('#action_fields') do
-        click_button "Add"
-        select "Flat Rate", from: I18n.t('spree.admin.promotions.actions.calculator_label')
-        fill_in "Amount", with: "5"
-      end
-      within('#actions_container') { click_button "Update" }
-      expect(page).to have_text 'successfully updated'
-
-      promotion = Spree::Promotion.find_by(name: "SAVE SAVE SAVE")
-      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: "SAVE SAVE SAVE"
-      choose "Apply to all orders"
-      click_button "Create"
-      expect(page).to have_title("SAVE SAVE SAVE - Promotions")
-
-      select "Item Total", from: "Discount Rules"
-      within('#rule_fields') { click_button "Add" }
-
-      find('[id$=_preferred_amount]').set(30)
-      within('#rule_fields') { click_button "Update" }
-
-      select "Create whole-order adjustment", from: "Adjustment type"
-      within('#action_fields') do
-        click_button "Add"
-        select "Flat Percent", from: I18n.t('spree.admin.promotions.actions.calculator_label')
-        fill_in "Flat Percent", with: "10"
-      end
-      within('#actions_container') { click_button "Update" }
-      expect(page).to have_text 'successfully updated'
-
-      promotion = Spree::Promotion.find_by(name: "SAVE SAVE SAVE")
-      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: "SAVE SAVE SAVE"
-      choose "Apply to all orders"
-      click_button "Create"
-      expect(page).to have_title("SAVE SAVE SAVE - Promotions")
-
-      select "Product(s)", from: "Discount Rules"
-      within("#rule_fields") { click_button "Add" }
-      select2_search "RoR Mug", from: "Choose products"
-      within('#rule_fields') { click_button "Update" }
-
-      select "Create per-line-item adjustment", from: "Adjustment type"
-      within('#action_fields') do
-        click_button "Add"
-        select "Percent Per Item", from: I18n.t('spree.admin.promotions.actions.calculator_label')
-        fill_in "Percent", with: "10"
-      end
-      within('#actions_container') { click_button "Update" }
-      expect(page).to have_text 'successfully updated'
-
-      promotion = Spree::Promotion.find_by(name: "SAVE SAVE SAVE")
-      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
-
-    it "should allow an admin to create an automatic promotion with free shipping (no code)" do
-      fill_in "Name", with: "SAVE SAVE SAVE"
-      choose "Apply to all orders"
-      click_button "Create"
-      expect(page).to have_title("SAVE SAVE SAVE - Promotions")
-
-      select "Item Total", from: "Discount Rules"
-      within('#rule_fields') { click_button "Add" }
-      find('[id$=_preferred_amount]').set(30)
-      within('#rule_fields') { click_button "Update" }
-
-      select "Free Shipping", from: "Adjustment type"
-      within('#action_fields') { click_button "Add" }
-      expect(page).to have_content('Makes all shipments for the order free')
-
-      promotion = Spree::Promotion.find_by(name: "SAVE SAVE SAVE")
-      expect(promotion.codes).to be_empty
-      expect(promotion.rules.first).to be_a(Spree::Promotion::Rules::ItemTotal)
-      expect(promotion.actions.first).to be_a(Spree::Promotion::Actions::FreeShipping)
-    end
-
-    it "disables the button at submit", :js do
-      page.execute_script "$('form').submit(function(e) { e.preventDefault()})"
-      fill_in "Name", with: "SAVE SAVE SAVE"
-      choose "Apply to all orders"
-      click_button "Create"
-
-      expect(page).to have_button("Create", disabled: true)
-    end
-
-    it "should allow an admin to create an automatic promotion" do
-      fill_in "Name", with: "SAVE SAVE SAVE"
-      choose "Apply to all orders"
-      click_button "Create"
-      expect(page).to have_title("SAVE SAVE SAVE - Promotions")
-
-      promotion = Spree::Promotion.find_by(name: "SAVE SAVE SAVE")
-      expect(promotion).to be_apply_automatically
-      expect(promotion.path).to be_nil
-      expect(promotion.codes).to be_empty
-      expect(promotion.rules).to be_blank
-    end
-
-    it "should allow an admin to create a promo with generated codes" do
-      fill_in "Name", with: "SAVE SAVE SAVE"
-      choose "Multiple promotion codes"
-      fill_in "Base code", with: "testing"
-      fill_in "Number of codes", with: "10"
-
-      perform_enqueued_jobs {
-        click_button "Create"
-        expect(page).to have_title("SAVE SAVE SAVE - Promotions")
-      }
-
-      promotion = Spree::Promotion.find_by(name: "SAVE SAVE SAVE")
-      expect(promotion.path).to be_nil
-      expect(promotion).not_to be_apply_automatically
-      expect(promotion.rules).to be_blank
-
-      expect(promotion.codes.count).to eq(10)
-    end
-
-    it "ceasing to be eligible for a promotion with item total rule then becoming eligible again" do
-      fill_in "Name", with: "SAVE SAVE SAVE"
-      choose "Apply to all orders"
-      click_button "Create"
-      expect(page).to have_title("SAVE SAVE SAVE - Promotions")
-
-      select "Item Total", from: "Discount Rules"
-      within('#rule_fields') { click_button "Add" }
-      find('[id$=_preferred_amount]').set(50)
-      within('#rule_fields') { click_button "Update" }
-
-      select "Create whole-order adjustment", from: "Adjustment type"
-      within('#action_fields') do
-        click_button "Add"
-        select "Flat Rate", from: I18n.t('spree.admin.promotions.actions.calculator_label')
-        fill_in "Amount", with: "5"
-      end
-      within('#actions_container') { click_button "Update" }
-      expect(page).to have_text 'successfully updated'
-
-      promotion = Spree::Promotion.find_by(name: "SAVE SAVE SAVE")
-
-      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
-
-    context 'creating a promotion with promotion action that has a calculator with complex preferences' do
-      before do
-        class ComplexCalculator < Spree::Calculator
-          preference :amount, :decimal
-          preference :currency, :string
-          preference :mapping, :hash
-          preference :list, :array
-
-          def self.description
-            "Complex Calculator"
-          end
-        end
-        @calculators = Rails.application.config.spree.calculators.promotion_actions_create_item_adjustments
-        Rails.application.config.spree.calculators.promotion_actions_create_item_adjustments = [ComplexCalculator]
-      end
-
-      after do
-        Rails.application.config.spree.calculators.promotion_actions_create_item_adjustments = @calculators
-      end
-
-      it "does not show array and hash form fields" do
-        fill_in "Name", with: "SAVE SAVE SAVE"
-        choose "Apply to all orders"
-        click_button "Create"
-        expect(page).to have_title("SAVE SAVE SAVE - Promotions")
-
-        select "Create per-line-item adjustment", from: "Adjustment type"
-        within('#action_fields') do
-          click_button "Add"
-          select "Complex Calculator", from: I18n.t('spree.admin.promotions.actions.calculator_label')
-        end
-        within('#actions_container') { click_button "Update" }
-        expect(page).to have_text 'successfully updated'
-
-        within('#action_fields') do
-          expect(page).to have_field('Amount')
-          expect(page).to have_field('Currency')
-          expect(page).to_not have_field('Mapping')
-          expect(page).to_not have_field('List')
-        end
-      end
-    end
-  end
-end