solidus_backend 2.9.6 → 2.11.0

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

@@ -1,56 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe "Log entries", type: :feature do
-  stub_authorization!
-
-  let!(:payment) { create(:payment) }
-
-  context "with a successful log entry" do
-    before do
-      response = ActiveMerchant::Billing::Response.new(
-        true,
-        "Transaction successful",
-        transid: "ABCD1234"
-      )
-
-      payment.log_entries.create(
-        details: response.to_yaml
-      )
-    end
-
-    it "shows a successful attempt" do
-      visit spree.admin_order_payments_path(payment.order)
-      click_on payment.number
-
-      within("#listing_log_entries") do
-        expect(page).to have_content("Transaction successful")
-      end
-    end
-  end
-
-  context "with a failed log entry" do
-    before do
-      response = ActiveMerchant::Billing::Response.new(
-        false,
-        "Transaction failed",
-        transid: "ABCD1234"
-      )
-
-      payment.log_entries.create(
-        source: payment.source,
-        details: response.to_yaml
-      )
-    end
-
-    it "shows a failed attempt" do
-      visit spree.admin_order_payments_path(payment.order)
-      click_on payment.number
-
-      within("#listing_log_entries") do
-        expect(page).to have_content("Transaction failed")
-      end
-    end
-  end
-end

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

