I have added the before_destroy callback to the Crop model to destroy all CropCompanion records where the crop is crop_b. (#4266)

I have added a new test to `spec/models/crop_spec.rb` to verify that deleting a crop also destroys the associated `CropCompanion` records.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>

Author: CloCkWeRX

app/models/crop.rb

@@ -165,8 +165,14 @@ def all_companions
     (companions + parent.companions).uniq
   end
 
+  before_destroy :destroy_reverse_companionships
+
   private
 
+  def destroy_reverse_companionships
+    CropCompanion.where(crop_b: self).destroy_all
+  end
+
   def count_uses_of_property(col_name)
     plantings.unscoped
       .where(crop_id: id)

spec/models/crop_spec.rb

@@ -544,6 +544,20 @@
     end
   end
 
+  context "destroying a crop" do
+    let!(:crop_a) { FactoryBot.create(:crop) }
+    let!(:crop_b) { FactoryBot.create(:crop) }
+
+    before do
+      CropCompanion.create(crop_a: crop_a, crop_b: crop_b)
+      CropCompanion.create(crop_a: crop_b, crop_b: crop_a)
+    end
+
+    it "destroys companion links" do
+      expect { crop_a.destroy }.to change { CropCompanion.count }.from(2).to(0)
+    end
+  end
+
   context "crop rejections" do
     let!(:rejected_reason) do
       FactoryBot.create(:crop, name:                 'tomato',