Merge pull request #1078 from voxpupuli/modulesync

modulesync 10.6.0

Author: bastelfreak

.devcontainer/devcontainer.json

@@ -1,4 +1,3 @@
-// For format details, see https://aka.ms/devcontainer.json. For config options, see the
 {
 	"name": "VoxBox",
 	"image": "ghcr.io/voxpupuli/voxbox:latest"

.msync.yml

@@ -2,4 +2,4 @@
 # Managed by modulesync - DO NOT EDIT
 # https://voxpupuli.org/docs/updating-files-managed-with-modulesync/
 
-modulesync_config_version: '10.5.0'
+modulesync_config_version: '10.6.0'

.rubocop_todo.yml

@@ -1,22 +1,36 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config --no-auto-gen-timestamp`
-# using RuboCop version 1.50.2.
+# using RuboCop version 1.85.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
-# Offense count: 8
+# Offense count: 5
 # This cop supports unsafe autocorrection (--autocorrect-all).
 RSpec/BeEq:
   Exclude:
     - 'spec/unit/puppet/provider/rabbitmq_plugin/rabbitmqctl_spec.rb'
     - 'spec/unit/puppet/provider/rabbitmq_policy/rabbitmqctl_spec.rb'
     - 'spec/unit/puppet/provider/rabbitmq_user_permissions/rabbitmqctl_spec.rb'
-    - 'spec/unit/puppet/provider/rabbitmq_vhost/rabbitmqctl_spec.rb'
     - 'spec/unit/puppet/type/rabbitmq_user_permissions_spec.rb'
 
-# Offense count: 185
+# Offense count: 13
+RSpec/LeakyLocalVariable:
+  Exclude:
+    - 'spec/classes/rabbitmq_spec.rb'
+    - 'spec/unit/puppet/provider/rabbitmq_binding/rabbitmqadmin_spec.rb'
+    - 'spec/unit/puppet/provider/rabbitmq_cli_spec.rb'
+    - 'spec/unit/puppet/provider/rabbitmq_cluster/rabbitmqctl_spec.rb'
+    - 'spec/unit/puppet/provider/rabbitmq_exchange/rabbitmqadmin_spec.rb'
+    - 'spec/unit/puppet/provider/rabbitmq_parameter/rabbitmqctl_federation_spec.rb'
+    - 'spec/unit/puppet/provider/rabbitmq_parameter/rabbitmqctl_spec.rb'
+    - 'spec/unit/puppet/provider/rabbitmq_plugin/rabbitmqctl_spec.rb'
+    - 'spec/unit/puppet/provider/rabbitmq_queue/rabbitmqadmin_spec.rb'
+    - 'spec/unit/puppet/provider/rabbitmq_user/rabbitmqctl_spec.rb'
+    - 'spec/unit/puppet/provider/rabbitmq_vhost/rabbitmqctl_spec.rb'
+
+# Offense count: 172
 # Configuration parameters: .
 # SupportedStyles: have_received, receive
 RSpec/MessageSpies:
@@ -37,7 +51,7 @@ RSpec/RepeatedExampleGroupDescription:
   Exclude:
     - 'spec/classes/rabbitmq_spec.rb'
 
-# Offense count: 125
+# Offense count: 118
 RSpec/StubbedMock:
   Exclude:
     - 'spec/unit/facter/util/fact_rabbitmq_clustername_spec.rb'
@@ -54,4 +68,11 @@ RSpec/StubbedMock:
     - 'spec/unit/puppet/provider/rabbitmq_policy/rabbitmqctl_spec.rb'
     - 'spec/unit/puppet/provider/rabbitmq_queue/rabbitmqadmin_spec.rb'
     - 'spec/unit/puppet/provider/rabbitmq_user_permissions/rabbitmqctl_spec.rb'
-    - 'spec/unit/puppet/provider/rabbitmq_vhost/rabbitmqctl_spec.rb'
+
+# Offense count: 1
+# This cop supports unsafe autocorrection (--autocorrect-all).
+# Configuration parameters: AllowedReceivers.
+# AllowedReceivers: Thread.current
+Style/HashEachMethods:
+  Exclude:
+    - 'lib/puppet/type/rabbitmq_parameter.rb'

Gemfile

@@ -4,7 +4,7 @@
 source ENV['GEM_SOURCE'] || 'https://rubygems.org'
 
 group :test do
-  gem 'voxpupuli-test', '~> 13.0',  :require => false
+  gem 'voxpupuli-test', '~> 14.0',  :require => false
   gem 'puppet_metadata', '~> 6.0',  :require => false
 end
 
@@ -18,7 +18,7 @@ group :system_tests do
 end
 
 group :release do
-  gem 'voxpupuli-release', '~> 5.0',  :require => false
+  gem 'voxpupuli-release', '~> 5.3',  :require => false
 end
 
 gem 'rake', :require => false

lib/puppet/provider/rabbitmq_binding/rabbitmqadmin.rb

@@ -54,7 +54,7 @@ def self.instances
           routing_key: routing_key,
           arguments: JSON.parse(arguments),
           ensure: :present,
-          name: hashed_name
+          name: hashed_name,
         }
         resources << new(binding) if binding[:name]
       end

lib/puppet/provider/rabbitmq_cluster/rabbitmqctl.rb

@@ -3,7 +3,7 @@
 require File.expand_path(File.join(File.dirname(__FILE__), '..', 'rabbitmq_cli'))
 Puppet::Type.type(:rabbitmq_cluster).provide(
   :rabbitmqctl,
-  parent: Puppet::Provider::RabbitmqCli
+  parent: Puppet::Provider::RabbitmqCli,
 ) do
   desc 'Rabbitmqctl provider for rabbitmq cluster'
   confine feature: :posix
@@ -19,7 +19,7 @@ def create
     local_node = @resource[:local_node].to_s.gsub(%r{^.*@}, '')
 
     if local_node == init_node || [Facter.value(:networking)['hostname'], Facter.value(:networking)['fqdn']].include?(init_node)
-      return rabbitmqctl('set_cluster_name', @resource[:name]) unless cluster_name == resource[:name].to_s
+      rabbitmqctl('set_cluster_name', @resource[:name]) unless cluster_name == resource[:name].to_s
     else
       rabbitmqctl('stop_app')
       rabbitmqctl('join_cluster', "rabbit@#{init_node}", "--#{storage_type}")

lib/puppet/provider/rabbitmq_exchange/rabbitmqadmin.rb

@@ -49,7 +49,7 @@ def self.instances
           durable: durable,
           auto_delete: auto_delete,
           name: format('%s@%s', name, vhost),
-          arguments: JSON.parse(arguments)
+          arguments: JSON.parse(arguments),
         }
         resources << new(exchange) if exchange[:type]
       end

lib/puppet/provider/rabbitmq_parameter/rabbitmqctl.rb

@@ -11,7 +11,7 @@
   mk_resource_methods
 
   def initialize(value = {})
-    super(value)
+    super
     @property_flush = {}
   end
 
@@ -33,7 +33,7 @@ def self.instances
           ensure: :present,
           component_name: Regexp.last_match(1),
           name: format('%s@%s', Regexp.last_match(2), vhost),
-          value: JSON.parse(Regexp.last_match(3))
+          value: JSON.parse(Regexp.last_match(3)),
         }
         resources << new(parameter)
       end

lib/puppet/provider/rabbitmq_policy/rabbitmqctl.rb

@@ -46,7 +46,7 @@ def self.policies(vhost, name)
           applyto: applyto,
           pattern: pattern,
           definition: definition,
-          priority: priority
+          priority: priority,
         }
       end
     end

lib/puppet/provider/rabbitmq_queue/rabbitmqadmin.rb

@@ -42,7 +42,7 @@ def self.instances
           auto_delete: auto_delete,
           arguments: JSON.parse(arguments),
           ensure: :present,
-          name: format('%s@%s', name, vhost)
+          name: format('%s@%s', name, vhost),
         }
         resources << new(queue) if queue[:name]
       end