I think it will be hard to modify in the future because it is really hard to follow.
It will be better if you'll define a list of dicts and iterate through them in the parametrized test function.
Something like this:

[{'cn': value1, 'attr': value1, 'positive': value1, 'negative': value1}, {'cn': value2, 'attr': value2, 'positive': value2, 'negative': value2}, ...]

Also, please, change the commit message to something like this:
https://pagure.io/389-ds-base/c/7596ca048c14d4ebcdd77454aa15f272b30482a5?branch=master

@spichugi changes are done as per your suggestion

Small nitpick but I think it can be a good improvement.
Could you please construct cn out of attr name? It follows the same pattern so it's kind of redundant now.

@spichugi changes are done as per your suggestion

Could you please also modify test_valid_invalid_attributes the same way?
Also test_valid_invalid_modes and test_valid_invalid_attributes have the same docstring.

@spichugi changes are done as per your suggestion

LGTM

Please, fix the grammar in Commit message.

Add more test cases to the match test suite

@spichugi done

Something went wrong

@spichugi Fixed

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

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.