You can add it as a fixture -
def test_basic_anonymous_search(topology_st, test_users):

@pytest.fixture
def test_users(topology_st):

Fixed and added one more test case. Please review. Thanks.

LGTM, but probably best to wait for @spichugi

Fixed and added one more test case. Please review. Thanks.

I think we can remove 'Verify bug 915801 - ' part because we already mentioned ds604

I am happy with the code. But we will add more test cases as Amita said.

Added one more test case to verify bug 192901, did not find its trac ticket number. Please review.

Please use format 'bz123456', as existing test cases, not 'bug123456'.

Please use 'example.com'

Let's move this number to pytest.mark.bz

typo

@vashirov @spichugi Thanks for the review. Modified the code and added more test cases. Please check. Thanks.

LGTM!

Okay, tests pass and the code looks good.

So please, make a rebase and also change the first commit line...
It should be something like this

Issue 48056 - Add more test cases to the basic suite

instead of this

Test basic LDAP operations

The commit message should always follow the same structure. Use the imperative mood in the subject line.
A hint, it should sounds like '(This commit will) Add more test cases to the basic suite'

Thanks @spichugi for the review.
Please check the latest commit.

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

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.