adding /var/run/redis for Debian based hosts

* Fixes failing beaker test * Add spec for new setting for Debian
voxpupuli · Nov 30, 2016 · 4a36463 · 4a36463
1 parent c2717f8
commit 4a36463
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 1 deletion.
diff --git a/manifests/config.pp b/manifests/config.pp
@@ -115,6 +115,13 @@
         owner  => $::redis::config_owner,
       }
 
+      file { '/var/run/redis':
+        ensure => 'directory',
+        owner  => $::redis::config_owner,
+        group  => $::redis::config_group,
+        mode   => '0755',
+      }
+
       if $::redis::ulimit {
         augeas { 'redis ulimit' :
           context => '/files/etc/default/redis-server',

diff --git a/spec/classes/redis_spec.rb b/spec/classes/redis_spec.rb
@@ -315,6 +315,15 @@
               })
             end
 
+            it do
+              is_expected.to contain_file('/var/run/redis').with({
+                :ensure => 'directory',
+                :owner  => 'redis',
+                :group  => 'redis',
+                :mode   => '0755',
+              })
+            end
+
           end
 
         when 'Ubuntu'
@@ -323,6 +332,15 @@
 
           it { is_expected.to contain_apt__ppa(ppa_repo) }
 
+          it do
+            is_expected.to contain_file('/var/run/redis').with({
+              :ensure => 'directory',
+              :owner  => 'redis',
+              :group  => 'redis',
+              :mode   => '0755',
+            })
+          end
+
         when 'RedHat', 'CentOS', 'Scientific', 'OEL', 'Amazon'
 
           it { is_expected.to contain_class('epel') }
@@ -1071,4 +1089,3 @@
   end
 
 end
-