From 7ca6434b1410b9b2eb831a4592dd96131b996f59 Mon Sep 17 00:00:00 2001
From: Jean-Baptiste Bellet <jb.bellet@gmail.com>
Date: Fri, 4 Mar 2022 11:34:19 +0100
Subject: [PATCH] Improve the test to delete a cloned order_cycle

---
 .../admin/order_cycles_controller_spec.rb       | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/spec/controllers/admin/order_cycles_controller_spec.rb b/spec/controllers/admin/order_cycles_controller_spec.rb
index 0db0b83dfb..747eb6107a 100644
--- a/spec/controllers/admin/order_cycles_controller_spec.rb
+++ b/spec/controllers/admin/order_cycles_controller_spec.rb
@@ -402,18 +402,31 @@ module Admin
         end
       end
 
-      describe "when an order cycle has any coordinator_fees", :debug do
+      describe "when an order cycle has any coordinator_fees" do
         let(:enterprise_fee1) { create(:enterprise_fee) }
 
         before do
           oc.coordinator_fees << enterprise_fee1
         end
 
-        it "displays an error message when we attempt to delete it" do
+        it "actually delete the order cycle" do
           get :destroy, params: { id: oc.id }
           expect(OrderCycle.find_by(id: oc.id)).to be nil
           expect(response).to redirect_to admin_order_cycles_path
         end
+
+        describe "when the order_cycle was previously cloned" do
+          let(:cloned) { oc.clone! }
+
+          it "actually delete the order cycle" do
+            get :destroy, params: { id: cloned.id }
+
+            expect(OrderCycle.find_by(id: cloned.id)).to be nil
+            expect(OrderCycle.find_by(id: oc.id)).to_not be nil
+            expect(EnterpriseFee.find_by(id: enterprise_fee1.id)).to_not be nil
+            expect(response).to redirect_to admin_order_cycles_path
+          end
+        end
       end
     end
   end