kpm 0.7.2 → 0.10.0

data/lib/kpm/system_helpers/os_information.rb

@@ -1,66 +1,64 @@
+# frozen_string_literal: true
+
 module KPM
   module SystemProxy
-    module OsInformation
-      class << self
+    class OsInformation
+      attr_reader :os_info, :labels
 
-        def fetch
-          os_information  = nil
-          if OS.windows?
-            os_information  = fetch_windows
-          elsif OS.linux?
-            os_information  = fetch_linux
-          elsif OS.mac?
-            os_information  = fetch_mac
-          end
+      def initialize
+        @os_info = fetch
+        @labels =  [{ label: :os_detail },
+                    { label: :value }]
+      end
 
-          os_information
-        end
+      private
 
-        def get_labels
-          labels = [{:label => :os_detail},
-                    {:label => :value}]
-          labels
+      def fetch
+        os_information = nil
+        if OS.windows?
+          os_information = fetch_windows
+        elsif OS.linux?
+          os_information = fetch_linux
+        elsif OS.mac?
+          os_information = fetch_mac
         end
 
-        private
-          def fetch_linux
-            os_data = `lsb_release -a 2>&1`
+        os_information
+      end
 
-            if os_data.nil? || os_data.include?('lsb_release: not found')
-              os_data = `cat /etc/issue 2>&1`
-              os_data = 'Description:'+os_data.gsub('\n \l','')
-            end
+      def fetch_linux
+        os_data = `lsb_release -a 2>&1`
 
-            os = get_hash(os_data)
-            os
-          end
+        if os_data.nil? || os_data.include?('lsb_release: not found')
+          os_data = `cat /etc/issue 2>&1`
+          os_data = 'Description:' + os_data.gsub('\n \l', '')
+        end
 
-          def fetch_mac
-            os_data = `sw_vers`
-            os = get_hash(os_data)
-            os
-          end
+        build_hash(os_data)
+      end
 
-          def fetch_windows
-            os_data = `systeminfo | findstr /C:"OS"`
-            os = get_hash(os_data)
-            os
-          end
+      def fetch_mac
+        os_data = `sw_vers`
+        build_hash(os_data)
+      end
 
-          def get_hash(data)
-            os = Hash.new
+      def fetch_windows
+        os_data = `systeminfo | findstr /C:"OS"`
+        build_hash(os_data)
+      end
 
-            unless data.nil?
-              data.split("\n").each do |info|
-                infos = info.split(':')
-                os[infos[0].to_s.strip] = {:os_detail => infos[0].to_s.strip, :value => infos[1].to_s.strip}
-              end
-            end
+      def build_hash(data)
+        os = {}
 
-            os
+        unless data.nil?
+          data.split("\n").each do |info|
+            infos = info.split(':')
+            os[infos[0].to_s.strip] = { os_detail: infos[0].to_s.strip, value: infos[1].to_s.strip }
           end
+        end
 
+        os
       end
     end
   end
-end
+end

data/lib/kpm/system_helpers/system_proxy.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'cpu_information'
 require_relative 'memory_information'
 require_relative 'disk_space_information'
@@ -5,24 +7,22 @@ require_relative 'entropy_available'
 require_relative 'os_information'
 module KPM
   module SystemProxy
-
     module OS
-      def OS.windows?
-        (/cygwin|mswin|mingw|bccwin|wince|emx/ =~ RbConfig::CONFIG["host_os"]) != nil
+      def self.windows?
+        (/cygwin|mswin|mingw|bccwin|wince|emx/ =~ RbConfig::CONFIG['host_os']) != nil
       end
 
-      def OS.mac?
-        (/darwin/ =~ RbConfig::CONFIG["host_os"]) != nil
+      def self.mac?
+        (/darwin/ =~ RbConfig::CONFIG['host_os']) != nil
       end
 
-      def OS.unix?
+      def self.unix?
         !OS.windows?
       end
 
-      def OS.linux?
-        OS.unix? and not OS.mac?
+      def self.linux?
+        OS.unix? && !OS.mac?
       end
     end
-
   end
-end
+end

data/lib/kpm/tasks.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'highline'
 require 'logger'
 require 'thor'
@@ -10,81 +12,112 @@ module KPM
     def self.included(base)
       base.send :include, ::Thor::Actions
       base.class_eval do
-
-
-        desc 'KPM version', 'Return current KPM version.'
+        desc 'version', 'Return current KPM version.'
         def version
           say "KPM version #{KPM::VERSION}"
         end
 
         class_option :overrides,
-                     :type    => :hash,
-                     :default => nil,
-                     :desc    => "A hashed list of overrides. Available options are 'url', 'repository', 'username', and 'password'."
+                     type: :hash,
+                     default: nil,
+                     desc: "A hashed list of overrides. Available options are 'url', 'repository', 'username', and 'password'."
 
         class_option :ssl_verify,
-                     :type    => :boolean,
-                     :default => true,
-                     :desc    => 'Set to false to disable SSL Verification.'
+                     type: :boolean,
+                     default: true,
+                     desc: 'Set to false to disable SSL Verification.'
 
         method_option :force_download,
-                      :type    => :boolean,
-                      :default => false,
-                      :desc    => 'Force download of the artifact even if it exists'
+                      type: :boolean,
+                      default: false,
+                      desc: 'Force download of the artifact even if it exists'
         method_option :verify_sha1,
-                      :type    => :boolean,
-                      :default => true,
-                      :desc    => 'Validate sha1 sum'
+                      type: :boolean,
+                      default: true,
+                      desc: 'Validate sha1 sum'
         desc 'install config_file', 'Install Kill Bill server and plugins according to the specified YAML configuration file.'
