solidus_backend 2.10.3 → 2.11.0

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

@@ -1,45 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe "Cancelling + Resuming", type: :feature do
-  stub_authorization!
-
-  let(:user) { build_stubbed(:user, id: 123, spree_api_key: 'fake') }
-
-  before do
-    allow(user).to receive(:has_spree_role?).and_return(true)
-    allow_any_instance_of(Spree::Admin::BaseController).to receive(:try_spree_current_user).and_return(user)
-  end
-
-  let(:order) do
-    order = create(:order)
-    order.update_columns({
-      state: 'complete',
-      completed_at: Time.current
-    })
-    order
-  end
-
-  it "can cancel an order" do
-    visit spree.edit_admin_order_path(order.number)
-    click_button 'Cancel'
-    within(".additional-info") do
-      expect(find('dt#order_status + dd')).to have_content("Canceled")
-    end
-  end
-
-  context "with a cancelled order" do
-    before do
-      order.update_column(:state, 'canceled')
-    end
-
-    it "can resume an order" do
-      visit spree.edit_admin_order_path(order.number)
-      click_button 'Resume'
-      within(".additional-info") do
-        expect(find('dt#order_status + dd')).to have_content("Resumed")
-      end
-    end
-  end
-end

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

@@ -1,50 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe "Cancel items" do
-  stub_authorization!
-
-  let!(:order) do
-    create(
-      :order_ready_to_ship,
-      number: "R100",
-      state: "complete",
-      line_items_count: 1
-    )
-  end
-
-  def visit_order
-    visit spree.admin_path
-    click_link "Orders"
-    within_row(1) do
-      click_link "R100"
-    end
-  end
-
-  context "when some items are cancelable" do
-    it "can cancel the item" do
-      visit_order
-
-      click_link 'Cancel Items'
-
-      within_row(1) do
-        check 'inventory_unit_ids[]'
-      end
-
-      click_button "Cancel Items"
-      expect(page).to have_content("Inventory canceled")
-      expect(page).to have_content("1 x Canceled")
-    end
-  end
-
-  context "when all items are not cancelable" do
-    before { order.inventory_units.each(&:cancel!) }
-
-    it "does not display the link to cancel items" do
-      visit_order
-
-      expect(page).to have_no_content('Cancel Items')
-    end
-  end
-end

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

@@ -1,203 +0,0 @@
-# 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
-      stub_spree_preferences(
-        default_country_iso: country.iso,
-        company: true
-      )
-
-      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!(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
-        stub_spree_preferences(default_country_iso: brazil.iso)
-      end
-
-      it "sets default country when displaying form" do
-        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/customer_returns_spec.rb

@@ -1,41 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_helper'
-
-describe 'Customer returns', type: :feature do
-  stub_authorization!
-
-  context 'when the order has more than one line item' do
-    let(:order) { create :shipped_order, line_items_count: 2 }
-
-    def create_customer_return
-      find('#select-all').click
-      page.execute_script "$('select.add-item').val('receive')"
-      select 'NY Warehouse', from: 'Stock Location'
-      click_button 'Create'
-    end
-
-    before do
-      visit spree.new_admin_order_customer_return_path(order)
-    end
-
-    context 'when creating a return with state "Received"' do
-      it 'marks the order as returned', :js do
-        create_customer_return
-
-        expect(page).to have_content 'Customer Return has been successfully created'
-        within 'dd.order-state' do
-          expect(page).to have_content 'Returned'
-        end
-      end
-    end
-
-    it 'disables the button at submit', :js do
-      page.execute_script "$('form').submit(function(e) { e.preventDefault()})"
-
-      create_customer_return
-
-      expect(page).to have_button("Create", disabled: true)
-    end
-  end
-end

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

@@ -1,52 +0,0 @@
-# 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

@@ -1,186 +0,0 @@
-# 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
-          stub_spree_preferences(orders_per_page: 1)
-        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