tests: adding gpo customer test scenario to use the ldap attribute name

[danlavu]

No description provided.

[spoore1]

Small nitpick on assert message.

[spoore1]

LGTM. Ignore previous comment about assert message.

[alexey-tikhonov]

tests/test_gpo.py::test_gpo__ldap_user_name_attribute_mapping[ssh] (samba) FAILED [ 82%]
tests/test_gpo.py::test_gpo__ldap_user_name_attribute_mapping[su] (samba) FAILED [ 82%]

[danlavu]

Samba topology needs to be dropped from the test case, because the name attribute is not populated. It will take a small-ish patch in the framework to get it working. Let's get this merged and moving, we'll worry about the samba stuff later.

Local testing AD passes against sssd-client-2.10.0-1.fc41.x86_64

============================= test session starts ==============================
collecting ... 

Selected tests will use the following hosts:
  client: client.test
  ad: dc.ad.test
  nfs: nfs.test

collected 639 items / 637 deselected / 2 selected

tests/test_gpo.py::test_gpo__ldap_user_name_attribute_mapping[ssh] (ad) 
tests/test_gpo.py::test_gpo__ldap_user_name_attribute_mapping[su] (ad) 

====================== 2 passed, 637 deselected in 44.10s ======================
PASSED [ 50%]PASSED [100%]
Process finished with exit code 0

[alexey-tikhonov]

Samba topology needs to be dropped from the test case, because the name attribute is not populated. It will take a small-ish patch in the framework to get it working.

I'd say: let's fix framework.
But your call, @thalman

[thalman]

Looks great. Thanks.

[danlavu]

The failure is not related to the patch.

[alexey-tikhonov]

Pushed PR: #7621

• master
  • 3054970 - tests: adding gpo customer test scenario to use the ldap attribute name
• sssd-2-10
  • be90cc6 - tests: adding gpo customer test scenario to use the ldap attribute name
• sssd-2-9
  • 2df6ff9 - tests: adding gpo customer test scenario to use the ldap attribute name