<html> <head> <meta content="text/html; charset=windows-1252" http-equiv="Content-Type"> </head> <body bgcolor="#FFFFFF" text="#000000"> <div class="moz-cite-prefix">On 16/03/15 15:32, Milan Kubik wrote: </div> <blockquote cite="mid:5506E983.4020807@redhat.com" type="cite"> <meta content="text/html; charset=windows-1252" http-equiv="Content-Type"> <div class="moz-cite-prefix">On 03/16/2015 12:03 PM, Milan Kubik wrote: </div> <blockquote cite="mid:5506B87B.6050600@redhat.com" type="cite"> <meta content="text/html; charset=windows-1252" http-equiv="Content-Type"> <div class="moz-cite-prefix">On 03/13/2015 02:59 PM, Milan Kubik wrote: </div> <blockquote cite="mid:5502ED38.9020302@redhat.com" type="cite">Hi, this is a patch with port of [1] to pytest. [1]: <a moz-do-not-send="true" class="moz-txt-link-freetext" href="https://github.com/spacekpe/freeipa-pkcs11/blob/master/python/run.py">https://github.com/spacekpe/freeipa-pkcs11/blob/master/python/run.py</a> Cheers, Milan <fieldset class="mimeAttachmentHeader"></fieldset> </blockquote> Added few more asserts in methods where the test could fail and cause other errors. <fieldset class="mimeAttachmentHeader"></fieldset> </blockquote> New version of the patch after brief discussion with Martin Basti. Removed unnecessary variable assignments and separated a new test case. <fieldset class="mimeAttachmentHeader"></fieldset> </blockquote> Hello, thank you for the patch. I have a few nitpicks: 1) You can remove this and use just hexlify(s) +def str_to_hex(s): + return ''.join("{:02x}".format(ord(c)) for c in s) 2) + def test_find_secret_key(self, p11): + assert p11.find_keys(_ipap11helper.KEY_CLASS_SECRET_KEY, label=u"žžž-aest") In tests before you tested the exact number of expected IDs returned by find_keys method, why not here? Martin^2 </body> </html>