Made unit tests more thourough (voxpupuli#104).

cegeka · Sep 11, 2015 · 7e2978e · 7e2978e
1 parent 8a8b946
commit 7e2978e
Show file tree

Hide file tree

Showing 2 changed files with 48 additions and 8 deletions.
diff --git a/spec/classes/cassandra_template1_spec.rb b/spec/classes/cassandra_template1_spec.rb
@@ -149,9 +149,29 @@
     it { should contain_file('/etc/cassandra.yaml').with_content(/saved_caches_directory: saved_caches_directory/) }
     it { should contain_file('/etc/cassandra.yaml').with_content(/    - datadir1/) }
     it { should contain_file('/etc/cassandra.yaml').with_content(/    - datadir2/) }
-    it { should contain_file('commitlog_directory') }
-    it { should contain_file('saved_caches_directory') }
-    it { should contain_file('datadir1') }
-    it { should contain_file('datadir2') }
+    it { should contain_file('commitlog_directory').with({
+        'ensure' => 'directory',
+        'owner'  => 'cassandra',
+        'group'  => 'cassandra',
+        'mode'   => '0750'
+      }) }
+    it { should contain_file('saved_caches_directory').with({
+        'ensure' => 'directory',
+        'owner'  => 'cassandra',
+        'group'  => 'cassandra',
+        'mode'   => '0750'
+      }) }
+    it { should contain_file('datadir1').with({
+        'ensure' => 'directory',
+        'owner'  => 'cassandra',
+        'group'  => 'cassandra',
+        'mode'   => '0750'
+      }) }
+    it { should contain_file('datadir2').with({
+        'ensure' => 'directory',
+        'owner'  => 'cassandra',
+        'group'  => 'cassandra',
+        'mode'   => '0750'
+      }) }
   end
 end
diff --git a/spec/classes/cassandra_template2_spec.rb b/spec/classes/cassandra_template2_spec.rb
@@ -165,9 +165,29 @@
     it { should contain_file('/etc/cassandra.yaml').with_content(/saved_caches_directory: saved_caches_directory/) }
     it { should contain_file('/etc/cassandra.yaml').with_content(/    - datadir1/) }
     it { should contain_file('/etc/cassandra.yaml').with_content(/    - datadir2/) }
-    it { should contain_file('commitlog_directory') }
-    it { should contain_file('saved_caches_directory') }
-    it { should contain_file('datadir1') }
-    it { should contain_file('datadir2') }
+    it { should contain_file('commitlog_directory').with({
+        'ensure' => 'directory',
+        'owner'  => 'cassandra',
+        'group'  => 'cassandra',
+        'mode'   => '0750'
+      }) }
+    it { should contain_file('saved_caches_directory').with({
+        'ensure' => 'directory',
+        'owner'  => 'cassandra',
+        'group'  => 'cassandra',
+        'mode'   => '0750'
+      }) }
+    it { should contain_file('datadir1').with({
+        'ensure' => 'directory',
+        'owner'  => 'cassandra',
+        'group'  => 'cassandra',
+        'mode'   => '0750'
+      }) }
+    it { should contain_file('datadir2').with({
+        'ensure' => 'directory',
+        'owner'  => 'cassandra',
+        'group'  => 'cassandra',
+        'mode'   => '0750'
+      }) }
   end
 end