From 2105c0d0eadbe170b8cea91c95fb49fe2db4f943 Mon Sep 17 00:00:00 2001 From: Maikel Linke Date: Tue, 14 Mar 2023 14:21:10 +1100 Subject: [PATCH] Ignore master variants exporting to DFC --- engines/dfc_provider/app/services/variant_fetcher.rb | 2 +- engines/dfc_provider/spec/services/variant_fetcher_spec.rb | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/engines/dfc_provider/app/services/variant_fetcher.rb b/engines/dfc_provider/app/services/variant_fetcher.rb index 3947a12b65..e99eb8dd02 100644 --- a/engines/dfc_provider/app/services/variant_fetcher.rb +++ b/engines/dfc_provider/app/services/variant_fetcher.rb @@ -9,7 +9,7 @@ class VariantFetcher end def scope - Spree::Variant. + Spree::Variant.not_master. joins(product: :supplier). where('enterprises.id' => @enterprise.id) end diff --git a/engines/dfc_provider/spec/services/variant_fetcher_spec.rb b/engines/dfc_provider/spec/services/variant_fetcher_spec.rb index 8a7adebc15..84a916c43f 100644 --- a/engines/dfc_provider/spec/services/variant_fetcher_spec.rb +++ b/engines/dfc_provider/spec/services/variant_fetcher_spec.rb @@ -14,8 +14,6 @@ describe VariantFetcher do it "returns the variants of a supplier" do product = create(:product, supplier: enterprise) - pending "ignoring of the master variant" - # it currently returns two variants instead of one expect(subject.scope.count).to eq 1 expect(subject.scope).to eq product.variants end