Merge pull request #161 from raoulbhatia/fix-spec-for-file-permissions

Spec: Follow puppetlabs-apt "MODULES-10548: make files readonly"
voxpupuli · Mar 1, 2020 · 3a79efd · 3a79efd
2 parents af3024e + 086e0e5
commit 3a79efd
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 19 deletions.
diff --git a/spec/classes/debian_spec.rb b/spec/classes/debian_spec.rb
@@ -15,7 +15,7 @@
       is_expected.to create_file(file_periodic).with(
         owner: 'root',
         group: 'root',
-        mode: '0644'
+        mode: '0444'
       ).with_content(
         /APT::Periodic::Enable "1";/
       ).with_content(
@@ -52,7 +52,7 @@
       is_expected.to create_file(file_options).with(
         owner: 'root',
         group: 'root',
-        mode:  '0644'
+        mode:  '0444'
       ).with_content(
         /^Dpkg::Options\s{/
       ).with_content(
@@ -84,7 +84,7 @@
               is_expected.to create_file(file_unattended).with(
                 owner: 'root',
                 group: 'root',
-                mode: '0644'
+                mode: '0444'
               ).with_content(
                 # This section varies for different releases
                 /\Unattended-Upgrade::Allowed-Origins\ {\n
@@ -100,7 +100,7 @@
               is_expected.to create_file(file_unattended).with(
                 owner: 'root',
                 group: 'root',
-                mode: '0644'
+                mode: '0444'
               ).with_content(
                 # This section varies for different releases
                 /\Unattended-Upgrade::Allowed-Origins\ {\n
@@ -116,7 +116,7 @@
               is_expected.to create_file(file_unattended).with(
                 owner: 'root',
                 group: 'root',
-                mode: '0644'
+                mode: '0444'
               ).with_content(
                 # This section varies for different releases
                 /\Unattended-Upgrade::Origins-Pattern\ {\n
@@ -131,7 +131,7 @@
               is_expected.to create_file(file_unattended).with(
                 owner: 'root',
                 group: 'root',
-                mode: '0644'
+                mode: '0444'
               ).with_content(
                 # This section varies for different releases
                 /\Unattended-Upgrade::Origins-Pattern\ {\n
@@ -146,7 +146,7 @@
               is_expected.to create_file(file_unattended).with(
                 owner: 'root',
                 group: 'root',
-                mode: '0644'
+                mode: '0444'
               ).with_content(
                 # This section varies for different releases
                 /\Unattended-Upgrade::Origins-Pattern\ {\n

diff --git a/spec/classes/other_debians_spec.rb b/spec/classes/other_debians_spec.rb
@@ -25,7 +25,7 @@
       is_expected.to create_file(file_unattended).with(
         owner: 'root',
         group: 'root',
-        mode: '0644'
+        mode: '0444'
       )
     end
   end
@@ -52,7 +52,7 @@
       is_expected.to create_file(file_unattended).with(
         'owner' => 'root',
         'group' => 'root',
-        'mode'  => '0644'
+        'mode'  => '0444'
       ).with_content(
         # This is the only section that's different for Ubuntu compared to Debian
         %r{\Unattended-Upgrade::Allowed-Origins\ {\n
@@ -84,7 +84,7 @@
       is_expected.to create_file(file_unattended).with(
         'owner' => 'root',
         'group' => 'root',
-        'mode'  => '0644'
+        'mode'  => '0444'
       ).with_content(
         # This is the only section that's different for Ubuntu compared to Debian
         %r{\Unattended-Upgrade::Allowed-Origins\ {\n
@@ -116,7 +116,7 @@
       is_expected.to create_file(file_unattended).with(
         'owner' => 'root',
         'group' => 'root',
-        'mode'  => '0644'
+        'mode'  => '0444'
       ).with_content(
         # This is the only section that's different for Ubuntu compared to Debian
         %r{\Unattended-Upgrade::Allowed-Origins\ {\n

diff --git a/spec/classes/ubuntu_spec.rb b/spec/classes/ubuntu_spec.rb
@@ -15,7 +15,7 @@
       is_expected.to create_file(file_periodic).with(
         owner: 'root',
         group: 'root',
-        mode: '0644'
+        mode: '0444'
       ).with_content(
         /APT::Periodic::Enable "1";/
       ).with_content(
@@ -52,7 +52,7 @@
       is_expected.to create_file(file_options).with(
         owner: 'root',
         group: 'root',
-        mode:  '0644'
+        mode:  '0444'
       ).with_content(
         /^Dpkg::Options\s{/
       ).with_content(
@@ -84,7 +84,7 @@
               is_expected.to create_file(file_unattended).with(
                 owner: 'root',
                 group: 'root',
-                mode: '0644'
+                mode: '0444'
               ).with_content(
                 # This is the only section that's different for Ubuntu compared to Debian
                 /\Unattended-Upgrade::Allowed-Origins\ {\n
@@ -99,7 +99,7 @@
               is_expected.to create_file(file_unattended).with(
                 owner: 'root',
                 group: 'root',
-                mode: '0644'
+                mode: '0444'
               ).with_content(
                 # This is the only section that's different for Ubuntu compared to Debian
                 /\Unattended-Upgrade::Allowed-Origins\ {\n
@@ -114,7 +114,7 @@
               is_expected.to create_file(file_unattended).with(
                 owner: 'root',
                 group: 'root',
-                mode: '0644'
+                mode: '0444'
               ).with_content(
                 # This is the only section that's different for Ubuntu compared to Debian
                 /\Unattended-Upgrade::Allowed-Origins\ {\n

diff --git a/spec/classes/unattended_upgrades_spec.rb b/spec/classes/unattended_upgrades_spec.rb
@@ -113,7 +113,7 @@
         is_expected.to create_file(file_unattended).with(
           owner: 'root',
           group: 'root',
-          mode: '0644'
+          mode: '0444'
         ).with_content(
           /Unattended-Upgrade::Allowed-Origins {\n\t"bananas";\n};/
         ).with_content(
@@ -147,7 +147,7 @@
         is_expected.to create_file(file_periodic).with(
           owner: 'root',
           group: 'root',
-          mode: '0644'
+          mode: '0444'
         ).with_content(
           /APT::Periodic::Enable "1";/
         ).with_content(
@@ -181,7 +181,7 @@
         is_expected.to create_file(file_options).with(
           owner: 'root',
           group: 'root',
-          mode:  '0644'
+          mode:  '0444'
         ).with_content(
           /^Dpkg::Options\s{/
         ).without_content(