On Wed, Aug 03, 2016 at 09:56:40AM +0200, Lukas Slebodnik wrote: > On (13/07/16 17:48), Lukas Slebodnik wrote: > >ehlo, > > > >attched patch is an integration test for regression #3093. > >I prepared a test and I let someone else to fix it :-) > > > >I will try to find more bugs in downstream tests. > > > >BTW we might move test from test_ts_cache somewhere else. > >But it was the fastest way how to write a test without enumeration. > > > > As I previously wrote test_ts_cache is not the best place for this > test. Attached are patches which some changes in intg tests. > There is also test for grups with special characters > > LS
Thank you, the only comment I have is that we should remove ldap_auth_disable_tls_never_use_in_production option. We don't use it in tests (we don't test authentication there at all at the moment) and it's better to not advertise the option. I'll give a formal ACK once the CI run finishes. _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org
