On 09/30/2015 06:15 PM, Nikolai Kondrashov wrote:
-    create_conf_fixture(request, conf)
+    create_conf_fixture(request, format_basic_conf(ldap_conn, False, True))

Actually, I wanted to do one more thing with this and forgot:
name the "bis" and "enum" parameters explicitly on each invocation to make it
clear what's happening. E.g.:

    create_conf_fixture(request,
                        format_basic_conf(ldap_conn, bis=False, enum=True))

Anyone in favor of/against this?

Nick
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to