solidus_backend 2.9.6 → 2.11.0

data/spec/features/admin/promotions/option_value_rule_spec.rb

@@ -1,90 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-feature 'Promotion with option value rule' do
-  stub_authorization!
-
-  given(:variant) { create :variant }
-  given!(:product) { variant.product }
-  given!(:option_value) { variant.option_values.first }
-
-  given(:promotion) { create :promotion }
-
-  background do
-    visit spree.edit_admin_promotion_path(promotion)
-  end
-
-  scenario "adding an option value rule", js: true do
-    select "Option Value(s)", from: "Discount Rules"
-    within("#rules_container") { click_button "Add" }
-
-    within("#rules_container .promotion-block") do
-      expect(page).to have_content("Product")
-      expect(page).to have_content("Option Values")
-
-      click_button "Add"
-    end
-
-    within('.promo-rule-option-value') do
-      targetted_select2_search product.name, from: '.js-promo-rule-option-value-product-select'
-      targetted_select2_search option_value.name, from: '.js-promo-rule-option-value-option-values-select'
-    end
-
-    within('#rules_container') { click_button "Update" }
-
-    expect(page).to have_content("has been successfully updated")
-
-    first_rule = promotion.rules.reload.first
-    expect(first_rule.class).to eq Spree::Promotion::Rules::OptionValue
-    expect(first_rule.preferred_eligible_values).to eq Hash[product.id => [option_value.id]]
-  end
-
-  context "with an attempted XSS" do
-    let(:xss_string) { %(<script>throw("XSS")</script>) }
-    before do
-      option_value.update!(name: xss_string)
-    end
-    scenario "adding an option value rule", js: true do
-      select "Option Value(s)", from: "Discount Rules"
-      within("#rules_container") { click_button "Add" }
-
-      within("#rules_container .promotion-block") do
-        click_button "Add"
-      end
-
-      within('.promo-rule-option-value') do
-        targetted_select2_search product.name, from: '.js-promo-rule-option-value-product-select'
-        targetted_select2_search option_value.name, from: '.js-promo-rule-option-value-option-values-select'
-      end
-    end
-  end
-
-  context "with an existing option value rule" do
-    given(:variant1) { create :variant }
-    given(:variant2) { create :variant }
-    background do
-      rule = Spree::Promotion::Rules::OptionValue.new
-      rule.promotion = promotion
-      rule.preferred_eligible_values = Hash[
-        variant1.product_id => variant1.option_values.pluck(:id),
-        variant2.product_id => variant2.option_values.pluck(:id)
-      ]
-      rule.save!
-
-      visit spree.edit_admin_promotion_path(promotion)
-    end
-
-    scenario "deleting a product", js: true do
-      expect(page).to have_css('.promo-rule-option-value', count: 2)
-      all('.promo-rule-option-value')[1].find('.remove').click
-
-      within('#rules_container') { click_button "Update" }
-
-      expect(page).to have_content("has been successfully updated")
-
-      first_rule = promotion.rules.reload.first
-      expect(first_rule.preferred_eligible_values).to eq Hash[variant1.product_id => variant1.option_values.pluck(:id)]
-    end
-  end
-end

data/spec/features/admin/promotions/product_rule_spec.rb

@@ -1,35 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-feature 'Promotion with product rule', js: true do
-  stub_authorization!
-
-  given!(:product) { create :product, name: "BlamCo Mac & Cheese", sku: "PRODUCT_SKU" }
-  given!(:variant) { create :variant, sku: "VARIANT_SKU", product: product }
-  given!(:option_value) { variant.option_values.first }
-
-  given(:promotion) { create :promotion }
-
-  def add_promotion_rule_of_type(type)
-    select type, from: "Discount Rules"
-    within("#rules_container") { click_button "Add" }
-  end
-
-  background do
-    visit spree.edit_admin_promotion_path(promotion)
-    add_promotion_rule_of_type("Product(s)")
-  end
-
-  it "can select by product sku" do
-    select2_search product.name, from: "Choose products", search: "PRODUCT_SKU"
-  end
-
-  it "can select by variant sku" do
-    select2_search product.name, from: "Choose products", search: "VARIANT_SKU"
-  end
-
-  it "can select by product name" do
-    select2_search product.name, from: "Choose products"
-  end
-end

data/spec/features/admin/promotions/promotion_categories_spec.rb

