Route: Don't run `#calculate_costs!` during a referential merge6410-route--don,t-run-calculate_costs-during-referential-mer

This callback got triggered during a referential merge, but it isn't
needed at that point. It should only be executed as a result of a user
updating the route (via the web interface).

Using Alban's suggestion, we can detect that we're in the process of a
merge if the associated referential is in a `ReferentialSuite`. Use this
as a condition to run the callback.

Refs #6410

1 files changed, 19 insertions, 0 deletions

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