@@ -1,331 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe "New Order", type: :feature do
-  include OrderFeatureHelper
-
-  let!(:product) { create(:product_in_stock) }
-  let!(:state) { create(:state, state_code: 'CA') }
-  let!(:store) { create(:store) }
-  let!(:user) { create(:user, ship_address: create(:address), bill_address: create(:address)) }
-  let!(:payment_method) { create(:check_payment_method) }
-  let!(:shipping_method) { create(:shipping_method, cost: 0.0) }
-
-  stub_authorization!
-
-  before do
-    visit spree.admin_path
-    click_on "Orders"
-    click_on "New Order"
-  end
-
-  it "does check if you have a billing address before letting you add shipments" do
-    click_on "Shipments"
-    expect(page).to have_content 'Please fill in customer info'
-    expect(current_path).to eql(spree.edit_admin_order_customer_path(Spree::Order.last))
-  end
-
-  it "default line item quantity is 1", js: true do
-    within ".line-items" do
-      expect(page).to have_field 'quantity', with: '1'
-    end
-  end
-
-  it "completes new order succesfully without using the cart", js: true do
-    add_line_item product.name
-
-    click_on "Customer"
-
-    within "#select-customer" do
-      targetted_select2_search user.email, from: "#s2id_customer_search"
-    end
-
-    expect(page).to have_checked_field('order_use_billing')
-    fill_in_address
-    click_on "Update"
-
-    click_on "Payments"
-    click_on "Update"
-
-    expect(current_path).to eql(spree.admin_order_payments_path(Spree::Order.last))
-
-    click_on "Confirm"
-    click_on "Complete Order"
-
-    expect(current_path).to eql(spree.edit_admin_order_path(Spree::Order.last))
-
-    click_on "Payments"
-    click_icon "capture"
-
-    click_on "Shipments"
-    click_on "Ship"
-
-    within '.carton-state' do
-      expect(page).to have_content('Shipped')
-    end
-  end
-
-  it 'can create split payments', js: true do
-    add_line_item product.name
-
-    click_on "Customer"
-
-    within "#select-customer" do
-      targetted_select2_search user.email, from: "#s2id_customer_search"
-    end
-
-    expect(page).to have_checked_field('order_use_billing')
-    fill_in_address
-    click_on "Update"
-
-    click_on "Payments"
-    fill_in "Amount", with: '10.00'
-    click_on 'Update'
-
-    click_on 'New Payment'
-    fill_in "Amount", with: '29.98'
-    click_on 'Update'
-
-    expect(page).to have_content("$10.00")
-    expect(page).to have_content("$29.98")
-  end
-
-  context "adding new item to the order", js: true do
-    it "inventory items show up just fine and are also registered as shipments" do
-      add_line_item product.name
-
-      within(".line-items") do
-        expect(page).to have_content(product.name)
-      end
-
-      click_on "Customer"
-
-      within "#select-customer" do
-        targetted_select2_search user.email, from: "#s2id_customer_search"
-      end
-
-      expect(page).to have_checked_field('order_use_billing')
-      fill_in_address
-      click_on "Update"
-
-      click_on "Shipments"
-
-      within(".stock-contents") do
-        expect(page).to have_content(product.name)
-      end
-    end
-  end
-
-  # Regression test for https://github.com/spree/spree/issues/3958
-  context "without a delivery step", js: true do
-    before do
-      allow(Spree::Order).to receive_messages checkout_step_names: [:address, :payment, :confirm, :complete]
-    end
-
-    it "can still see line items" do
-      add_line_item product.name
-
-      within(".line-items") do
-        within(".line-item-name") do
-          expect(page).to have_content(product.name)
-        end
-        within(".line-item-qty-show") do
-          expect(page).to have_content("1")
-        end
-        within(".line-item-price") do
-          expect(page).to have_content(product.price)
-        end
-      end
-    end
-  end
-
-  # Regression test for https://github.com/spree/spree/issues/3336
-  context "start by customer address" do
-    it "completes order fine", js: true do
-      click_on "Customer"
-
-      within "#select-customer" do
-        targetted_select2_search user.email, from: "#s2id_customer_search"
-      end
-
-      expect(page).to have_checked_field('order_use_billing')
-      fill_in_address
-      click_on "Update"
-
-      # Automatically redirected to Shipments page
-      select2_search product.name, from: I18n.t('spree.name_or_sku')
-
-      click_icon :plus
-
-      expect(page).to have_css('.stock-item')
-
-      click_on "Payments"
-      click_on "Continue"
-
-      within(".additional-info") do
-        expect(page).to have_content("Confirm")
-      end
-    end
-  end
-
-  # Regression test for https://github.com/spree/spree/issues/5327
-  context "customer with default credit card", js: true do
-    let!(:credit_card) { create(:credit_card, user: user) }
-
-    before do
-      user.wallet.add(credit_card)
-    end
-
-    it "transitions to delivery not to complete" do
-      add_line_item product.name
-
-      expect(page).to have_css('.line-item')
-
-      click_link "Customer"
-      targetted_select2 user.email, from: "#s2id_customer_search"
-      click_button "Update"
-      expect(page).to have_css('.order-state', text: 'Delivery')
-    end
-  end
-
-  context "customer with attempted XSS", js: true do
-    let(:xss_string) { %(<script>throw("XSS")</script>) }
-    before do
-      user.update!(email: xss_string)
-    end
-    it "displays the user's email escaped without executing" do
-      click_on "Customer"
-      targetted_select2_search user.email, from: "#s2id_customer_search"
-      expect(page).to have_field("Customer E-Mail", with: xss_string)
-    end
-  end
-
-  context 'with a checkout_zone set as the country of Canada' do
-    let!(:canada) { create(:country, iso: 'CA', states_required: true) }
-    let!(:canada_state) { create(:state, country: canada) }
-    let!(:checkout_zone) { create(:zone, name: 'Checkout Zone', countries: [canada]) }
-
-    before do
-      Spree::Country.update_all(states_required: true)
-      stub_spree_preferences(checkout_zone: checkout_zone.name)
-    end
-
-    context 'and default_country_iso of the United States' do
-      before do
-        stub_spree_preferences(default_country_iso: Spree::Country.find_by!(iso: 'US').iso)
-      end
-
-      it 'the shipping address country select includes only options for Canada' do
-        visit spree.new_admin_order_path
-        click_link 'Customer'
-        within '#shipping' do
-          expect(page).to have_select(
-            'Country',
-            options: ['Canada']
-          )
-        end
-      end
-
-      it 'does not show any shipping address state' do
-        visit spree.new_admin_order_path
-        click_link 'Customer'
-        within '#shipping' do
-          expect(page).to have_select(
-            'State',
-            disabled: true,
-            visible: false,
-            options: ['']
-          )
-        end
-      end
-
-      it 'the billing address country select includes only options for Canada' do
-        visit spree.new_admin_order_path
-        click_link 'Customer'
-        within '#billing' do
-          expect(page).to have_select(
-            'Country',
-            options: ['Canada']
-          )
-        end
-      end
-
-      it 'does not show any billing address state' do
-        visit spree.new_admin_order_path
-        click_link 'Customer'
-        within '#billing' do
-          expect(page).to have_select(
-            'State',
-            disabled: true,
-            visible: false,
-            options: ['']
-          )
-        end
-      end
-    end
-
-    context 'and default_country_iso of Canada' do
-      before do
-        stub_spree_preferences(default_country_iso: Spree::Country.find_by!(iso: 'CA').iso)
-      end
-
-      it 'defaults the shipping address country to Canada' do
-        visit spree.new_admin_order_path
-        click_link 'Customer'
-        within '#shipping' do
-          expect(page).to have_select(
-            'Country',
-            selected: 'Canada',
-            options: ['Canada']
-          )
-        end
-      end
-
-      it 'shows relevant shipping address states' do
-        visit spree.new_admin_order_path
-        click_link 'Customer'
-        within '#shipping' do
-          expect(page).to have_select(
-            'State',
-            options: [''] + canada.states.map(&:name)
-          )
-        end
-      end
-
-      it 'defaults the billing address country to Canada' do
-        visit spree.new_admin_order_path
-        click_link 'Customer'
-        within '#billing' do
-          expect(page).to have_select(
-            'Country',
-            selected: 'Canada',
-            options: ['Canada']
-          )
-        end
-      end
-
-      it 'shows relevant billing address states' do
-        visit spree.new_admin_order_path
-        click_link 'Customer'
-        within '#billing' do
-          expect(page).to have_select(
-            'State',
-            options: [''] + canada.states.map(&:name)
-          )
-        end
-      end
-    end
-  end
-
-  def fill_in_address
-    fill_in "First Name",                with: "John 99"
-    fill_in "Last Name",                 with: "Doe"
-    fill_in "Street Address",            with: "100 first lane"
-    fill_in "Street Address (cont'd)",   with: "#101"
-    fill_in "City",                      with: "Bethesda"
-    fill_in "Zip Code",                  with: "20170"
-    select state.name, from: "State"
-    fill_in "Phone", with: "123-456-7890"
-  end
-end

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

