hammer_cli_foreman_virt_who_configure 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aa67ec96782ac99c36a99614216817e60422d1b7b7ef99f21152c806a610560f
-  data.tar.gz: 3f3b9692d4eb19636b444aed17ae030a4dcb31b5a5963eadaab16305c7740832
+  metadata.gz: fce0d55b84baaf7c230353820665c88c6053b7f62253231a3e0d17d57663d0bc
+  data.tar.gz: 22219b792cfe77a3500a149641a579fd28100ca3b084b75e99137b2415cf26f8
 SHA512:
-  metadata.gz: 7b4f624d8ef5af1d65781be5511f292cde27dbc52ba095eec6fd3871b3244466af51fe67d7ca19b36fcea2ca05a51d28d93da195d6fa67759bea337b3019456a
-  data.tar.gz: 5180c3415c4d94a6721b589a17761cbd04da023ed433075177efb61a73b2cde5fb7c91a7004ff61aed04524684c3c68285a6b116e6a0d12fdc49af470063a930
+  metadata.gz: 71786def3416e05021baa313753105a35e6eee7faa2d6fe259a228539a42e8240bce786a7ca5271b9ec2245a450b404f598d3e802879ebf4cb347ea7c47f2536
+  data.tar.gz: 8cf7a835c06daedd8c44b7907167b22aa245c727e508f9847c60540a9d81d3067ace69d29f983934bfacf49935b563f8ca9cafdc0b2883c847f074f6bcc6dab4

data/lib/hammer_cli_foreman_virt_who_configure/config.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'hammer_cli_foreman_virt_who_configure/system_caller'
 
 module HammerCLIForemanVirtWhoConfigure
@@ -19,34 +21,34 @@ module HammerCLIForemanVirtWhoConfigure
 
     def self.format_status(status)
       case status
-        when 'unknown'
-          _('No Report Yet')
-        when 'ok', 'out_of_date'
-          _('OK')
-        when 'error'
-          _('Error')
-        else
-          _('Unknown configuration status')
+      when 'unknown'
+        _('No Report Yet')
+      when 'ok', 'out_of_date'
+        _('OK')
+      when 'error'
+        _('Error')
+      else
+        _('Unknown configuration status')
       end
     end
 
     def self.format_filtering_mode(mode)
       case mode
-        when MODE_UNLIMITED
-          _('Unlimited')
-        when MODE_WHITELIST
-          _('Whitelist')
-        when MODE_BLACKLIST
-          _('Blacklist')
-        else
-          _('Unknown listing mode')
+      when MODE_UNLIMITED
+        _('Unlimited')
+      when MODE_WHITELIST
+        _('Whitelist')
+      when MODE_BLACKLIST
+        _('Blacklist')
+      else
+        _('Unknown listing mode')
       end
     end
 
     def self.validate_hypervisor_options(conf)
-      options = conf["hypervisor_type"] == 'kubevirt' ? %w(hypervisor_server hypervisor_username) : %w(kubeconfig_path)
-      options.append("prism_flavor", "ahv_internal_debug") unless conf["hypervisor_type"] == 'ahv'
-      conf.delete_if { |k, v| options.include?(k) }
+      options = conf['hypervisor_type'] == 'kubevirt' ? %w(hypervisor_server hypervisor_username) : %w(kubeconfig_path)
+      options.append('prism_flavor', 'ahv_internal_debug') unless conf['hypervisor_type'] == 'ahv'
+      conf.delete_if { |k, _v| options.include?(k) }
     end
 
     class ListCommand < HammerCLIForeman::ListCommand
@@ -113,12 +115,12 @@ module HammerCLIForemanVirtWhoConfigure
         if conf['filtering_mode'] != MODE_WHITELIST
           conf['whitelist'] = nil
         else
-          conf['whitelist'] ||= " "
+          conf['whitelist'] ||= ' '
         end
         if conf['filtering_mode'] != MODE_BLACKLIST
           conf['blacklist'] = nil
         else
-          conf['blacklist'] ||= " "
+          conf['blacklist'] ||= ' '
         end
         VirtWhoConfig.validate_hypervisor_options(conf)
       end
@@ -127,7 +129,7 @@ module HammerCLIForemanVirtWhoConfigure
     end
 
     class FetchCommand < HammerCLIForeman::Command
-      command_name "fetch"
+      command_name 'fetch'
       action :deploy_script
 
       failure_message _('Could not fetch the Virt Who configuration')
@@ -142,11 +144,11 @@ module HammerCLIForemanVirtWhoConfigure
             # this could be a security issue, the file should be readable only by the user
             output.print_error(
               _('Could not save the script'),
-              _("File at %{path} already exists, please specify a different path.") % { :path => path }
+              _('File at %{path} already exists, please specify a different path.') % { :path => path }
             )
             return HammerCLI::EX_USAGE
           else
-            File.write(path, data, perm: 0700, mode: File::RDWR|File::CREAT|File::EXCL)
+            File.write(path, data, perm: 0700, mode: File::RDWR | File::CREAT | File::EXCL)
             return HammerCLI::EX_OK
           end
         else
@@ -163,7 +165,7 @@ module HammerCLIForemanVirtWhoConfigure
     end
 
     class DeployCommand < HammerCLIForeman::Command
-      command_name "deploy"
+      command_name 'deploy'
       desc _('Download and execute script for the specified virt-who configuration')
       action :deploy_script
 
@@ -194,7 +196,7 @@ module HammerCLIForemanVirtWhoConfigure
         'none' => 0,
         'whitelist' => 1,
         'blacklist' => 2
-      }
+      }.freeze
 
       def self.included(base)
         base.option '--filtering-mode', 'MODE', _('Hypervisor filtering mode'),

data/lib/hammer_cli_foreman_virt_who_configure/i18n.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'hammer_cli/i18n'
 
 module HammerCLIForemanVirtWhoConfigure

data/lib/hammer_cli_foreman_virt_who_configure/system_caller.rb

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 module HammerCLIForemanVirtWhoConfigure
   class SystemCaller
-    WHITELISTED_VARS = %w[HOME USER LANG].freeze
+    WHITELISTED_VARS = %w(HOME USER LANG).freeze
 
     def initialize(tempfile = nil)
       @tempfile = tempfile || Tempfile.new('virt_who')

data/lib/hammer_cli_foreman_virt_who_configure/version.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 module HammerCLIForemanVirtWhoConfigure
   def self.version
-    @version ||= Gem::Version.new '0.1.0'
+    @version ||= Gem::Version.new '0.1.1'
   end
 end

data/lib/hammer_cli_foreman_virt_who_configure.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 require 'hammer_cli'
 require 'hammer_cli_foreman'
 require 'hammer_cli_foreman_virt_who_configure/i18n'
 
 module HammerCLIForemanVirtWhoConfigure
-  HammerCLI::MainCommand.lazy_subcommand('virt-who-config', _("Manage Virt Who configurations"),
-    'HammerCLIForemanVirtWhoConfigure::VirtWhoConfig', 'hammer_cli_foreman_virt_who_configure/config'
-  )
+  HammerCLI::MainCommand.lazy_subcommand('virt-who-config', _('Manage Virt Who configurations'),
+  'HammerCLIForemanVirtWhoConfigure::VirtWhoConfig', 'hammer_cli_foreman_virt_who_configure/config'
+                                        )
 end