Skip to content

Tests: Change multiple user attributes at once#1664

Merged
ikerexxe merged 1 commit into
shadow-maint:masterfrom
aborah-sudo:usermod
Jul 1, 2026
Merged

Tests: Change multiple user attributes at once#1664
ikerexxe merged 1 commit into
shadow-maint:masterfrom
aborah-sudo:usermod

Conversation

@aborah-sudo

Copy link
Copy Markdown
Contributor

This is the transformation to Python of the test located in tests/usertools/01/09_usermod_change_user_info.test which checks that usermod can change multiple user attributes at once

@ikerexxe ikerexxe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Some minor comments inline.

In addition, the test is failing in openSUSE and I suspect it's because there isn't a group with ID 1, but I'm not 100% sure so I recommend you to investigate it

Comment thread tests/system/tests/test_usermod.py Outdated
Comment thread tests/system/tests/test_usermod.py Outdated
Comment thread tests/system/tests/test_usermod.py Outdated
Comment thread tests/system/tests/test_usermod.py Outdated
This is the transformation to Python of the test located in
`tests/usertools/01/09_usermod_change_user_info.test`
which checks that `usermod` can change multiple user attributes at once

@ikerexxe ikerexxe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

@ikerexxe ikerexxe merged commit f333efe into shadow-maint:master Jul 1, 2026
16 checks passed
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.

2 participants