RubyGems - puppet-lint - Versions diffs - 2.5.2 → 3.0.0 - Mend

puppet-lint 2.5.2 → 3.0.0

Files changed (112) hide show

data/spec/{puppet-lint → unit/puppet-lint}/plugins/check_whitespace/trailing_whitespace_spec.rb RENAMED Viewed

@@ -7,11 +7,11 @@ describe 'trailing_whitespace' do
     context 'line with trailing whitespace' do
       let(:code) { 'foo ' }
-      it 'should only detect a single problem' do
+      it 'only detects a single problem' do
         expect(problems).to have(1).problem
       end
-      it 'should create an error' do
+      it 'creates an error' do
         expect(problems).to contain_error(msg).on_line(1).in_column(4)
       end
     end
@@ -25,37 +25,37 @@ describe 'trailing_whitespace' do
         ].join("\n")
       end
-      it 'should only detect a single problem' do
+      it 'only detects a single problem' do
         expect(problems).to have(1).problem
       end
-      it 'should create an error' do
+      it 'creates an error' do
         expect(problems).to contain_error(msg).on_line(2).in_column(1)
       end
     end
   end
   context 'with fix enabled' do
-    before do
+    before(:each) do
       PuppetLint.configuration.fix = true
     end
-    after do
+    after(:each) do
       PuppetLint.configuration.fix = false
     end
     context 'single line with trailing whitespace' do
       let(:code) { 'foo ' }
-      it 'should only detect a single problem' do
+      it 'only detects a single problem' do
         expect(problems).to have(1).problem
       end
-      it 'should fix the manifest' do
+      it 'fixes the manifest' do
         expect(problems).to contain_fixed(msg).on_line(1).in_column(4)
       end
-      it 'should remove the trailing whitespace' do
+      it 'removes the trailing whitespace' do
         expect(manifest).to eq('foo')
       end
     end
@@ -63,15 +63,15 @@ describe 'trailing_whitespace' do
     context 'multiple lines with trailing whitespace' do
       let(:code) { "foo    \nbar" }
-      it 'should only detect a single problem' do
+      it 'only detects a single problem' do
         expect(problems).to have(1).problem
       end
-      it 'should fix the manifest' do
+      it 'fixes the manifest' do
         expect(problems).to contain_fixed(msg).on_line(1).in_column(4)
       end
-      it 'should remove the trailing whitespace' do
+      it 'removes the trailing whitespace' do
         expect(manifest).to eq("foo\nbar")
       end
     end
@@ -93,15 +93,15 @@ describe 'trailing_whitespace' do
         ].join("\n")
       end
-      it 'should only detect a single problem' do
+      it 'only detects a single problem' do
         expect(problems).to have(1).problem
       end
-      it 'should create an error' do
+      it 'creates an error' do
         expect(problems).to contain_fixed(msg).on_line(2).in_column(1)
       end
-      it 'should remove the trailing whitespace' do
+      it 'removes the trailing whitespace' do
         expect(manifest).to eq(fixed)
       end
     end

data/spec/unit/puppet-lint/puppet-lint_spec.rb ADDED Viewed

@@ -0,0 +1,18 @@
+# rubocop: disable Naming/FileName
+require 'spec_helper'
+describe PuppetLint do
+  subject(:linter) { described_class.new }
+  it 'accepts manifests as a string' do
+    linter.code = 'class foo { }'
+    expect(linter.code).not_to be_nil
+  end
+  it 'returns empty manifest when empty one given as the input' do
+    linter.code = ''
+    linter.run
+    expect(linter.manifest).to eq('')
+  end
+end

metadata CHANGED Viewed

@@ -1,20 +1,19 @@
 --- !ruby/object:Gem::Specification
 name: puppet-lint
 version: !ruby/object:Gem::Version
-  version: 2.5.2
+  version: 3.0.0
 platform: ruby
 authors:
 - Tim Sharpe
 - Puppet, Inc.
 - Community Contributors
-autorequire:
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-09-14 00:00:00.000000000 Z
+date: 2022-10-13 00:00:00.000000000 Z
 dependencies: []
