<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi,<br>
<br>
<div class="moz-cite-prefix">On 03/16/2015 05:23 PM, Martin Basti
wrote:<br>
</div>
<blockquote cite="mid:55070370.1010200@redhat.com" type="cite">
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<div class="moz-cite-prefix">On 16/03/15 15:32, Milan Kubik wrote:<br>
</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:<br>
</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:<br>
</div>
<blockquote cite="mid:5502ED38.9020302@redhat.com" type="cite">Hi,
<br>
<br>
this is a patch with port of [1] to pytest. <br>
<br>
[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>
<br>
<br>
Cheers, <br>
Milan <br>
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
</blockquote>
Added few more asserts in methods where the test could fail
and cause other errors.<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
</blockquote>
New version of the patch after brief discussion with Martin
Basti. Removed unnecessary variable assignments and separated a
new test case.<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
</blockquote>
Hello,<br>
<br>
thank you for the patch.<br>
I have a few nitpicks:<br>
1)<br>
You can remove this and use just hexlify(s)<br>
+def str_to_hex(s):<br>
+ return ''.join("{:02x}".format(ord(c)) for c in s)<br>
</blockquote>
done<br>
<blockquote cite="mid:55070370.1010200@redhat.com" type="cite"> <br>
2)<br>
+ def test_find_secret_key(self, p11):<br>
+ assert p11.find_keys(_ipap11helper.KEY_CLASS_SECRET_KEY,
label=u"žžž-aest")<br>
<br>
In tests before you tested the exact number of expected IDs
returned by find_keys method, why not here?<br>
</blockquote>
Lack of attention.<br>
Fixed the assert in `test_search_for_master_key` which does the same
thing. Merged `test_find_secret_key` with
`test_search_for_master_key` where it belongs.<br>
<blockquote cite="mid:55070370.1010200@redhat.com" type="cite"> <br>
Martin^2<br>
</blockquote>
<br>
Milan<br>
</body>
</html>