URL: https://github.com/SSSD/sssd/pull/532 Title: #532: TESTS: Fix E501 pep8 issues on test_netgroup.py
jhrozek commented: """ Feel free to push the patch as emergency to unblock the CI, but I admit I don't find the syntax very readable. What about: ``` --- a/src/tests/intg/test_netgroup.py +++ b/src/tests/intg/test_netgroup.py @@ -253,7 +253,9 @@ def test_add_tripled_netgroup(add_tripled_netgroup): assert sorted(netgrps) == sorted([("host1", "user1", "domain1"), ("host2", "user2", "domain2")]) - res, _, netgrps = sssd_netgroup.get_sssd_netgroups("tripled_netgroup_no_domain") + res, _, netgrps = \ + sssd_netgroup.get_sssd_netgroups("tripled_netgroup_no_domain") + assert res == sssd_netgroup.NssReturnCode.SUCCESS assert netgrps == [("host", "user", "")] ``` """ See the full comment at https://github.com/SSSD/sssd/pull/532#issuecomment-372270891
_______________________________________________ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org