[SSSD] [PATCH] nested groups unit test

Jakub Hrozek jhrozek at redhat.com
Tue Feb 18 15:14:24 UTC 2014


On Tue, Feb 18, 2014 at 01:46:14PM +0100, Pavel Březina wrote:
> Rebased again atop Nikolai's DEBUG patches. I hope it is the last time :-)
> 
> 

[PATCH 01/13] sdap: move non async functions from sdap_async.c to sdap_utils.c
ACK

[PATCH 02/13] sdap: move non async functions from sdap_async_connection.c to sdap_utils.c
ACK, just can we napespace deref_string_to_val in the future?

[PATCH 03/13] sdap: move sdap_get_id_specific_filter() to sdap_utils.c
ACK

[PATCH 04/13] ldap: move options related content from ldap_common.c to ldap_options.c
ACK

[PATCH 05/13] ldap: move domain related content from ldap_common.c to sdap_domain.c
ACK

[PATCH 06/13] make make_realm_upper_case() static
ACK

[PATCH 07/13] tests: add confdb_path to sss_test_ctx
ACK

[PATCH 08/13] tests: mock SDAP
Shouldn't the file be called mock_sdap_async.c given that most if
not all functions the module mocks come from sdap_async.c ? But ACK to
the content of the files.

You need to add common_mock_sdap.h to noinst_HEADERS to avoid breaking
distcheck.

[PATCH 09/13] tests: mock sysdb users and groups
I think the shortcut is acceptable in tests as it makes the unit test
itself easier to read. Also the ldb values are ultimately just strings, 
so I don't think the danger here is too big.

You need to add common_mock_sysdb_objects.h to noinst_HEADERS to
avoid breaking distcheck.

mock_sysdb_user() is unused, do you plan on using it in future?

[PATCH 10/13] tests: prepare makefile for provider related unit
ACK. I just verified that SSSD builds fine when this patch is the tip of
the tree to make sure we don't break bisect.

[PATCH 11/13] tests: new macro sss_will_return_always
ACK, same simple test as above

[PATCH 12/13] tests: nested groups unit test
ACK. Do you plan on adding more tests in the future with different group
nestings etc? Then even mock_sysdb_user could be useful..

[PATCH 13/13] tests: don't print debug message when test dir does
ACK



More information about the sssd-devel mailing list