kanrisuru 0.15.0 → 0.16.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e77a2eee0eb08495f947f9dc3b130a7c47cd713ab2f970dcc8fd17b7b52cd8b3
-  data.tar.gz: 76d3ec684fe307aafce2f3be2c594f47a38c50f2b0bb394998c1c83c06d8a47b
+  metadata.gz: af1a581f630e30f0d4e45864f8adba335b5a4c39ba77ca81c3de97e62d1b63bc
+  data.tar.gz: b34344cd94480bac72c5c78db9fe2b331884366a68c30d81b001c4f6d2d648c8
 SHA512:
-  metadata.gz: cdd8b0d400e2557525fca889bbd3ef8b8eef67774f0143ea925fa9c5ac091acabe2986de25d0f69a7005185425cf94edcd873b19cacbe56aa823c9c64fa607b3
-  data.tar.gz: c1d802b26491e276ffbcdc3e3dd4e4272c9b48ab63a5e6b7283fb5e5f7b706ce64d5216728946aa3783f88396a0ec9a0582c045571cd78de050f42496479a392
+  metadata.gz: 437e92131a90f509f116067b7f0f5fb311c7ddc835d554732479d3895d1ce23b63fec937df6dc6d2ecb048deaaff1411c0c9956019c4d2f26b10e0cfeb781d86
+  data.tar.gz: 100a3f6f404a6665cf56ab0b6159724f775f8e25c76ab103ba491764916a067d4ce55a4043db671f58251dfed7ba25d09fa5c6b63efee7ae5d9a8facfc612248

data/CHANGELOG.md

@@ -1,3 +1,26 @@
+## Kanrisuru 0.16.3 (December 25, 2021) ##
+*  Refactor `Kanrisuru::Mode::Permission` class into separate file.
+
+## Kanrisuru 0.16.2 (December 19, 2021) ##
+*  Organize functional ip specs
+*  Log in realtime, debug output the stdout from the remote server, as opposed to waiting until after the command is done.
+*  Fix the `opensuse_leap` for the stubnetwork rspec helper.
+*  Refactor and cleanup the `append_array` method for the `Kanrisuru::Command` instance
+*  Move `gpg_opts` for `zypper` command into re-usable method
+*  Add functional test cases for `zypper` command. 
+
+## Kanrisuru 0.16.1 (December 16, 2021) ##
+*  Fix require class for `os_package`.
+
+## Kanrisuru 0.16.0 (December 14, 2021) ##
+*  Fix `append_array` for the `Kanrisuru::Command` class with negated check on Array type.
+*  Add `to_f` method for `Kanrisuru::Command` and `Kanrisuru::Result` with unit test cases.
+*  Refactor `lsblk_version` into seperate command namespace with it's own parser.
+*  Add functional test cases for several core modules.
+*  Add `return_value` option to `stub_command!` method for non-result return methods. These are few and far between, but need to be tested appropriately. 
+*  Refactor several core modules with use of `command` calls for legibility.
+*  Fix several issues with `create_user`, `update_user`, and `delete_user`.
+
 ## Kanrisuru 0.15.0 (December 13, 2021) ##
 *  Add opts for `ip` command.
 *  Refactor `ip` object commands into smaller methods. Refactor `version` for consistent use and stub ability in funcitonal test cases.

data/kanrisuru.gemspec

@@ -8,9 +8,12 @@ Gem::Specification.new do |gem|
   gem.author      = 'Ryan Mammina'
   gem.email       = 'ryan@avamia.com'
   gem.license     = 'MIT'
-  gem.summary     = 'Manage remote servers over ssh with ruby.'
-  gem.description = 'Kanrisuru helps manage remote servers with objected oriented ruby. Results come back as structured data, parsed, prepared and ready.'
   gem.homepage    = 'https://github.com/avamia/kanrisuru'
+  gem.summary     = 'Manage remote servers over ssh with ruby.'
+  gem.description = <<-EOF
+    Kanrisuru helps manage remote servers with objected oriented ruby.#{' '}
+    Results come back as structured data, parsed, prepared and ready.
+  EOF
 
   gem.required_ruby_version = '>= 2.5.0'
 
