puppet-lint-absolute_classname-check 3.1.0 → 4.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 86140cad9c7e991e50d45bb4bc896788b16a64776749c5cbbb80f29032bc1692
-  data.tar.gz: 716c53d528e10c3bcda030e1c19cc54c017346556fb98ff580a698fc32542680
+  metadata.gz: 44d9484de2d04ad14e0eb4ed9c17424b72c319a94b49fb823622a98fa85771d5
+  data.tar.gz: 589a21445dbf0baade77d8bc94c0217573b0767c838e86dd71d365d6dc43f8f4
 SHA512:
-  metadata.gz: 51b2d31cff4c997644a034a97c909c950ac67e59c3ddec2135b21bcc1a096e0357d48e61fd15d89ddbdfb0643bdac37e3e1f32c956f748c89c79ba348a6e3b52
-  data.tar.gz: 0ce3da50d95b4643d37b99e526b887c0ae7fa876abbb2c60917420a8c10b69efaa5c59714c09f57f4a42ef18d0cd5df7bc48aaa6620ab6322ca92d0067e90ea8
+  metadata.gz: 57dc43ecd8829bc7baff9186b430a47050c51513f59326db2b4e28c8e99cf50340d21776ea7d0f09578e4a9772a621a195f21d2ab0d739932ccb0c4ec951e513
+  data.tar.gz: 4c37050af7045255fba3f396ff56fcaae3e8160f22e2ce2786ab68ba230756f5ac7d87d42ad1bc3a22134d117d6febfe028195886366815276a5658b538ec14a

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

@@ -2,8 +2,8 @@ PuppetLint.new_check(:relative_classname_inclusion) do
   def check
     message = 'class included by absolute name (::$class)'
 
-    tokens.each_with_index do |token, token_idx|
-      if [:NAME,:FUNCTION_NAME].include?(token.type) && ['include','contain','require'].include?(token.value)
+    tokens.each_with_index do |token, _token_idx|
+      if %i[NAME FUNCTION_NAME].include?(token.type) && %w[include contain require].include?(token.value)
         s = token.next_code_token
         next if s.nil?
         next if s.type == :FARROW
@@ -11,7 +11,7 @@ PuppetLint.new_check(:relative_classname_inclusion) do
         in_function = 0
         while s.type != :NEWLINE
           n = s.next_code_token
-          if [:NAME, :FUNCTION_NAME, :SSTRING,].include?(s.type)
+          if %i[NAME FUNCTION_NAME SSTRING].include?(s.type)
             if n && n.type == :LPAREN
               in_function += 1
             elsif in_function > 0 && n && n.type == :RPAREN
@@ -21,7 +21,7 @@ PuppetLint.new_check(:relative_classname_inclusion) do
                 message: message,
                 line: s.line,
                 column: s.column,
-                token: s
+                token: s,
               }
             end
           end
@@ -35,7 +35,7 @@ PuppetLint.new_check(:relative_classname_inclusion) do
               message: message,
               line: s.line,
               column: s.column,
-              token: s
+              token: s,
             }
           end
           s = s.next_token
@@ -47,7 +47,7 @@ PuppetLint.new_check(:relative_classname_inclusion) do
             message: message,
             line: s.line,
             column: s.column,
-            token: s
+            token: s,
           }
         end
       end

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

@@ -2,20 +2,20 @@ PuppetLint.new_check(:relative_classname_reference) do
   def check
     message = 'absolute class name reference'
 
-    tokens.each_with_index do |token, token_idx|
-      if token.type == :TYPE and token.value == 'Class' and token.next_code_token.type == :LBRACK
-        s = token.next_code_token
-        while s.type != :RBRACK
-          if (s.type == :NAME || s.type == :SSTRING) && s.value.start_with?('::')
-            notify :warning, {
-              message: message,
-              line: s.line,
-              column: s.column,
-              token: s
-            }
-          end
-          s = s.next_token
+    tokens.each_with_index do |token, _token_idx|
+      next unless token.type == :TYPE and token.value == 'Class' and token.next_code_token.type == :LBRACK
+
+      s = token.next_code_token
+      while s.type != :RBRACK
+        if (s.type == :NAME || s.type == :SSTRING) && s.value.start_with?('::')
+          notify :warning, {
+            message: message,
+            line: s.line,
+            column: s.column,
+            token: s,
+          }
         end
+        s = s.next_token
       end
     end
   end

data/spec/puppet-lint/plugins/check_absolute_classname/relative_classname_inclusion_spec.rb

@@ -35,11 +35,11 @@ describe 'relative_classname_inclusion' do
         EOS
       end
 
-      it 'should detect 12 problems' do
+      it 'detects 12 problems' do
         expect(problems).to have(12).problems
       end
 
-      it 'should create warnings' do
+      it 'creates warnings' do
         expect(problems).to contain_warning(msg).on_line(1).in_column(17)
         expect(problems).to contain_warning(msg).on_line(2).in_column(17)
         expect(problems).to contain_warning(msg).on_line(6).in_column(17)
@@ -70,7 +70,7 @@ describe 'relative_classname_inclusion' do
         EOS
       end
 
