spree_backend 3.2.9 → 3.3.0.rc1

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

@@ -0,0 +1,70 @@
+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

@@ -0,0 +1,57 @@
+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

@@ -0,0 +1,36 @@
+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

@@ -0,0 +1,62 @@
+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

@@ -0,0 +1,61 @@
+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

@@ -0,0 +1,63 @@
+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

@@ -0,0 +1,80 @@
+# 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

@@ -0,0 +1,152 @@
+# 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

@@ -0,0 +1,91 @@
+require 'spec_helper'
+
+describe 'Stock Transfers', type: :feature, js: true do
+  stub_authorization!
+
+  it 'shows variants with options text' do
+    create(:stock_location_with_items, name: 'NY')
+
+    product = Spree::Product.first
+    variant = create(:variant, product: product)
+    variant.set_option_value('Color', 'Green')
+
+    visit spree.admin_stock_transfers_path
+    click_on 'New Stock Transfer'
+
+    select2_search variant.sku, from: 'Variant'
+
+    content = "#{variant.name} - #{variant.sku} (#{variant.options_text})"
+    expect(page).to have_content(content)
+  end
+
+  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