Please change the wording in the test name and in the description that the test is about passwordchange attribute.

Please add comments here or log.info with the step information.

Is this second assignment needed?

Please also add comments or log.info here for steps.

I assume that this is step "Reset server". Here you're changing all pw policy attributes, but only some of these were changed by the test.

And here also please add some comments or log.info.

Please change the wording in the test name and in the description that the test is about passwordchange attribute.

Done

Please add comments here or log.info with the step information.

Done

Please also add comments or log.info here for steps.

Done

I assume that this is step "Reset server". Here you're changing all pw policy attributes, but only some of these were changed by the test.

Done

And here also please add some comments or log.info.

Done

Is this second assignment needed?

Removed

Thanks, LGTM.

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

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.