Pass feature to the switchable method

README.md

@@ -183,7 +183,7 @@ class UserPreferenceStrategy < Flipflop::Strategies::AbstractStrategy
     end
   end
 
-  def switchable?
+  def switchable?(feature)
     # Can only switch features on/off if we have the user's session.
     # The `request` method is provided by AbstractStrategy.
     request?

app/views/flipflop/features/index.html.erb

@@ -50,20 +50,20 @@
                       name: "commit",
                       value: "1",
                       class: @feature_set.strategy_status(strategy, feature) == :enabled ? "active" : nil,
-                      disabled: !strategy.switchable?
+                      disabled: !strategy.switchable?(feature)
                     -%>
 
                     <%= button_tag t(:disabled, scope: [:flipflop, :feature_states]),
                       type: "submit",
                       name: "commit",
                       value: "0",
                       class: @feature_set.strategy_status(strategy, feature) == :disabled ? "active" : nil,
-                      disabled: !strategy.switchable?
+                      disabled: !strategy.switchable?(feature)
                     -%>
                   </div>
                 <% end -%>
 
-                <% if strategy.switchable? -%>
+                <% if strategy.switchable?(feature) -%>
                   <div class="group">
                     <% unless @feature_set.strategy_status(strategy, feature).blank? -%>
                       <%= form_tag(@feature_set.switch_url(strategy, feature), method: :delete) do -%>

lib/flipflop/strategies/abstract_strategy.rb

@@ -58,7 +58,7 @@ def hidden?
 
       # Return true iff this strategy is able to switch features on/off.
       # Return false otherwise.
-      def switchable?
+      def switchable?(feature)
         false
       end
 

lib/flipflop/strategies/active_record_strategy.rb

@@ -15,7 +15,7 @@ def initialize(**options)
         super(**options)
       end
 
-      def switchable?
+      def switchable?(feature)
         true
       end
 

lib/flipflop/strategies/cookie_strategy.rb

@@ -14,7 +14,7 @@ def initialize(**options)
         super(**options)
       end
 
-      def switchable?
+      def switchable?(feature)
         request?
       end
 

lib/flipflop/strategies/redis_strategy.rb

@@ -13,7 +13,7 @@ def initialize(**options)
         super(**options)
       end
 
-      def switchable?
+      def switchable?(feature)
         true
       end
 

lib/flipflop/strategies/session_strategy.rb

@@ -12,7 +12,7 @@ def initialize(**options)
         super(**options)
       end
 
-      def switchable?
+      def switchable?(feature)
         request?
       end
 

lib/flipflop/strategies/test_strategy.rb

@@ -8,7 +8,7 @@ def initialize(**options)
         super(**options)
       end
 
-      def switchable?
+      def switchable?(feature)
         true
       end
 

test/unit/strategies/abstract_strategy_test.rb

@@ -23,7 +23,7 @@
     end
 
     it "should not be switchable" do
-      assert_equal false, subject.switchable?
+      assert_equal false, subject.switchable?(:test)
     end
 
     it "should not be hidden" do

test/unit/strategies/active_record_strategy_test.rb

@@ -56,7 +56,7 @@ def save!
     end
 
     it "should be switchable" do
-      assert_equal true, subject.switchable?
+      assert_equal true, subject.switchable?(:test)
     end
 
     it "should have unique key" do

test/unit/strategies/cookie_strategy_test.rb

@@ -28,7 +28,7 @@
     end
 
     it "should be switchable" do
-      assert_equal true, subject.switchable?
+      assert_equal true, subject.switchable?(:test)
     end
 
     it "should have unique key" do
@@ -124,7 +124,7 @@
     end
 
     it "should not be switchable" do
-      assert_equal false, subject.switchable?
+      assert_equal false, subject.switchable?(:test)
     end
 
     it "should not be able to switch feature on" do

test/unit/strategies/default_strategy_test.rb

@@ -28,7 +28,7 @@
     end
 
     it "should not be switchable" do
-      assert_equal false, subject.switchable?
+      assert_equal false, subject.switchable?(:test)
     end
 
     it "should have unique key" do

test/unit/strategies/lambda_strategy_test.rb

@@ -20,7 +20,7 @@
     end
 
     it "should not be switchable" do
-      assert_equal false, subject.switchable?
+      assert_equal false, subject.switchable?(:test)
     end
 
     it "should have unique key" do

test/unit/strategies/query_string_strategy_test.rb

@@ -28,7 +28,7 @@
     end
 
     it "should not be switchable" do
-      assert_equal false, subject.switchable?
+      assert_equal false, subject.switchable?(:test)
     end
 
     it "should have unique key" do
@@ -84,7 +84,7 @@
     end
 
     it "should not be switchable" do
-      assert_equal false, subject.switchable?
+      assert_equal false, subject.switchable?(:test)
     end
   end
 end

test/unit/strategies/redis_strategy_test.rb

@@ -26,7 +26,7 @@
     end
 
     it "should be switchable" do
-      assert_equal true, subject.switchable?
+      assert_equal true, subject.switchable?(:test)
     end
 
     it "should have unique key" do

test/unit/strategies/session_strategy_test.rb

@@ -28,7 +28,7 @@
     end
 
     it "should be switchable" do
-      assert_equal true, subject.switchable?
+      assert_equal true, subject.switchable?(:test)
     end
 
     it "should have unique key" do
@@ -109,7 +109,7 @@
     end
 
     it "should not be switchable" do
-      assert_equal false, subject.switchable?
+      assert_equal false, subject.switchable?(:test)
     end
 
     it "should not be able to switch feature on" do

test/unit/strategies/test_strategy_test.rb

@@ -19,7 +19,7 @@
     end
 
     it "should be switchable" do
-      assert_equal true, subject.switchable?
+      assert_equal true, subject.switchable?(:test)
     end
 
     it "should have unique key" do