solidus_backend_devise_token_auth 2.8.0.alpha.2

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

@@ -0,0 +1,206 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "Customer Details", type: :feature, js: true do
+  include OrderFeatureHelper
+
+  stub_authorization!
+
+  let(:country) { create(:country, name: "Kangaland") }
+  let(:state) { create(:state, name: "Alabama", country: country) }
+  let!(:shipping_method) { create(:shipping_method) }
+  let!(:order) { create(:order, ship_address: ship_address, bill_address: bill_address, state: 'complete', completed_at: "2011-02-01 12:36:15") }
+  let!(:product) { create(:product_in_stock) }
+
+  # We need a unique name that will appear for the customer dropdown
+  let!(:ship_address) { create(:address, country: country, state: state, first_name: "Rumpelstiltskin") }
+  let!(:bill_address) { create(:address, country: country, state: state, first_name: "Rumpelstiltskin") }
+
+  let!(:user) { create(:user, email: 'foobar@example.com', ship_address: ship_address, bill_address: bill_address) }
+
+  context "brand new order" do
+    let(:quantity) { 1 }
+
+    before do
+      visit spree.admin_path
+      click_link "Orders"
+      click_link "New Order"
+
+      add_line_item product.name, quantity: quantity
+
+      expect(page).to have_css('.line-item')
+      click_link "Customer"
+      targetted_select2 "foobar@example.com", from: "#s2id_customer_search"
+    end
+
+    # Regression test for https://github.com/spree/spree/issues/3335 and https://github.com/spree/spree/issues/5317
+    it "associates a user when not using guest checkout" do
+      # 5317 - Address prefills using user's default.
+      expect(page).to have_field('First Name', with: user.bill_address.firstname)
+      expect(page).to have_field('Last Name', with: user.bill_address.lastname)
+      expect(page).to have_field('Street Address', with: user.bill_address.address1)
+      expect(page).to have_field("Street Address (cont'd)", with: user.bill_address.address2)
+      expect(page).to have_field('City', with: user.bill_address.city)
+      expect(page).to have_field('Zip Code', with: user.bill_address.zipcode)
+      expect(page).to have_select('Country', selected: "United States of America", visible: false)
+      expect(page).to have_select('State', selected: user.bill_address.state.name, visible: false)
+      expect(page).to have_field('Phone', with: user.bill_address.phone)
+      click_button "Update"
+      expect(Spree::Order.last.user).not_to be_nil
+    end
+
+    # Regression test for https://github.com/solidusio/solidus/pull/2176
+    it "does not reset guest checkout to true when returning to customer tab" do
+      click_button "Update"
+      click_link "Customer"
+      expect(find('#guest_checkout_true')).not_to be_checked
+    end
+
+    context "when required quantity is more than available" do
+      let(:quantity) { 11 }
+      let!(:product) { create(:product_not_backorderable) }
+
+      it "displays an error" do
+        click_button "Update"
+        expect(page).to have_content I18n.t('spree.insufficient_stock_for_order')
+      end
+    end
+  end
+
+  context "editing an order" do
+    before do
+      configure_spree_preferences do |config|
+        config.default_country_iso = country.iso
+        config.company = true
+      end
+
+      visit spree.admin_path
+      click_link "Orders"
+      within('table#listing_orders') { click_icon(:edit) }
+    end
+
+    context "selected country has no state" do
+      before { create(:country, iso: "BRA", name: "Brazil") }
+
+      it "changes state field to text input" do
+        click_link "Customer"
+
+        within("#billing") do
+          select "Brazil", from: "Country"
+          fill_in "order_bill_address_attributes_state_name", with: "Piaui"
+        end
+
+        click_button "Update"
+        expect(page).to have_content "Customer Details Updated"
+        click_link "Customer"
+        expect(page).to have_field("order_bill_address_attributes_state_name", with: "Piaui")
+      end
+    end
+
+    it "should be able to update customer details for an existing order" do
+      order.ship_address = create(:address)
+      order.save!
+
+      click_link "Customer"
+      within("#shipping") { fill_in_address }
+      within("#billing") { fill_in_address }
+
+      click_button "Update"
+      click_link "Customer"
+
+      # Regression test for https://github.com/spree/spree/issues/2950 and https://github.com/spree/spree/issues/2433
+      # This act should transition the state of the order as far as it will go too
+      within("#order_tab_summary") do
+        expect(find("dt#order_status + dd")).to have_content("Complete")
+      end
+    end
+
+    it "should show validation errors" do
+      order.update_attributes!(ship_address_id: nil)
+      click_link "Customer"
+      click_button "Update"
+      expect(page).to have_content("Shipping address first name can't be blank")
+    end
+
+    context "for an order in confirm state with a user" do
+      let(:user) { order.user }
+
+      before do
+        order.update_columns(
+          ship_address_id: ship_address.id,
+          bill_address_id: bill_address.id,
+          state: "confirm",
+          completed_at: nil
+        )
+      end
+
+      it "updating order email works" do
+        click_link "Customer"
+        fill_in "order_email", with: "newemail@example.com"
+        click_button "Update"
+        expect(page).to have_content 'Customer Details Updated'
+        click_link "Customer"
+        expect(page).to have_field 'Customer E-Mail', with: order.reload.email
+        within '#order_user_link' do
+          expect(page).to have_link user.email
+        end
+      end
+    end
+
+    context "country associated was removed" do
+      let(:brazil) { create(:country, iso: "BR", name: "Brazil") }
+
+      before do
+        order.bill_address.country.destroy
+        configure_spree_preferences do |config|
+          config.default_country_iso = brazil.iso
+        end
+      end
+
+      it "sets default country when displaying form" do
+        click_link "Cart"
+        click_link "Customer"
+        expect(page).to have_field("order_bill_address_attributes_country_id", with: brazil.id, visible: false)
+      end
+    end
+
+    # Regression test for https://github.com/spree/spree/issues/942
+    context "errors when no shipping methods are available" do
+      before do
+        Spree::ShippingMethod.delete_all
+      end
+
+      specify do
+        click_link "Customer"
+        # Need to fill in valid information so it passes validations
+        fill_in "order_ship_address_attributes_firstname",  with: "John 99"
+        fill_in "order_ship_address_attributes_lastname",   with: "Doe"
+        fill_in "order_ship_address_attributes_lastname",   with: "Company"
+        fill_in "order_ship_address_attributes_address1",   with: "100 first lane"
+        fill_in "order_ship_address_attributes_address2",   with: "#101"
+        fill_in "order_ship_address_attributes_city",       with: "Bethesda"
+        fill_in "order_ship_address_attributes_zipcode",    with: "20170"
+
+        within("#shipping") do
+          select 'Alabama', from: "State"
+        end
+
+        fill_in "order_ship_address_attributes_phone", with: "123-456-7890"
+        click_button "Update"
+      end
+    end
+  end
+
+  def fill_in_address
+    fill_in "First Name",              with: "John 99"
+    fill_in "Last Name",               with: "Doe"
+    fill_in "Company",                 with: "Company"
+    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 'Alabama', from: "State"
+    fill_in "Phone", with: "123-456-7890"
+  end
+end

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

