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