Remove validate command. Not needed.

Author: MemberIT

manifests/fact.pp

@@ -14,12 +14,11 @@
   $facter_data = { "${title}" => $value }
 
   file { "${facterbasepath}/facts.d/${title}.yaml":
-    ensure       => $ensure,
-    owner        => 'root',
-    group        => $::puppet::defaults::puppet_group,
-    mode         => '0640',
-    validate_cmd => "/usr/bin/env ruby -ryaml -e \"YAML.load_file '%'\"",
-    content      => template('puppet/fact.yaml.erb'),
+    ensure  => $ensure,
+    owner   => 'root',
+    group   => $::puppet::defaults::puppet_group,
+    mode    => '0640',
+    content => template('puppet/fact.yaml.erb'),
   }
 
 }

manifests/facts.pp

@@ -16,12 +16,11 @@
   }
 
   file { "${facterbasepath}/facts.d/local.yaml":
-    ensure       => file,
-    owner        => 'root',
-    group        => $::puppet::defaults::puppet_group,
-    mode         => '0640',
-    validate_cmd => "/usr/bin/env ruby -ryaml -e \"YAML.load_file '%'\"",
-    content      => template('puppet/local_facts.yaml.erb'),
+    ensure  => file,
+    owner   => 'root',
+    group   => $::puppet::defaults::puppet_group,
+    mode    => '0640',
+    content => template('puppet/local_facts.yaml.erb'),
   }
 
 }

spec/classes/puppet_facts_spec.rb

@@ -102,8 +102,6 @@
             /FQDN my.fq.hostname/
           ).with_content(
             /---/
-          ).with_validate_cmd(
-            "/usr/bin/env ruby -ryaml -e \"YAML.load_file '%'\""
           )
           should contain_file("#{facterbasepath}/facts.d/local.yaml").with_content(
             /Environment production/
@@ -129,8 +127,6 @@
             /key1: val1/
           ).with_content(
             /key2: val2/
-          ).with_validate_cmd(
-            "/usr/bin/env ruby -ryaml -e \"YAML.load_file '%'\""
           )
           should contain_file("#{facterbasepath}/facts.d/local.yaml").with_content(
             /Environment production/
@@ -164,8 +160,6 @@
             /- val21/
           ).with_content(  
             /- val22/
-          ).with_validate_cmd(
-            "/usr/bin/env ruby -ryaml -e \"YAML.load_file '%'\""
           )
           should contain_file("#{facterbasepath}/facts.d/local.yaml").with_content(
             /Environment production/
@@ -195,8 +189,6 @@
             /key2:/
           ).with_content(
             /key21: val21/
-          ).with_validate_cmd(
-            "/usr/bin/env ruby -ryaml -e \"YAML.load_file '%'\""
           )
           should contain_file("#{facterbasepath}/facts.d/local.yaml").with_content(
             /Environment production/

spec/defines/fact_spec.rb

@@ -66,8 +66,6 @@
             /---/
           ).with_content(
             /my_fact: my_val/
-          ).with_validate_cmd(
-            "/usr/bin/env ruby -ryaml -e \"YAML.load_file '%'\""
           )
         end
       end
@@ -91,8 +89,6 @@
             /- my_val0/
           ).with_content(
             /- my_val1/
-          ).with_validate_cmd(
-            "/usr/bin/env ruby -ryaml -e \"YAML.load_file '%'\""
           )
         end
       end
@@ -116,8 +112,6 @@
             /my_key0: my_val0/
           ).with_content(
             /my_key1: my_val1/
-          ).with_validate_cmd(
-            "/usr/bin/env ruby -ryaml -e \"YAML.load_file '%'\""
           )
         end
       end