@@ -1,83 +0,0 @@
-require 'spec_helper'
-
-describe 'Promotion Categories', type: :feature do
-  stub_authorization!
-
-  context "index" do
-    before do
-      create(:promotion_category, name: 'name1', code: 'code1')
-      create(:promotion_category, name: 'name2', code: 'code2')
-      visit spree.admin_promotion_categories_path
-    end
-
-    context "listing promotion categories" do
-      it "should list the existing promotion categories" do
-        within_row(1) do
-          expect(column_text(1)).to eq("name1")
-          expect(column_text(2)).to eq("code1")
-        end
-
-        within_row(2) do
-          expect(column_text(1)).to eq("name2")
-          expect(column_text(2)).to eq("code2")
-        end
-      end
-    end
-  end
-
-  context "create" do
-    before do
-      visit spree.admin_promotion_categories_path
-      click_on "New Promotion Category"
-    end
-
-    it "should allow an admin to create a new promotion category" do
-      fill_in "promotion_category_name", with: "promotion test"
-      fill_in "promotion_category_code", with: "prtest"
-      click_button "Create"
-      expect(page).to have_content("successfully created!")
-    end
-
-    it "should not allow admin to create promotion category when invalid data" do
-      fill_in "promotion_category_name", with: ""
-      fill_in "promotion_category_code", with: "prtest"
-      click_button "Create"
-      expect(page).to have_content("Name can't be blank")
-    end
-  end
-
-  context "edit" do
-    before(:each) do
-      create(:promotion_category, name: 'name1')
-      visit spree.admin_promotion_categories_path
-      within_row(1) { click_icon :edit }
-    end
-
-    it "should allow an admin to edit an existing promotion category" do
-      fill_in "promotion_category_name", with: "name 99"
-      click_button "Update"
-      expect(page).to have_content("successfully updated!")
-      expect(page).to have_content("name 99")
-    end
-
-    it "should show validation errors" do
-      fill_in "promotion_category_name", with: ""
-      click_button "Update"
-      expect(page).to have_content("Name can't be blank")
-    end
-  end
-
-  context "delete" do
-    before(:each) do
-      create(:promotion_category, name: 'name1')
-      visit spree.admin_promotion_categories_path
-    end
-
-    it "should allow an admin to delete an existing promotion category", js: true do
-      accept_alert do
-        click_icon :trash
-      end
-      expect(page).to have_content("successfully removed!")
-    end
-  end
-end

data/spec/features/admin/promotions/promotion_spec.rb

@@ -1,52 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-feature 'Promotions' do
-  stub_authorization!
-
-  context 'index' do
-    context 'when no promotions' do
-      scenario 'shows no promotions found message' do
-        visit spree.admin_promotions_path
-        expect(page).to have_content('No Promotions found.')
-      end
-    end
-
-    context 'when promotion is active' do
-      given!(:promotion) { create :promotion }
-
-      scenario 'promotion status is active' do
-        visit spree.admin_promotions_path
-
-        within_row(1) do
-          expect(column_text(3)).to eq("Active")
-        end
-      end
-    end
-
-    context 'when promotion is in the future' do
-      given!(:promotion) { create :promotion, starts_at: 1.day.after }
-
-      scenario 'promotion status is not started' do
-        visit spree.admin_promotions_path
-
-        within_row(1) do
-          expect(column_text(3)).to eq("Not started")
-        end
-      end
-    end
-
-    context 'when promotion is in the past' do
-      given!(:promotion) { create :promotion, expires_at: 1.day.ago }
-
-      scenario 'promotion status is expired' do
-        visit spree.admin_promotions_path
-
-        within_row(1) do
-          expect(column_text(3)).to eq("Expired")
-        end
-      end
-    end
-  end
-end

data/spec/features/admin/promotions/tiered_calculator_spec.rb

