New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Port backend_test.py lib389 test to the new DSLdapObject implementation #2465
Comments
Comment from mreynolds (@mreynolds389) at 2017-10-26 18:03:04 Metadata Update from @mreynolds389:
|
Comment from akshayshivekar at 2017-11-13 09:40:27 |
Comment from firstyear (@Firstyear) at 2017-11-14 04:32:34 Hey there,
Avoid the use of backend_suffix and backend_name : they are there for legacy compat, but they should be removed. Use "nsslapd-suffix" and "cn" instead :)
I don't like this syntax either. While it can be convinient to say "topo.backends..." I think it's cleaner to say: backends = Backends(topology.standalone) But that's my personal preference.
Don't check this, because the value could CHANGE in the background due to the way this monitor works. Check the .dn is the same is sufficent. Otherwise looks pretty good, @droideck may have some more comments, Thanks! |
Comment from spichugi (@droideck) at 2018-04-18 16:22:03 Metadata Update from @droideck:
|
Comment from spichugi (@droideck) at 2018-04-25 12:09:38 |
Comment from spichugi (@droideck) at 2018-04-25 12:09:54 Metadata Update from @droideck:
|
Cloned from Pagure issue: https://pagure.io/389-ds-base/issue/49406
Port the test cases from lib389/tests/backend_test.py to new Backends(DSLdapObject).
Legacy object test cases should be at lib389/tests/backendLegacy_test.py
test_create can be ported with the same structure, but the docstrings should be adjusted to the proper format.
Additionally, we need to test the next methods: lint, create_sample_entries, get_monitor, get_indexes. Just test the very basic behavior:
The text was updated successfully, but these errors were encountered: