Skip void delete calls

#3

CHANGELOG.md

@@ -1,6 +1,9 @@
 # Changelog
 
 ## Unreleased
+### Fixed
+- fixed unnecessary void DELETE calls for empty relations
+  - thanks to [Richard Nienaber](https://github.com/rjnienaber) for the report
 
 ## v1.2.1
 ### Fixed

lib/delete_recursively.rb

@@ -40,7 +40,7 @@ def delete_recursively(reflection, _legacy_arg, owner_ids, seen: [], force: fals
 
         if dest_method = destructive_method(reflection, force: force)
           record_ids ||= DependentIdFinder.call(owner_ids, reflection, assoc_class)
-          assoc_class.send(dest_method, record_ids)
+          assoc_class.send(dest_method, record_ids) unless record_ids.empty?
         end
       end
     end

spec/delete_recursively_spec.rb

@@ -30,6 +30,16 @@
       blog.destroy!
     end
 
+    it 'does not trigger DB calls for empty relations' do
+      expect(Post).to receive(:delete)
+      expect(Comment).to receive(:destroy)
+      blog_with_posts_and_comments.destroy!
+
+      expect(Post).not_to receive(:delete)
+      expect(Comment).not_to receive(:destroy)
+      Blog.create!.destroy!
+    end
+
     it 'works on has_one: associations' do
       delivery_service = DeliveryService.create!
       pizza = delivery_service.pizzas.new