@@ -30,6 +33,7 @@ Gem::Specification.new do |gem|
 
   gem.metadata = {
     'source_code_uri' => 'https://github.com/avamia/kanrisuru/',
-    'changelog_uri' => 'https://github.com/avamia/kanrisuru/blob/main/CHANGELOG.md'
+    'changelog_uri' => 'https://github.com/avamia/kanrisuru/blob/main/CHANGELOG.md',
+    'rubygems_mfa_required' => 'true'
   }
 end

data/lib/kanrisuru/command.rb

@@ -23,6 +23,10 @@ module Kanrisuru
       to_a.join.to_i
     end
 
+    def to_f
+      to_a.join.to_f
+    end
+
     def to_s
       to_a.join(' ')
     end
@@ -84,11 +88,10 @@ module Kanrisuru
       append_value("| #{value}")
     end
 
-    def append_array(value)
+    def append_array(value, separator = ' ')
       return unless Kanrisuru::Util.present?(value)
 
-      value = [value] if value.instance_of?(String)
-      append_value(value.join(' '))
+      append_value(Kanrisuru::Util.array_join_string(value, separator))
     end
 
     def append_value(value)

data/lib/kanrisuru/core/disk/commands/lsblk.rb

@@ -10,12 +10,12 @@ module Kanrisuru
 
         command = Kanrisuru::Command.new('lsblk')
 
-        version = lsblk_version
+        version = lsblk_version.to_f
 
         ## lsblk after version 2.26 handles json parsing.
         ## TODO: parse nested children for earlier version of lsblk
-        if version >= 2.27
-          command.append_flag('--json') if version >= 2.27
+        if version >= LSBK_VERSION
+          command.append_flag('--json') if version >= LSBK_VERSION
         else
           command.append_flag('-i')
           command.append_flag('-P')
@@ -40,14 +40,9 @@ module Kanrisuru
         command = Kanrisuru::Command.new('lsblk --version')
         execute(command)
 
-        version = 0.00
-        regex = Regexp.new(/\d+(?:[,.]\d+)?/)
-
-        raise 'lsblk command not found' if command.failure?
-
-        version = command.to_s.scan(regex)[0].to_f unless regex.match(command.to_s).nil?
-
-        version
+        Kanrisuru::Result.new(command) do |cmd|
+          Parser::LsblkVersion.parse(cmd)
+        end
       end
     end
   end

data/lib/kanrisuru/core/disk/constants.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+module Kanrisuru
+  module Core
+    module Disk
+      LSBK_VERSION = 2.27
+    end
+  end
+end

data/lib/kanrisuru/core/disk/parser.rb

@@ -3,4 +3,5 @@
 require_relative 'parsers/blkid'
 require_relative 'parsers/df'
 require_relative 'parsers/du'
+require_relative 'parsers/lsblk_version'
 require_relative 'parsers/lsblk'

data/lib/kanrisuru/core/disk/parsers/lsblk_version.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Kanrisuru
+  module Core
+    module Disk
+      module Parser
+        class LsblkVersion
+          def self.parse(command)
+            version = 0.00
+            regex = Regexp.new(/\d+(?:[,.]\d+)?/)
+
+            raise 'lsblk command not found' if command.failure?
+
+            version = command.to_s.scan(regex)[0].to_f unless regex.match(command.to_s).nil?
+            version
+          end
+        end
+      end
+    end
+  end
+end

data/lib/kanrisuru/core/disk.rb

@@ -2,6 +2,7 @@
 
 require 'json'
 
+require_relative 'disk/constants'
 require_relative 'disk/types'
 require_relative 'disk/parser'
 require_relative 'disk/commands'

data/lib/kanrisuru/core/dmi/commands/dmi.rb

@@ -20,7 +20,7 @@ module Kanrisuru
         return unless Kanrisuru::Util.present?(opts[:types])
 
         types = opts[:types]
-        types = [types] if types.instance_of?(String)
+        types = [types] unless types.instance_of?(Array)
 
         types.each do |type|
           type = get_dmi_type(type)

data/lib/kanrisuru/core/file/commands/chmod.rb

@@ -7,7 +7,7 @@ module Kanrisuru
         recursive = opts[:recursive]
 
         command =
-          if mode.instance_of?(String) && (mode.include?(',') || /[=+-]/.match(mode))
+          if mode.instance_of?(String) && (mode.include?(',') || /[=+-]/.match(mode)) && mode.length < 10
             Kanrisuru::Command.new("chmod #{mode} #{path}")
           elsif mode.instance_of?(Kanrisuru::Mode)
             Kanrisuru::Command.new("chmod #{mode.numeric} #{path}")

