Fixes command

Correct command is `test -e $filename`

Also % replacement in validate_cmd is not supported in a released version of puppetlabs-stdlib, it's fixed in 4.5.x but not not released yet (See: puppetlabs/puppetlabs-stdlib#377)

Author: petems

manifests/init.pp

@@ -196,7 +196,7 @@
         }
       }
       else {
-        validate_cmd("${fragdir}/${concat_name}", $validate_cmd)
+        fail("validate_cmd is a limted to Puppet > 3.5, you are on ${::puppetversion}")
       }
     }
 

spec/unit/defines/concat_spec.rb

@@ -19,7 +19,6 @@
       :replace        => true,
       :order          => 'alpha',
       :ensure_newline => false,
-      :validate_cmd   => nil,
     }.merge(params)
 
     safe_name            = title.gsub('/', '_')
@@ -391,19 +390,28 @@
     end
 
     context '/usr/bin/test -e' do
-      it_behaves_like 'concat', '/etc/foo.bar', { :validate_cmd => '/usr/bin/test % -e' }
-    end
-
-    [ 1234, true ].each do |cmd|
-      context cmd do
+      if (Puppet.version >= '3.5.0')
+        let(:facts) {{ :puppetversion => Puppet.version }}
+        it_behaves_like 'concat', '/etc/foo.bar', { :validate_cmd => '/usr/bin/test -e' }
+      else
         let(:title) { '/etc/foo.bar' }
-        let(:params) {{ :validate_cmd => cmd }}
+        let(:facts) {
+          {
+            :puppetversion => Puppet.version,
+            :concat_basedir => '/var/lib/puppet/concat',
+          }
+      }
+        let(:params) {
+          {
+            :validate_cmd => '/usr/bin/test -e'
+          }
+      }
         it 'should fail' do
-          expect { should }.to raise_error(Puppet::Error, /\$validate_cmd must be a string/)
+          expect { should }.to raise_error(/validate_cmd is a limted to Puppet > 3.5, you are on/)
         end
       end
     end
-  end # validate_cmd =>
+  end
 
   describe 'deprecated parameter' do
     context 'gnu =>' do