Merge pull request #487 from af83/6410-route--don,t-run-calculate_costs-during-referential-mer

Route: Don't run `#calculate_costs!` during a referential merge. Fixes #6410

2 files changed, 24 insertions, 1 deletions

diff --git a/app/models/chouette/route.rb b/app/models/chouette/route.rb
index c5e909d7e..a5eab3002 100644
--- a/app/models/chouette/route.rb
+++ b/app/models/chouette/route.rb
@@ -80,7 +80,11 @@ module Chouette
     validates :wayback, inclusion: { in: self.wayback.values }
     after_commit :calculate_costs!,
       on: [:create, :update],
-      if: ->() { TomTom.enabled? }
+      if: ->() {
+        # Ensure the call back doesn't run during a referential merge
+        !referential.in_referential_suite? &&
+          TomTom.enabled?
+      }
 
     def duplicate opposite=false
       overrides = {
diff --git a/spec/models/chouette/route/route_base_spec.rb b/spec/models/chouette/route/route_base_spec.rb
index 43ff28c40..e76f10a13 100644
--- a/spec/models/chouette/route/route_base_spec.rb
+++ b/spec/models/chouette/route/route_base_spec.rb
@@ -77,5 +77,24 @@ RSpec.describe Chouette::Route, :type => :model do
       expect(route).not_to receive(:calculate_costs!)
       route.save
     end
+
+    it "doesn't call #calculate_costs! after_commit if in a ReferentialSuite",
+        truncation: true do
+      begin
+        allow(TomTom).to receive(:enabled?).and_return(true)
+
+        referential_suite = create(:referential_suite)
+        referential = create(:referential, referential_suite: referential_suite)
+
+        referential.switch do
+          route = build(:route)
+
+          expect(route).not_to receive(:calculate_costs!)
+          route.save
+        end
+      ensure
+        referential.destroy
+      end
+    end
   end
 end