-      it 'should not detect a problem' do
+      it 'does not detect a problem' do
         expect(problems).to have(0).problems
       end
     end
@@ -85,7 +85,7 @@ describe 'relative_classname_inclusion' do
         EOS
       end
 
-      it 'should detect no problems' do
+      it 'detects no problems' do
         expect(problems).to have(0).problems
       end
     end
@@ -102,7 +102,7 @@ describe 'relative_classname_inclusion' do
         EOS
       end
 
-      it 'should detect no problems' do
+      it 'detects no problems' do
         expect(problems).to have(0).problems
       end
     end
@@ -148,11 +148,11 @@ describe 'relative_classname_inclusion' do
         EOS
       end
 
-      it 'should detect 12 problems' do
+      it 'detects 12 problems' do
         expect(problems).to have(12).problems
       end
 
-      it 'should fix the problems' do
+      it 'fixes the problems' do
         expect(problems).to contain_fixed(msg).on_line(1).in_column(17)
         expect(problems).to contain_fixed(msg).on_line(2).in_column(17)
         expect(problems).to contain_fixed(msg).on_line(6).in_column(17)
@@ -167,9 +167,9 @@ describe 'relative_classname_inclusion' do
         expect(problems).to contain_fixed(msg).on_line(24).in_column(31)
       end
 
-      it 'should should remove colons' do
+      it 'shoulds remove colons' do
         expect(manifest).to eq(
-        <<-EOS
+          <<-EOS,
         include foobar
         include('foobar')
         include(foobar(baz))
@@ -215,7 +215,7 @@ describe 'relative_classname_inclusion' do
         EOS
       end
 
-      it 'should not detect any problems' do
+      it 'does not detect any problems' do
         expect(problems).to have(0).problems
       end
     end
@@ -230,7 +230,7 @@ describe 'relative_classname_inclusion' do
       EOS
     end
 
-    it 'should not detect any problems' do
+    it 'does not detect any problems' do
       expect(problems).to have(0).problems
     end
   end

data/spec/puppet-lint/plugins/check_classname_reference/relative_classname_reference_spec.rb

@@ -21,11 +21,11 @@ describe 'relative_classname_reference' do
         EOS
       end
 
-      it 'should detect 6 problems' do
+      it 'detects 6 problems' do
         expect(problems).to have(6).problems
       end
 
-      it 'should create warnings' do
+      it 'creates warnings' do
         expect(problems).to contain_warning(msg).on_line(1).in_column(15)
         expect(problems).to contain_warning(msg).on_line(1).in_column(31)
         expect(problems).to contain_warning(msg).on_line(5).in_column(28)
@@ -50,7 +50,7 @@ describe 'relative_classname_reference' do
         EOS
       end
 
-      it 'should not detect a problem' do
+      it 'does not detect a problem' do
         expect(problems).to have(0).problems
       end
     end
@@ -82,11 +82,11 @@ describe 'relative_classname_reference' do
         EOS
       end
 
-      it 'should detect 6 problems' do
+      it 'detects 6 problems' do
         expect(problems).to have(6).problems
       end
 
-      it 'should fix the problems' do
+      it 'fixes the problems' do
         expect(problems).to contain_fixed(msg).on_line(1).in_column(15)
         expect(problems).to contain_fixed(msg).on_line(1).in_column(31)
         expect(problems).to contain_fixed(msg).on_line(5).in_column(28)
@@ -95,9 +95,9 @@ describe 'relative_classname_reference' do
         expect(problems).to contain_fixed(msg).on_line(10).in_column(43)
       end
 
-      it 'should should remove colons' do
+      it 'shoulds remove colons' do
         expect(manifest).to eq(
-        <<-EOS
+          <<-EOS,
         Class[foo] -> Class['bar']
 
         file { '/path':
@@ -129,7 +129,7 @@ describe 'relative_classname_reference' do
         EOS
       end
 
-      it 'should not detect any problems' do
+      it 'does not detect any problems' do
         expect(problems).to have(0).problems
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puppet-lint-absolute_classname-check
 version: !ruby/object:Gem::Version
-  version: 3.1.0
+  version: 4.0.0
 platform: ruby
 authors:
 - Vox Pupuli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-11-29 00:00:00.000000000 Z
+date: 2023-04-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: puppet-lint
@@ -16,76 +16,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '3.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '4'
+        version: '5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '3.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '4'
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: rspec-collection_matchers
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: rspec-its
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
+        version: '5'
 description: "    A puppet-lint plugin to check that classes are not included or referenced
   by their absolute name.\n"
 email: voxpupuli@groups.io
@@ -112,7 +56,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.1.0
+      version: 2.7.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -124,7 +68,4 @@ signing_key:
 specification_version: 4
 summary: A puppet-lint plugin to check that classes are not included or referenced
   by their absolute name.
-test_files:
-- spec/puppet-lint/plugins/check_absolute_classname/relative_classname_inclusion_spec.rb
-- spec/puppet-lint/plugins/check_classname_reference/relative_classname_reference_spec.rb
-- spec/spec_helper.rb
+test_files: []