URL: https://github.com/SSSD/sssd/pull/5689
Title: #5689: Tests: Add tests ported from bash for AD Parameters
pbrezina commented:
"""
Pushed PR: https://github.com/SSSD/sssd/pull/5689
* `master`
* a5716cd74a7c8bdf7a64a48625380884b483 - Tests: Add AD Parameters tests
ported from bash.
URL: https://github.com/SSSD/sssd/pull/5689
Title: #5689: Tests: Add tests ported from bash for AD Parameters
sidecontrol commented:
"""
Really nice, clean code, thank you Jakub, LG2M!
"""
See the full comment at
https://github.com/SSSD/sssd/pull/5689#issuecomment-921412041
URL: https://github.com/SSSD/sssd/pull/5689
Title: #5689: Tests: Add tests ported from bash for AD Parameters
jakub-vavra-cz commented:
"""
I was able to execute and pass all of the tests apart from the sasl ones:
`PHASE START: test
Phase timeout: 14400s
UPSTREAM TESTS STEP START:
URL: https://github.com/SSSD/sssd/pull/5689
Title: #5689: Tests: Add tests ported from bash for AD Parameters
jakub-vavra-cz commented:
"""
Here is the execution in jenkins:
https://ci-jenkins-csb-idmops.apps.ocp-c1.prod.psi.redhat.com/job/trigger-test-suite-tool/578/console
"""
See the full
URL: https://github.com/SSSD/sssd/pull/5689
Title: #5689: Tests: Add tests ported from bash for AD Parameters
jakub-vavra-cz commented:
"""
I was able to execute and pass all of the tests apart from the sasl ones:
`PHASE START: test
Phase timeout: 14400s
UPSTREAM TESTS STEP START:
URL: https://github.com/SSSD/sssd/pull/5689
Title: #5689: Tests: Add tests ported from bash for AD Parameters
jakub-vavra-cz commented:
"""
`PHASE START: test
Phase timeout: 14400s
UPSTREAM TESTS STEP START: src/tests/multihost/ad/test_adparameters_ported.py
= test
URL: https://github.com/SSSD/sssd/pull/5689
Title: #5689: Tests: Add tests ported from bash for AD Parameters Domain
sidecontrol commented:
"""
I forgot, please go through the pep8 failures and correct them.
"""
See the full comment at