<div dir="ltr"><div><div>New tests added to success tests.<br></div>Please review it.<br><br></div>Thanks,<br><div><div><div><br></div></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Mar 22, 2013 at 4:48 PM, Stephen Gallagher <span dir="ltr">&lt;<a href="mailto:sgallagh@redhat.com" target="_blank">sgallagh@redhat.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
</div><div class="im">On Fri 22 Mar 2013 04:32:52 AM EDT, Abhishek Singh wrote:<br>
&gt;<br>
&gt; Hi,<br>
&gt;<br>
&gt; I have attached a patch that contains cmocka unittest for io.c .<br>
&gt; Kindly review it.<br>
&gt;<br>
<br>
<br>
</div>This is a good start, but I&#39;d like to see some more comprehensive<br>
tests. In the success tests, could you please use fcntl(F_GETFD) to<br>
confirm that the file descriptor actually has the correct flags? You<br>
can take a look at the sss_open_cloexec() function implementation for<br>
inspiration here.<br>
<div class="im">-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v1.4.13 (GNU/Linux)<br>
Comment: Using GnuPG with Thunderbird - <a href="http://www.enigmail.net/" target="_blank">http://www.enigmail.net/</a><br>
<br>
</div>iEYEARECAAYFAlFMPg0ACgkQeiVVYja6o6MHsACfTSehzUmCKYeRVmFeBnOKFUco<br>
dJIAnj6CGoKH3psZ97dchvTil5jwEIvE<br>
=hEyx<br>
-----END PGP SIGNATURE-----<br>
</blockquote></div><br></div>