Skip to content

tests: standardize HBAC test name format#8279

Merged
madhuriupadhye merged 1 commit intoSSSD:masterfrom
madhuriupadhye:minor_hbac
Dec 8, 2025
Merged

tests: standardize HBAC test name format#8279
madhuriupadhye merged 1 commit intoSSSD:masterfrom
madhuriupadhye:minor_hbac

Conversation

@madhuriupadhye
Copy link
Contributor

Rename test_ipa_hbac_user_host_service_category_equals_all to test_ipa__hbac_user_host_service_category_equals_all to match the double-underscore naming convention used for IPA HBAC tests and improve test discoverability.

Rename test_ipa_hbac_user_host_service_category_equals_all
to test_ipa__hbac_user_host_service_category_equals_all to
match the double-underscore naming convention used for
IPA HBAC tests and improve test discoverability.

Signed-off-by: Madhuri Upadhye <mupadhye@redhat.com>
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 standardizes the name of an HBAC test function to align with the project's naming convention for IPA tests. The change is straightforward and correct, renaming test_ipa_hbac_user_host_service_category_equals_all to test_ipa__hbac_user_host_service_category_equals_all. This improves consistency and test discoverability as intended. The change is approved.

@madhuriupadhye madhuriupadhye added Waiting for review Trivial A single reviewer is sufficient to review the Pull Request labels Dec 8, 2025
@madhuriupadhye madhuriupadhye added no-backport This should go to target branch only. Tests labels Dec 8, 2025
@madhuriupadhye madhuriupadhye merged commit 790228c into SSSD:master Dec 8, 2025
33 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-backport This should go to target branch only. Tests Trivial A single reviewer is sufficient to review the Pull Request Waiting for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants