[SSSD] SSSD Test Suite Coverage

Abhishek Singh abhishekkumarsingh.cse at gmail.com
Mon Mar 18 19:01:09 UTC 2013


Changes done.

My public repo of sssd : https://github.com/AbhishekKumarSingh/sssd


On Mon, Mar 18, 2013 at 7:02 PM, Stephen Gallagher <sgallagh at redhat.com>wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On 03/18/2013 09:17 AM, Abhishek Singh wrote:
> > Hi Jackub,
> >
> > I have attached a patch. Kindly check it.
> >
> > thanks,
> >
>
>
> An excellent start, Abhishek!
>
> I have a few comments (mostly style):
>
> Please make sure to maintain the same whitespace that was used in the
> original code you modify. For example, you added tabs in the
> HAVE_CMOCKA section of Makefile.am where the other code is indented
> with spaces. Also, you have trailing whitespace after --coverage in
> the _CFLAGS section, as well as two lines in
> test_check_if_uid_is_active_fail() that consist of nothing but
> whitespace. Many of us use editors that warn of such things with red
> overlays. If you're using VIM, I can recommend a .vimrc setting that
> will display them to you as well.
>
>
> Also, in main(), when you're creating the list of tests, it would be
> more readable in the form:
>
> const UnitTest tests[] = {
>     unit_test(test_check_if_uid_is_active_success),
>     unit_test(test_check_if_uid_is_active_fail),
>     unit_test(test_get_uid_table)
> };
>
> When I first glanced at it, I didn't notice the _fail test because it
> was on the same line as the _success test.
>
>
> All tests pass when run and it builds without warnings.
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.13 (GNU/Linux)
> Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
>
> iEYEARECAAYFAlFHF28ACgkQeiVVYja6o6MSsQCgm6rpXpiKTCrf/Y4JYLQfClTY
> kKkAnRdupgXGNusltLWZkVII2nVaOvxk
> =ZKm1
> -----END PGP SIGNATURE-----
> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130319/ba751387/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-cmocka-unittest-for-find_uid-added.patch
Type: application/octet-stream
Size: 4233 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130319/ba751387/attachment.obj>


More information about the sssd-devel mailing list