data/lib/kanrisuru/core/file/commands/copy.rb

@@ -29,9 +29,7 @@ module Kanrisuru
         if opts[:preserve].instance_of?(TrueClass)
           command.append_flag('-p')
         elsif Kanrisuru::Util.present?(opts[:preserve])
-          preserve =
-            (opts[:preserve].join(',') if opts[:preserve].instance_of?(Array))
-
+          preserve = Kanrisuru::Util.array_join_string(opts[:preserve], ',')
           command << "--preserve=#{preserve}"
         end
 

data/lib/kanrisuru/core/file/commands/mkdir.rb

@@ -8,16 +8,21 @@ module Kanrisuru
         group     = opts[:group]
         recursive = opts[:recursive]
 
-        command = Kanrisuru::Command.new("mkdir #{path}")
+        command = Kanrisuru::Command.new('mkdir')
+
+        command.append_array(path)
         command.append_flag('-p', opts[:silent])
 
         if Kanrisuru::Util.present?(opts[:mode])
           mode = opts[:mode]
-          if mode.instance_of?(Kanrisuru::Mode)
-            mode = mode.numeric
-          elsif mode.instance_of?(String) && (mode.include?(',') || /[=+-]/.match(mode))
-            mode = Kanrisuru::Mode.new(mode).numeric
-          end
+
+          mode = if mode.instance_of?(Kanrisuru::Mode)
+                   mode.numeric
+                 elsif mode.instance_of?(String) && (mode.include?(',') || /[=+-]/.match(mode))
+                   mode
+                 else
+                   Kanrisuru::Mode.new(mode).numeric
+                 end
 
           command.append_arg('-m', mode)
         end

data/lib/kanrisuru/core/file/commands/rm.rb

@@ -10,7 +10,10 @@ module Kanrisuru
           raise ArgumentError, "Can't delete root path" if path == '/' || realpath(path).path == '/'
         end
 
-        command = Kanrisuru::Command.new("rm #{paths.join(' ')} --preserve-root")
+        command = Kanrisuru::Command.new('rm')
+        command.append_array(paths)
+
+        command.append_flag('--preserve-root')
         command.append_flag('-f', opts[:force])
         command.append_flag('-r', opts[:recursive])
 

data/lib/kanrisuru/core/file/commands/touch.rb

@@ -7,7 +7,8 @@ module Kanrisuru
         date = opts[:date]
 
         paths = [paths] if paths.instance_of?(String)
-        command = Kanrisuru::Command.new("touch #{paths.join(' ')}")
+        command = Kanrisuru::Command.new('touch')
+        command.append_array(paths)
 
         command.append_flag('-a', opts[:atime])
         command.append_flag('-m', opts[:mtime])

data/lib/kanrisuru/core/system/commands/kill.rb

@@ -11,7 +11,7 @@ module Kanrisuru
 
         command = Kanrisuru::Command.new('kill')
         command << "-#{signal}"
-        command << (pids.instance_of?(Array) ? pids.join(' ') : pids)
+        command.append_array(pids)
 
         execute_shell(command)
 

data/lib/kanrisuru/core/user/commands/create_user.rb

@@ -4,33 +4,25 @@ module Kanrisuru
   module Core
     module User
       def create_user(user, opts = {})
-        uid           = opts[:uid]
         group         = opts[:group]
         groups        = opts[:groups]
-        home          = opts[:home]
         shell         = opts[:shell] || '/bin/false'
-        createhome    = opts[:createhome]
-        system_opt    = opts[:system]
-        skeleton      = opts[:skeleton]
-        non_unique    = opts[:non_unique]
-        password      = opts[:password]
-        expires       = opts[:expires] ## YYYY-MM-DD
 
         command = Kanrisuru::Command.new("useradd #{user}")
 
-        if Kanrisuru::Util.present?(uid)
-          command.append_arg('-u', uid)
-          command.append_flag('-o', non_unique)
+        if Kanrisuru::Util.present?(opts[:uid])
+          command.append_arg('-u', opts[:uid])
+          command.append_flag('-o', opts[:non_unique])
         end
 
