Route: Don't run `#calculate_costs!` on callback if TomTom disabled

We say `TomTom` is disabled when no API key is present. If this is the
case, the `after_save` callback that uses it shouldn't be executed.

I had to change my `API_KEY` constant to an instance variable to be able
to change it for testing.

Refs #6095

1 files changed, 18 insertions, 0 deletions

diff --git a/spec/models/chouette/route/route_base_spec.rb b/spec/models/chouette/route/route_base_spec.rb
index 98cb3e358..d24ad6348 100644
--- a/spec/models/chouette/route/route_base_spec.rb
+++ b/spec/models/chouette/route/route_base_spec.rb
@@ -61,4 +61,22 @@ RSpec.describe Chouette::Route, :type => :model do
       end
     end
   end
+
+  context "callbacks" do
+    it "calls #calculate_costs! after_save when TomTom is enabled" do
+      allow(TomTom).to receive(:enabled?).and_return(true)
+      route = create(:route)
+
+      expect(route).to receive(:calculate_costs!)
+      route.save
+    end
+
+    it "doesn't call #calculate_costs! after_save if TomTom is disabled" do
+      allow(TomTom).to receive(:enabled?).and_return(false)
+      route = create(:route)
+
+      expect(route).not_to receive(:calculate_costs!)
+      route.save
+    end
+  end
 end