OrganizationalUnit is unused import. Please, don't forget to run pylint, it tells these things

Typo

Looks like this step is not described in the docstring. Or do I miss something?

It is not a default log level. Please, get the initial log level and then set it back in the end

You are not creating any instance or entries in this test case

You are not creating any instance or entries in this test case

I have merged the last test case with
second one it was part of it was checking indexing works after restart

@spichugi changes are done as per your suggestion

You are not creating an instance in the test case itselt. It is already mentioned in Setup

Why do you define this function here and not in the outer scope?

@spichugi changes are done as per your suggestion

I was asking the question to understand your way of thought :)

Personally, I think it makes sense to put the content of the function here because it is used only once.

As I mentioned before, please, get the previous value and then reset it back at the end of the test case.

@spichugi changes are done as per your suggestion

Please, wrap whole part of the test in which you change nsslapd-errorlog-level with try: and finally:, and reset the attribute to the original value in the finally: section. This makes sure the value is always re-set.

@mhonek changes are done as per your suggestion

@aborah Not sure why you moved the point where the value is reset but if you are sure this is ok then I am fine with it. The test passes for me. LGTM.

@spichugi ACK?

Sure, ACK
Please, squash the commits, rebase and we can merge

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/3410

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.