@@ -0,0 +1,52 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+# Tests for https://github.com/spree/spree/issues/3958's features
+describe "Order Line Items", type: :feature, js: true do
+  stub_authorization!
+
+  before do
+    # Removing the delivery step causes the order page to render a different
+    # partial, called _line_items, which shows line items rather than shipments
+    allow(Spree::Order).to receive_messages checkout_step_names: [:address, :payment, :confirm, :complete]
+  end
+
+  let!(:order) do
+    order = create(:order_with_line_items, line_items_count: 1)
+    order.shipments.destroy_all
+    order
+  end
+
+  it "can edit a line item's quantity" do
+    visit spree.cart_admin_order_path(order)
+    within(".line-items") do
+      find(".edit-line-item").click
+      fill_in "quantity", with: 10
+      find(".save-line-item").click
+    end
+    within '.line-item-qty-show' do
+      expect(page).to have_content("10")
+    end
+    within '.line-item-total' do
+      expect(page).to have_content("$100.00")
+    end
+  end
+
+  it "can delete a line item" do
+    visit spree.cart_admin_order_path(order)
+
+    product_name = find(".line-items tr:nth-child(1) .line-item-name").text
+
+    within(".line-items") do
+      within_row(1) do
+        accept_alert do
+          find(".delete-line-item").click
+        end
+      end
+    end
+
+    expect(find('#order_total')).to have_content("$0.00")
+    expect(page).not_to have_content(product_name)
+  end
+end

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

