Merge pull request #69 from alexiri/fix_erb

Only try to validate .erb or .epp files

lib/puppet-syntax/tasks/puppet-syntax.rb

@@ -15,7 +15,7 @@ def filelist_manifests
     end
 
     def filelist_templates
-      filelist("**/templates/**/*")
+      filelist(["**/templates/**/*.erb", "**/templates/**/*.epp"])
     end
 
     def filelist_hiera_yaml

lib/puppet-syntax/templates.rb

@@ -14,7 +14,7 @@ def check(filelist)
       filelist.each do |file|
         if File.extname(file) == '.epp' or PuppetSyntax.epp_only
           errors.concat validate_epp(file)
-        else
+        elsif File.extname(file) == '.erb'
           errors.concat validate_erb(file)
         end
       end

spec/fixtures/test_module/templates/ignore.tpl

@@ -0,0 +1,3 @@
+This is plain text
+<% This is not valid Ruby %>
+This is also plain text

spec/puppet-syntax/templates_spec.rb

@@ -60,6 +60,13 @@
     expect(res).to match([])
   end
 
+  it 'should ignore files without .erb extension' do
+    files = fixture_templates('ignore.tpl')
+    res = subject.check(files)
+
+    expect(res).to match([])
+  end
+
   if Puppet::PUPPETVERSION.to_f < 3.7
     context 'on Puppet < 3.7' do
       it 'should throw an exception when parsing EPP files' do