Do not retry when refreshing cache on deleted OC

app/jobs/refresh_products_cache_job.rb

@@ -3,6 +3,8 @@ require 'open_food_network/products_renderer'
 RefreshProductsCacheJob = Struct.new(:distributor_id, :order_cycle_id) do
   def perform
     Rails.cache.write(key, products_json)
+  rescue ActiveRecord::RecordNotFound
+    true
   end
 
   private

spec/jobs/refresh_products_cache_job_spec.rb

@@ -5,12 +5,34 @@ describe RefreshProductsCacheJob do
   let(:distributor) { create(:distributor_enterprise) }
   let(:order_cycle) { create(:simple_order_cycle) }
 
-  it "renders products and writes them to cache" do
-    RefreshProductsCacheJob.any_instance.stub(:products_json) { 'products' }
+  context 'when the enterprise and the order cycle exist' do
+    it "renders products and writes them to cache" do
+      RefreshProductsCacheJob.any_instance.stub(:products_json) { 'products' }
 
-    run_job RefreshProductsCacheJob.new distributor.id, order_cycle.id
+      run_job RefreshProductsCacheJob.new distributor.id, order_cycle.id
 
-    expect(Rails.cache.read("products-json-#{distributor.id}-#{order_cycle.id}")).to eq 'products'
+      expect(Rails.cache.read("products-json-#{distributor.id}-#{order_cycle.id}")).to eq 'products'
+    end
+  end
+
+  context 'when the order cycle does not exist' do
+    before do
+      allow(OrderCycle)
+        .to receive(:find)
+        .with(order_cycle.id)
+        .and_raise(ActiveRecord::RecordNotFound)
+    end
+
+    it 'does not raise' do
+      expect {
+        run_job RefreshProductsCacheJob.new(distributor.id, order_cycle.id)
+      }.not_to raise_error(/ActiveRecord::RecordNotFound/)
+    end
+
+    it 'returns true' do
+      refresh_products_cache_job = RefreshProductsCacheJob.new(distributor.id, order_cycle.id)
+      expect(refresh_products_cache_job.perform).to eq(true)
+    end
   end
 
   describe "fetching products JSON" do