-        def install(config_file=nil)
+        def install(config_file = nil)
           help = Installer.from_file(config_file).install(options[:force_download], options[:verify_sha1])
           help = JSON(help)
           say help['help'], :green unless help['help'].nil?
         end
 
         method_option :destination,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'A different folder other than the default bundles directory.'
+                      type: :string,
+                      default: nil,
+                      desc: 'A different folder other than the default bundles directory.'
         method_option :force,
-                      :type    => :boolean,
-                      :default => nil,
-                      :desc    => 'Don\'t ask for confirmation while deleting multiple versions of a plugin.'
+                      type: :boolean,
+                      default: nil,
+                      desc: 'Don\'t ask for confirmation while deleting multiple versions of a plugin.'
+        method_option :version,
+                      type: :string,
+                      default: nil,
+                      desc: 'Specific plugin version to uninstall'
         desc 'uninstall plugin', 'Uninstall the specified plugin, identified by its name or key, from current deployment'
         def uninstall(plugin)
-          say 'Done!' if Uninstaller.new(options[:destination]).uninstall_plugin(plugin, options[:force])
+          Uninstaller.new(options[:destination]).uninstall_plugin(plugin, options[:force], options[:version])
         end
 
         method_option :destination,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'A different folder other than the current working directory.'
+                      type: :string,
+                      default: nil,
+                      desc: 'A different folder other than the default bundles directory.'
+        method_option :dry_run,
+                      type: :boolean,
+                      default: false,
+                      desc: 'Print the plugins which would be deleted'
+        desc 'cleanup', 'Delete old plugins'
+        def cleanup
+          Uninstaller.new(options[:destination]).uninstall_non_default_plugins(options[:dry_run])
+        end
+
+        method_option :group_id,
+                      type: :string,
+                      default: KillbillServerArtifact::KILLBILL_GROUP_ID,
+                      desc: 'The Kill Bill war artifact group-id'
+        method_option :artifact_id,
+                      type: :string,
+                      default: KillbillServerArtifact::KILLBILL_ARTIFACT_ID,
+                      desc: 'The Kill Bill war artifact id'
+        method_option :packaging,
+                      type: :string,
+                      default: KillbillServerArtifact::KILLBILL_PACKAGING,
+                      desc: 'The Kill Bill war packaging'
+        method_option :classifier,
+                      type: :string,
+                      default: KillbillServerArtifact::KILLBILL_CLASSIFIER,
+                      desc: 'The Kill Bill war classifier'
+        method_option :destination,
+                      type: :string,
+                      default: nil,
+                      desc: 'A different folder other than the current working directory.'
         method_option :bundles_dir,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'The location where bundles will be installed (along with sha1 file)'
+                      type: :string,
+                      default: nil,
+                      desc: 'The location where bundles will be installed (along with sha1 file)'
         method_option :force_download,
-                      :type    => :boolean,
-                      :default => false,
-                      :desc    => 'Force download of the artifact even if it exists'
+                      type: :boolean,
+                      default: false,
+                      desc: 'Force download of the artifact even if it exists'
         method_option :verify_sha1,
-                      :type    => :boolean,
-                      :default => true,
-                      :desc    => 'Validate sha1 sum'
-        desc 'pull_kb_server_war <version>', 'Pulls Kill Bill server war from Sonatype and places it on your machine. If version was not specified it uses the latest released version.'
-        def pull_kb_server_war(version='LATEST')
+                      type: :boolean,
+                      default: true,
+                      desc: 'Validate sha1 sum'
+        desc 'pull_kb_server_war <version>', 'Pulls Kill Bill server war and places it on your machine. If version was not specified it uses the latest released version.'
+        def pull_kb_server_war(version = 'LATEST')
           installer = BaseInstaller.new(logger,
                                         options[:overrides],
                                         options[:ssl_verify])
-          response = installer.install_killbill_server(KillbillServerArtifact::KILLBILL_GROUP_ID,
-                                            KillbillServerArtifact::KILLBILL_ARTIFACT_ID,
-                                            KillbillServerArtifact::KILLBILL_PACKAGING,
-                                            KillbillServerArtifact::KILLBILL_CLASSIFIER,
-                                            version,
-                                            options[:destination],
-                                            options[:bundles_dir],
-                                            options[:force_download],
-                                            options[:verify_sha1])
+          response = installer.install_killbill_server(options[:group_id],
+                                                       options[:artifact_id],
+                                                       options[:packaging],
+                                                       options[:classifier],
+                                                       version,
+                                                       options[:destination],
+                                                       options[:bundles_dir],
+                                                       options[:force_download],
+                                                       options[:verify_sha1])
           say "Artifact has been retrieved and can be found at path: #{response[:file_path]}", :green
         end
 
@@ -97,98 +130,54 @@ module KPM
                                                                      options[:ssl_verify]).to_a.join(', ')}", :green
         end
 
-        method_option :destination,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'A different folder other than the current working directory.'
-        method_option :bundles_dir,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'The location where bundles will be installed (along with sha1 file)'
-        method_option :force_download,
-                      :type    => :boolean,
-                      :default => false,
-                      :desc    => 'Force download of the artifact even if it exists'
-        method_option :verify_sha1,
-                      :type    => :boolean,
-                      :default => true,
-                      :desc    => 'Validates sha1 sum'
-        desc 'pull_kp_server_war <version>', 'Pulls Kill Pay server war from Sonatype and places it on your machine. If version was not specified it uses the latest released version.'
-        def pull_kp_server_war(version='LATEST')
-          installer = BaseInstaller.new(logger,
-                                        options[:overrides],
-                                        options[:ssl_verify])
-          response = installer.install_killbill_server(KillbillServerArtifact::KILLBILL_GROUP_ID,
-                                            KillbillServerArtifact::KILLPAY_ARTIFACT_ID,
-                                            KillbillServerArtifact::KILLPAY_PACKAGING,
-                                            KillbillServerArtifact::KILLPAY_CLASSIFIER,
-                                            version,
-                                            options[:destination],
-                                            options[:bundles_dir],
-                                            options[:force_download],
-                                            options[:verify_sha1])
-          say "Artifact has been retrieved and can be found at path: #{response[:file_path]}", :green
-        end
-
-        desc 'search_for_kp_server', 'Searches for all versions of Kill Pay server and prints them to the screen.'
-        def search_for_kp_server
-          say "Available versions: #{KillbillServerArtifact.versions(KillbillServerArtifact::KILLPAY_ARTIFACT_ID,
-                                                                     KillbillServerArtifact::KILLPAY_PACKAGING,
-                                                                     KillbillServerArtifact::KILLPAY_CLASSIFIER,
-                                                                     options[:overrides],
-                                                                     options[:ssl_verify]).to_a.join(', ')}", :green
-        end
-
         method_option :group_id,
-                      :type    => :string,
-                      :default => KillbillPluginArtifact::KILLBILL_JAVA_PLUGIN_GROUP_ID,
-                      :desc    => 'The plugin artifact group-id'
+                      type: :string,
+                      default: KillbillPluginArtifact::KILLBILL_JAVA_PLUGIN_GROUP_ID,
+                      desc: 'The plugin artifact group-id'
         method_option :artifact_id,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'The plugin artifact id'
+                      type: :string,
+                      default: nil,
+                      desc: 'The plugin artifact id'
         method_option :version,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'The plugin artifact version'
+                      type: :string,
+                      default: nil,
+                      desc: 'The plugin artifact version'
         method_option :packaging,
-                      :type    => :string,
-                      :default => KillbillPluginArtifact::KILLBILL_JAVA_PLUGIN_PACKAGING,
-                      :desc    => 'The plugin artifact packaging'
+                      type: :string,
+                      default: KillbillPluginArtifact::KILLBILL_JAVA_PLUGIN_PACKAGING,
+                      desc: 'The plugin artifact packaging'
         method_option :classifier,
-                      :type    => :string,
-                      :default => KillbillPluginArtifact::KILLBILL_JAVA_PLUGIN_CLASSIFIER,
-                      :desc    => 'The plugin artifact classifier'
+                      type: :string,
+                      default: KillbillPluginArtifact::KILLBILL_JAVA_PLUGIN_CLASSIFIER,
+                      desc: 'The plugin artifact classifier'
         method_option :from_source_file,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Specify the plugin jar that should be used for the installation.'
+                      type: :string,
+                      default: nil,
+                      desc: 'Specify the plugin jar that should be used for the installation.'
         method_option :destination,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'A different folder other than the current working directory.'
+                      type: :string,
+                      default: nil,
+                      desc: 'A different folder other than the current working directory.'
         method_option :force_download,
-                      :type    => :boolean,
-                      :default => false,
-                      :desc    => 'Force download of the artifact even if it exists'
+                      type: :boolean,
+                      default: false,
+                      desc: 'Force download of the artifact even if it exists'
         method_option :sha1_file,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Location of the sha1 file'
+                      type: :string,
+                      default: nil,
+                      desc: 'Location of the sha1 file'
         method_option :verify_sha1,
-                      :type    => :boolean,
-                      :default => true,
-                      :desc    => 'Validates sha1 sum'
-        desc 'install_java_plugin plugin-key <kb-version>', 'Pulls a java plugin from Sonatype and installs it under the specified destination. If the kb-version has been specified, it is used to download the matching plugin artifact version; if not, it uses the specified plugin version or if null, the LATEST one.'
-        def install_java_plugin(plugin_key, kb_version='LATEST')
-
-
+                      type: :boolean,
+                      default: true,
+                      desc: 'Validates sha1 sum'
+        desc 'install_java_plugin plugin-key <kb-version>', 'Pulls a java plugin and installs it under the specified destination. If the kb-version has been specified, it is used to download the matching plugin artifact version; if not, it uses the specified plugin version or if null, the LATEST one.'
+        def install_java_plugin(plugin_key, kb_version = 'LATEST')
           installer = BaseInstaller.new(logger,
                                         options[:overrides],
                                         options[:ssl_verify])
 
