puppet-lint-template_file_extension-check 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c386f651f5d324b4ef4ee45599cca5e7d12349fe
-  data.tar.gz: 6f3677dfabf9e6a8dd418c8ccba30bafc4791a92
+  metadata.gz: ed1fc7b6dbe9b2dada9167beb0fd4031173a3636
+  data.tar.gz: 50c2a6bf683352e51518834475a13e792cb3e529
 SHA512:
-  metadata.gz: 358d8b7f6b3877b178061d4914d69cd9fd2cc540f2d2dc382c13f1f1216244833a4d940071ada74569f55afeed4a4e3e67084c023050fd1d6a536aa59651dc30
-  data.tar.gz: cc413fe0da5d88b9008cca8012d7162ae6fb2d54ac4608a1b1e1e404631dac435619c0ca9a0cc2b9157eb43fde1f1c4569ca243074ac6d162c871fe81b6a9b78
+  metadata.gz: 094fa6fb18f119a87be96d7977a1f328705772c8c97d1da39c547ef7ac8f867d74e1dee38372e5f8f76a6399202695acf979f8255f860716af6e8e5959be8b0b
+  data.tar.gz: 0cbb0f81f7abc8ce8dd5456ad0bc4156dd39da21ef0aa4405e2d5bf1292b4dd0be763cb0b24c0d09411509b3c26044f95e9c8f9122d826539be7287c63bbd2fb

data/lib/puppet-lint/plugins/template_file_extension.rb

@@ -21,7 +21,7 @@ PuppetLint.new_check(:template_file_extension) do
             current_token = value_token.next_token
 
             # iterate over all the code tokens until we hit the closing ')'
-            until current_token.type == :RPAREN
+            until current_token.type == :RPAREN || current_token.type == :LBRACE
               current_token = current_token.next_code_token
 
               if current_token.type == :SSTRING && !current_token.value.end_with?(extension)

data/spec/puppet-lint/plugins/puppet-lint-template_file_extension_spec.rb

@@ -38,6 +38,21 @@ describe 'template_file_extension' do
     end
   end
 
+  context 'epp function with one valid file name and parameters' do
+    let(:code) do
+      <<-EOS
+        class valid_epp_template_filename {
+          file { '/tmp/templated':
+            content => epp('mymodule/single_file.epp', {'key' => 'val'}),
+          }
+        }
+      EOS
+    end
+
+    it 'should not detect any problems' do
+      expect(problems).to have(0).problems
+    end
+  end
 
   ##########################
   # Invalid examples
@@ -72,7 +87,7 @@ describe 'template_file_extension' do
       <<-EOS
         class epp_multi_templated_file {
           file { '/tmp/templated':
-            content => epp('mymodule/first_file.epp', 'mymodule/second_file.conf'),
+            content => epp('mymodule/first_file', {'key' => 'val'}),
           }
         }
       EOS

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puppet-lint-template_file_extension-check
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Dean Wilson
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-06-06 00:00:00.000000000 Z
+date: 2017-11-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: puppet-lint
@@ -114,6 +114,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.4'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.15.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.15.0
 description: |2
       Extends puppet-lint to ensure all filenames used in template and epp functions
       end with the string '.erb' or '.epp' respectively
@@ -147,11 +161,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.2
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: puppet-lint template_file_extension check
 test_files:
-- spec/puppet-lint/plugins/puppet-lint-template_file_extension_spec.rb
 - spec/spec_helper.rb
-has_rdoc: 
+- spec/puppet-lint/plugins/puppet-lint-template_file_extension_spec.rb