Merge branch 'master' into test-from-PR-27

Author: ashtul

README.md

@@ -40,29 +40,32 @@ For more details, visit [http://redisearch.io](http://redisearch.io)
 from redisearch import Client, TextField
 
 # Creating a client with a given index name
-client = Client('myIndex')
+client = Client("myIndex")
 
 # Creating the index definition and schema
-client.create_index((TextField('title', weight=5.0), TextField('body')))
+client.create_index((TextField("title", weight=5.0), TextField("body")))
 
 # Indexing a document
-client.add_document('doc1', title = 'RediSearch', body = 'Redisearch impements a search engine on top of redis')
+client.add_document(
+    "doc1",
+    title="RediSearch",
+    body="Redisearch implements a search engine on top of redis",
+)
 
 # Simple search
 res = client.search("search engine")
 
 # Searching with snippets
-res = client.search("search engine", snippet_sizes = {'body': 50})
+res = client.search("search engine", snippet_sizes={"body": 50})
 
-# Searching with complext parameters:
-q = Query("search engine").verbatim().no_content().with_scores().paging(0,5)
+# Searching with complex parameters:
+q = Query("search engine").verbatim().no_content().with_scores().paging(0, 5)
 res = client.search(q)
 
 
-# the result has the total number of results, and a list of documents
-print res.total # "1"
-print res.docs[0].title 
-
+# The result has the total number of results, and a list of documents
+print(res.total)  # "1"
+print(res.docs[0].title)
 ```
 
 ## Installing

redisearch/aggregation.py

@@ -86,8 +86,6 @@ class Group(object):
     This object automatically created in the `AggregateRequest.group_by()`
     """
     def __init__(self, fields, reducers):
-        if not fields:
-            raise ValueError('need at least one field')
         if not reducers:
             raise ValueError('Need at least one reducer')
 

test/test_builder.py

@@ -40,7 +40,11 @@ def testGroup(self):
         # Check the group class on its own
         self.assertRaises(ValueError, a.Group, [], [])
         self.assertRaises(ValueError, a.Group, ['foo'], [])
-        self.assertRaises(ValueError, a.Group, [], r.count())
+
+        # Zero fields, single reducer
+        g = a.Group([], r.count())
+        ret = g.build_args()
+        self.assertEqual(['GROUPBY', '0', 'REDUCE', 'COUNT', '0'], ret)
 
         # Single field, single reducer
         g = a.Group('foo', r.count())