Skip to content

Comments

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

Open
sssd-bot wants to merge 1 commit intoSSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr8451-to-sssd-2-9
Open

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

Conversation

@sssd-bot
Copy link
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.

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)
Copy link

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants