The three bottom test functions are almost identical. Please, squash them into one parametrized test.

@mhonek changes are done as per your suggestion

Please, do not use 'rebase-force-push'.
Check this for details - https://github.com/marcus2376/389wiki/blob/master/docs/389ds/contributing.md#getting-the-patch-ready

The name is confusing. Should be ou_inheritance

Everything, besides target and userattr, is constant. Please, use some constant to avoid repetition. It will increase readability

Everything, besides userattr here, is constant. Please, use some constant to avoid repetition. It will increase readability

@spichugi changes are done as per your suggestion

It is not what I meant...
(targetattr=*)(version 3.0; aci "Inheritance aci"; allow(all) is constant. You can put it to some variable and then use it everywhere

This code has a constant part too

@spichugi changes are done as per your suggestion

Please, do not use 'rebase-force-push'.

ErrorLog is unused

Besides that, LGTM

Please, create an additional separate commit for the change, I'll set ack
and then you can do interactive rebase in the end.

@spichugi changes are done as per your suggestion

Great, thanks!
We can merge it after you'll do

git rebase -i HEAD~2
git push myfork --force-with-lease

@spichugi changes are done as per your suggestion

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3395

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.