The permissions on this repository are being updated. This may take a while.
During this time, you or some of the project's contributors may not be able
to push to this repository.
e2810e7Ticket 49943 - rfc3673_all_oper_attrs_test is not strict enough
Ticket 49943 - rfc3673_all_oper_attrs_test is not strict enough
Bug Description:
Test suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic
does not reach constraints extensively. The asserts are too
benevolent.
The commit 6ef4eb5 changed 'normal user' ACIs, however these changes
introduced new attr 'modifiersName' which was supposed to be missing
when searching.
In the first case, assert checks only for 'objectClass' and
pseudo-randomly one more attr to be present which is not sufficient.
In the second case, recently changed assert introduced weaker check
than the one present before.
Fix Description:
Bring back previous ACI to explicitly test the difference when binding
as normal user and the DM.
In case of add_attr == '*', test for all expected_attrs to be in
found_attrs. In the other case bring back the strict comparison as
there used to be before.
https://pagure.io/389-ds-base/issue/49943
Author: mhonek
Review by: firstyear, spichugi (Thanks!)