URL:
https://github.com/SSSD/sssd/pull/289
Title: #289: Check the timestamp cache during the users/groups cleanup
fidencio commented:
"""
On Thu, Jun 15, 2017 at 11:19 AM, lslebodn <notifications(a)github.com> wrote:
Is there a reason why there is missing unit test for new public
function?
Lukáš,
I've started changing the cmocka/test_ldap_id_cleanup.c in order to add the
unit test, but the tests there are using the files provider, which doesn't
have/support the timestamp cache.
I actually started extending the files provider to support the timestamp
cache and then having a test for those, but ends up it's not actually
desired (extending the files provider, not the tests).
So, I've decided to go without the tests, at least for now and hope that it
wouldn't block those patches.
I would also appreciate integration test for this bug. we need to
prevent
regressions in future.
The integration test is something that can be done (and will) as soon as
the fire on other places is controlled. I'll file a ticket about this. Is
it okay for you?
"""
See the full comment at
https://github.com/SSSD/sssd/pull/289#issuecomment-308708682