@@ -0,0 +1,191 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "Orders Listing", type: :feature, js: true do
+  stub_authorization!
+
+  let!(:promotion) { create(:promotion_with_item_adjustment, code: "vnskseiw") }
+  let(:promotion_code) { promotion.codes.first }
+
+  before(:each) do
+    allow_any_instance_of(Spree::OrderInventory).to receive(:add_to_shipment)
+    @order1 = create(:order_with_line_items, created_at: 1.day.from_now, completed_at: 1.day.from_now, number: "R100")
+    @order2 = create(:order, created_at: 1.day.ago, completed_at: 1.day.ago, number: "R200")
+    visit spree.admin_orders_path
+  end
+
+  it 'displays the new order button' do
+    expect(page).to have_link('New Order')
+  end
+
+  context 'without create permission' do
+    custom_authorization! do |_user|
+      can :manage, Spree::Order
+      cannot :create, Spree::Order
+    end
+
+    it 'does not display the new order button' do
+      expect(page).to_not have_link('New Order')
+    end
+  end
+
+  context "listing orders" do
+    it "should list existing orders" do
+      within_row(1) do
+        expect(column_text(2)).to eq "R100"
+        expect(column_text(3)).to eq "Cart"
+      end
+
+      within_row(2) do
+        expect(column_text(2)).to eq "R200"
+      end
+    end
+
+    it "should be able to sort the orders listing" do
+      # default is completed_at desc
+      within_row(1) { expect(page).to have_content("R100") }
+      within_row(2) { expect(page).to have_content("R200") }
+
+      click_link "Completed At", exact: false
+
+      # Completed at desc
+      within_row(1) { expect(page).to have_content("R200") }
+      within_row(2) { expect(page).to have_content("R100") }
+
+      within('table#listing_orders thead') { click_link "Number" }
+
+      # number asc
+      within_row(1) { expect(page).to have_content("R100") }
+      within_row(2) { expect(page).to have_content("R200") }
+    end
+  end
+
+  context "searching orders" do
+    context "when there are multiple stores" do
+      let(:stores) { FactoryBot.create_pair(:store) }
+
+      before do
+        stores.each do |store|
+          FactoryBot.create(:completed_order_with_totals, number: "R#{store.id}999", store: store)
+        end
+      end
+
+      it "can find the orders belonging to a specific store" do
+        main_store, other_store = stores
+
+        click_on "Filter Results"
+        select main_store.name, from: I18n.t('spree.store')
+        click_on "Filter Results"
+
+        within_row(1) do
+          expect(page).to have_content("R#{main_store.id}999")
+        end
+
+        # Ensure that the other order doesn't show up
+        within("table#listing_orders") { expect(page).not_to have_content("R#{other_store.id}999") }
+      end
+    end
+
+    context "when there's a single store" do
+      it "should be able to search orders" do
+        click_on "Filter Results"
+        fill_in "q_number_start", with: "R200"
+        click_on 'Filter Results'
+        within_row(1) do
+          expect(page).to have_content("R200")
+        end
+
+        # Ensure that the other order doesn't show up
+        within("table#listing_orders") { expect(page).not_to have_content("R100") }
+      end
+
+      it "should be able to filter on variant_id" do
+        click_on "Filter Results"
+        select2_search @order1.products.first.sku, from: I18n.t('spree.variant')
+        click_on 'Filter Results'
+
+        within_row(1) do
+          expect(page).to have_content(@order1.number)
+        end
+
+        expect(page).not_to have_content(@order2.number)
+      end
+
+      context "when pagination is really short" do
+        before do
+          @old_per_page = Spree::Config[:orders_per_page]
+          Spree::Config[:orders_per_page] = 1
+        end
+
+        after do
+          Spree::Config[:orders_per_page] = @old_per_page
+        end
+
+        # Regression test for https://github.com/spree/spree/issues/4004
+        it "should be able to go from page to page for incomplete orders" do
+          10.times { Spree::Order.create email: "incomplete@example.com" }
+          click_on "Filter Results"
+          uncheck "q_completed_at_not_null"
+          click_on 'Filter Results'
+          within(".pagination", match: :first) do
+            click_link "2"
+          end
+          expect(page).to have_content("incomplete@example.com")
+          click_on "Filter Results"
+          expect(find("#q_completed_at_not_null")).not_to be_checked
+        end
+      end
+
+      it "should be able to search orders using only completed at input" do
+        click_on "Filter Results"
+        fill_in "q_created_at_gt", with: Date.current
+
+        click_on 'Filter Results'
+        within_row(1) { expect(page).to have_content("R100") }
+
+        # Ensure that the other order doesn't show up
+        within("table#listing_orders") { expect(page).not_to have_content("R200") }
+      end
+
+      context "filter on promotions" do
+        before(:each) do
+          @order1.order_promotions.build(
+            promotion: promotion,
+            promotion_code: promotion_code
+          )
+          @order1.save
+          visit spree.admin_orders_path
+        end
+
+        it "only shows the orders with the selected promotion" do
+          click_on "Filter Results"
+          fill_in "q_order_promotions_promotion_code_value_start", with: promotion.codes.first.value
+          click_on 'Filter Results'
+          within_row(1) { expect(page).to have_content("R100") }
+          within("table#listing_orders") { expect(page).not_to have_content("R200") }
+        end
+      end
+
+      context "when toggling the completed orders checkbox" do
+        before do
+          create(:order, number: 'R300', completed_at: nil, state: 'cart')
+        end
+
+        it "shows both complete and incomplete orders" do
+          check "q_completed_at_not_null"
+          click_on "Filter Results"
+
+          expect(page).to have_content("R200")
+          expect(page).to_not have_content("R300")
+
+          uncheck "q_completed_at_not_null"
+          click_on 'Filter Results'
+
+          expect(page).to have_content("R200")
+          expect(page).to have_content("R300")
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,56 @@
+# 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
+      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)
+      click_on payment.number
+      click_link "Logs"
+      within("#listing_log_entries") do
+        expect(page).to have_content("Transaction failed")
+      end
+    end
+  end
+end