-description: |-
-  Checks your Puppet manifests against the Puppetlabs
-    style guide and alerts you to any discrepancies.
+description: "    Checks your Puppet manifests against the Puppetlabs style guide
+  and alerts you to any discrepancies.'\n"
 email:
 - tim@sharpe.id.au
 - modules-team@puppet.com
@@ -23,8 +22,7 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- CHANGELOG.md
-- HISTORY.md
+- ".rubocop.yml"
 - LICENSE
 - README.md
 - bin/puppet-lint
@@ -76,11 +74,15 @@ files:
 - lib/puppet-lint/plugins/check_whitespace/80chars.rb
 - lib/puppet-lint/plugins/check_whitespace/arrow_alignment.rb
 - lib/puppet-lint/plugins/check_whitespace/hard_tabs.rb
+- lib/puppet-lint/plugins/check_whitespace/line_length.rb
 - lib/puppet-lint/plugins/check_whitespace/trailing_whitespace.rb
+- lib/puppet-lint/report/github.rb
+- lib/puppet-lint/report/sarif_template.json
 - lib/puppet-lint/tasks/gemfile_rewrite.rb
 - lib/puppet-lint/tasks/puppet-lint.rb
 - lib/puppet-lint/tasks/release_test.rb
 - lib/puppet-lint/version.rb
+- spec/acceptance/puppet_lint_spec.rb
 - spec/fixtures/test/manifests/fail.pp
 - spec/fixtures/test/manifests/ignore.pp
 - spec/fixtures/test/manifests/ignore_multiple_block.pp
@@ -93,58 +95,60 @@ files:
 - spec/fixtures/test/manifests/unterminated_control_comment.pp
 - spec/fixtures/test/manifests/url_interpolation.pp
 - spec/fixtures/test/manifests/warning.pp
-- spec/puppet-lint/bin_spec.rb
-- spec/puppet-lint/checks_spec.rb
-- spec/puppet-lint/configuration_spec.rb
-- spec/puppet-lint/data_spec.rb
-- spec/puppet-lint/ignore_overrides_spec.rb
-- spec/puppet-lint/lexer/string_slurper_spec.rb
-- spec/puppet-lint/lexer/token_spec.rb
-- spec/puppet-lint/lexer_spec.rb
-- spec/puppet-lint/plugins/check_classes/arrow_on_right_operand_line_spec.rb
-- spec/puppet-lint/plugins/check_classes/autoloader_layout_spec.rb
-- spec/puppet-lint/plugins/check_classes/class_inherits_from_params_class_spec.rb
-- spec/puppet-lint/plugins/check_classes/code_on_top_scope_spec.rb
-- spec/puppet-lint/plugins/check_classes/inherits_across_namespaces_spec.rb
-- spec/puppet-lint/plugins/check_classes/name_contains_uppercase_spec.rb
-- spec/puppet-lint/plugins/check_classes/names_containing_dash_spec.rb
-- spec/puppet-lint/plugins/check_classes/nested_classes_or_defines_spec.rb
-- spec/puppet-lint/plugins/check_classes/parameter_order_spec.rb
-- spec/puppet-lint/plugins/check_classes/right_to_left_relationship_spec.rb
-- spec/puppet-lint/plugins/check_classes/variable_scope_spec.rb
-- spec/puppet-lint/plugins/check_comments/slash_comments_spec.rb
-- spec/puppet-lint/plugins/check_comments/star_comments_spec.rb
-- spec/puppet-lint/plugins/check_conditionals/case_without_default_spec.rb
-- spec/puppet-lint/plugins/check_conditionals/selector_inside_resource_spec.rb
-- spec/puppet-lint/plugins/check_documentation/documentation_spec.rb
-- spec/puppet-lint/plugins/check_nodes/unquoted_node_name_spec.rb
-- spec/puppet-lint/plugins/check_resources/duplicate_params_spec.rb
-- spec/puppet-lint/plugins/check_resources/ensure_first_param_spec.rb
-- spec/puppet-lint/plugins/check_resources/ensure_not_symlink_target_spec.rb
-- spec/puppet-lint/plugins/check_resources/file_mode_spec.rb
-- spec/puppet-lint/plugins/check_resources/unquoted_file_mode_spec.rb
-- spec/puppet-lint/plugins/check_resources/unquoted_resource_title_spec.rb
-- spec/puppet-lint/plugins/check_strings/double_quoted_strings_spec.rb
-- spec/puppet-lint/plugins/check_strings/only_variable_string_spec.rb
-- spec/puppet-lint/plugins/check_strings/puppet_url_without_modules_spec.rb
-- spec/puppet-lint/plugins/check_strings/quoted_booleans_spec.rb
-- spec/puppet-lint/plugins/check_strings/single_quote_string_with_variables_spec.rb
-- spec/puppet-lint/plugins/check_strings/variables_not_enclosed_spec.rb
-- spec/puppet-lint/plugins/check_variables/variable_contains_dash_spec.rb
-- spec/puppet-lint/plugins/check_variables/variable_is_lowercase_spec.rb
-- spec/puppet-lint/plugins/check_whitespace/140chars_spec.rb
-- spec/puppet-lint/plugins/check_whitespace/2sp_soft_tabs_spec.rb
-- spec/puppet-lint/plugins/check_whitespace/80chars_spec.rb
-- spec/puppet-lint/plugins/check_whitespace/arrow_alignment_spec.rb
-- spec/puppet-lint/plugins/check_whitespace/hard_tabs_spec.rb
-- spec/puppet-lint/plugins/check_whitespace/trailing_whitespace_spec.rb
-- spec/puppet-lint_spec.rb
 - spec/spec_helper.rb
