From d5174032349a81f161d8a494a345c783d9f1c9a2 Mon Sep 17 00:00:00 2001
From: decabeza <alberto@decabeza.es>
Date: Tue, 12 Apr 2022 18:37:04 +0200
Subject: [PATCH] Show budgets map only if feature is enabled

---
 app/components/budgets/map_component.rb       |  2 +-
 spec/components/budgets/map_component_spec.rb | 13 ++++++++++++-
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/app/components/budgets/map_component.rb b/app/components/budgets/map_component.rb
index be83fcce4..1da16c137 100644
--- a/app/components/budgets/map_component.rb
+++ b/app/components/budgets/map_component.rb
@@ -7,7 +7,7 @@ class Budgets::MapComponent < ApplicationComponent
   end
 
   def render?
-    !budget.informing?
+    feature?(:map) && !budget.informing?
   end
 
   private
diff --git a/spec/components/budgets/map_component_spec.rb b/spec/components/budgets/map_component_spec.rb
index 662507bb1..7d9d3ffb8 100644
--- a/spec/components/budgets/map_component_spec.rb
+++ b/spec/components/budgets/map_component_spec.rb
@@ -4,7 +4,8 @@ describe Budgets::MapComponent do
   let(:budget) { build(:budget) }
 
   describe "#render?" do
-    it "is rendered after the informing phase" do
+    it "is rendered after the informing phase when the map feature is enabled" do
+      Setting["feature.map"] = true
       budget.phase = "accepting"
 
       render_inline Budgets::MapComponent.new(budget)
@@ -13,11 +14,21 @@ describe Budgets::MapComponent do
     end
 
     it "is not rendered during the informing phase" do
+      Setting["feature.map"] = true
       budget.phase = "informing"
 
       render_inline Budgets::MapComponent.new(budget)
 
       expect(page).not_to be_rendered
     end
+
+    it "is not rendered when the map feature is disabled" do
+      Setting["feature.map"] = false
+      budget.phase = "accepting"
+
+      render_inline Budgets::MapComponent.new(budget)
+
+      expect(page).not_to be_rendered
+    end
   end
 end