From f20a15e955fab5da8bfd30822eb23df70b8479ce Mon Sep 17 00:00:00 2001
From: Pau Perez <saulopefa@gmail.com>
Date: Wed, 10 Apr 2019 16:16:00 +0200
Subject: [PATCH] Add specs for the cache refresh on variant removal

This is failing in `master` but thanks to the VariantDeleter it does work
in v2. This specs prove it.
---
 .../spree/api/variants_controller_spec.rb     | 24 ++++++++++++++++---
 1 file changed, 21 insertions(+), 3 deletions(-)

diff --git a/spec/controllers/spree/api/variants_controller_spec.rb b/spec/controllers/spree/api/variants_controller_spec.rb
index af50d783e3..4f5be2dbf2 100644
--- a/spec/controllers/spree/api/variants_controller_spec.rb
+++ b/spec/controllers/spree/api/variants_controller_spec.rb
@@ -55,15 +55,24 @@ module Spree
         lambda { variant.reload }.should_not raise_error
         variant.deleted_at.should be_nil
       end
+
+      context 'when the variant is not the master' do
+        before { variant.update_attribute(:is_master, false) }
+
+        it 'refreshes the cache' do
+          expect(OpenFoodNetwork::ProductsCache).to receive(:variant_destroyed).with(variant)
+          spree_delete :soft_delete, variant_id: variant.id, product_id: variant.product.permalink, format: :json
+        end
+      end
     end
 
     context "as an administrator" do
       sign_in_as_admin!
 
-      it "soft deletes a variant" do
-        product = create(:product)
-        variant = product.master
+      let(:product) { create(:product) }
+      let(:variant) { product.master }
 
+      it "soft deletes a variant" do
         spree_delete :soft_delete, {variant_id: variant.to_param, product_id: product.to_param, format: :json}
         response.status.should == 204
         lambda { variant.reload }.should_not raise_error
@@ -79,6 +88,15 @@ module Spree
         expect(variant.reload).to_not be_deleted
         expect(assigns(:variant).errors[:product]).to include "must have at least one variant"
       end
+
+      context 'when the variant is not the master' do
+        before { variant.update_attribute(:is_master, false) }
+
+        it 'refreshes the cache' do
+          expect(OpenFoodNetwork::ProductsCache).to receive(:variant_destroyed).with(variant)
+          spree_delete :soft_delete, variant_id: variant.id, product_id: variant.product.permalink, format: :json
+        end
+      end
     end
   end
 end