Handle tab-separated values better

Ericsson · Dec 12, 2023 · 0179024 · 0179024
1 parent bbdddf0
commit 0179024
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/manifests/init.pp b/manifests/init.pp
@@ -100,7 +100,7 @@
       $qvalue = shellquote("${value}")
       # lint:endignore
       exec { "enforce-sysctl-value-${qtitle}":
-        unless  => "test \"$(sysctl -n ${qtitle})\" = ${qvalue}",
+        unless  => "test \"$(sysctl -n ${qtitle} | tr -s '[ \t]' ' ')\" = ${qvalue}",
         command => "sysctl -w ${qtitle}=${qvalue}",
         path    => ['/usr/sbin', '/sbin', '/usr/bin', '/bin'],
       }

diff --git a/spec/defines/init_spec.rb b/spec/defines/init_spec.rb
@@ -59,7 +59,7 @@
 
       it do
         is_expected.to contain_exec('enforce-sysctl-value-net.ipv4.ip_forward').only_with(
-          unless:  'test "$(sysctl -n net.ipv4.ip_forward)" = ""',
+          unless:  "test \"$(sysctl -n net.ipv4.ip_forward | tr -s '[ \t]' ' ')\" = \"\"",
           command: 'sysctl -w net.ipv4.ip_forward=""',
           path:    ['/usr/sbin', '/sbin', '/usr/bin', '/bin'],
         )
@@ -99,7 +99,7 @@
 
         it do
           is_expected.to contain_exec('enforce-sysctl-value-net.ipv4.ip_forward').only_with(
-            unless:  'test "$(sysctl -n net.ipv4.ip_forward)" = 1',
+            unless:  "test \"$(sysctl -n net.ipv4.ip_forward | tr -s '[ \t]' ' ')\" = 1",
             command: 'sysctl -w net.ipv4.ip_forward=1',
             path:    ['/usr/sbin', '/sbin', '/usr/bin', '/bin'],
           )