solidus_backend_devise_token_auth 2.8.0.alpha.2

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

@@ -0,0 +1,572 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "Order Details", type: :feature, js: true do
+  include OrderFeatureHelper
+
+  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(:line_item) { order.line_items.first }
+
+  let!(:shipment1) do
+    order.shipments.create(stock_location_id: stock_location.id)
+  end
+
+  before do
+    order.contents.add(product.master, 2)
+    # order.contents.add causes things (like line items & shipments) to get
+    # cached, and these are going to change during this spec so we go ahead and
+    # reload now
+    order.reload
+  end
+
+  context 'as Admin' do
+    stub_authorization!
+
+    context "cart edit page" do
+      let(:track_inventory) { true }
+      let(:backorderable) { true }
+      let(:count_on_hand) { 100 }
+
+      before do
+        product.master.update_columns(track_inventory: track_inventory)
+        product.master.stock_items.update_all(count_on_hand: count_on_hand, backorderable: backorderable)
+
+        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('tr', text: line_item.sku) do
+          click_icon :edit
+          fill_in "quantity", with: "1"
+        end
+        click_icon :ok
+
+        within("#order_total") do
+          expect(page).to have_content("$20.00")
+        end
+      end
+
+      it "can add an item" do
+        add_line_item "spree t-shirt", quantity: 2
+
+        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('tr', text: line_item.sku) do
+          accept_confirm "Are you sure you want to delete this record?" do
+            click_icon :trash
+          end
+        end
+
+        expect(page).not_to have_content("spree t-shirt")
+
+        # Should have a new item row
+        expect(page).to have_field('quantity')
+      end
+
+      # Regression test for https://github.com/spree/spree/issues/3862
+      it "can cancel removing an item from a shipment" do
+        expect(page).to have_content("spree t-shirt")
+
+        within('tr', text: line_item.sku) do
+          # Click "cancel" on confirmation dialog
+          dismiss_confirm "Are you sure you want to delete this record?" do
+            click_icon :trash
+          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("tr", text: "Tracking Number") do
+          click_icon :edit
+          fill_in "tracking", with: "FOOBAR"
+          click_icon :check
+
+          expect(page).not_to have_css("input")
+          expect(page).to have_content("Tracking Number FOOBAR")
+        end
+      end
+
+      context "with a completed order" do
+        let!(:order) { create(:completed_order_with_totals) }
+        let(:shipment1) { order.shipments[0] }
+
+        it "can change the shipping method" do
+          visit spree.edit_admin_order_path(order)
+
+          within("tr", text: "Shipping Method") do
+            click_icon :edit
+            select "UPS Ground $100.00"
+            click_icon :check
+          end
+
+          expect(page).not_to have_css('#selected_shipping_rate_id')
+          expect(page).to have_content("UPS Ground")
+        end
+
+        it "can use admin-only shipping methods" do
+          create(:shipping_method, name: "Admin Free Shipping", cost: 0, available_to_users: false)
+
+          visit spree.edit_admin_order_path(order)
+
+          within("tr", text: "Shipping Method") do
+            click_icon :edit
+            select "Admin Free Shipping $0.00"
+            click_icon :check
+          end
+
+          expect(page).not_to have_css('#selected_shipping_rate_id')
+          expect(page).to have_no_content("UPS Ground")
+          expect(page).to have_content("Admin Free Shipping")
+        end
+
+        it "will show the variant sku" do
+          visit spree.edit_admin_order_path(order)
+          sku = order.line_items.first.variant.sku
+          expect(page).to have_content("SKU: #{sku}")
+        end
+      end
+
+      context "with special_instructions present" do
+        let(:order) { create(:order, state: 'complete', completed_at: "2011-02-01 12:36:15", number: "R100", special_instructions: "Very special instructions here") }
+        it "will show the special_instructions" do
+          visit spree.edit_admin_order_path(order)
+          expect(page).to have_content("Very special instructions here")
+        end
+      end
+
+      context "variant doesn't track inventory" do
+        let(:track_inventory) { false }
+        let(:backorderable) { false }
+        let(:count_on_hand) { 0 }
+
+        it "adds variant to order just fine" do
+          add_line_item "spree t-shirt"
+
+          within(".line-items") do
+            expect(page).to have_content("spree t-shirt")
+          end
+        end
+      end
+
+      context "variant out of stock and not backorderable" do
+        let(:backorderable) { false }
+        let(:count_on_hand) { 0 }
+
+        it "doesn't display the out of stock variant in the search results" do
+          click_on 'Add Item'
+          select2_search_without_selection product.name, from: ".select-variant"
+
+          expect(page).to have_selector('.select2-no-results')
+          within(".select2-no-results") do
+            expect(page).to have_content("No matches found")
+          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
+        # 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)
+
+            visit spree.edit_admin_order_path(order)
+
+            within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+            complete_split_to(stock_location2)
+
+            expect(page).to have_css('.shipment', count: 2)
+
+            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
+            visit spree.edit_admin_order_path(order)
+
+            expect(order.shipments.first.stock_location.id).to eq(stock_location.id)
+
+            within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+            complete_split_to(stock_location2, quantity: 2)
+
+            expect(page).to have_content("Pending package from 'Clarksville'")
+
+            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 not allow me to split more than I had in the original shipment' do
+            visit spree.edit_admin_order_path(order)
+
+            expect(order.shipments.first.stock_location.id).to eq(stock_location.id)
+
+            within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+            complete_split_to(stock_location2, quantity: 5)
+
+            expect(page).to have_content("Pending package from 'Clarksville'")
+
+            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 not allow less than or equal to zero qty' do
+            visit spree.edit_admin_order_path(order)
+
+            expect(order.shipments.first.stock_location.id).to eq(stock_location.id)
+
+            within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+
+            accept_alert "Quantity must be greater than 0" do
+              complete_split_to(stock_location2, quantity: 0)
+            end
+
+            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
+
+            accept_alert "Quantity must be greater than 0" do
+              click_icon :ok
+            end
+
+            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' 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-arrows-h')
+              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)
+
+              visit spree.edit_admin_order_path(order)
+
+              within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+              accept_alert "Desired shipment not enough stock in desired stock location" do
+                complete_split_to(stock_location2, quantity: 2)
+              end
+
+              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)
+
+              visit spree.edit_admin_order_path(order)
+
+              within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+              complete_split_to(stock_location2, quantity: 2)
+
+              expect(page).to have_content("Pending package from 'Clarksville'")
+
+              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)
+            order.reload
+
+            visit spree.edit_admin_order_path(order)
+
+            expect(order.shipments.count).to eq(1)
+            expect(order.shipments.first.manifest.count).to eq(2)
+
+            within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+            complete_split_to(stock_location2)
+
+            expect(page).to have_css('.shipment', count: 2)
+
+            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
+      end
+
+      context 'removing an item' do
+        let!(:shipment2) { order.shipments.create(stock_location_id: stock_location2.id) }
+
+        it "removes only the one item" do
+          order.line_items[0].inventory_units[0].update!(shipment: shipment2)
+          visit spree.edit_admin_order_path(order)
+
+          expect(page).to have_css('.stock-item', count: 2)
+
+          within '[data-hook=admin_shipment_form]', text: shipment2.number do
+            accept_confirm "Are you sure you want to delete this record?" do
+              click_icon :trash
+            end
+          end
+
+          expect(page).to have_css('.stock-item', count: 1)
+        end
+      end
+
+      context 'splitting to shipment' do
+        let!(:shipment2) { order.shipments.create(stock_location_id: stock_location2.id) }
+
+        it 'should delete the old shipment if enough are split off' do
+          expect(order.shipments.count).to eq(2)
+
+          visit spree.edit_admin_order_path(order)
+
+          within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+          complete_split_to(shipment2, quantity: 2)
+
+          expect(page).not_to have_content(/Move .* to/)
+
+          expect(page).to have_css("#shipment_#{shipment2.id}", count: 1)
+
+          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)
+
+            visit spree.edit_admin_order_path(order)
+            expect(page).to have_css('.item-name', text: product.name, count: 1)
+
+            within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+            complete_split_to(shipment2, quantity: 1)
+
+            expect(page).to have_css('.item-name', text: product.name, count: 2)
+
+            within(all('.stock-contents', count: 2).first) do
+              within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+
+              accept_alert("Desired shipment not enough stock in desired stock location") do
+                complete_split_to(shipment2, quantity: 200)
+              end
+            end
+
+            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
+            visit spree.edit_admin_order_path(order)
+            within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+            click_on 'Choose location'
+            within '.select2-results' do
+              expect(page).to have_content(shipment2.number)
+              expect(page).not_to have_content(shipment1.number)
+            end
+          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)
+            order.reload
+
+            visit spree.edit_admin_order_path(order)
+
+            within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+            complete_split_to(shipment2, quantity: 1)
+
+            expect(page).not_to have_content(/Move .* to/)
+            expect(page).to have_css('.shipment', count: 2)
+
+            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
+            shipment1.inventory_units.update_all(state: :on_hand)
+            product.master.stock_items.last.update_column(:backorderable, true)
+            product.master.stock_items.last.update_column(:count_on_hand, 0)
+            expect(shipment2.reload).not_to be_backordered
+
+            visit spree.edit_admin_order_path(order)
+
+            within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+            complete_split_to(shipment2, quantity: 1)
+
+            expect(page).to have_content("1 x Backordered")
+
+            within('.stock-contents', text: "1 x On hand") do
+              within('tr', text: line_item.sku) { click_icon 'arrows-h' }
+              complete_split_to(shipment2, quantity: 1)
+            end
+
+            # Empty shipment should be removed
+            expect(page).to have_css('.stock-contents', count: 1)
+            expect(page).to have_content("2 x Backordered")
+          end
+        end
+      end
+
+      describe 'line item sort order' do
+        let(:product2) { create(:product) }
+        let(:product3) { create(:product) }
+
+        before do
+          # grab this one first and then create others that should end up before
+          # and after
+          @middle_line_item = order.line_items[0]
+
+          @first_line_item  = order.contents.add(product2.master)
+          @first_line_item.update_columns(created_at: 1.day.ago)
+          @last_line_item = order.contents.add(product3.master)
+          @last_line_item.update_columns(created_at: 1.day.from_now)
+        end
+
+        it 'orders the items in a shipment by created_at' do
+          visit spree.edit_admin_order_path(order)
+
+          stock_items = page.all(:css, '.stock-item', count: 3)
+
+          expect(stock_items[0]).to have_text(@first_line_item.variant.sku)
+          expect(stock_items[1]).to have_text(@middle_line_item.variant.sku)
+          expect(stock_items[2]).to have_text(@last_line_item.variant.sku)
+        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('Adjustments')
+      expect(page).not_to have_link('Payments')
+      expect(page).not_to have_link('Return Authorizations')
+
+      # 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-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
+
+    stub_api_authentication!
+
+    it 'should not display order tabs or edit buttons without ability' do
+      visit spree.edit_admin_order_path(order)
+
+      # Order Tabs
+      expect(page).not_to have_link('Adjustments')
+      expect(page).not_to have_link('Payments')
+      expect(page).not_to have_link('Return Authorizations')
+    end
+
+    it "can change the shipping method" do
+      order = create(:completed_order_with_totals)
+      visit spree.edit_admin_order_path(order)
+      within("tr", text: "Shipping Method") do
+        click_icon :edit
+      end
+      select "UPS Ground $100.00"
+      click_icon :check
+
+      expect(page).not_to have_css('#selected_shipping_rate_id')
+      expect(page).to have_content("UPS Ground")
+    end
+
+    it 'can ship' do
+      order = create(:order_ready_to_ship)
+      order.refresh_shipment_rates
+      visit spree.edit_admin_order_path(order)
+
+      find(".ship-shipment-button").click
+
+      within '.carton-state' do
+        expect(page).to have_content('Shipped')
+      end
+    end
+  end
+end