create user/group as system accounts

voxpupuli · Jun 22, 2015 · 1559c1e · 1559c1e
1 parent a196ac3
commit 1559c1e
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/manifests/install.pp b/manifests/install.pp
@@ -69,11 +69,13 @@
   if $consul::manage_user {
     user { $consul::user:
       ensure => 'present',
+      system => true,
     }
   }
   if $consul::manage_group {
     group { $consul::group:
       ensure => 'present',
+      system => true,
     }
   }
 }
diff --git a/spec/classes/init_spec.rb b/spec/classes/init_spec.rb
@@ -154,8 +154,16 @@
   end
 
   context "By default, a user and group should be installed" do
-    it { should contain_user('consul').with(:ensure => :present) }
-    it { should contain_group('consul').with(:ensure => :present) }
+    it { should contain_user('consul').with(:ensure => :present, :system => true) }
+    it { should contain_group('consul').with(:ensure => :present, :system => true) }
+  end
+
+  context "Allow creating non-system accounts" do
+    let(:params) {{
+      :system_accounts => false,
+    }}
+    it { should contain_user('consul').with(:ensure => :present, :system => false) }
+    it { should contain_group('consul').with(:ensure => :present, :system => false) }
   end
 
   context "When data_dir is provided" do