[SSSD] [PATCH v2] intg: Add more LDAP tests

Nikolai Kondrashov Nikolai.Kondrashov at redhat.com
Thu Oct 22 14:07:16 UTC 2015


On 10/22/2015 03:04 PM, Michal Židek wrote:
> On 10/12/2015 03:31 PM, Nikolai Kondrashov wrote:
>> Here's another version of the patch, which is still interfering with the
>> memory cache tests. However, I did some digging around and added a test
>> in a
>> separate patch which makes it easier to reproduce the issue. In this
>> test it
>> seems that the cache file is not invalidated with "sss_cache -E", if
>> there was
>> a LDAP enumeration refresh before that.
>>
>> Other changes are:
>>
>>      * Add full PEP8 cleanup for integration tests prior to adding more
>> tests
>
> Could you please send the PEP8 changes in a separate thread? They
> LGTM, but I would like to ACK them in a separate thread so that they
> are not blocked by review of the other 2 patches (btw. I will look at
> those as well soon).

Done. Thanks!

Nick


More information about the sssd-devel mailing list