-        command.append_flag('-r', system_opt)
+        command.append_flag('-r', opts[:system])
         command.append_arg('-s', shell)
-        command.append_arg('-d', home)
+        command.append_arg('-d', opts[:home])
 
-        case createhome
+        case opts[:createhome]
         when true
           command.append_flag('-m')
-          command.append_arg('-k', skeleton)
+          command.append_arg('-k', opts[:skeleton])
         when false
           command.append_flag('-M')
         end
@@ -43,8 +35,8 @@ module Kanrisuru
 
         command.append_arg('-G', groups.join(',')) if Kanrisuru::Util.present?(groups)
 
-        command.append_arg('-p', password)
-        command.append_arg('-e', expires)
+        command.append_arg('-p', opts[:password])
+        command.append_arg('-e', opts[:expires]) ## YYYY-MM-DD
 
         execute_shell(command)
 

data/lib/kanrisuru/core/user/commands/delete_user.rb

@@ -6,7 +6,7 @@ module Kanrisuru
       def delete_user(user, opts = {})
         force = opts[:force]
 
-        return false unless get_uid(user)
+        return false if get_uid(user).failure?
 
         command = Kanrisuru::Command.new("userdel #{user}")
         command.append_flag('-f', force)

data/lib/kanrisuru/core/user/commands/update_user.rb

@@ -4,52 +4,43 @@ module Kanrisuru
   module Core
     module User
       def update_user(user, opts = {})
-        uid           = opts[:uid]
         group         = opts[:group]
         groups        = opts[:groups]
-        append        = opts[:append]
-        home          = opts[:home]
-        move_home     = opts[:move_home]
-        shell         = opts[:shell] || '/bin/false'
-        non_unique    = opts[:non_unique]
-        password      = opts[:password]
-        expires       = opts[:expires] ## YYYY-MM-DD
-        locked        = opts[:locked]
 
         command = Kanrisuru::Command.new("usermod #{user}")
 
-        if Kanrisuru::Util.present?(home)
-          command.append_arg('-d', home)
-          command.append_flag('-m', move_home)
+        if Kanrisuru::Util.present?(opts[:home])
+          command.append_arg('-d', opts[:home])
+          command.append_flag('-m', opts[:move_home])
         end
 
-        command.append_arg('-s', shell)
+        command.append_arg('-s', opts[:shell])
 
-        if Kanrisuru::Util.present?(uid)
-          command.append_arg('-u', uid)
-          command.append_flag('-o', non_unique)
+        if Kanrisuru::Util.present?(opts[:uid])
+          command.append_arg('-u', opts[:uid])
+          command.append_flag('-o', opts[:non_unique])
         end
 
-        command.append_arg('-g', group) if Kanrisuru::Util.present?(group) && group_exists?(group)
+        command.append_arg('-g', group) if Kanrisuru::Util.present?(group) && group?(group)
 
         if Kanrisuru::Util.present?(groups)
-          command.append_arg('-G', groups.join(','))
-          command.append_flag('-a', append)
+          command.append_arg('-G', Kanrisuru::Util.array_join_string(groups, ','))
+          command.append_flag('-a', opts[:append])
         end
 
-        case locked
+        case opts[:locked]
         when true
           command.append_flag('-L')
           command.append_arg('-e', 1)
         when false
-          command.append_arg('-U')
+          command.append_flag('-U')
           command.append_arg('-e', 99_999)
         else
           ## Ensure expires isn't added twice.
-          command.append_arg('-e', expires)
+          command.append_arg('-e', opts[:expires]) ## YYYY-MM-DD
 
           ## Can't use password with lock / unlock flag.
-          command.append_arg('-p', password)
+          command.append_arg('-p', opts[:password])
         end
 
         execute_shell(command)

data/lib/kanrisuru/core/zypper/commands/add_repo.rb

@@ -17,16 +17,9 @@ module Kanrisuru
         command.append_flag('--no-refresh', opts[:no_refresh])
         command.append_flag('--keep-packages', opts[:keep_packages])
         command.append_flag('--no-keep-packages', opts[:no_keep_packages])
