cherry pick: 8120 to 2.10 - Migrate missing tests to new framework#8291
cherry pick: 8120 to 2.10 - Migrate missing tests to new framework#8291thalman merged 1 commit intoSSSD:sssd-2-10from
Conversation
There was a problem hiding this comment.
Code Review
This pull request migrates several tests from the old integration test framework to the new system test framework, which is a good improvement. I've found a couple of critical issues related to missing imports in test_authentication.py that will cause the tests to fail at runtime. I've also found a minor bug in an assertion message in the new test_nss.py file. Please address these issues.
4b3812b to
91dcad1
Compare
This PR implements tests from integ/test_ldap.py to new system tests Reviewed-by: Dan Lavu <dlavu@redhat.com> Reviewed-by: Jakub Vávra <jvavra@redhat.com> Reviewed-by: Tomáš Halman <thalman@redhat.com>
|
The pull request was accepted by @thalman with the following PR CI status: 🟢 CodeQL (success) There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging. |
91dcad1 to
3bd5025
Compare
This PR implements tests from integ/test_ldap.py to new system tests
Reviewed-by: Dan Lavu dlavu@redhat.com
Reviewed-by: Jakub Vávra jvavra@redhat.com