From 1b9d64ad5ea0f2bb4d78bfa523d38b2c1c76eda0 Mon Sep 17 00:00:00 2001 From: Ahmed Ejaz Date: Sat, 21 Jun 2025 16:39:15 +0500 Subject: [PATCH] Refactor search functionality in variants controller spec to include order_id for improved filtering --- .../spree/admin/variants_controller_spec.rb | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/spec/controllers/spree/admin/variants_controller_spec.rb b/spec/controllers/spree/admin/variants_controller_spec.rb index 0eb2350265..4e74f61278 100644 --- a/spec/controllers/spree/admin/variants_controller_spec.rb +++ b/spec/controllers/spree/admin/variants_controller_spec.rb @@ -137,13 +137,6 @@ RSpec.describe Spree::Admin::VariantsController do expect(variant).to have_received(:destroy) end - describe "#search" do - it "filters by distributor and supplier1 products" do - spree_get :search, q: 'Prod', distributor_id: d.id.to_s, search_variants_as: 'supplier' - expect(assigns(:variants)).to eq([v1]) - end - end - it 'shows a success flash message' do spree_delete :destroy, id: variant.id, product_id: variant.product.id, format: 'html' @@ -186,7 +179,12 @@ RSpec.describe Spree::Admin::VariantsController do describe "#search" do it "filters by distributor and supplier1 products" do - spree_get :search, q: 'Prod', distributor_id: d.id.to_s + order = d.distributed_orders.first + spree_get :search, + q: 'Prod', + distributor_id: d.id.to_s, + search_variants_as: 'supplier', + order_id: order.id.to_s expect(assigns(:variants)).to eq([v1]) end end