-          if options[:from_source_file].nil?
-            response = installer.install_plugin(plugin_key,
+          response = if options[:from_source_file].nil?
+                       installer.install_plugin(plugin_key,
                                                 kb_version,
                                                 options[:group_id],
                                                 options[:artifact_id],
@@ -200,64 +189,61 @@ module KPM
                                                 options[:force_download],
                                                 options[:verify_sha1],
                                                 false)
-          else
-            response = installer.install_plugin_from_fs(plugin_key, options[:from_source_file], nil, options[:version], options[:destination], 'java')
-          end
+                     else
+                       installer.install_plugin_from_fs(plugin_key, options[:from_source_file], nil, options[:version], options[:destination], 'java')
+                     end
 
           say "Artifact has been retrieved and can be found at path: #{response[:file_path]}", :green
         end
 
-
-
-
         method_option :group_id,
-                      :type    => :string,
-                      :default => KillbillPluginArtifact::KILLBILL_RUBY_PLUGIN_GROUP_ID,
-                      :desc    => 'The plugin artifact group-id'
+                      type: :string,
+                      default: KillbillPluginArtifact::KILLBILL_RUBY_PLUGIN_GROUP_ID,
+                      desc: 'The plugin artifact group-id'
         method_option :artifact_id,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'The plugin artifact id'
+                      type: :string,
+                      default: nil,
+                      desc: 'The plugin artifact id'
         method_option :version,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'The plugin artifact version'
+                      type: :string,
+                      default: nil,
+                      desc: 'The plugin artifact version'
         method_option :packaging,
-                      :type    => :string,
-                      :default => KillbillPluginArtifact::KILLBILL_RUBY_PLUGIN_PACKAGING,
-                      :desc    => 'The plugin artifact packaging'
+                      type: :string,
+                      default: KillbillPluginArtifact::KILLBILL_RUBY_PLUGIN_PACKAGING,
+                      desc: 'The plugin artifact packaging'
         method_option :classifier,
-                      :type    => :string,
-                      :default => KillbillPluginArtifact::KILLBILL_RUBY_PLUGIN_CLASSIFIER,
-                      :desc    => 'The plugin artifact classifier'
+                      type: :string,
+                      default: KillbillPluginArtifact::KILLBILL_RUBY_PLUGIN_CLASSIFIER,
+                      desc: 'The plugin artifact classifier'
         method_option :from_source_file,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Specify the ruby plugin archive that should be used for the installation.'
+                      type: :string,
+                      default: nil,
+                      desc: 'Specify the ruby plugin archive that should be used for the installation.'
         method_option :destination,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'A different folder other than the current working directory.'
+                      type: :string,
+                      default: nil,
+                      desc: 'A different folder other than the current working directory.'
         method_option :force_download,
-                      :type    => :boolean,
-                      :default => false,
-                      :desc    => 'Force download of the artifact even if it exists'
+                      type: :boolean,
+                      default: false,
+                      desc: 'Force download of the artifact even if it exists'
         method_option :sha1_file,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Location of the sha1 file'
+                      type: :string,
+                      default: nil,
+                      desc: 'Location of the sha1 file'
         method_option :verify_sha1,
-                      :type    => :boolean,
-                      :default => true,
-                      :desc    => 'Validates sha1 sum'
-        desc 'install_ruby_plugin plugin-key <kb-version>', 'Pulls a ruby plugin from Sonatype and installs it under the specified destination. If the kb-version has been specified, it is used to download the matching plugin artifact version; if not, it uses the specified plugin version or if null, the LATEST one.'
-        def install_ruby_plugin(plugin_key, kb_version='LATEST')
+                      type: :boolean,
+                      default: true,
+                      desc: 'Validates sha1 sum'
+        desc 'install_ruby_plugin plugin-key <kb-version>', 'Pulls a ruby plugin and installs it under the specified destination. If the kb-version has been specified, it is used to download the matching plugin artifact version; if not, it uses the specified plugin version or if null, the LATEST one.'
+        def install_ruby_plugin(plugin_key, kb_version = 'LATEST')
           installer = BaseInstaller.new(logger,
-                            options[:overrides],
-                            options[:ssl_verify])
+                                        options[:overrides],
+                                        options[:ssl_verify])
 
