Merge branch 'asherf-feature/sadd_error' into develop

Author: Jesse Myers

CHANGES.md

@@ -1,3 +1,5 @@
+ - SADD will raise an exception if an empty list is passed to it
+
 Version 2.9.0.8
 
  - Add inclusive syntax (parenthesis) support for zero sets ZRANGEBYSCORE, ZREVRANGEBYSCORE  & ZREMRANGEBYSCORE

mockredis/client.py

@@ -830,6 +830,8 @@ def value_function():
 
     def sadd(self, key, *values):
         """Emulate sadd."""
+        if len(values) == 0:
+            raise ResponseError("wrong number of arguments for 'sadd' command")
         redis_set = self._get_set(key, 'SADD', create=True)
         before_count = len(redis_set)
         redis_set.update(map(str, values))

mockredis/tests/test_set.py

@@ -1,5 +1,6 @@
 from nose.tools import assert_raises, eq_, ok_
 
+from mockredis.exceptions import ResponseError
 from mockredis.tests.fixtures import setup
 
 
@@ -9,6 +10,12 @@ class TestRedisSet(object):
     def setup(self):
         setup(self)
 
+    def test_sadd_empty(self):
+        key = "set"
+        values = []
+        with assert_raises(ResponseError):
+            self.redis.sadd(key, *values)
+
     def test_sadd(self):
         key = "set"
         values = ["one", "uno", "two", "three"]