spree_backend 3.2.1 → 3.2.2

data/spec/features/admin/orders/order_details_spec.rb

@@ -1,662 +0,0 @@
-# coding: utf-8
-require 'spec_helper'
-
-describe "Order Details", type: :feature, js: true do
-  let!(:stock_location) { create(:stock_location_with_items) }
-  let!(:product) { create(:product, name: 'spree t-shirt', price: 20.00) }
-  let(:order) { create(:order, state: 'complete', completed_at: "2011-02-01 12:36:15", number: "R100") }
-  let(:state) { create(:state) }
-  let!(:shipping_method) { create(:shipping_method, name: "Default") }
-
-  before do
-    order.shipments.create!(stock_location_id: stock_location.id)
-    order.contents.add(product.master, 2)
-  end
-
-  context 'as Admin' do
-    stub_authorization!
-
-    context "cart edit page" do
-      before do
-        product.master.stock_items.first.update_column(:count_on_hand, 100)
-        visit spree.cart_admin_order_path(order)
-      end
-
-      it "should allow me to edit order details" do
-        expect(page).to have_content("spree t-shirt")
-        expect(page).to have_content("$40.00")
-
-        within_row(1) do
-          click_icon :edit
-          fill_in "quantity", with: "1"
-        end
-        click_icon :save
-
-        within("#order_total") do
-          expect(page).to have_content("$20.00")
-        end
-      end
-
-      it "can add an item to a shipment" do
-        select2_search "spree t-shirt", from: Spree.t(:name_or_sku)
-        within("table.stock-levels") do
-          fill_in "variant_quantity", with: 2
-          click_icon :add
-        end
-
-        within("#order_total") do
-          expect(page).to have_content("$80.00")
-        end
-      end
-
-      it "can remove an item from a shipment" do
-        expect(page).to have_content("spree t-shirt")
-
-        within_row(1) do
-          accept_alert do
-            click_icon :delete
-          end
-        end
-
-        # Click "ok" on confirmation dialog
-        expect(page).not_to have_content("spree t-shirt")
-      end
-
-      # Regression test for #3862
-      it "can cancel removing an item from a shipment" do
-        expect(page).to have_content("spree t-shirt")
-
-        within_row(1) do
-          # Click "cancel" on confirmation dialog
-          dismiss_alert do
-            click_icon :delete
-          end
-        end
-
-        expect(page).to have_content("spree t-shirt")
-      end
-
-      it "can add tracking information" do
-        visit spree.edit_admin_order_path(order)
-
-        within(".show-tracking") do
-          click_icon :edit
-        end
-        fill_in "tracking", with: "FOOBAR"
-        click_icon :save
-
-        expect(page).not_to have_css("input[name=tracking]")
-        expect(page).to have_content("Tracking: FOOBAR")
-      end
-
-      it "can change the shipping method" do
-        order = create(:completed_order_with_totals)
-        visit spree.edit_admin_order_path(order)
-        within("table.table tr.show-method") do
-          click_icon :edit
-        end
-        select2 "Default", from: "Shipping Method"
-        click_icon :save
-
-        expect(page).not_to have_css('#selected_shipping_rate_id')
-        expect(page).to have_content("Default")
-      end
-
-      it "can assign a back-end only shipping method" do
-        create(:shipping_method, name: "Backdoor", display_on: "back_end")
-        order = create(
-          :completed_order_with_totals,
-          shipping_method_filter: Spree::ShippingMethod::DISPLAY_ON_FRONT_AND_BACK_END
-        )
-        visit spree.edit_admin_order_path(order)
-        within("table tr.show-method") do
-          click_icon :edit
-        end
-        select2 "Backdoor", from: "Shipping Method"
-        click_icon :save
-
-        expect(page).not_to have_css('#selected_shipping_rate_id')
-        expect(page).to have_content("Backdoor")
-      end
-
-      it "will show the variant sku", js: false do
-        order = create(:completed_order_with_totals)
-        visit spree.edit_admin_order_path(order)
-        sku = order.line_items.first.variant.sku
-        expect(page).to have_content("SKU: #{sku}")
-      end
-
-      context "with special_instructions present" do
-        before(:each) do
-          order.update_column(:special_instructions, "Very special instructions here")
-        end
-
-        it "will show the special_instructions", js: false do
-          visit spree.edit_admin_order_path(order)
-          expect(page).to have_content("Very special instructions here")
-        end
-      end
-
-      context 'when not tracking inventory' do
-        let(:tote) { create(:product, name: "Tote", price: 15.00) }
-
-        context "variant doesn't track inventory" do
-          before do
-            tote.master.update_column :track_inventory, false
-            # make sure there's no stock level for any item
-            tote.master.stock_items.update_all count_on_hand: 0, backorderable: false
-          end
-
-          it "adds variant to order just fine" do
-            select2_search tote.name, from: Spree.t(:name_or_sku)
-            within("table.stock-levels") do
-              fill_in "variant_quantity", with: 1
-              click_icon :add
-            end
-
-            wait_for_ajax
-
-            within(".line-items") do
-              expect(page).to have_content(tote.name)
-            end
-          end
-        end
-
-        context "site doesn't track inventory" do
-          before do
-            Spree::Config[:track_inventory_levels] = false
-            tote.master.update_column(:track_inventory, true)
-            # make sure there's no stock level for any item
-            tote.master.stock_items.update_all count_on_hand: 0, backorderable: true
-          end
-
-          it "adds variant to order just fine" do
-            select2_search tote.name, from: Spree.t(:name_or_sku)
-            within("table.stock-levels") do
-              fill_in "variant_quantity", with: 1
-              click_icon :add
-            end
-
-            wait_for_ajax
-
-            within(".line-items") do
-              expect(page).to have_content(tote.name)
-            end
-          end
-
-          after { Spree::Config[:track_inventory_levels] = true }
-        end
-      end
-
-      context "variant out of stock and not backorderable" do
-        before do
-          product.master.stock_items.first.update_column(:backorderable, false)
-          product.master.stock_items.first.update_column(:count_on_hand, 0)
-        end
-
-        it "displays out of stock instead of add button" do
-          select2_search product.name, from: Spree.t(:name_or_sku)
-
-          within("table.stock-levels") do
-            expect(page).to have_content(Spree.t(:out_of_stock))
-          end
-        end
-      end
-    end
-
-    context 'Shipment edit page' do
-      let!(:stock_location2) { create(:stock_location_with_items, name: 'Clarksville') }
-
-      before do
-        product.master.stock_items.first.update_column(:backorderable, true)
-        product.master.stock_items.first.update_column(:count_on_hand, 100)
-        product.master.stock_items.last.update_column(:count_on_hand, 100)
-      end
-
-      context 'splitting to location' do
-        before { visit spree.edit_admin_order_path(order) }
-        # can not properly implement until poltergeist supports checking alert text
-        # see https://github.com/teampoltergeist/poltergeist/pull/516
-        it 'should warn you if you have not selected a location or shipment'
-
-        context 'there is enough stock at the other location' do
-          it 'should allow me to make a split' do
-            expect(order.shipments.count).to eq(1)
-            expect(order.shipments.first.inventory_units_for(product.master).count).to eq(2)
-
-            within_row(1) { click_icon 'split' }
-            targetted_select2 stock_location2.name, from: '#s2id_item_stock_location'
-            click_icon :save
-
-            wait_for_ajax
-            order.reload
-
-            expect(order.shipments.count).to eq(2)
-            expect(order.shipments.last.backordered?).to eq(false)
-            expect(order.shipments.first.inventory_units_for(product.master).count).to eq(1)
-            expect(order.shipments.last.inventory_units_for(product.master).count).to eq(1)
-          end
-
-          it 'should allow me to make a transfer via splitting off all stock' do
-            expect(order.shipments.first.stock_location.id).to eq(stock_location.id)
-
-            within_row(1) { click_icon 'split' }
-            targetted_select2 stock_location2.name, from: '#s2id_item_stock_location'
-            fill_in 'item_quantity', with: 2
-            click_icon :save
-
-            wait_for_ajax
-            order.reload
-
-            expect(order.shipments.count).to eq(1)
-            expect(order.shipments.last.backordered?).to eq(false)
-            expect(order.shipments.first.inventory_units_for(product.master).count).to eq(2)
-            expect(order.shipments.first.stock_location.id).to eq(stock_location2.id)
-          end
-
-          it 'should allow me to split more than I have if available there' do
-            expect(order.shipments.first.stock_location.id).to eq(stock_location.id)
-
-            within_row(1) { click_icon 'split' }
-            targetted_select2 stock_location2.name, from: '#s2id_item_stock_location'
-            fill_in 'item_quantity', with: 5
-            click_icon :save
-
-            wait_for_ajax
-            order.reload
-
-            expect(order.shipments.count).to eq(1)
-            expect(order.shipments.last.backordered?).to eq(false)
-            expect(order.shipments.first.inventory_units_for(product.master).count).to eq(5)
-            expect(order.shipments.first.stock_location.id).to eq(stock_location2.id)
-          end
-
-          it 'should not split anything if the input quantity is garbage' do
-            expect(order.shipments.first.stock_location.id).to eq(stock_location.id)
-
-            within_row(1) { click_icon 'split' }
-            targetted_select2 stock_location2.name, from: '#s2id_item_stock_location'
-            fill_in 'item_quantity', with: 'ff'
-            click_icon :save
-
-            wait_for_ajax
-
-            expect(order.shipments.count).to eq(1)
-            expect(order.shipments.first.inventory_units_for(product.master).count).to eq(2)
-            expect(order.shipments.first.stock_location.id).to eq(stock_location.id)
-          end
-
-          it 'should not allow less than or equal to zero qty' do
-            expect(order.shipments.first.stock_location.id).to eq(stock_location.id)
-
-            within_row(1) { click_icon 'split' }
-            targetted_select2 stock_location2.name, from: '#s2id_item_stock_location'
-            fill_in 'item_quantity', with: 0
-            click_icon :save
-
-            wait_for_ajax
-
-            expect(order.shipments.count).to eq(1)
-            expect(order.shipments.first.inventory_units_for(product.master).count).to eq(2)
-            expect(order.shipments.first.stock_location.id).to eq(stock_location.id)
-
-
-            fill_in 'item_quantity', with: -1
-            click_icon :save
-
-            wait_for_ajax
-
-            expect(order.shipments.count).to eq(1)
-            expect(order.shipments.first.inventory_units_for(product.master).count).to eq(2)
-            expect(order.shipments.first.stock_location.id).to eq(stock_location.id)
-          end
-
-          context 'A shipment has shipped' do
-
-            it 'should not show or let me back to the cart page, nor show the shipment edit buttons', js: false do
-              order = create(:order, state: 'payment')
-              order.shipments.create!(stock_location_id: stock_location.id, state: 'shipped')
-
-              visit spree.cart_admin_order_path(order)
-
-              expect(page.current_path).to eq(spree.edit_admin_order_path(order))
-              expect(page).not_to have_text 'Cart'
-              expect(page).not_to have_selector('.fa-split')
-              expect(page).not_to have_selector('.fa-trash')
-            end
-
-          end
-        end
-
-        context 'there is not enough stock at the other location' do
-          context 'and it cannot backorder' do
-            it 'should not allow me to split stock' do
-              product.master.stock_items.last.update_column(:backorderable, false)
-              product.master.stock_items.last.update_column(:count_on_hand, 0)
-
-              within_row(1) { click_icon 'split' }
-              targetted_select2 stock_location2.name, from: '#s2id_item_stock_location'
-              fill_in 'item_quantity', with: 2
-              click_icon :save
-
-              wait_for_ajax
-
-              expect(order.shipments.count).to eq(1)
-              expect(order.shipments.first.inventory_units_for(product.master).count).to eq(2)
-              expect(order.shipments.first.stock_location.id).to eq(stock_location.id)
-            end
-
-          end
-
-          context 'but it can backorder' do
-            it 'should allow me to split and backorder the stock' do
-              product.master.stock_items.last.update_column(:count_on_hand, 0)
-              product.master.stock_items.last.update_column(:backorderable, true)
-
-              within_row(1) { click_icon 'split' }
-              targetted_select2 stock_location2.name, from: '#s2id_item_stock_location'
-              fill_in 'item_quantity', with: 2
-              click_icon :save
-
-              wait_for_ajax
-              order.reload
-
-              expect(order.shipments.count).to eq(1)
-              expect(order.shipments.first.inventory_units_for(product.master).count).to eq(2)
-              expect(order.shipments.first.stock_location.id).to eq(stock_location2.id)
-            end
-          end
-        end
-
-        context 'multiple items in cart' do
-          it 'should have no problem splitting if multiple items are in the from shipment' do
-            order.contents.add(create(:variant), 2)
-            expect(order.shipments.count).to eq(1)
-            expect(order.shipments.first.manifest.count).to eq(2)
-
-            within_row(1) { click_icon 'split' }
-            targetted_select2 stock_location2.name, from: '#s2id_item_stock_location'
-            click_icon :save
-
-            wait_for_ajax
-            order.reload
-
-            expect(order.shipments.count).to eq(2)
-            expect(order.shipments.last.backordered?).to eq(false)
-            expect(order.shipments.first.inventory_units_for(product.master).count).to eq(1)
-            expect(order.shipments.last.inventory_units_for(product.master).count).to eq(1)
-          end
-        end
-
-        context 'when not tracking inventory' do
-          let(:tote) { create(:product, name: "Tote", price: 15.00) }
-
-          context "variant doesn't track inventory" do
-            before do
-              tote.master.update_column :track_inventory, false
-              # make sure there's no stock level for any item
-              tote.master.stock_items.update_all count_on_hand: 0, backorderable: false
-            end
-
-            it "adds variant to order just fine" do
-              select2_search tote.name, from: Spree.t(:name_or_sku)
-              within("table.stock-levels") do
-                fill_in "stock_item_quantity", with: 1
-                click_icon :add
-              end
-
-              wait_for_ajax
-
-              within("[data-hook=admin_order_form_fields]") do
-                expect(page).to have_content(tote.name)
-              end
-            end
-          end
-
-          context "site doesn't track inventory" do
-            before do
-              Spree::Config[:track_inventory_levels] = false
-              tote.master.update_column(:track_inventory, true)
-              # make sure there's no stock level for any item
-              tote.master.stock_items.update_all count_on_hand: 0, backorderable: true
-            end
-
-            it "adds variant to order just fine" do
-              select2_search tote.name, from: Spree.t(:name_or_sku)
-              within("table.stock-levels") do
-                fill_in "stock_item_quantity", with: 1
-                click_icon :add
-              end
-
-              wait_for_ajax
-
-              within("[data-hook=admin_order_form_fields]") do
-                expect(page).to have_content(tote.name)
-              end
-            end
-
-            after { Spree::Config[:track_inventory_levels] = true }
-          end
-        end
-
-        context "variant out of stock and not backorderable" do
-          before do
-            product.master.stock_items.first.update_column(:backorderable, false)
-            product.master.stock_items.first.update_column(:count_on_hand, 0)
-          end
-
-          it "displays out of stock instead of add button" do
-            select2_search product.name, from: Spree.t(:name_or_sku)
-
-            within("table.stock-levels") do
-              expect(page).to have_content(Spree.t(:out_of_stock))
-            end
-          end
-        end
-      end
-
-      context 'splitting to shipment' do
-        before do
-          @shipment2 = order.shipments.create(stock_location_id: stock_location2.id)
-          visit spree.edit_admin_order_path(order)
-        end
-
-        it 'should delete the old shipment if enough are split off' do
-          expect(order.shipments.count).to eq(2)
-
-          within_row(1) { click_icon 'split' }
-          targetted_select2 @shipment2.number, from: '#s2id_item_stock_location'
-          fill_in 'item_quantity', with: 2
-          click_icon :save
-
-          wait_for_ajax
-          order.reload
-
-          expect(order.shipments.count).to eq(1)
-          expect(order.shipments.last.inventory_units_for(product.master).count).to eq(2)
-        end
-
-        context 'receiving shipment can not backorder' do
-          before { product.master.stock_items.last.update_column(:backorderable, false) }
-
-          it 'should not allow a split if the receiving shipment qty plus the incoming is greater than the count_on_hand' do
-            expect(order.shipments.count).to eq(2)
-
-            within_row(1) { click_icon 'split' }
-            targetted_select2 @shipment2.number, from: '#s2id_item_stock_location'
-            fill_in 'item_quantity', with: 1
-            click_icon :save
-
-            wait_for_ajax
-
-            within_row(1) { click_icon 'split' }
-            targetted_select2 @shipment2.number, from: '#s2id_item_stock_location'
-            fill_in 'item_quantity', with: 200
-            click_icon :save
-
-            wait_for_ajax
-
-            expect(order.shipments.count).to eq(2)
-            expect(order.shipments.first.inventory_units_for(product.master).count).to eq(1)
-            expect(order.shipments.last.inventory_units_for(product.master).count).to eq(1)
-          end
-
-          it 'should not allow a shipment to split stock to itself' do
-            within_row(1) { click_icon 'split' }
-            targetted_select2 order.shipments.first.number, from: '#s2id_item_stock_location'
-            fill_in 'item_quantity', with: 1
-            click_icon :save
-
-            wait_for_ajax
-
-            expect(order.shipments.count).to eq(2)
-            expect(order.shipments.first.inventory_units_for(product.master).count).to eq(2)
-          end
-
-          it 'should split fine if more than one line_item is in the receiving shipment' do
-            variant2 = create(:variant)
-            order.contents.add(variant2, 2, shipment: @shipment2)
-
-            within_row(1) { click_icon 'split' }
-            targetted_select2 @shipment2.number, from: '#s2id_item_stock_location'
-            fill_in 'item_quantity', with: 1
-            click_icon :save
-
-            wait_for_ajax
-
-            expect(order.shipments.count).to eq(2)
-            expect(order.shipments.first.inventory_units_for(product.master).count).to eq 1
-            expect(order.shipments.last.inventory_units_for(product.master).count).to eq 1
-            expect(order.shipments.first.inventory_units_for(variant2).count).to eq 0
-            expect(order.shipments.last.inventory_units_for(variant2).count).to eq 2
-          end
-        end
-
-        context 'receiving shipment can backorder' do
-          it 'should add more to the backorder' do
-            product.master.stock_items.last.update_column(:backorderable, true)
-            product.master.stock_items.last.update_column(:count_on_hand, 0)
-            expect(@shipment2.reload.backordered?).to eq(false)
-
-
-            within_row(1) { click_icon 'split' }
-            targetted_select2 @shipment2.number, from: '#s2id_item_stock_location'
-            fill_in 'item_quantity', with: 1
-            click_icon :save
-
-            wait_for_ajax
-
-            expect(@shipment2.reload.backordered?).to eq(true)
-
-            within_row(1) { click_icon 'split' }
-            targetted_select2 @shipment2.number, from: '#s2id_item_stock_location'
-            fill_in 'item_quantity', with: 1
-            click_icon :save
-
-            wait_for_ajax
-
-            expect(order.shipments.count).to eq(1)
-            expect(order.shipments.last.inventory_units_for(product.master).count).to eq(2)
-            expect(@shipment2.reload.backordered?).to eq(true)
-          end
-        end
-      end
-    end
-  end
-
-  context 'with only read permissions' do
-    before do
-      allow_any_instance_of(Spree::Admin::BaseController).to receive(:spree_current_user).and_return(nil)
-    end
-
-    custom_authorization! do |user|
-      can [:admin, :index, :read, :edit], Spree::Order
-    end
-
-    it "should not display forbidden links" do
-      visit spree.edit_admin_order_path(order)
-
-      expect(page).not_to have_button('cancel')
-      expect(page).not_to have_button('Resend')
-
-      # Order Tabs
-      expect(page).not_to have_link('Details')
-      expect(page).not_to have_link('Customer')
-      expect(page).not_to have_link('Adjustments')
-      expect(page).not_to have_link('Payments')
-      expect(page).not_to have_link('Returns')
-
-      # Order item actions
-      expect(page).not_to have_css('.delete-item')
-      expect(page).not_to have_css('.split-item')
-      expect(page).not_to have_css('.edit-item')
-      expect(page).not_to have_css('.edit-tracking')
-
-      expect(page).not_to have_css('#add-line-item')
-    end
-  end
-
-  context 'as Fakedispatch' do
-    custom_authorization! do |user|
-      # allow dispatch to :admin, :index, and :edit on Spree::Order
-      can [:admin, :edit, :index, :read], Spree::Order
-      # allow dispatch to :index, :show, :create and :update shipments on the admin
-      can [:admin, :manage, :read, :ship], Spree::Shipment
-    end
-
-    before do
-      allow(Spree.user_class).to receive(:find_by).
-                                   with(hash_including(:spree_api_key)).
-                                   and_return(Spree.user_class.new)
-    end
-
-    it 'should not display order tabs or edit buttons without ability', js: false do
-      visit spree.edit_admin_order_path(order)
-
-      # Order Form
-      expect(page).not_to have_css('.edit-item')
-      # Order Tabs
-      expect(page).not_to have_link('Details')
-      expect(page).not_to have_link('Customer')
-      expect(page).not_to have_link('Adjustments')
-      expect(page).not_to have_link('Payments')
-      expect(page).not_to have_link('Returns')
-    end
-
-    it "can add tracking information" do
-      visit spree.edit_admin_order_path(order)
-      within("table.table tr:nth-child(5)") do
-        click_icon :edit
-      end
-      fill_in "tracking", with: "FOOBAR"
-      click_icon :save
-
-      expect(page).not_to have_css("input[name=tracking]")
-      expect(page).to have_content("Tracking: FOOBAR")
-    end
-
-    it "can change the shipping method" do
-      order = create(:completed_order_with_totals)
-      visit spree.edit_admin_order_path(order)
-      within("table.table tr.show-method") do
-        click_icon :edit
-      end
-      select2 "Default", from: "Shipping Method"
-      click_icon :save
-
-      expect(page).not_to have_css('#selected_shipping_rate_id')
-      expect(page).to have_content("Default")
-    end
-
-    it 'can ship' do
-      order = create(:order_ready_to_ship)
-      order.refresh_shipment_rates
-      visit spree.edit_admin_order_path(order)
-      click_on 'Ship'
-      wait_for_ajax
-      within '.shipment-state' do
-        expect(page).to have_content('shipped')
-      end
-    end
-  end
-end