@@ -1,76 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-feature "Tiered Calculator Promotions" do
-  stub_authorization!
-
-  let(:promotion) { create :promotion }
-
-  background do
-    visit spree.edit_admin_promotion_path(promotion)
-  end
-
-  scenario "adding a tiered percent calculator", js: true do
-    select "Create whole-order adjustment", from: "Adjustment type"
-    within('#action_fields') { click_button "Add" }
-
-    select "Tiered Percent", from: I18n.t('spree.admin.promotions.actions.calculator_label')
-    within('#actions_container') { click_button "Update" }
-
-    within("#actions_container .settings") do
-      expect(page).to have_content("Base Percent")
-      expect(page).to have_content("Tiers")
-
-      page.find('a.button').click
-    end
-
-    fill_in "Base Percent", with: 5
-
-    within(".tier") do
-      find("input:last-child").set(100)
-      find("input:first-child").set(10)
-    end
-
-    within('#actions_container') { click_button "Update" }
-
-    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::TieredPercent
-    expect(first_action_calculator.preferred_base_percent).to eq 5
-    expect(first_action_calculator.preferred_tiers).to eq(BigDecimal(100) => BigDecimal(10))
-  end
-
-  context "with an existing tiered flat rate calculator" do
-    let(:promotion) { create :promotion, :with_order_adjustment }
-
-    background do
-      action = promotion.actions.first
-
-      action.calculator = Spree::Calculator::TieredFlatRate.new
-      action.calculator.preferred_base_amount = 5
-      action.calculator.preferred_tiers = { 100 => 10, 200 => 15, 300 => 20 }
-      action.calculator.save!
-
-      visit spree.edit_admin_promotion_path(promotion)
-    end
-
-    scenario "deleting a tier", js: true do
-      within(".tier:nth-child(2)") do
-        find(".remove").click
-      end
-
-      within('#actions_container') { click_button "Update" }
-
-      expect(page).to have_text('has been successfully updated!')
-
-      calculator = promotion.actions.first.calculator
-      expect(calculator.preferred_tiers).to eq({
-        BigDecimal(100) => 10,
-        BigDecimal(300) => 20
-      })
-    end
-  end
-end

data/spec/features/admin/promotions/user_rule_spec.rb

@@ -1,42 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-feature 'Promotion with user rule', js: true do
-  stub_authorization!
-
-  given(:promotion) { create :promotion }
-
-  background do
-    visit spree.edit_admin_promotion_path(promotion)
-  end
-
-  context "multiple users" do
-    let!(:user) { create(:user, email: 'foo@example.com') }
-    let!(:other_user) { create(:user, email: 'bar@example.com') }
-
-    scenario "searching a user" do
-      select "User", from: "Discount Rules"
-      within("#rules_container") { click_button "Add" }
-
-      select2_search "foo", from: "Choose users", select: false
-
-      expect(page).to have_content('foo@example.com')
-      expect(page).not_to have_content('bar@example.com')
-    end
-  end
-
-  context "with an attempted XSS" do
-    let(:xss_string) { %(<script>throw("XSS")</script>) }
-    given!(:user) { create(:user, email: xss_string) }
-
-    scenario "adding an option value rule" do
-      select "User", from: "Discount Rules"
-      within("#rules_container") { click_button "Add" }
-
-      select2_search "<script>", from: "Choose users"
-
-      expect(page).to have_content(xss_string)
-    end
-  end
-end

data/spec/features/admin/reimbursements_spec.rb

@@ -1,18 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe 'Promotions', type: :feature do
-  stub_authorization!
-  let!(:reimbursement) { create(:reimbursement) }
-
-  it "should display the reimbursements table" do
-    visit spree.admin_order_reimbursement_path(reimbursement.order, reimbursement)
-    expect(page).to have_css('table thead tr th', text: 'Product')
-    expect(page).to have_css('table thead tr th', text: 'Preferred Reimbursement Type')
-    expect(page).to have_css('table thead tr th', text: 'Reimbursement Type Override')
-    expect(page).to have_css('table thead tr th', text: 'Exchange For')
-    expect(page).to have_css('table thead tr th', text: 'Amount Before Sales Tax')
-    expect(page).to have_css('table thead tr th', text: 'Total')
-  end
-end

data/spec/features/admin/store_credits_spec.rb

