- Make sure that sending an empty list to batch() doesn't error out but rather returns an empty list.

pboucher · pboucher · commit 875c1a8f10b1 · 2012-08-23T15:34:26.000-04:00
- Tests to validate new functionality.
diff --git a/shotgun_api3/shotgun.py b/shotgun_api3/shotgun.py
@@ -708,6 +708,11 @@ def batch(self, requests):
             raise ShotgunError("batch() expects a list.  Instead was sent "\
                 "a %s" % type(requests))
 
+        # If we have no requests, just return an empty list immediately.
+        # Nothing to process means nothing to get results of.
+        if len(requests) == 0:
+            return []
+
         calls = []
 
         def _required_keys(message, required_keys, data):
diff --git a/tests/test_api.py b/tests/test_api.py
@@ -66,6 +66,11 @@ def test_batch(self):
         result = self.sg.batch(requests)[0]
         self.assertEqual(True, result)
 
+    def test_empty_batch(self):
+        """Empty list sent to .batch()"""
+        result = self.sg.batch([])
+        self.assertEqual([], result)
+
     def test_create_update_delete(self):
         """Called create, update, delete, revive"""
         data = {
