The same issue as here - https://pagure.io/389-ds-base/pull-request/50540#comment-97761

The docstring doesn't describe what you actually test (match count).
The same is with other docstrings in the file (could you please extend them?)

The same issue as here - https://pagure.io/389-ds-base/pull-request/50540#comment-97761

This one is different ,

for attr, value in LIST_ATTR:
    cos.create(properties={
        'cn': attr,
        'attr' + attr.split('entry')[1][:-1]: value
    })

say,

       attr = "entryoctetStringMatch0" and  value = "AAAAAAAAAAAAAAE="  
       cos.create(properties={
             'cn': "entryoctetStringMatch0",
             'attroctetStringMatch": "AAAAAAAAAAAAAAE="
        })

Here i am creating "attroctetStringMatch" from 'attr' + attr.split('entry')[1][:-1]

I need "attroctetStringMatch" attribute in the created entry , other wise test will fail .

Please refer original TET filter test cases for more clarification.

The same issue as here - https://pagure.io/389-ds-base/pull-request/50540#comment-97761

Replied

The docstring doesn't describe what you actually test (match count).
The same is with other docstrings in the file (could you please extend them?)

Done

The code looks good

Thank you, Anuj! Merged.

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

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.