Skip to content
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

tests: remove multihost basic tests #7565

Merged
merged 1 commit into from
Sep 5, 2024

Conversation

jakub-vavra-cz
Copy link
Contributor

  • test_ifp.py test are now convered in system/test_infopipe.py
  • test_kcm.py test are now covered in system/test_kdm.py and authselect/system/test_sssd.py , the functional credential delegation ** a functional test has been added to the test plan
  • test_ldapapi.py tests are low priority with a larger effort to move. ** test configures ldap, using the 389 slapd file for it's URI, this test can only be performed on a server and does not offer much value. this test has been added to the test plan and will be re-implemented if approved.

Reviewed-by: Jakub Vávra [email protected]
Reviewed-by: Pavel Březina [email protected]
Reviewed-by: Scott Poore [email protected]

* test_ifp.py test are now convered in system/test_infopipe.py
* test_kcm.py test are now covered in system/test_kdm.py and
  authselect/system/test_sssd.py , the functional credential delegation
** a functional test has been added to the test plan
* test_ldapapi.py tests are low priority with a larger effort to move.
** test configures ldap, using the 389 slapd file for it's URI, this
test can only be performed on a server and does not offer much value.
this test has been added to the test plan and will be re-implemented if
approved.

Reviewed-by: Jakub Vávra <[email protected]>
Reviewed-by: Pavel Březina <[email protected]>
Reviewed-by: Scott Poore <[email protected]>
@alexey-tikhonov
Copy link
Member

Let's wait for CI to finish.

@jakub-vavra-cz jakub-vavra-cz merged commit 6bb9e7c into SSSD:sssd-2-9 Sep 5, 2024
25 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants