* Add examples in README.md file.

* Correct indentation, double quote, ...

CHANGELOG.md

@@ -1,3 +1,8 @@
+---------------------------------------
+
+##### Changes
+* Add examples in README.md file.
+* Correct indentation, double quote, ...
 
 ---------------------------------------
 

README.md

@@ -56,11 +56,20 @@ class { 'proxmox::hypervisor':
 
 * `proxmox::hypervisor::group`: Manage groups for Proxmox WebGUI and set permissions.
 
-  `proxmox::hypervisor::group { 'sysadmin': role => "Administrator", users => [ 'user1@pam', 'toto@pve' ], }`
+```
+proxmox::hypervisor::group { 'sysadmin':
+  role  => "Administrator",
+  users => [ 'user1@pam', 'toto@pve' ],
+}
+```
 
 * `proxmox::hypervisor::user`: Manage user for Proxmox WebGUI.
 
-  `proxmox::hypervisor::user { 'marvin': group => 'sysadmin', }
+```
+proxmox::hypervisor::user { 'marvin':
+  group => 'sysadmin',
+}
+```
 
   Mainly used by the `proxmox::hypervisor::group` defined type to create the group, permissions and also create/add the users to a group. Because to add a user to a group via this defined type, the group should already exist.
 

manifests/hypervisor/group.pp

@@ -29,7 +29,7 @@
 #
 #   The user list members of this group. A user will be created if not exist.
 #
-define proxmox::hypervisor::group ( $group = $title, $role, $acl_path = '/', $permission_file = '/etc/pve/user.cfg', $users = '' ) {
+define proxmox::hypervisor::group ( $group = $title, $acl_path = '/', $permission_file = '/etc/pve/user.cfg', $users = '', $role ) {
 
   File {
     owner  => root,

tests/group_management.pp

@@ -1,2 +1,8 @@
-proxmox::hypervisor::group { 'sysadmin': role => "Administrator", users => [ 'user1@pam', 'toto@pve' ], }
+proxmox::hypervisor::group { 'sysadmin':
-proxmox::hypervisor::group { 'audit': role => "PVEAuditor", users => [ 'user2@pam' ], }
+  role  => "Administrator",
+  users => [ 'user1@pam', 'toto@pve' ],
+}
+proxmox::hypervisor::group { 'audit':
+  role  => "PVEAuditor",
+  users => [ 'user2@pam' ],
+}