+- spec/spec_helper_acceptance.rb
+- spec/spec_helper_acceptance_local.rb
+- spec/unit/puppet-lint/bin_spec.rb
+- spec/unit/puppet-lint/checks_spec.rb
+- spec/unit/puppet-lint/configuration_spec.rb
+- spec/unit/puppet-lint/data_spec.rb
+- spec/unit/puppet-lint/ignore_overrides_spec.rb
+- spec/unit/puppet-lint/lexer/string_slurper_spec.rb
+- spec/unit/puppet-lint/lexer/token_spec.rb
+- spec/unit/puppet-lint/lexer_spec.rb
+- spec/unit/puppet-lint/plugins/check_classes/arrow_on_right_operand_line_spec.rb
+- spec/unit/puppet-lint/plugins/check_classes/autoloader_layout_spec.rb
+- spec/unit/puppet-lint/plugins/check_classes/class_inherits_from_params_class_spec.rb
+- spec/unit/puppet-lint/plugins/check_classes/code_on_top_scope_spec.rb
+- spec/unit/puppet-lint/plugins/check_classes/inherits_across_namespaces_spec.rb
+- spec/unit/puppet-lint/plugins/check_classes/name_contains_uppercase_spec.rb
+- spec/unit/puppet-lint/plugins/check_classes/names_containing_dash_spec.rb
+- spec/unit/puppet-lint/plugins/check_classes/nested_classes_or_defines_spec.rb
+- spec/unit/puppet-lint/plugins/check_classes/parameter_order_spec.rb
+- spec/unit/puppet-lint/plugins/check_classes/right_to_left_relationship_spec.rb
+- spec/unit/puppet-lint/plugins/check_classes/variable_scope_spec.rb
+- spec/unit/puppet-lint/plugins/check_comments/slash_comments_spec.rb
+- spec/unit/puppet-lint/plugins/check_comments/star_comments_spec.rb
+- spec/unit/puppet-lint/plugins/check_conditionals/case_without_default_spec.rb
+- spec/unit/puppet-lint/plugins/check_conditionals/selector_inside_resource_spec.rb
+- spec/unit/puppet-lint/plugins/check_documentation/documentation_spec.rb
+- spec/unit/puppet-lint/plugins/check_nodes/unquoted_node_name_spec.rb
+- spec/unit/puppet-lint/plugins/check_resources/duplicate_params_spec.rb
+- spec/unit/puppet-lint/plugins/check_resources/ensure_first_param_spec.rb
+- spec/unit/puppet-lint/plugins/check_resources/ensure_not_symlink_target_spec.rb
+- spec/unit/puppet-lint/plugins/check_resources/file_mode_spec.rb
+- spec/unit/puppet-lint/plugins/check_resources/unquoted_file_mode_spec.rb
+- spec/unit/puppet-lint/plugins/check_resources/unquoted_resource_title_spec.rb
+- spec/unit/puppet-lint/plugins/check_strings/double_quoted_strings_spec.rb
+- spec/unit/puppet-lint/plugins/check_strings/only_variable_string_spec.rb
+- spec/unit/puppet-lint/plugins/check_strings/puppet_url_without_modules_spec.rb
+- spec/unit/puppet-lint/plugins/check_strings/quoted_booleans_spec.rb
+- spec/unit/puppet-lint/plugins/check_strings/single_quote_string_with_variables_spec.rb
+- spec/unit/puppet-lint/plugins/check_strings/variables_not_enclosed_spec.rb
+- spec/unit/puppet-lint/plugins/check_variables/variable_contains_dash_spec.rb
+- spec/unit/puppet-lint/plugins/check_variables/variable_is_lowercase_spec.rb
+- spec/unit/puppet-lint/plugins/check_whitespace/140chars_spec.rb
+- spec/unit/puppet-lint/plugins/check_whitespace/2sp_soft_tabs_spec.rb
+- spec/unit/puppet-lint/plugins/check_whitespace/80chars_spec.rb
+- spec/unit/puppet-lint/plugins/check_whitespace/arrow_alignment_spec.rb
+- spec/unit/puppet-lint/plugins/check_whitespace/hard_tabs_spec.rb
+- spec/unit/puppet-lint/plugins/check_whitespace/trailing_whitespace_spec.rb
+- spec/unit/puppet-lint/puppet-lint_spec.rb
 homepage: https://github.com/puppetlabs/puppet-lint/
 licenses:
 - MIT
 metadata: {}
