ZOOKEEPER-1141. zkpython fails tests under python 2.4. (phunt via mahadev)

git-svn-id: https://svn.apache.org/repos/asf/zookeeper/trunk@1157686 13f79535-47bb-0310-9956-ffa450edef68

CHANGES.txt

@@ -288,6 +288,8 @@ BUGFIXES:
   ZOOKEEPER-1055. check for duplicate ACLs in addACL() and create().
   (Eugene Koontz via mahadev)
 
+  ZOOKEEPER-1141. zkpython fails tests under python 2.4. (phunt via mahadev)
+
 IMPROVEMENTS:
   ZOOKEEPER-724. Improve junit test integration - log harness information 
   (phunt via mahadev)

src/contrib/zkpython/src/test/connection_test.py

@@ -76,7 +76,7 @@ class ConnectionTest(zktestbase.TestBase):
         cv.wait(15.0)
         cv.release()
         self.assertEqual(self.connected, True, "Connection timed out to " + self.host)
-        self.assertEqual(True, all( [ zookeeper.state(handle) == zookeeper.CONNECTED_STATE for handle in handles ] ),
+        self.assertEqual(True, self.all( [ zookeeper.state(handle) == zookeeper.CONNECTED_STATE for handle in handles ] ),
                          "Not all connections succeeded")
         oldhandle = handles[3]
         zookeeper.close(oldhandle)
@@ -115,7 +115,7 @@ class ConnectionTest(zktestbase.TestBase):
             path = "/zkpython-test-handles-%s" % str(i)
             self.assertEqual(path, zookeeper.create(h, path, "", [ZOO_OPEN_ACL_UNSAFE], zookeeper.EPHEMERAL))
 
-        self.assertEqual(True, all( zookeeper.close(h) == zookeeper.OK for h in handles ))
+        self.assertEqual(True, self.all( zookeeper.close(h) == zookeeper.OK for h in handles ))
 
     def testversionstringexists(self):
         self.assertTrue(hasattr(zookeeper, '__version__'))

src/contrib/zkpython/src/test/zktestbase.py

@@ -93,3 +93,9 @@ class TestBase(unittest.TestCase):
     def tearDown(self):
         if self.connected:
             zookeeper.close(self.handle)
+
+    def all(self, iterable):
+        for element in iterable:
+            if not element:
+                return False
+        return True