Fixing a bug where a group activated in redis but not defined in roll…

…out causes user_in_active_group? to fail

Conflicts:

	lib/rollout.rb

lib/rollout.rb

@@ -62,7 +62,7 @@ def percentage_key(name)
     end
 
     def user_in_active_group?(feature, user)
-      @redis.smembers(group_key(feature)).any? { |group| @groups[group].call(user) }
+      @redis.smembers(group_key(feature)).any? { |group| @groups.key?(group) && @groups[group].call(user) }
     end
 
     def user_active?(feature, user)

spec/rollout_spec.rb

@@ -19,6 +19,11 @@
     it "is not active for users for which the block evaluates to false" do
       @rollout.should_not be_active(:chat, stub(:id => 1))
     end
+
+    it "is not active if a group is found in Redis but not defined in Rollout" do
+      @rollout.activate_group(:chat, :fake)
+      @rollout.should_not be_active(:chat, stub(:id => 1))
+    end
   end
 
   describe "the default all group" do