diff mbox series

[v2,2/2] add unit test for user creation with groups

Message ID 20230727032415.3366650-2-felix.moessbauer@siemens.com
State Accepted, archived
Headers show
Series [v2,1/2] fix group splitting in user creation | expand

Commit Message

MOESSBAUER, Felix July 27, 2023, 3:24 a.m. UTC 
Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
---
 testsuite/unittests/test_image_account_extension.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/testsuite/unittests/test_image_account_extension.py b/testsuite/unittests/test_image_account_extension.py
index b7ad0c5c..08021a4a 100644
--- a/testsuite/unittests/test_image_account_extension.py
+++ b/testsuite/unittests/test_image_account_extension.py
@@ -39,12 +39,15 @@  class TestImageAccountExtensionImageCreateUsers(TestImageAccountExtensionCommon)
     def test_new_user(self):
         test_user = "new"
         d, rootfs = self.setup(test_user)
+        # make the list a bit clumsy to simulate appends and removals to that var
+        d.setVarFlag('USER_{}'.format(test_user), 'groups', 'dialout render  foo ')
 
         with patch.object(bb.process, "run") as run_mock:
             image_create_users(d)
 
         run_mock.assert_called_once_with(
-            ["sudo", "-E", "chroot", rootfs.path(), "/usr/sbin/useradd", test_user])
+            ["sudo", "-E", "chroot", rootfs.path(), "/usr/sbin/useradd",
+             '--groups', 'dialout,render,foo', test_user])
 
     def test_existing_user_no_change(self):
         test_user = "test"