@@ -1,132 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe "Store credits admin" do
-  stub_authorization!
-  let!(:admin_user)   { create(:admin_user) }
-  let!(:store_credit) { create(:store_credit) }
-  let(:user)          { store_credit.user }
-
-  before do
-    allow(Spree.user_class).to receive(:find_by).
-      with(hash_including(:id)).
-      and_return(store_credit.user)
-  end
-
-  describe "visiting the store credits page" do
-    before do
-      visit spree.admin_path
-      click_link "Users"
-    end
-
-    it "should be on the store credits page" do
-      click_link store_credit.user.email
-      click_link "Store Credit"
-      expect(page.current_path).to eq spree.admin_user_store_credits_path(store_credit.user)
-
-      store_credit_table = page.find(".sc-table")
-      expect(store_credit_table).to have_css('tr', count: 1)
-      expect(store_credit_table).to have_content(Spree::Money.new(store_credit.amount).to_s)
-      expect(store_credit_table).to have_content(Spree::Money.new(store_credit.amount_used).to_s)
-      expect(store_credit_table).to have_content(store_credit.category_name)
-      expect(store_credit_table).to have_content(store_credit.created_by_email)
-    end
-  end
-
-  describe "creating store credit" do
-    before do
-      visit spree.admin_path
-      click_link "Users"
-      click_link store_credit.user.email
-      click_link "Store Credit"
-      allow_any_instance_of(Spree::Admin::StoreCreditsController).to receive_messages(try_spree_current_user: admin_user)
-    end
-
-    it "should create store credit and associate it with the user" do
-      click_link "Add store credit"
-      page.fill_in "store_credit_amount", with: "102.00"
-      select "Exchange", from: "store_credit_category_id"
-      click_button "Create"
-
-      expect(page.current_path).to eq spree.admin_user_store_credits_path(store_credit.user)
-      store_credit_table = page.find(".sc-table")
-      expect(store_credit_table).to have_css('tr', count: 2)
-      expect(Spree::StoreCredit.count).to eq 2
-    end
-  end
-
-  describe "displaying a store credit details page" do
-    before do
-      visit spree.admin_path
-      click_link "Users"
-      click_link store_credit.user.email
-      click_link "Store Credit"
-      page.find(".sc-table td.actions a.fa-edit").click
-    end
-
-    it "shows the store credit's information" do
-      within ".content" do
-        expect(page).to have_content "Credited #{store_credit.display_amount}"
-        expect(page).to have_content "Created By #{store_credit.created_by.email}"
-        expect(page).to have_content "Type #{store_credit.category_name}"
-        expect(page).to have_content 'Store credit history'
-      end
-    end
-
-    it "lets edit store credit's memo", js: true do
-      allow_any_instance_of(Spree::Admin::StoreCreditsController)
-        .to receive(:try_spree_current_user)
-        .and_return(admin_user)
-
-      # When there are no errors
-      within '.store-credit-memo-row' do
-        click_button 'Edit'
-        fill_in 'store_credit_memo', with: 'Lottery Won'
-        click_button 'Save'
-        expect(page).to have_content "Memo Lottery Won", normalize_ws: true
-      end
-      expect(page).to have_content "Store Credit has been successfully updated!"
-
-      # When there are errors
-      allow_any_instance_of(Spree::StoreCredit).to receive(:save) { false }
-      allow_any_instance_of(Spree::StoreCredit)
-        .to receive_message_chain(:errors, :full_messages)
-        .and_return(["Memo is not valid"])
-
-      within '.store-credit-memo-row' do
-        find(:css, ".edit-memo").click
-        fill_in 'store_credit_memo', with: 'Lottery Won Twice'
-        find(:css, ".save-memo").click
-        expect(page).to have_content "Memo Lottery Won", normalize_ws: true
-        expect(page).not_to have_content "Lottery Won Twice"
-      end
-      expect(page).to have_content "Unable to update store credit [\"Memo is not valid\"]"
-    end
-  end
-
-  describe "updating store credit" do
-    let(:updated_amount) { "99.0" }
-    let!(:store_credit_reason) { create(:store_credit_reason) }
-
-    before do
-      visit spree.admin_path
-      click_link "Users"
-      click_link store_credit.user.email
-      click_link "Store Credit"
-      allow_any_instance_of(Spree::Admin::StoreCreditsController).to receive_messages(try_spree_current_user: admin_user)
-    end
-
-    it "updates the store credit's amount" do
-      page.find(".sc-table td.actions a.fa-edit").click
-      expect(page).to have_content 'Store credit history'
-      click_link "Change amount"
-      expect(page).to have_content 'Editing store credit amount'
-      page.fill_in 'store_credit_amount', with: updated_amount
-      page.select store_credit_reason.name, from: 'store_credit_reason_id'
-      click_button "Update"
-      expect(page.find('#sc-detail-table')).to have_content "$99.00"
-      expect(store_credit.reload.amount.to_f).to eq updated_amount.to_f
-    end
-  end
-end