-        command.append_flag('--gpgcheck', opts[:gpgcheck])
-        command.append_flag('--gpgcheck-strict', opts[:gpgcheck_strict])
-        command.append_flag('--gpgcheck-allow-unsigned', opts[:gpgcheck_allow_unsigned])
-        command.append_flag('--gpgcheck-allow-unsigned-repo', opts[:gpgcheck_allow_unsigned_repo])
-        command.append_flag('--gpgcheck-allow-unsigned-package', opts[:gpgcheck_allow_unsigned_package])
-        command.append_flag('--no-gpgcheck', opts[:no_gpgcheck])
-        command.append_flag('--default-gpgcheck', opts[:default_gpgcheck])
-
         command.append_arg('--priority', opts[:priority])
 
+        zypper_gpg_opts(command, opts)
         zypper_repos_opt(command, opts)
 
         execute_shell(command)

data/lib/kanrisuru/core/zypper/commands/add_service.rb

@@ -8,14 +8,16 @@ module Kanrisuru
         zypper_global_opts(command, opts)
         command << 'addservice'
 
-        command.append_arg('--name', opts[:name])
+        name = ("'#{opts[:name]}'" if Kanrisuru::Util.present?(opts[:name]))
 
+        command.append_arg('--name', name)
         command.append_flag('--enable', opts[:enable])
         command.append_flag('--disable', opts[:disable])
         command.append_flag('--refresh', opts[:refresh])
         command.append_flag('--no-refresh', opts[:no_refresh])
 
-        command.append_array(opts[:services])
+        command << opts[:service]
+        command << opts[:alias]
 
         execute_shell(command)
 

data/lib/kanrisuru/core/zypper/commands/info.rb

@@ -11,8 +11,7 @@ module Kanrisuru
         zypper_repos_opt(command, opts)
         zypper_package_type_opt(command, opts)
 
-        packages = Kanrisuru::Util.array_join_string(opts[:packages], ' ')
-        command << packages
+        command.append_array(opts[:packages])
 
         execute_shell(command)
 

data/lib/kanrisuru/core/zypper/commands/install.rb

@@ -12,7 +12,7 @@ module Kanrisuru
         zypper_package_type_opt(command, opts)
 
         command.append_arg('-n', opts[:name])
-        command.append_arg('-f', opts[:force])
+        command.append_flag('-f', opts[:force])
         command.append_flag('--oldpackage', opts[:oldpackage])
         command.append_arg('--from', opts[:from])
         command.append_arg('--capability', opts[:capability])
@@ -27,8 +27,7 @@ module Kanrisuru
         zypper_download_and_install_opts(command, opts)
         zypper_expert_opts(command, opts)
 
-        packages = Kanrisuru::Util.array_join_string(opts[:packages], ' ')
-        command << packages
+        command.append_array(opts[:packages])
 
         execute_shell(command)
 

data/lib/kanrisuru/core/zypper/commands/modify_repo.rb

@@ -19,13 +19,7 @@ module Kanrisuru
         command.append_flag('--keep-packages', opts[:keep_packages])
         command.append_flag('--no-keep-packages', opts[:no_keep_packages])
 
-        command.append_flag('--gpgcheck', opts[:gpgcheck])
-        command.append_flag('--gpgcheck-strict', opts[:gpgcheck_strict])
-        command.append_flag('--gpgcheck-allow-unsigned', opts[:gpgcheck_allow_unsigned])
-        command.append_flag('--gpgcheck-allow-unsigned-repo', opts[:gpgcheck_allow_unsigned_repo])
-        command.append_flag('--gpgcheck-allow-unsigned-package', opts[:gpgcheck_allow_unsigned_package])
-        command.append_flag('--no-gpgcheck', opts[:no_gpgcheck])
-        command.append_flag('--default-gpgcheck', opts[:default_gpgcheck])
+        zypper_gpg_opts(command, opts)
 
         command.append_flag('--all', opts[:all])
         command.append_flag('--local', opts[:local])

data/lib/kanrisuru/core/zypper/commands/modify_service.rb

@@ -8,8 +8,8 @@ module Kanrisuru
         zypper_global_opts(command, opts)
 
         command << 'modifyservice'
-        command.append_arg('--name', opts[:name])
 
+        command.append_arg('--name', opts[:name])
         command.append_flag('--enable', opts[:enable])
         command.append_flag('--disable', opts[:disable])
         command.append_flag('--refresh', opts[:refresh])
