puppet-lint autofix with voxpupuli-test 5

Author: ekohl

manifests/config.pp

@@ -1,7 +1,6 @@
 # @summary Configure the foreman proxy
 # @api private
 class foreman_proxy::config {
-
   # Ensure SSL certs from the puppetmaster are available
   # Relationship is duplicated there as defined() is parse-order dependent
   if $foreman_proxy::ssl and defined(Class['puppet::server::config']) {
@@ -110,7 +109,7 @@
 
     if $foreman_proxy::use_sudoersd {
       if $uses_sudo and $foreman_proxy::manage_sudoersd {
-        ensure_resource('file', "${foreman_proxy::sudoers}.d", {'ensure' => 'directory'})
+        ensure_resource('file', "${foreman_proxy::sudoers}.d", { 'ensure' => 'directory' })
       }
 
       file { "${foreman_proxy::sudoers}.d/foreman-proxy":

manifests/install.pp

@@ -1,7 +1,7 @@
 # @summary Install the foreman proxy
 # @api private
 class foreman_proxy::install {
-  package {'foreman-proxy':
+  package { 'foreman-proxy':
     ensure => $foreman_proxy::version,
   }
 

manifests/plugin.pp

@@ -6,7 +6,7 @@
 # @param package
 #   The package to install. Underscores are replaced with dashes on Debian
 #
-define foreman_proxy::plugin(
+define foreman_proxy::plugin (
   $version = $foreman_proxy::params::plugin_version,
   $package = "${foreman_proxy::params::plugin_prefix}${title}",
 ) {

manifests/plugin/ansible.pp

@@ -62,7 +62,7 @@
   $foreman_ssl_ca = pick($foreman_proxy::foreman_ssl_ca, $foreman_proxy::ssl_ca)
   $proxy_url = $foreman_proxy::real_registered_proxy_url
 
-  file {"${foreman_proxy::config_dir}/ansible.cfg":
+  file { "${foreman_proxy::config_dir}/ansible.cfg":
     ensure  => file,
     content => template('foreman_proxy/plugin/ansible.cfg.erb'),
     owner   => 'root',

manifests/plugin/ansible/runner.pp

@@ -4,11 +4,10 @@
 #
 # $package_name:: Name of the package to install to provide 'ansible-runner' command
 #
-class foreman_proxy::plugin::ansible::runner(
+class foreman_proxy::plugin::ansible::runner (
   Boolean $manage_runner_repo = $foreman_proxy::plugin::ansible::manage_runner_repo,
   String  $package_name = $foreman_proxy::plugin::ansible::runner_package_name,
 ) {
-
   if $manage_runner_repo {
     case $facts['os']['family'] {
       'Debian': {
@@ -36,5 +35,4 @@
   package { $package_name:
     ensure => 'installed',
   }
-
 }

manifests/plugin/discovery.pp

@@ -20,15 +20,15 @@
   Stdlib::HTTPUrl $source_url = $foreman_proxy::plugin::discovery::params::source_url,
   String $image_name = $foreman_proxy::plugin::discovery::params::image_name,
 ) inherits foreman_proxy::plugin::discovery::params {
-  foreman_proxy::plugin {'discovery':
+  foreman_proxy::plugin { 'discovery':
   }
 
   foreman_proxy::feature { 'Discovery': }
 
   if $install_images {
     $tftp_root_clean = regsubst($tftp_root, '/$', '')
 
-    foreman_proxy::remote_file {"${tftp_root_clean}/boot/${image_name}":
+    foreman_proxy::remote_file { "${tftp_root_clean}/boot/${image_name}":
       remote_location => "${source_url}${image_name}",
       mode            => '0644',
     } ~> exec { "untar ${image_name}":

manifests/plugin/pulp.pp

@@ -32,8 +32,7 @@
   Array[String[1], 1] $client_authentication = ['client_certificate'],
   Stdlib::HTTPUrl $rhsm_url = $foreman_proxy::plugin::pulp::params::rhsm_url,
 ) inherits foreman_proxy::plugin::pulp::params {
-
-  foreman_proxy::plugin {'pulp':
+  foreman_proxy::plugin { 'pulp':
     version => $version,
   }
   -> [

manifests/plugin/remote_execution/script.pp

@@ -79,5 +79,4 @@
   }
 
   Class['foreman_proxy::config'] ~> Class['foreman_proxy::plugin::remote_execution::mosquitto']
-
 }

manifests/plugin/remote_execution/ssh_user.pp

@@ -16,9 +16,8 @@
   Enum['sudo', 'su'] $effective_user_method = $foreman_proxy::plugin::remote_execution::ssh_user::params::effective_user_method,
   Array[String[1]]   $ssh_keys              = $foreman_proxy::plugin::remote_execution::ssh_user::params::ssh_keys,
 ) inherits foreman_proxy::plugin::remote_execution::ssh_user::params {
-
   # Manage the user
-  if $manage_user and $ssh_user != 'root'{
+  if $manage_user and $ssh_user != 'root' {
     user { $ssh_user:
       ensure           => present,
       home             => "/home/${ssh_user}",

manifests/plugin/remote_execution/ssh_user/params.pp

@@ -1,7 +1,6 @@
 # Remote Execution SSH user default parameters
 # @api private
 class foreman_proxy::plugin::remote_execution::ssh_user::params {
-
   # `getvar()` is used because the top scope variables being accessed here are *not* facts.
   # They come from the ENC, (foreman), and are not present in `$facts`.
   # `$::var` can't safely be used either as catalog compilation will fail if `strict_variables = true`