-post_install_message:
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -152,75 +156,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.7'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project:
-rubygems_version: 2.7.6.3
-signing_key:
+rubygems_version: 3.1.6
+signing_key:
 specification_version: 4
 summary: Ensure your Puppet manifests conform with the Puppetlabs style guide
-test_files:
-- spec/fixtures/test/manifests/fail.pp
-- spec/fixtures/test/manifests/ignore.pp
-- spec/fixtures/test/manifests/ignore_multiple_block.pp
-- spec/fixtures/test/manifests/ignore_multiple_line.pp
-- spec/fixtures/test/manifests/ignore_reason.pp
-- spec/fixtures/test/manifests/init.pp
-- spec/fixtures/test/manifests/malformed.pp
-- spec/fixtures/test/manifests/mismatched_control_comment.pp
-- spec/fixtures/test/manifests/two_warnings.pp
-- spec/fixtures/test/manifests/unterminated_control_comment.pp
-- spec/fixtures/test/manifests/url_interpolation.pp
-- spec/fixtures/test/manifests/warning.pp
-- spec/puppet-lint/bin_spec.rb
-- spec/puppet-lint/checks_spec.rb
-- spec/puppet-lint/configuration_spec.rb
-- spec/puppet-lint/data_spec.rb
-- spec/puppet-lint/ignore_overrides_spec.rb
-- spec/puppet-lint/lexer/string_slurper_spec.rb
-- spec/puppet-lint/lexer/token_spec.rb
-- spec/puppet-lint/lexer_spec.rb
-- spec/puppet-lint/plugins/check_classes/arrow_on_right_operand_line_spec.rb
-- spec/puppet-lint/plugins/check_classes/autoloader_layout_spec.rb
-- spec/puppet-lint/plugins/check_classes/class_inherits_from_params_class_spec.rb
-- spec/puppet-lint/plugins/check_classes/code_on_top_scope_spec.rb
-- spec/puppet-lint/plugins/check_classes/inherits_across_namespaces_spec.rb
-- spec/puppet-lint/plugins/check_classes/name_contains_uppercase_spec.rb
-- spec/puppet-lint/plugins/check_classes/names_containing_dash_spec.rb
-- spec/puppet-lint/plugins/check_classes/nested_classes_or_defines_spec.rb
-- spec/puppet-lint/plugins/check_classes/parameter_order_spec.rb
-- spec/puppet-lint/plugins/check_classes/right_to_left_relationship_spec.rb
-- spec/puppet-lint/plugins/check_classes/variable_scope_spec.rb
-- spec/puppet-lint/plugins/check_comments/slash_comments_spec.rb
-- spec/puppet-lint/plugins/check_comments/star_comments_spec.rb
-- spec/puppet-lint/plugins/check_conditionals/case_without_default_spec.rb
-- spec/puppet-lint/plugins/check_conditionals/selector_inside_resource_spec.rb
-- spec/puppet-lint/plugins/check_documentation/documentation_spec.rb
-- spec/puppet-lint/plugins/check_nodes/unquoted_node_name_spec.rb
-- spec/puppet-lint/plugins/check_resources/duplicate_params_spec.rb
-- spec/puppet-lint/plugins/check_resources/ensure_first_param_spec.rb
-- spec/puppet-lint/plugins/check_resources/ensure_not_symlink_target_spec.rb
-- spec/puppet-lint/plugins/check_resources/file_mode_spec.rb
-- spec/puppet-lint/plugins/check_resources/unquoted_file_mode_spec.rb
-- spec/puppet-lint/plugins/check_resources/unquoted_resource_title_spec.rb
-- spec/puppet-lint/plugins/check_strings/double_quoted_strings_spec.rb
-- spec/puppet-lint/plugins/check_strings/only_variable_string_spec.rb
-- spec/puppet-lint/plugins/check_strings/puppet_url_without_modules_spec.rb
-- spec/puppet-lint/plugins/check_strings/quoted_booleans_spec.rb
-- spec/puppet-lint/plugins/check_strings/single_quote_string_with_variables_spec.rb
-- spec/puppet-lint/plugins/check_strings/variables_not_enclosed_spec.rb
-- spec/puppet-lint/plugins/check_variables/variable_contains_dash_spec.rb
-- spec/puppet-lint/plugins/check_variables/variable_is_lowercase_spec.rb
-- spec/puppet-lint/plugins/check_whitespace/140chars_spec.rb
-- spec/puppet-lint/plugins/check_whitespace/2sp_soft_tabs_spec.rb
-- spec/puppet-lint/plugins/check_whitespace/80chars_spec.rb
-- spec/puppet-lint/plugins/check_whitespace/arrow_alignment_spec.rb
-- spec/puppet-lint/plugins/check_whitespace/hard_tabs_spec.rb
-- spec/puppet-lint/plugins/check_whitespace/trailing_whitespace_spec.rb
-- spec/puppet-lint_spec.rb
-- spec/spec_helper.rb
+test_files: []

