spree_backend 3.0.5 → 3.0.6

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

@@ -0,0 +1,55 @@
+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(
+        :source => payment.source,
+        :details => response.to_yaml
+      )
+    end
+
+    it "shows a successful attempt" do
+      visit spree.admin_order_payments_path(payment.order)
+      find("#payment_#{payment.id} a").click
+      click_link "Logs"
+      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)
+      find("#payment_#{payment.id} a").click
+      click_link "Logs"
+      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

@@ -0,0 +1,161 @@
+require 'spec_helper'
+
+describe "New Order", :type => :feature do
+  let!(:product) { create(:product_in_stock) }
+  let!(:state) { create(:state) }
+  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) }
+
+  stub_authorization!
+
+  before do
+    # create default store
+    create(:store)
+    visit spree.new_admin_order_path
+  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 "completes new order succesfully without using the cart", js: true do
+    select2_search product.name, from: Spree.t(:name_or_sku)
+    click_icon :add
+    wait_for_ajax
+    click_on "Customer"
+
+    within "#select-customer" do
+      targetted_select2_search user.email, from: "#s2id_customer_search"
+    end
+
+    check "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_icon "capture"
+
+    click_on "Shipments"
+    click_on "Ship"
+    wait_for_ajax
+
+    expect(page).to have_content("shipped")
+  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
+      select2_search product.name, from: Spree.t(:name_or_sku)
+
+      within("table.stock-levels") do
+        fill_in "variant_quantity", with: 2
+        click_icon :add
+      end
+
+      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
+
+      check "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 #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
+      select2_search product.name, from: Spree.t(:name_or_sku)
+      click_icon :add
+      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 #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
+
+      check "order_use_billing"
+      fill_in_address
+      click_on "Update"
+
+      click_on "Shipments"
+      select2_search product.name, from: Spree.t(:name_or_sku)
+      click_icon :add
+      wait_for_ajax
+
+      click_on "Payments"
+      click_on "Continue"
+
+      within(".additional-info .state") do
+        expect(page).to have_content("complete")
+      end
+    end
+  end
+
+  # Regression test for #5327
+  context "customer with default credit card", js: true do
+    before do
+      create(:credit_card, default: true, user: user)
+    end
+    it "transitions to delivery not to complete" do
+      select2_search product.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
+      click_link "Customer"
+      targetted_select2 user.email, from: "#s2id_customer_search"
+      click_button "Update"
+      expect(Spree::Order.last.state).to eq 'delivery'
+    end
+  end
+
+  def fill_in_address(kind = "bill")
+    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",                       with: "20170"
+    targetted_select2_search state.name, from: "#s2id_order_#{kind}_address_attributes_state_id"
+    fill_in "Phone",                     with: "123-456-7890"
+  end
+end

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

@@ -0,0 +1,620 @@
+# 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!(:tote) { create(:product, name: "Tote", price: 15.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" 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
+        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
+        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
+
+          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.set track_inventory_levels: false
+          product.master.update_column(:track_inventory, true)
+          product.master.stock_items.first.update_column(:backorderable, true)
+          product.master.stock_items.first.update_column(:count_on_hand, 0)
+        end
+
+        it "adds variant to order just fine" do
+          select2_search product.name, from: Spree.t(:name_or_sku)
+          within("table.stock-levels") do
+            fill_in "variant_quantity", with: 1
+            click_icon :add
+          end
+
+          within(".line-items") do
+            expect(page).to have_content(product.name)
+          end
+        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' 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 "site doesn't track inventory" do
+          before do
+            Spree::Config.set track_inventory_levels: false
+            product.master.update_column(:track_inventory, true)
+            product.master.stock_items.first.update_column(:backorderable, true)
+            product.master.stock_items.first.update_column(:count_on_hand, 0)
+          end
+
+          it "adds variant to order just fine" do
+            select2_search product.name, from: Spree.t(:name_or_sku)
+            within("table.stock-levels") do
+              fill_in "stock_item_quantity", with: 1
+              click_icon :add
+            end
+
+            within(".stock-contents") do
+              expect(page).to have_content(product.name)
+            end
+          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' 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