fix(association): return bad request when associated record is not destroy (#688)

Author: nicolasalexandre9

app/controllers/forest_liana/associations_controller.rb

@@ -76,6 +76,8 @@ def dissociate
         dissociator.perform
 
         head :no_content
+      rescue ActiveRecord::RecordNotDestroyed => error
+        render json: { errors: [{ status: :bad_request, detail: error.message }] }, status: :bad_request
       rescue => error
         FOREST_REPORTER.report error
         FOREST_LOGGER.error "Association Dissociate error: #{error}\n#{format_stacktrace(error)}"

app/services/forest_liana/has_many_dissociator.rb

@@ -31,7 +31,12 @@ def perform
 
         if @with_deletion
           record_ids = record_ids.select { |record_id| @association.klass.exists?(record_id) }
-          @association.klass.destroy(record_ids)
+          @resource.transaction do
+            record_ids.each do |id|
+              record = @association.klass.find(id)
+              record.destroy!
+            end
+          end
         end
       end
     end