Merge pull request #365 from radix/construct-exceptions-with-saner-arguments

construct empty exceptions with no arguments, instead of () and {}

Author: bbangert

kazoo/exceptions.py

@@ -52,10 +52,7 @@ def _invalid_error_code():
 
 def _zookeeper_exception(code):
     def decorator(klass):
-        def create(*args, **kwargs):
-            return klass(args, kwargs)
-
-        EXCEPTIONS[code] = create
+        EXCEPTIONS[code] = klass
         klass.code = code
         return klass
 

kazoo/tests/test_exceptions.py

@@ -20,3 +20,9 @@ def test_exceptions_code(self):
     def test_invalid_code(self):
         module = self._get()
         self.assertRaises(RuntimeError, module.EXCEPTIONS.__getitem__, 666)
+
+    def test_exceptions_construction(self):
+        module = self._get()
+        exc = module.EXCEPTIONS[-101]()
+        assert type(exc) is module.NoNodeError
+        assert exc.args == ()