data/CHANGELOG.md DELETED Viewed

@@ -1,33 +0,0 @@
-# Changelog
-## [2.5.2](https://github.com/puppetlabs/puppet-lint/tree/2.5.2) (2021-09-14)
-[Full Changelog](https://github.com/puppetlabs/puppet-lint/compare/2.5.1...2.5.2)
-**Closed issues:**
-- Release 2.5.1 contains stuff in bin/ [\#22](https://github.com/puppetlabs/puppet-lint/issues/22)
-## [2.5.1](https://github.com/puppetlabs/puppet-lint/tree/2.5.1) (2021-09-13) [YANKED]
-> **NOTE:** This release was yanked as there were unexpected artefacts published in the build
-[Full Changelog](https://github.com/puppetlabs/puppet-lint/compare/2.5.0...2.5.1)
-**Fixed bugs:**
-- double\_quoted\_strings does not permit space escape sequence [\#16](https://github.com/puppetlabs/puppet-lint/issues/16)
-**Closed issues:**
-- SPIKE: Add Puppetlabs automated release workflow [\#7](https://github.com/puppetlabs/puppet-lint/issues/7)
-- Update testing to test against latest puppet/ruby versions [\#5](https://github.com/puppetlabs/puppet-lint/issues/5)
-- Add in Puppetlabs Github Actions for testing [\#4](https://github.com/puppetlabs/puppet-lint/issues/4)
-- Prepare gem for release [\#3](https://github.com/puppetlabs/puppet-lint/issues/3)
-**Merged pull requests:**
-- \(GH-16\) Do not warn on "\s" string literal [\#20](https://github.com/puppetlabs/puppet-lint/pull/20) ([sanfrancrisko](https://github.com/sanfrancrisko))
-- \(maint\) Prepare repo and gem for next release [\#15](https://github.com/puppetlabs/puppet-lint/pull/15) ([da-ar](https://github.com/da-ar))
-\* *This Changelog was automatically generated by [github_changelog_generator](https://github.com/github-changelog-generator/github-changelog-generator)*