@@ -1,35 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-RSpec.describe 'New Refund creation', :js do
-  stub_authorization!
-
-  let(:order) { create :order_ready_to_ship }
-  let(:payment) { order.payments.first }
-  let(:amount) { '10.99' }
-  let!(:reason) { create :refund_reason }
-
-  it 'creates a new refund' do
-    visit spree.new_admin_order_payment_refund_path(order, payment)
-    expect(page).not_to have_selector 'td', text: amount
-    within '.new_refund' do
-      fill_in 'refund_amount', with: amount
-      select reason.name, from: 'Reason'
-      click_button 'Refund'
-    end
-    expect(page).to have_content 'Refund has been successfully created!'
-    expect(page).to have_selector 'td', text: amount
-  end
-
-  it 'disables the button at submit' do
-    visit spree.new_admin_order_payment_refund_path(order, payment)
-    page.execute_script "$('form').submit(function(e) { e.preventDefault()})"
-    within '.new_refund' do
-      fill_in 'refund_amount', with: amount
-      select reason.name, from: 'Reason'
-      click_button 'Refund'
-      expect(find('input[type="submit"]')).to be_disabled
-    end
-  end
-end

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

@@ -1,581 +0,0 @@
-# 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
-        it 'should warn you if you have not selected a location or shipment' do
-          visit spree.edit_admin_order_path(order)
-
-          within('tr', text: line_item.sku) { click_icon 'arrows-h' }
-          accept_alert 'Please select the split destination.' do
-            click_icon :ok
-          end
-        end
-
-        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(:try_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
-
-    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' 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