spree_backend 3.2.1 → 3.2.2

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

@@ -1,70 +0,0 @@
-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
-    select2 "Create whole-order adjustment", from: "Add action of type"
-    within('#action_fields') { click_button "Add" }
-
-    select2 "Tiered Percent", from: "Calculator"
-    within('#actions_container') { click_button "Update" }
-
-    within("#actions_container .settings") do
-      expect(page.body).to have_content("Base Percent")
-      expect(page.body).to have_content("Tiers")
-
-      click_button "Add"
-    end
-
-    fill_in "Base Percent", with: 5
-
-    within(".tier") do
-      find(".js-base-input").set(100)
-      page.execute_script("$('.js-base-input').change();")
-      find(".js-value-input").set(10)
-      page.execute_script("$('.js-value-input').change();")
-    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 Hash[100.0 => 10.0]
-  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 = Hash[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
-        click_icon :delete
-      end
-
-      within('#actions_container') { click_button "Update" }
-
-      calculator = promotion.actions.first.calculator
-      expect(calculator.preferred_tiers).to eq Hash[100.0 => 10.0, 300.0 => 20.0]
-    end
-  end
-end

data/spec/features/admin/refund_reasons/refund_reasons_spec.rb

@@ -1,57 +0,0 @@
-require 'spec_helper'
-
-describe 'RefundReason', type: :feature, js: true do
-  stub_authorization!
-
-  let!(:amount) { 100.0 }
-  let!(:payment_amount) { amount * 2 }
-  let!(:payment_method) { create(:credit_card_payment_method) }
-  let!(:payment) { create(:payment, amount: payment_amount, payment_method: payment_method) }
-  let!(:refund_reason) { create(:default_refund_reason, name: 'Reason #1', mutable: true) }
-  let!(:refund_reason2) { create(:refund_reason, name: 'Reason #2', mutable: true) }
-  let!(:refund) { create(:refund, payment: payment, amount: amount, reason: refund_reason, transaction_id: nil) }
-
-  before { visit spree.admin_refund_reasons_path }
-
-  describe 'destroy' do
-    it 'has refund reasons' do
-      within('.table #refund_reasons') do
-        expect(page).to have_content(refund_reason.name)
-        expect(page).to have_content(refund_reason2.name)
-      end
-    end
-
-    context 'should not destroy an associated option type' do
-      before { within_row(1) { delete_product_property } }
-
-      it 'has persisted refund reasons' do
-        within('.table #refund_reasons') do
-          expect(page).to have_content(refund_reason.name)
-          expect(page).to have_content(refund_reason2.name)
-        end
-      end
-
-      it(js: false) { expect(Spree::RefundReason.all).to include(refund_reason) }
-      it(js: false) { expect(Spree::RefundReason.all).to include(refund_reason2) }
-    end
-
-    context 'should allow an admin to destroy a non associated option type' do
-      before { within_row(2) { delete_product_property } }
-
-      it 'has persisted refund reasons' do
-        within('.table #refund_reasons') do
-          expect(page).to have_content(refund_reason.name)
-          expect(page).not_to have_content(refund_reason2.name)
-        end
-      end
-
-      it(js: false) { expect(Spree::RefundReason.all).to include(refund_reason) }
-      it(js: false) { expect(Spree::RefundReason.all).not_to include(refund_reason2) }
-    end
-
-    def delete_product_property
-      click_icon :delete
-      wait_for_ajax
-    end
-  end
-end

data/spec/features/admin/reimbursement_type/edit_reimbursement_type_spec.rb

@@ -1,36 +0,0 @@
-require 'spec_helper'
-
-describe "edit reimbursement type", type: :feature do
-  stub_authorization!
-  let(:r_type) do
-    create(:reimbursement_type,
-           name: 'Exchange',
-           type: 'Spree::ReimbursementType::Exchange',
-           active: true,
-           mutable: true)
-  end
-
-  before do
-    visit "/admin/reimbursement_types/#{r_type.id}/edit"
-  end
-
-  context "with valid attributes" do
-    it "change name, active and mutable" do
-      fill_in "Name", with: "New Credit"
-      uncheck "Mutable"
-      uncheck "Active"
-
-      expect { click_button "Create" }.not_to change(Spree::ReimbursementType, :count)
-
-      r_type.reload
-
-      expect(r_type.active).to eq false
-      expect(r_type.mutable).to eq false
-      expect(page).to have_content("New Credit")
-    end
-  end
-
-  it "view should have select field" do
-    expect(page).not_to have_css("div#reimbursement_type_type_field.form-group.field")
-  end
-end

data/spec/features/admin/reimbursement_type/new_reimbursement_type_spec.rb

@@ -1,62 +0,0 @@
-require 'spec_helper'
-
-describe "new reimbursement type", type: :feature do
-  stub_authorization!
-
-  before do
-    visit "/admin/reimbursement_types/new"
-  end
-
-  it "view should have select field" do
-    expect(page).to have_css("div#reimbursement_type_type_field.form-group.field")
-  end
-
-  context "with valid attributes" do
-    it "credit type" do
-      fill_in "Name", with: "Credit"
-      select "Spree::ReimbursementType::Credit", from: "reimbursement_type_type"
-
-      expect { click_button "Create" }.to change(Spree::ReimbursementType, :count).by(1)
-
-      expect(page).to have_content("Credit")
-    end
-
-    it "exchange type" do
-      fill_in "Name", with: "Exchange"
-      select "Spree::ReimbursementType::Exchange", from: "reimbursement_type_type"
-
-      expect { click_button "Create" }.to change(Spree::ReimbursementType, :count).by(1)
-
-      expect(page).to have_content("Exchange")
-    end
-
-    it "original payment type" do
-      fill_in "Name", with: "OriginalPayment"
-      select "Spree::ReimbursementType::OriginalPayment", from: "reimbursement_type_type"
-
-      expect { click_button "Create" }.to change(Spree::ReimbursementType, :count).by(1)
-
-      expect(page).to have_content("OriginalPayment")
-    end
-
-    it "store credit type" do
-      fill_in "Name", with: "StoreCredit"
-      select "Spree::ReimbursementType::StoreCredit", from: "reimbursement_type_type"
-
-      expect { click_button "Create" }.to change(Spree::ReimbursementType, :count).by(1)
-
-      expect(page).to have_content("StoreCredit")
-    end
-  end
-
-  context "with invalid params" do
-    it "without name" do
-      fill_in "Name", with: ""
-      select "Spree::ReimbursementType::StoreCredit", from: "reimbursement_type_type"
-
-      expect { click_button "Create" }.not_to change(Spree::ReimbursementType, :count)
-
-      expect(page).to have_content("Name can't be blank")
-    end
-  end
-end

data/spec/features/admin/reports_spec.rb

@@ -1,61 +0,0 @@
-require 'spec_helper'
-
-describe "Reports", type: :feature do
-  stub_authorization!
-
-  context "visiting the admin reports page" do
-    it "should have the right content" do
-      visit spree.admin_path
-      click_link "Reports"
-      click_link "Sales Total"
-
-      expect(page).to have_content("Sales Totals")
-      expect(page).to have_content("Item Total")
-      expect(page).to have_content("Adjustment Total")
-      expect(page).to have_content("Sales Total")
-    end
-  end
-
-  context "searching the admin reports page" do
-    before do
-      order = create(:order)
-      order.update_columns({adjustment_total: 100})
-      order.completed_at = Time.current
-      order.save!
-
-      order = create(:order)
-      order.update_columns({adjustment_total: 200})
-      order.completed_at = Time.current
-      order.save!
-
-      #incomplete order
-      order = create(:order)
-      order.update_columns({adjustment_total: 50})
-      order.save!
-
-      order = create(:order)
-      order.update_columns({adjustment_total: 200})
-      order.completed_at = 3.years.ago
-      order.created_at = 3.years.ago
-      order.save!
-
-      order = create(:order)
-      order.update_columns({adjustment_total: 200})
-      order.completed_at = 3.years.from_now
-      order.created_at = 3.years.from_now
-      order.save!
-    end
-
-    it "should allow me to search for reports" do
-      visit spree.admin_path
-      click_link "Reports"
-      click_link "Sales Total"
-
-      fill_in "q_completed_at_gt", with: 1.week.ago
-      fill_in "q_completed_at_lt", with: 1.week.from_now
-      click_button "Search"
-
-      expect(page).to have_content("$300.00")
-    end
-  end
-end

data/spec/features/admin/return_authorization_reasons/return_authorization_reasons_spec.rb

@@ -1,63 +0,0 @@
-require 'spec_helper'
-
-describe 'ReturnAuthorizationReason', type: :feature, js: true do
-  stub_authorization!
-
-  let!(:order) { create(:shipped_order) }
-  let!(:stock_location) { create(:stock_location) }
-  let!(:rma_reason) { create(:return_authorization_reason, name: 'Defect #1', mutable: true) }
-  let!(:rma_reason2) { create(:return_authorization_reason, name: 'Defect #2', mutable: true) }
-
-  let!(:return_authorization) do
-    create(
-      :return_authorization,
-      order: order,
-      stock_location: stock_location,
-      reason: rma_reason
-    )
-  end
-
-  before { visit spree.admin_return_authorization_reasons_path }
-
-  describe 'destroy' do
-    it 'has return authorization reasons' do
-      within('.table #return_authorization_reasons') do
-        expect(page).to have_content(rma_reason.name)
-        expect(page).to have_content(rma_reason2.name)
-      end
-    end
-
-    context 'should not destroy an associated option type' do
-      before { within_row(1) { delete_product_property } }
-
-      it 'has persisted return authorization reasons' do
-        within('.table #return_authorization_reasons') do
-          expect(page).to have_content(rma_reason.name)
-          expect(page).to have_content(rma_reason2.name)
-        end
-      end
-
-      it(js: false) { expect(Spree::ReturnAuthorizationReason.all).to include(rma_reason) }
-      it(js: false) { expect(Spree::ReturnAuthorizationReason.all).to include(rma_reason2) }
-    end
-
-    context 'should allow an admin to destroy a non associated option type' do
-      before { within_row(2) { delete_product_property } }
-
-      it 'has persisted return authorization reasons' do
-        within('.table #return_authorization_reasons') do
-          expect(page).to have_content(rma_reason.name)
-          expect(page).not_to have_content(rma_reason2.name)
-        end
-      end
-
-      it(js: false) { expect(Spree::ReturnAuthorizationReason.all).to include(rma_reason) }
-      it(js: false) { expect(Spree::ReturnAuthorizationReason.all).not_to include(rma_reason2) }
-    end
-
-    def delete_product_property
-      click_icon :delete
-      wait_for_ajax
-    end
-  end
-end

data/spec/features/admin/returns/customer_returns_spec.rb

@@ -1,80 +0,0 @@
-# encoding: utf-8
-require 'spec_helper'
-
-describe "Customer Returns", type: :feature do
-  stub_authorization!
-  let!(:customer_return) { create(:customer_return, created_at: Time.current) }
-
-  describe "listing" do
-    let!(:customer_return_2) { create(:customer_return, created_at: Time.current - 1.day) }
-
-    before(:each) do
-      visit spree.admin_customer_returns_path
-    end
-
-    it "lists sorted by created_at" do
-      within_row(1) { expect(page).to have_content(customer_return.number) }
-      within_row(2) { expect(page).to have_content(customer_return_2.number) }
-    end
-
-    it "displays pre tax total" do
-      within_row(1) { expect(page).to have_content(customer_return.display_pre_tax_total.to_html) }
-    end
-
-    it "displays order number" do
-      within_row(1) { expect(page).to have_content(customer_return.order.number) }
-    end
-
-    it "displays customer return number" do
-      within_row(1) { expect(page).to have_content(customer_return.number) }
-    end
-
-    it 'displays status' do
-      within_row(1) { expect(page).to have_content(Spree.t(:incomplete)) }
-    end
-
-    it 'has edit link' do
-      expect(page).to have_css('.icon-edit')
-    end
-  end
-
-  describe "searching" do
-    let!(:customer_return_2) { create(:customer_return) }
-
-    it "searches on number" do
-      visit spree.admin_customer_returns_path
-
-      click_on 'Filter'
-      fill_in "q_number_cont", with: customer_return.number
-      click_on 'Search'
-
-      expect(page).to have_content(customer_return.number)
-      expect(page).not_to have_content(customer_return_2.number)
-
-      click_on 'Filter'
-      fill_in "q_number_cont", with: customer_return_2.number
-      click_on 'Search'
-
-      expect(page).to have_content(customer_return_2.number)
-      expect(page).not_to have_content(customer_return.number)
-    end
-  end
-
-  describe 'link' do
-    describe 'order number' do
-      it 'opens orders edit page' do
-        visit spree.admin_customer_returns_path
-        click_link customer_return.order.number
-        expect(page).to have_content("Orders / #{customer_return.order.number}")
-      end
-    end
-
-    describe 'customer return number' do
-      it 'opens customer return edit page' do
-        visit spree.admin_customer_returns_path
-        click_link customer_return.number
-        expect(page).to have_content("Customer Return ##{customer_return.number}")
-      end
-    end
-  end
-end

data/spec/features/admin/returns/return_authorizations_spec.rb

@@ -1,152 +0,0 @@
-# encoding: utf-8
-require 'spec_helper'
-
-describe "Return Authorizations", type: :feature do
-  stub_authorization!
-
-  describe "listing" do
-    let!(:return_authorization) { create(:return_authorization, created_at: Time.current) }
-    let!(:return_authorization_2) { create(:return_authorization, created_at: Time.current - 1.day) }
-
-    before(:each) do
-      visit spree.admin_return_authorizations_path
-    end
-
-    it "lists return authorizations sorted by created_at" do
-      within_row(1) { expect(page).to have_content(return_authorization.number) }
-      within_row(2) { expect(page).to have_content(return_authorization_2.number) }
-    end
-
-    it "displays order number" do
-      within_row(1) { expect(page).to have_content(return_authorization.order.number) }
-    end
-
-    it "displays return authorization number" do
-      within_row(1) { expect(page).to have_content(return_authorization.number) }
-    end
-
-    it 'displays state' do
-      return_authorization_state = Spree.t("return_authorization_states.#{return_authorization.state}")
-      within_row(1) { expect(page).to have_content(return_authorization_state) }
-    end
-
-    it 'has edit link' do
-      expect(page).to have_css('.icon-edit')
-    end
-  end
-
-  describe "searching" do
-    let!(:return_authorization) { create(:return_authorization, state: 'authorized') }
-    let!(:return_authorization_2) { create(:return_authorization, state: 'canceled') }
-
-    it "searches on number" do
-      visit spree.admin_return_authorizations_path
-
-      click_on 'Filter'
-      fill_in "q_number_cont", with: return_authorization.number
-      click_on 'Search'
-
-      expect(page).to have_content(return_authorization.number)
-      expect(page).not_to have_content(return_authorization_2.number)
-
-      click_on 'Filter'
-      fill_in "q_number_cont", with: return_authorization_2.number
-      click_on 'Search'
-
-      expect(page).to have_content(return_authorization_2.number)
-      expect(page).not_to have_content(return_authorization.number)
-    end
-
-    it "searches on status" do
-      visit spree.admin_return_authorizations_path
-
-      click_on 'Filter'
-      select Spree.t("return_authorization_states.#{return_authorization.state}"), from: 'Status'
-      click_on 'Search'
-
-      expect(page).to have_content(return_authorization.number)
-      expect(page).not_to have_content(return_authorization_2.number)
-
-      click_on 'Filter'
-      select Spree.t("return_authorization_states.#{return_authorization_2.state}"), from: 'Status'
-      click_on 'Search'
-
-      expect(page).to have_content(return_authorization_2.number)
-      expect(page).not_to have_content(return_authorization.number)
-    end
-  end
-
-  describe 'link' do
-    let!(:return_authorization) { create(:return_authorization) }
-
-    describe 'order number' do
-      it 'opens orders edit page' do
-        visit spree.admin_return_authorizations_path
-        click_link return_authorization.order.number
-        expect(page).to have_content("Orders / #{return_authorization.order.number}")
-      end
-    end
-
-    describe 'return authorization number' do
-      it 'opens return authorization edit page' do
-        visit spree.admin_return_authorizations_path
-        click_link return_authorization.number
-        expect(page).to have_content(return_authorization.number)
-      end
-    end
-
-    describe 'authorized' do
-      let!(:return_authorization) { create(:return_authorization, state: 'authorized') }
-      let!(:return_authorization_2) { create(:return_authorization, state: 'canceled') }
-
-      it 'only shows authorized return authorizations' do
-        visit spree.admin_return_authorizations_path
-        within('.nav-tabs') do
-          click_link 'Authorized'
-        end
-
-        expect(page).to have_content(return_authorization.number)
-        expect(page).not_to have_content(return_authorization_2.number)
-      end
-
-      it 'preselects authorized status in filter' do
-        visit spree.admin_return_authorizations_path
-        within('.nav-tabs') do
-          click_link 'Authorized'
-        end
-
-        within('#table-filter') do
-          return_authorization_state = Spree.t("return_authorization_states.#{return_authorization.state}")
-          expect(page).to have_select("Status", selected: return_authorization_state)
-        end
-      end
-    end
-
-    describe 'canceled' do
-      let!(:return_authorization) { create(:return_authorization, state: 'canceled') }
-      let!(:return_authorization_2) { create(:return_authorization, state: 'authorized') }
-
-      it 'only shows canceled return authorizations' do
-        visit spree.admin_return_authorizations_path
-        within('.nav-tabs') do
-          click_link 'Canceled'
-        end
-
-        expect(page).to have_content(return_authorization.number)
-        expect(page).not_to have_content(return_authorization_2.number)
-      end
-
-      it 'preselects canceled status in filter' do
-        visit spree.admin_return_authorizations_path
-        within('.nav-tabs') do
-          click_link 'Canceled'
-        end
-
-        within('#table-filter') do
-          return_authorization_state = Spree.t("return_authorization_states.#{return_authorization.state}")
-          expect(page).to have_select("Status", selected: return_authorization_state)
-        end
-      end
-    end
-  end
-end

data/spec/features/admin/stock_transfer_spec.rb

@@ -1,75 +0,0 @@
-require 'spec_helper'
-
-describe 'Stock Transfers', type: :feature, js: true do
-  stub_authorization!
-
-  it 'transfer between 2 locations' do
-    source_location = create(:stock_location_with_items, name: 'NY')
-    destination_location = create(:stock_location, name: 'SF')
-    variant = Spree::Variant.last
-
-    visit spree.admin_stock_transfers_path
-    click_on 'New Stock Transfer'
-    fill_in 'reference', with: 'PO 666'
-
-    select2_search variant.name, from: 'Variant'
-
-    click_button 'Add'
-    click_button 'Transfer Stock'
-
-    expect(page).to have_content('Reference PO 666')
-    expect(page).to have_content('NY')
-    expect(page).to have_content('SF')
-    expect(page).to have_content(variant.name)
-
-    transfer = Spree::StockTransfer.last
-    expect(transfer.stock_movements.size).to eq 2
-  end
-
-  describe 'received stock transfer' do
-    def it_is_received_stock_transfer(page)
-      expect(page).to have_content('Reference PO 666')
-      expect(page).not_to have_selector("#stock-location-source")
-      expect(page).to have_selector("#stock-location-destination")
-
-      transfer = Spree::StockTransfer.last
-      expect(transfer.stock_movements.size).to eq 1
-      expect(transfer.source_location).to be_nil
-    end
-
-    it 'receive stock to a single location' do
-      source_location = create(:stock_location_with_items, name: 'NY')
-      destination_location = create(:stock_location, name: 'SF')
-      variant = Spree::Variant.last
-
-      visit spree.new_admin_stock_transfer_path
-
-      fill_in 'reference', with: 'PO 666'
-      check 'transfer_receive_stock'
-      select('NY', from: 'transfer_destination_location_id')
-      select2_search variant.name, from: 'Variant'
-
-      click_button 'Add'
-      click_button 'Transfer Stock'
-
-      it_is_received_stock_transfer page
-    end
-
-    it 'forced to only receive there is only one location' do
-      source_location = create(:stock_location_with_items, name: 'NY')
-      variant = Spree::Variant.last
-
-      visit spree.new_admin_stock_transfer_path
-
-      fill_in 'reference', with: 'PO 666'
-
-      select('NY', from: 'transfer_destination_location_id')
-      select2_search variant.name, from: 'Variant'
-
-      click_button 'Add'
-      click_button 'Transfer Stock'
-
-      it_is_received_stock_transfer page
-    end
-  end
-end