-          if options[:from_source_file].nil?
-            response = installer.install_plugin(plugin_key,
+          response = if options[:from_source_file].nil?
+                       installer.install_plugin(plugin_key,
                                                 kb_version,
                                                 options[:group_id],
                                                 options[:artifact_id],
@@ -269,28 +255,27 @@ module KPM
                                                 options[:force_download],
                                                 options[:verify_sha1],
                                                 true)
-          else
-            response = installer.install_plugin_from_fs(plugin_key, options[:from_source_file], nil, nil, options[:destination], 'ruby')
-          end
+                     else
+                       installer.install_plugin_from_fs(plugin_key, options[:from_source_file], nil, nil, options[:destination], 'ruby')
+                     end
 
           say "Artifact has been retrieved and can be found at path: #{response[:file_path]}", :green
-
         end
 
         method_option :destination,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'A different folder other than the default bundles directory.'
+                      type: :string,
+                      default: nil,
+                      desc: 'A different folder other than the default bundles directory.'
         method_option :force_download,
-                      :type    => :boolean,
-                      :default => false,
-                      :desc    => 'Force download of the artifact even if it exists'
+                      type: :boolean,
+                      default: false,
+                      desc: 'Force download of the artifact even if it exists'
         method_option :verify_sha1,
-                      :type    => :boolean,
-                      :default => true,
-                      :desc    => 'Validates sha1 sum'
-        desc 'pull_defaultbundles <kb-version>', 'Pulls the default OSGI bundles from Sonatype and places it on your machine. If the kb-version has been specified, it is used to download the matching platform artifact; if not, it uses the latest released version.'
-        def pull_defaultbundles(kb_version='LATEST')
+                      type: :boolean,
+                      default: true,
+                      desc: 'Validates sha1 sum'
+        desc 'pull_defaultbundles <kb-version>', 'Pulls the default OSGI bundles and places it on your machine. If the kb-version has been specified, it is used to download the matching platform artifact; if not, it uses the latest released version.'
+        def pull_defaultbundles(kb_version = 'LATEST')
           response = BaseInstaller.new(logger,
                                        options[:overrides],
                                        options[:ssl_verify])
@@ -318,23 +303,23 @@ module KPM
         end
 
         method_option :destination,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'A different folder other than the current working directory.'
+                      type: :string,
+                      default: nil,
+                      desc: 'A different folder other than the current working directory.'
         method_option :force_download,
-                      :type    => :boolean,
-                      :default => false,
-                      :desc    => 'Force download of the artifact even if it exists'
+                      type: :boolean,
+                      default: false,
+                      desc: 'Force download of the artifact even if it exists'
         method_option :sha1_file,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Location of the sha1 file'
+                      type: :string,
+                      default: nil,
+                      desc: 'Location of the sha1 file'
         method_option :verify_sha1,
-                      :type    => :boolean,
-                      :default => true,
-                      :desc    => 'Validates sha1 sum'
-        desc 'pull_kaui_war <version>', 'Pulls Kaui war from Sonatype and places it on your machine. If version was not specified it uses the latest released version.'
-        def pull_kaui_war(version='LATEST')
+                      type: :boolean,
+                      default: true,
+                      desc: 'Validates sha1 sum'
+        desc 'pull_kaui_war <version>', 'Pulls Kaui war and places it on your machine. If version was not specified it uses the latest released version.'
+        def pull_kaui_war(version = 'LATEST')
           response = KauiArtifact.pull(logger,
                                        KauiArtifact::KAUI_GROUP_ID,
                                        KauiArtifact::KAUI_ARTIFACT_ID,
@@ -356,39 +341,54 @@ module KPM
         end
 
         method_option :version,
-                      :type => :string,
-                      :default => 'LATEST',
-                      :desc => 'Kill Bill version'
+                      type: :string,
+                      default: 'LATEST',
+                      desc: 'Kill Bill version'
+        method_option :force_download,
+                      type: :boolean,
+                      default: false,
+                      desc: 'Ignore local cache'
+        method_option :sha1_file,
+                      type: :string,
+                      default: nil,
+                      desc: 'Location of the sha1 file'
+        method_option :verify_sha1,
+                      type: :boolean,
+                      default: true,
+                      desc: 'Validates sha1 sum'
+        method_option :as_json,
+                      type: :boolean,
+                      default: false,
+                      desc: 'Set the output format as JSON when true'
         desc 'info', 'Describe information about a Kill Bill version'
         def info
-
-          say "Fetching info for version #{options[:version]}...\n"
-
           versions_info = KillbillServerArtifact.info(options[:version],
                                                       options[:sha1_file],
                                                       options[:force_download],
                                                       options[:verify_sha1],
                                                       options[:overrides],
                                                       options[:ssl_verify])
-          say "Dependencies for version #{options[:version]}\n  " + (versions_info.map {|k,v| "#{k} #{v}"}).join("\n  "), :green
-          say "\n\n"
-
           resolved_kb_version = versions_info['killbill']
-          kb_version = resolved_kb_version.split('.').slice(0,2).join(".")
+          kb_version = resolved_kb_version.split('.').slice(0, 2).join('.')
 
           plugins_info = KPM::PluginsDirectory.list_plugins(true, kb_version)
 
-          say "Known plugin for KB version #{options[:version]}\n  " + (plugins_info.map {|k,v| "#{k} #{v}"}).join("\n  "), :green
+          if options[:as_json]
+            puts({ 'killbill' => versions_info, 'plugins' => plugins_info }.to_json)
+          else
+            say "Dependencies for version #{options[:version]}\n  " + (versions_info.map { |k, v| "#{k} #{v}" }).join("\n  "), :green
+            say "Known plugins for KB version #{options[:version]}\n  " + (plugins_info.map { |k, v| "#{k} #{v}" }).join("\n  "), :green
+          end
         end
 
         method_option :destination,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Folder where to download migration files.'
+                      type: :string,
+                      default: nil,
+                      desc: 'Folder where to download migration files.'
         method_option :token,
-                      :type => :string,
-                      :default => nil,
-                      :desc => 'GitHub OAuth token.'
+                      type: :string,
+                      default: nil,
+                      desc: 'GitHub OAuth token.'
         desc 'migrations repository from to', 'Download migration files for Kill Bill or a plugin'
         def migrations(repository, from, to = nil)
           full_repo = repository.include?('/') ? repository : "killbill/#{repository}"
@@ -397,315 +397,258 @@ module KPM
         end
 
         method_option :destination,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'A different folder other than the default bundles directory.'
+                      type: :string,
+                      default: nil,
+                      desc: 'A different folder other than the default bundles directory.'
+        method_option :as_json,
+                      type: :boolean,
+                      default: false,
+                      desc: 'Set the output format as JSON when true'
         desc 'inspect', 'Inspect current deployment'
         def inspect
           inspector = KPM::Inspector.new
-          puts options[:destination]
           all_plugins = inspector.inspect(options[:destination])
-          #puts all_plugins.to_json
-          inspector.format(all_plugins)
+          options[:as_json] ? puts(all_plugins.to_json) : inspector.format(all_plugins)
         end
 
         method_option :bundles_dir,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'A different folder other than the default bundles directory.'
+                      type: :string,
+                      default: nil,
+                      desc: 'A different folder other than the default bundles directory.'
         method_option :config_file,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'KPM configuration file (yml file)'
+                      type: :string,
+                      default: nil,
+                      desc: 'KPM configuration file (yml file)'
         method_option :as_json,
-                      :type    => :boolean,
-                      :default => false,
-                      :desc    => 'Set the output format as JSON when true'
+                      type: :boolean,
+                      default: false,
+                      desc: 'Set the output format as JSON when true'
         method_option :kaui_web_path,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Path for the KAUI web app'
+                      type: :string,
+                      default: nil,
+                      desc: 'Path for the KAUI web app'
         method_option :killbill_web_path,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Path for the killbill web app'
+                      type: :string,
+                      default: nil,
+                      desc: 'Path for the killbill web app'
         desc 'system', 'Gather information about the system'
         def system
-          system = KPM::System.new
+          system = KPM::System.new(logger)
           information = system.information(options[:bundles_dir], options[:as_json], options[:config_file], options[:kaui_web_path],
-                             options[:killbill_web_path])
-
-          if options[:as_json]
-            puts information
-          end
+                                           options[:killbill_web_path])
 
+          puts information if options[:as_json]
         end
 
         method_option :export,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'export account for a provided id.'
+                      type: :string,
+                      default: nil,
+                      desc: 'export account for a provided id.'
         method_option :import,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'import account for a previously exported file.'
+                      type: :string,
+                      default: nil,
+                      desc: 'import account for a previously exported file.'
         method_option :tenant_record_id,
-                      :type    => :numeric,
-                      :default => nil,
-                      :desc    => 'replace the tenant_record_id before importing data.'
+                      type: :numeric,
+                      default: nil,
+                      desc: 'replace the tenant_record_id before importing data.'
         method_option :generate_record_id,
-                      :type    => :boolean,
-                      :default => false,
-                      :desc    => 'The generate_record_id will instruct to generate the tables record_ids that were exported'
+                      type: :boolean,
+                      default: false,
+                      desc: 'The generate_record_id will instruct to generate the tables record_ids that were exported'
         method_option :skip_payment_methods,
-                      :type    => :boolean,
-                      :default => false,
-                      :desc    => 'Skip or swap payment types other than __EXTERNAL_PAYMENT__.'
+                      type: :boolean,
+                      default: false,
+                      desc: 'Skip or swap payment types other than __EXTERNAL_PAYMENT__.'
         method_option :config_file,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Yml that contains killbill api connection and DB connection'
+                      type: :string,
+                      default: nil,
+                      desc: 'Yml that contains killbill api connection and DB connection'
         method_option :killbill_api_credentials,
-                      :type    => :array,
-                      :default => nil,
-                      :desc    => 'Killbill api credentials <api_key> <api_secrets>'
+                      type: :array,
+                      default: nil,
+                      desc: 'Killbill api credentials <api_key> <api_secrets>'
         method_option :killbill_credentials,
-                      :type    => :array,
-                      :default => nil,
-                      :desc    => 'Killbill credentials <user> <password>'
+                      type: :array,
+                      default: nil,
+                      desc: 'Killbill credentials <user> <password>'
         method_option :killbill_url,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Killbill URL ex. http://127.0.0.1:8080'
+                      type: :string,
+                      default: nil,
+                      desc: 'Killbill URL ex. http://127.0.0.1:8080'
         method_option :database_name,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'DB name to connect'
+                      type: :string,
+                      default: nil,
+                      desc: 'DB name to connect'
         method_option :database_credentials,
-                      :type    => :array,
-                      :default => nil,
-                      :desc    => 'DB credentials <user> <password>'
+                      type: :array,
+                      default: nil,
+                      desc: 'DB credentials <user> <password>'
         method_option :data_delimiter,
-                      :type    => :string,
-                      :default => "|",
-                      :desc    => 'Data delimiter'
+                      type: :string,
+                      default: '|',
+                      desc: 'Data delimiter'
         method_option :database_host,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Database Host name'
+                      type: :string,
+                      default: nil,
+                      desc: 'Database Host name'
         method_option :database_port,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Database port'
+                      type: :string,
+                      default: nil,
+                      desc: 'Database port'
         desc 'account', 'export/import accounts'
         def account
-          begin
-            config_file = nil
-            if options[:killbill_url] && /https?:\/\/[\S]+/.match(options[:killbill_url]).nil?
-              raise Interrupt,'--killbill_url, required format -> http(s)://something'
-            end
+          config_file = nil
+          raise Interrupt, '--killbill_url, required format -> http(s)://something' if options[:killbill_url] && %r{https?://\S+}.match(options[:killbill_url]).nil?
 
-            if options[:killbill_api_credentials] && options[:killbill_api_credentials].size != 2
-              raise Interrupt,'--killbill_api_credentials, required format -> <api_key> <api_secrets>'
-            end
+          raise Interrupt, '--killbill_api_credentials, required format -> <api_key> <api_secrets>' if options[:killbill_api_credentials] && options[:killbill_api_credentials].size != 2
 
-            if options[:killbill_credentials] && options[:killbill_credentials].size != 2
-              raise Interrupt,'--killbill_credentials, required format -> <user> <password>'
-            end
-
-            if options[:database_credentials] && options[:database_credentials].size != 2
-              raise Interrupt,'--database_credentials, required format -> <user> <password>'
-            end
-
-            if options[:database_name] && options[:database_name] == :database_name.to_s
-              raise Interrupt,'--database_credentials, please provide a valid database name'
-            end
+          raise Interrupt, '--killbill_credentials, required format -> <user> <password>' if options[:killbill_credentials] && options[:killbill_credentials].size != 2
 
-            if options[:config_file] && options[:config_file] == :config_file.to_s
-              config_file = File.join(File.expand_path(File.dirname(__FILE__)), 'account_export_import.yml')
-            end
+          raise Interrupt, '--database_credentials, required format -> <user> <password>' if options[:database_credentials] && options[:database_credentials].size != 2
 
-            if options[:export].nil? && options[:import].nil?
-              raise Interrupt,'Need to specify an action'
-            end
+          raise Interrupt, '--database_credentials, please provide a valid database name' if options[:database_name] && options[:database_name] == :database_name.to_s
 
+          config_file = File.join(__dir__, 'account_export_import.yml') if options[:config_file] && options[:config_file] == :config_file.to_s
 
-            account = KPM::Account.new(config_file || options[:config_file],options[:killbill_api_credentials],options[:killbill_credentials],
-                                       options[:killbill_url],options[:database_name],options[:database_credentials],options[:database_host], options[:database_port],options[:data_delimiter], logger)
-            export_file = nil
-            round_trip_export_import = false
+          raise Interrupt, 'Need to specify an action' if options[:export].nil? && options[:import].nil?
 
-            if not options[:export].nil?
-              export_file = account.export_data(options[:export])
-              round_trip_export_import = true
-            end
+          account = KPM::Account.new(config_file || options[:config_file], options[:killbill_api_credentials], options[:killbill_credentials],
+                                     options[:killbill_url], options[:database_name], options[:database_credentials], options[:database_host], options[:database_port], options[:data_delimiter], logger)
+          export_file = nil
+          round_trip_export_import = false
 
-            if not options[:import].nil?
-              account.import_data(export_file || options[:import],options[:tenant_record_id], options[:skip_payment_methods],
-                                  round_trip_export_import, options[:generate_record_id])
-            end
+          unless options[:export].nil?
+            export_file = account.export_data(options[:export])
+            round_trip_export_import = true
+          end
 
-          rescue Exception => e
-            logger.error "\e[91;1m#{e.message}\e[0m"
-            if not e.is_a?(Interrupt)
-              logger.error e.backtrace.join("\n")
-            end
+          unless options[:import].nil?
+            account.import_data(export_file || options[:import], options[:tenant_record_id], options[:skip_payment_methods],
+                                round_trip_export_import, options[:generate_record_id])
           end
+        rescue StandardError => e
+          logger.error "\e[91;1m#{e.message}\e[0m"
+          logger.error e.backtrace.join("\n") unless e.is_a?(Interrupt)
         end
 
         method_option :key_prefix,
-                      :type    => :string,
-                      :default => nil,
-                      :enum => KPM::TenantConfig::KEY_PREFIXES,
-                      :desc    => 'Retrieve a per tenant key value based on key prefix'
+                      type: :string,
+                      default: nil,
+                      enum: KPM::TenantConfig::KEY_PREFIXES,
+                      desc: 'Retrieve a per tenant key value based on key prefix'
         method_option :killbill_api_credentials,
-                      :type    => :array,
-                      :default => nil,
-                      :desc    => 'Killbill api credentials <api_key> <api_secrets>'
+                      type: :array,
+                      default: nil,
+                      desc: 'Killbill api credentials <api_key> <api_secrets>'
         method_option :killbill_credentials,
-                      :type    => :array,
-                      :default => nil,
-                      :desc    => 'Killbill credentials <user> <password>'
+                      type: :array,
+                      default: nil,
+                      desc: 'Killbill credentials <user> <password>'
         method_option :killbill_url,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Killbill URL ex. http://127.0.0.1:8080'
+                      type: :string,
+                      default: nil,
+                      desc: 'Killbill URL ex. http://127.0.0.1:8080'
         desc 'tenant_config', 'export all tenant-level configs.'
         def tenant_config
-          begin
-
-            if options[:killbill_url] && /https?:\/\/[\S]+/.match(options[:killbill_url]).nil?
-              raise Interrupt,'--killbill_url, required format -> http(s)://something'
-            end
+          raise Interrupt, '--killbill_url, required format -> http(s)://something' if options[:killbill_url] && %r{https?://\S+}.match(options[:killbill_url]).nil?
 
-            if options[:killbill_api_credentials] && options[:killbill_api_credentials].size != 2
-              raise Interrupt,'--killbill_api_credentials, required format -> <api_key> <api_secrets>'
-            end
+          raise Interrupt, '--killbill_api_credentials, required format -> <api_key> <api_secrets>' if options[:killbill_api_credentials] && options[:killbill_api_credentials].size != 2
 
-            if options[:killbill_credentials] && options[:killbill_credentials].size != 2
-              raise Interrupt,'--killbill_credentials, required format -> <user> <password>'
-            end
+          raise Interrupt, '--killbill_credentials, required format -> <user> <password>' if options[:killbill_credentials] && options[:killbill_credentials].size != 2
 
-            if options[:key_prefix] === :key_prefix.to_s
-              raise Interrupt, "--key_prefix, posible values #{KPM::TenantConfig::KEY_PREFIXES.join(', ')}"
-            end
+          raise Interrupt, "--key_prefix, posible values #{KPM::TenantConfig::KEY_PREFIXES.join(', ')}" if options[:key_prefix] == :key_prefix.to_s
 
-            tenantConfig = KPM::TenantConfig.new(options[:killbill_api_credentials],options[:killbill_credentials],
-                                       options[:killbill_url], logger)
+          tenant_config = KPM::TenantConfig.new(options[:killbill_api_credentials], options[:killbill_credentials],
+                                                options[:killbill_url], logger)
 
-            tenantConfig.export(options[:key_prefix])
-
-          rescue Exception => e
-            logger.error "\e[91;1m#{e.message}\e[0m"
-            if not e.is_a?(Interrupt)
-              logger.error e.backtrace.join("\n")
-            end
-          end
+          tenant_config.export(options[:key_prefix])
+        rescue StandardError => e
+          logger.error "\e[91;1m#{e.message}\e[0m"
+          logger.error e.backtrace.join("\n") unless e.is_a?(Interrupt)
         end
 
-
         method_option :account_export,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'export account for a provided id.'
+                      type: :string,
+                      default: nil,
+                      desc: 'export account for a provided id.'
         method_option :log_dir,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => '(Optional) Log directory if the default tomcat location has changed'
+                      type: :string,
+                      default: nil,
+                      desc: '(Optional) Log directory if the default tomcat location has changed'
         method_option :config_file,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Yml that contains killbill api connection and DB connection'
+                      type: :string,
+                      default: nil,
+                      desc: 'Yml that contains killbill api connection and DB connection'
         method_option :killbill_api_credentials,
-                      :type    => :array,
-                      :default => nil,
-                      :desc    => 'Killbill api credentials <api_key> <api_secrets>'
+                      type: :array,
+                      default: nil,
+                      desc: 'Killbill api credentials <api_key> <api_secrets>'
         method_option :killbill_credentials,
-                      :type    => :array,
-                      :default => nil,
-                      :desc    => 'Killbill credentials <user> <password>'
+                      type: :array,
+                      default: nil,
+                      desc: 'Killbill credentials <user> <password>'
         method_option :killbill_url,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Killbill URL ex. http://127.0.0.1:8080'
+                      type: :string,
+                      default: nil,
+                      desc: 'Killbill URL ex. http://127.0.0.1:8080'
         method_option :database_name,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'DB name to connect'
+                      type: :string,
+                      default: nil,
+                      desc: 'DB name to connect'
         method_option :database_credentials,
-                      :type    => :array,
-                      :default => nil,
-                      :desc    => 'DB credentials <user> <password>'
+                      type: :array,
+                      default: nil,
+                      desc: 'DB credentials <user> <password>'
         method_option :database_host,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Database Host name'
+                      type: :string,
+                      default: nil,
+                      desc: 'Database Host name'
         method_option :database_port,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Database port'
+                      type: :string,
+                      default: nil,
+                      desc: 'Database port'
         method_option :kaui_web_path,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Path for the KAUI web app'
+                      type: :string,
+                      default: nil,
+                      desc: 'Path for the KAUI web app'
         method_option :killbill_web_path,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'Path for the killbill web app'
+                      type: :string,
+                      default: nil,
+                      desc: 'Path for the killbill web app'
         method_option :bundles_dir,
-                      :type    => :string,
-                      :default => nil,
-                      :desc    => 'A different folder other than the default bundles directory.'
+                      type: :string,
+                      default: nil,
+                      desc: 'A different folder other than the default bundles directory.'
         desc 'diagnostic', 'exports and \'zips\' the account data, system, logs and tenant configurations'
         def diagnostic
-          begin
-            if options[:account_export] && options[:account_export] == 'account_export'
-              raise Interrupt,'--account_export,  please provide a valid account id'
-            end
+          raise Interrupt, '--account_export,  please provide a valid account id' if options[:account_export] && options[:account_export] == 'account_export'
 
-            if options[:killbill_url] && /https?:\/\/[\S]+/.match(options[:killbill_url]).nil?
-              raise Interrupt,'--killbill_url, required format -> http(s)://something'
-            end
+          raise Interrupt, '--killbill_url, required format -> http(s)://something' if options[:killbill_url] && %r{https?://\S+}.match(options[:killbill_url]).nil?
 
-            if options[:killbill_api_credentials] && options[:killbill_api_credentials].size != 2
-              raise Interrupt,'--killbill_api_credentials, required format -> <api_key> <api_secrets>'
-            end
+          raise Interrupt, '--killbill_api_credentials, required format -> <api_key> <api_secrets>' if options[:killbill_api_credentials] && options[:killbill_api_credentials].size != 2
 
-            if options[:killbill_credentials] && options[:killbill_credentials].size != 2
-              raise Interrupt,'--killbill_credentials, required format -> <user> <password>'
-            end
-
-            if options[:database_credentials] && options[:database_credentials].size != 2
-              raise Interrupt,'--database_credentials, required format -> <user> <password>'
-            end
+          raise Interrupt, '--killbill_credentials, required format -> <user> <password>' if options[:killbill_credentials] && options[:killbill_credentials].size != 2
 
-            if options[:database_name] && options[:database_name] == :database_name.to_s
-              raise Interrupt,'--database_credentials, please provide a valid database name'
-            end
+          raise Interrupt, '--database_credentials, required format -> <user> <password>' if options[:database_credentials] && options[:database_credentials].size != 2
 
-            if options[:kaui_web_path] && options[:kaui_web_path] == :kaui_web_path.to_s
-              raise Interrupt,'--kaui_web_path, please provide a valid kaui web path '
-            end
+          raise Interrupt, '--database_credentials, please provide a valid database name' if options[:database_name] && options[:database_name] == :database_name.to_s
 
-            if options[:killbill_web_path] && options[:killbill_web_path] == :killbill_web_path.to_s
-              raise Interrupt,'--killbill_web_path, please provide a valid killbill web path'
-            end
+          raise Interrupt, '--kaui_web_path, please provide a valid kaui web path ' if options[:kaui_web_path] && options[:kaui_web_path] == :kaui_web_path.to_s
 
-            diagnostic = KPM::DiagnosticFile.new(options[:config_file],options[:killbill_api_credentials],options[:killbill_credentials],
-                                       options[:killbill_url],options[:database_name],options[:database_credentials],
-                                      options[:database_host], options[:database_port], options[:kaui_web_path], options[:killbill_web_path], options[:bundles_dir],logger)
-            diagnostic.export_data(options[:account_export],  options[:log_dir])
+          raise Interrupt, '--killbill_web_path, please provide a valid killbill web path' if options[:killbill_web_path] && options[:killbill_web_path] == :killbill_web_path.to_s
 
-          rescue Exception => e
-            logger.error "\e[91;1m#{e.message}\e[0m"
-            if not e.is_a?(Interrupt)
-              logger.error e.backtrace.join("\n")
-            end
-          end
+          diagnostic = KPM::DiagnosticFile.new(options[:config_file], options[:killbill_api_credentials], options[:killbill_credentials],
+                                               options[:killbill_url], options[:database_name], options[:database_credentials],
+                                               options[:database_host], options[:database_port], options[:kaui_web_path], options[:killbill_web_path], options[:bundles_dir], logger)
+          diagnostic.export_data(options[:account_export], options[:log_dir])
+        rescue StandardError => e
+          logger.error "\e[91;1m#{e.message}\e[0m"
+          logger.error e.backtrace.join("\n") unless e.is_a?(Interrupt)
         end
 
-        map :pull_ruby_plugin => :install_ruby_plugin,
-            :pull_java_plugin => :install_java_plugin
+        map pull_ruby_plugin: :install_ruby_plugin,
+            pull_java_plugin: :install_java_plugin
 
         private