[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-579?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Henry Robinson updated ZOOKEEPER-579:
-------------------------------------

    Attachment: zookeeper-579.patch

Patch adds test for aget_acl, aset_acl, set_acl and get_acl. 

Also adds tests in the C code for ACL objects that are roughly correctly 
formed, throwing InvalidACL where they are not found. This is to prevent an 
unpleasant bug where passing another object, say a callable, could silently 
succeed until a strange error condition would present. 

> zkpython needs more test coverage for ACL code paths
> ----------------------------------------------------
>
>                 Key: ZOOKEEPER-579
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-579
>             Project: Zookeeper
>          Issue Type: Improvement
>          Components: contrib-bindings
>    Affects Versions: 3.2.1
>            Reporter: Henry Robinson
>            Assignee: Henry Robinson
>         Attachments: zookeeper-579.patch
>
>
> zkpython's tests don't do a good enough job of exercising the ACL code paths. 
> A few new tests that confirm that setACL and friends are working correctly 
> are needed. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to