diff --git a/spec/features/admin/order_spec.rb b/spec/features/admin/order_spec.rb index f41b1df2e1..34921c68d1 100644 --- a/spec/features/admin/order_spec.rb +++ b/spec/features/admin/order_spec.rb @@ -174,28 +174,61 @@ feature ' expect(order.reload.line_items.first.quantity).to eq(1000) end - scenario "there is a variant override" do - # Move the order back to the cart state - order.state = 'cart' - order.completed_at = nil - variant = order.line_items.first.variant - variant.update_attribute(:on_demand, false) - variant.update_attribute(:on_hand, 0) - override = create(:variant_override, hub: order.distributor, variant: variant, count_on_hand: 100) + # Regression test for #7337 + context "creating a new order with a variant override" do + let!(:override) { create(:variant_override, hub: distributor, variant: product.variants.first, + count_on_hand: 100) } - login_as_admin_and_visit spree.edit_admin_order_path(order) + before do + product.variants.first.update(on_demand: false, on_hand: 0) - within("tr.stock-item", text: order.products.first.name) do - find("a.edit-item").click - expect(page).to have_input(:quantity) - fill_in(:quantity, with: 50) - find("a.save-item").click + login_as user + new_order_with_distribution(distributor, order_cycle) + expect(page).to have_content I18n.t('spree.add_product').upcase end - within("tr.stock-item", text: order.products.first.name) do - expect(page).to have_text("50 x") + it "creates order and shipment successfully and allows proceeding to payment" do + select2_select product.name, from: 'add_variant_id', search: true + + within("table.stock-levels") do + expect(page).to have_selector("#stock_item_quantity") + fill_in "stock_item_quantity", with: 50 + find("button.add_variant").click + end + + expect(page).to_not have_selector("table.stock-levels") + expect(page).to have_selector("table.stock-contents") + + within("tr.stock-item") do + expect(page).to have_text("50 x") + end + + order = Spree::Order.last + expect(order.line_items.first.quantity).to eq(50) + expect(order.shipments.count).to eq(1) + + click_button "Update And Recalculate Fees" + expect(page).to have_selector 'h1', text: "Customer Details" + + fill_in "order_email", with: "test@test.com" + check "order_use_billing" + fill_in "order_bill_address_attributes_firstname", with: "xxx" + fill_in "order_bill_address_attributes_lastname", with: "xxx" + fill_in "order_bill_address_attributes_address1", with: "xxx" + fill_in "order_bill_address_attributes_city", with: "xxx" + fill_in "order_bill_address_attributes_zipcode", with: "xxx" + select "Australia", from: "order_bill_address_attributes_country_id" + select "Victoria", from: "order_bill_address_attributes_state_id" + fill_in "order_bill_address_attributes_phone", with: "xxx" + + click_button "Update" + + expect(page).to have_content "Customer Details updated" + + click_link "Payments" + + expect(page).to have_content "New Payment" end - expect(order.reload.line_items.first.quantity).to eq(50) end scenario "can't change distributor or order cycle once order has been finalized" do