#49567 Add a test suite for ds-replcheck tool RFE
Closed by spichugi. Opened by spichugi.
spichugi/389-ds-base replcheck-rfe  into  master

Download 49567.patch

The script fails for me. Not sure if its an env issue (python version?), but I keep seeing these errors:

# PYTHONPATH=/home/mareynol/source/ds389/389-ds-base/src/lib389 py.test ./replcheck_test.py
...
E       TypeError: __init__() got an unexpected keyword argument 'encoding'
/usr/lib64/python2.7/subprocess.py:212: TypeError

As for my initial review, it looks good. Minor issue - we are not testing the verbose conflict entry data (you need the --conflict option), but that can get added later. I think you are also missing a lagtime setting for one of the commands (I'll add a comment in the file).

We might need the lag option set here: '-l', '1',

Lag option? '-l', '1',

2 new commits added

  • Add a test suite for ds-replcheck tool RFE
  • Issue 49043 - Add a test suite

I've fixed the test suite according to your comments.

Regarding your error, "/usr/lib64/python2.7/" - you run it with python 2 and the test case uses some features from Python 3.

One small thing. I think we need to mention '--conflict' flag in 'man' and http://www.port389.org/docs/389ds/design/repl-diff-tool-design.html

I'll upload more test cases soon. (for connections through ldaps, ldapi, StartTLS and ldif comparison)

Yeah I updated the man page in my most recent patch for the verbose conflict option (yet to be reviewed). I will also update the design doc.

Note - when I tried using python3 on F26 the CLI tests completely fail to run. is there something special I need to do? (we can discuss that offline if you want)

2 new commits added

  • Add a test suite for ds-replcheck tool RFE
  • Issue 49043 - Add a test suite

2 new commits added

  • Issue 49239 - Add a test suite for ds-replcheck tool RFE
  • Issue 49043 - Add a test suite

rebased onto f0d3dc8a8341fa92ecb9e5151cac514a20be1ee1

rebased onto 3d27d58c6e6ead82f53b90147387336a32799f52

I think this is the wrong variable name....

R-r-right. :)
Fixed.

rebased onto 9021e90cd639560fa2f57604287fabde2c78ed7f

ack, I'm happy with this now,

rebased onto 7d5c27ac57de91168a5ccd449229f56d67952a35

Pull-Request has been merged by spichugi

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

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.

Pull-Request has been closed by spichugi

Metadata