tests: reorganize infopipe tests by interface#8451
tests: reorganize infopipe tests by interface#8451aborah-sudo wants to merge 1 commit intoSSSD:masterfrom
Conversation
aborah-sudo
added
Waiting for review
Tests
Trivial
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request effectively reorganizes the infopipe tests into logical sections based on the D-Bus interface, which significantly improves test maintainability and readability as intended. The addition of new tests is also a great enhancement. I have one suggestion to improve the robustness of one of the new tests.
aborah-sudo
changed the title
aborah-sudo
added
Accepted
Ready to push
Group the infopipe tests into logical sections based on the D-Bus interface they exercise: - Infopipe (root object) - Domains - Users - Groups - Mixed/Combined interfaces This is a pure reorganization with no logic changes to improve test maintainability and readability. Reviewed-by: Jakub Vávra <jvavra@redhat.com>
|
The pull request was accepted by @aborah-sudo 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. |
3 participants
Group the infopipe tests into logical sections based on the D-Bus interface they exercise:
This is a pure reorganization with no logic changes to improve test maintainability and readability.