Skip to content

[autobackport: sssd-2-9] tests: reorganize infopipe tests by interface#8460

Merged
alexey-tikhonov merged 1 commit intoSSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr8451-to-sssd-2-9
Apr 24, 2026
Merged

[autobackport: sssd-2-9] tests: reorganize infopipe tests by interface#8460
alexey-tikhonov merged 1 commit intoSSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr8451-to-sssd-2-9

Conversation

@sssd-bot
Copy link
Copy Markdown
Contributor

This is an automatic backport of PR#8451 tests: reorganize infopipe tests by interface to branch sssd-2-9, created by @aborah-sudo.

Please make sure this backport is correct.

Note

The commits were cherry-picked without conflicts.

You can push changes to this pull request

git remote add sssd-bot git@github.com:sssd-bot/sssd.git
git fetch sssd-bot refs/heads/SSSD-sssd-backport-pr8451-to-sssd-2-9
git checkout SSSD-sssd-backport-pr8451-to-sssd-2-9
git push sssd-bot SSSD-sssd-backport-pr8451-to-sssd-2-9 --force

Original commits
1571946 - tests: reorganize infopipe tests by interface

Backported commits

  • 836c54c - tests: reorganize infopipe tests by interface

Original Pull Request Body

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.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request reorganizes the infopipe tests into logical sections based on the D-Bus interface they exercise. The changes are a pure reorganization with no logic changes, which improves test maintainability and readability as intended. The new structure is clear and the new tests for basic D-Bus functionality are well-written. I find no issues with this backport.

@alexey-tikhonov alexey-tikhonov added no-backport This should go to target branch only. Accepted labels Apr 24, 2026
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>
Reviewed-by: Shridhar Gadekar <sgadekar@redhat.com>
(cherry picked from commit 1571946)
@sssd-bot
Copy link
Copy Markdown
Contributor Author

The pull request was accepted by @alexey-tikhonov with the following PR CI status:


🟢 CodeQL (success)
🟢 rpm-build:centos-stream-9-x86_64:upstream (success)
🟢 ci / prepare (success)
🟢 ci / system (centos-9) (success)
🟢 Static code analysis / codeql (success)
🟢 Static code analysis / pre-commit (success)
🟢 Static code analysis / python-system-tests (success)


There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging.

@sssd-bot sssd-bot force-pushed the SSSD-sssd-backport-pr8451-to-sssd-2-9 branch from 836c54c to 5c89db9 Compare April 24, 2026 17:26
@alexey-tikhonov alexey-tikhonov merged commit d412ffa into SSSD:sssd-2-9 Apr 24, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Accepted no-backport This should go to target branch only. Tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants