groups

modules/generators/structured_content/account/secgen_local/local.rb

@@ -14,7 +14,7 @@ class AccountGenerator < StringEncoder
     self.module_name = 'Account Generator / Builder'
     self.username = ''
     self.password = ''
-    self.groups = ''
+    self.groups = []
     self.super_user = ''
     self.strings_to_leak = []
     self.data_to_leak = []
@@ -25,7 +25,7 @@ class AccountGenerator < StringEncoder
     account_hash = {}
     account_hash['username'] = self.username
     account_hash['password'] = self.password
-    account_hash['groups'] = self.password
+    account_hash['groups'] = self.groups
     account_hash['super_user'] = self.super_user
     account_hash['strings_to_leak'] = self.strings_to_leak
     account_hash['leaked_filenames'] = self.leaked_filenames

modules/utilities/unix/system/groups/manifests/init.pp

@@ -1,7 +1,7 @@
 class groups::init {
   $secgen_parameters = secgen_functions::get_parameters($::base64_inputs_file)
 
-  $groups = $secgen_parameters['accounts']
+  $groups = $secgen_parameters['groups']
   if $groups {
     $groups.each |$group| {
 

modules/utilities/unix/system/parameterised_accounts/manifests/init.pp

@@ -9,7 +9,7 @@ class parameterised_accounts::init {
       parameterised_accounts::account { "parameterised_user_$username":
         username        => $username,
         password        => $account['password'],
-        groups        => $account['groups'],
+        groups          => $account['groups'],
         super_user      => str2bool($account['super_user']),
         strings_to_leak => $account['strings_to_leak'],
         leaked_filenames => $account['leaked_filenames'],

scenarios/examples/services_utilities_examples/users_and_groups.xml

@@ -54,8 +54,8 @@
 
 			<input into_datastore="groups">
 				<value>staff</value>
-				<value>team2</value>
 				<value>team1</value>
+				<value>team2</value>
 			</input>
 
 	 		<!-- accounts on the desktop, with the main user as a sudoer, with no flags -->