@@ -31,6 +31,8 @@ module Kanrisuru
           command.append_arg('--medium-type', opts[:medium_type])
         end
 
+        command << opts[:service] if Kanrisuru::Util.present?(opts[:service])
+
         execute_shell(command)
 
         Kanrisuru::Result.new(command)

data/lib/kanrisuru/core/zypper/commands/remove.rb

@@ -15,8 +15,7 @@ module Kanrisuru
         zypper_package_type_opt(command, opts)
         zypper_solver_opts(command, opts)
 
-        packages = Kanrisuru::Util.array_join_string(opts[:packages], ' ')
-        command << packages
+        command.append_array(opts[:packages])
 
         execute_shell(command)
         Kanrisuru::Result.new(command)

data/lib/kanrisuru/core/zypper/commands/remove_repo.rb

@@ -15,10 +15,10 @@ module Kanrisuru
         command.append_flag('--local', opts[:local])
         command.append_flag('--remote', opts[:remote])
 
-        if Kanrisuru::Util.present?(opts[:media_type])
-          raise ArgumentError, 'Invalid media type' unless ZYPPER_MEDIA_TYPES.include?(opts[:media_type])
+        if Kanrisuru::Util.present?(opts[:medium_type])
+          raise ArgumentError, 'Invalid media type' unless MEDIUM_TYPES.include?(opts[:medium_type])
 
-          command.append_arg('--media-type', opts[:media_type])
+          command.append_arg('--medium-type', opts[:medium_type])
         end
 
         command.append_array(opts[:repos])

data/lib/kanrisuru/core/zypper/commands/remove_service.rb

@@ -4,13 +4,18 @@ module Kanrisuru
   module Core
     module Zypper
       def zypper_remove_service(opts)
+        service = opts[:service]
+
         command = Kanrisuru::Command.new('zypper')
         zypper_global_opts(command, opts)
 
         command << 'removeservice'
         command.append_flag('--loose-auth', opts[:loose_auth])
         command.append_flag('--loose-query', opts[:loose_query])
-        command.append_array(opts[:services])
+
+        service = "'#{service}'" if service.match(/\s/)
+
+        command << service
 
         execute_shell(command)
 

data/lib/kanrisuru/core/zypper/commands/search.rb

@@ -37,9 +37,7 @@ module Kanrisuru
 
         zypper_repos_opt(command, opts)
         zypper_package_type_opt(command, opts)
-
-        packages = Kanrisuru::Util.array_join_string(opts[:packages], ' ')
-        command << packages
+        command.append_array(opts[:packages])
 
         execute_shell(command)
 

data/lib/kanrisuru/core/zypper/commands/source_install.rb

@@ -13,6 +13,8 @@ module Kanrisuru
         command.append_flag('--no-build-deps', opts[:no_build_deps])
         command.append_flag('--download-only', opts[:download_only])
 
+        command.append_array(opts[:packages])
+
         execute_shell(command)
 
         Kanrisuru::Result.new(command)

data/lib/kanrisuru/core/zypper/commands.rb

@@ -76,6 +76,16 @@ module Kanrisuru
         command.append_flag('--no-allow-vendor-change', opts[:no_allow_vendor_change])
       end
 
+      def zypper_gpg_opts(command, opts)
+        command.append_flag('--gpgcheck', opts[:gpgcheck])
+        command.append_flag('--gpgcheck-strict', opts[:gpgcheck_strict])
+        command.append_flag('--gpgcheck-allow-unsigned', opts[:gpgcheck_allow_unsigned])
+        command.append_flag('--gpgcheck-allow-unsigned-repo', opts[:gpgcheck_allow_unsigned_repo])
+        command.append_flag('--gpgcheck-allow-unsigned-package', opts[:gpgcheck_allow_unsigned_package])
+        command.append_flag('--no-gpgcheck', opts[:no_gpgcheck])
+        command.append_flag('--default-gpgcheck', opts[:default_gpgcheck])
+      end
+
       def zypper_repos_opt(command, opts)
         zypper_array_opt(command, opts[:repos], '--repo')
       end
@@ -101,7 +111,6 @@ module Kanrisuru
 
       def zypper_package_type_opt(command, opts)
         type = opts[:type]
-
         command.append_arg('-t', type) if Kanrisuru::Util.present?(type) && PACKAGE_TYPES.include?(type)
       end
     end