kanrisuru 0.16.0 → 0.16.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7dcfedf8792427fccd9a7210d3f8dab6f95d2f1ba1b196ed90205c0d4a5b2c4f
-  data.tar.gz: 9e5d940a569f7b4c964aaf2a8869a07b4194a2aa9699dff9de780184403f65a2
+  metadata.gz: e33daa1aa8d51a7f8a9e6efb4e77933ef5a71e1ce3e428dabc7f44468bde79b3
+  data.tar.gz: 8cbc4f168fdf88bdd22c8fb237018c65427010356ed8d1bfe9771f2e8ecfc0ca
 SHA512:
-  metadata.gz: 8f90832958572df546c18e49f8354b3014dea58b780a4540bbd204196be96562652ea0768283912005aa025fc97a85d39ff9d96dd0c48f104f89bb10d086fb64
-  data.tar.gz: df414f1782efeb27eef49133c3af0c13e6966489adc19115403e3d78c69b6fe07aedaca8b21f78417bdd921860d455df9dbe7735823f2cdaeb427c55576e8a4d
+  metadata.gz: 992f48fbc7c55a67cd2bfa649cba991620ab0ee6aa97d4c49de918d099a1e9963a30dea99aa71fe738ad39982462e731caf4ab121263a0bbf4870a68ae5a1e83
+  data.tar.gz: 208b2389785ed015055a3c67b49b433dae7d5388ee32334af5d74f7a2f94db19900a1dba4396b02edc675987fadead7b90d3f961e3eddb9e5e7c35c946ac28fe

data/CHANGELOG.md

@@ -1,3 +1,20 @@
+## Kanrisuru 0.16.4 (December 25, 2021) ##
+*  Refactor `Kanrisuru::Fstab::Entry` and `Kanrisuru::Fstab::Options` classes into separate files.
+
+## 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.

data/lib/kanrisuru/command.rb

@@ -88,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] unless value.instance_of?(Array)
-      append_value(value.join(' '))
+      append_value(Kanrisuru::Util.array_join_string(value, separator))
     end
 
     def append_value(value)

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

data/lib/kanrisuru/mode/permission.rb

@@ -0,0 +1,103 @@
+# frozen_string_literal: true
+
+module Kanrisuru
+  class Mode
+    class Permission
+      attr_reader :symbolic
+
+      def initialize(numeric, symbolic)
+        @numeric = numeric
+        @symbolic = symbolic
+
+        update_symbolic_rwx
+      end
+
+      def to_i
+        numeric.to_i
+      end
+
+      def to_s
+        symbolic
+      end
+
+      def numeric
+        @numeric.to_s
+      end
+
+      def all?
+        read? && write? && execute?
+      end
+
+      def symbolic=(symbolic)
+        @symbolic = symbolic
+
+        update_symbolic_rwx
+        update_numeric
+      end
+
+      def numeric=(numeric)
+        @numeric = numeric
+
+        update_numeric_rwx
+        update_symbolic
+      end
+
+      def read=(boolean)
+        @readable = boolean
+
+        update_numeric
+        update_symbolic
+      end
+
+      def write=(boolean)
+        @writeable = boolean
+
+        update_numeric
+        update_symbolic
+      end
+
+      def execute=(boolean)
+        @executable = boolean
+
+        update_numeric
+        update_symbolic
+      end
+
+      def read?
+        @readable
+      end
+
+      def write?
+        @writeable
+      end
+
+      def execute?
+        @executable
+      end
+
+      private
+
+      def update_symbolic_rwx
+        @readable   = @symbolic.include?('r')
+        @writeable  = @symbolic.include?('w')
+        @executable = @symbolic.include?('x')
+      end
+
+      def update_numeric_rwx
+        mode = @numeric.to_i(8)
+
+        @readable   = ((mode >> 2) & 0b001) == 1
+        @writeable  = ((mode >> 1) & 0b001) == 1
+        @executable = ((mode >> 0) & 0b001) == 1
+      end
+
+      def update_numeric
+        @numeric = (((read? ? 1 : 0) << 2) + ((write? ? 1 : 0) << 1) + (execute? ? 1 : 0)).to_s
+      end
+
+      def update_symbolic
+        @symbolic = "#{read? ? 'r' : '-'}#{write? ? 'w' : '-'}#{execute? ? 'x' : '-'}"
+      end
+    end
+  end
+end

data/lib/kanrisuru/mode.rb

@@ -1,105 +1,9 @@
 # frozen_string_literal: true
 
+require_relative 'mode/permission'
+
 module Kanrisuru
   class Mode
-    class Permission
-      attr_reader :symbolic
-
-      def initialize(numeric, symbolic)
-        @numeric = numeric
-        @symbolic = symbolic
-
-        update_symbolic_rwx
-      end
-
-      def to_i
-        numeric.to_i
-      end
-
-      def to_s
-        symbolic
-      end
-
-      def numeric
-        @numeric.to_s
-      end
-
-      def all?
-        read? && write? && execute?
-      end
-
-      def symbolic=(symbolic)
-        @symbolic = symbolic
-
-        update_symbolic_rwx
-        update_numeric
-      end
-
-      def numeric=(numeric)
-        @numeric = numeric
-
-        update_numeric_rwx
-        update_symbolic
-      end
-
-      def read=(boolean)
-        @readable = boolean
-
-        update_numeric
-        update_symbolic
-      end
-
-      def write=(boolean)
-        @writeable = boolean
-
-        update_numeric
-        update_symbolic
-      end
-
-      def execute=(boolean)
-        @executable = boolean
-
-        update_numeric
-        update_symbolic
-      end
-
-      def read?
-        @readable
-      end
-
-      def write?
-        @writeable
-      end
-
-      def execute?
-        @executable
-      end
-
-      private
-
-      def update_symbolic_rwx
-        @readable   = @symbolic.include?('r')
-        @writeable  = @symbolic.include?('w')
-        @executable = @symbolic.include?('x')
-      end
-
-      def update_numeric_rwx
-        mode = @numeric.to_i(8)
-
-        @readable   = ((mode >> 2) & 0b001) == 1
-        @writeable  = ((mode >> 1) & 0b001) == 1
-        @executable = ((mode >> 0) & 0b001) == 1
-      end
-
-      def update_numeric
-        @numeric = (((read? ? 1 : 0) << 2) + ((write? ? 1 : 0) << 1) + (execute? ? 1 : 0)).to_s
-      end
-
-      def update_symbolic
-        @symbolic = "#{read? ? 'r' : '-'}#{write? ? 'w' : '-'}#{execute? ? 'x' : '-'}"
-      end
-    end
-
     attr_reader :owner, :group, :other
 
     def initialize(mode)

data/lib/kanrisuru/os_package.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'set'
+
 require_relative 'os_package/collection'
 require_relative 'os_package/define'
 require_relative 'os_package/include'

data/lib/kanrisuru/remote/fstab/entry.rb

@@ -0,0 +1,154 @@
+# frozen_string_literal: true
+
+module Kanrisuru
+  module Remote
+    class Fstab
+      class Entry
+        attr_reader :device, :uuid, :invalid, :label, :type, :opts, :freq, :passno
+        attr_accessor :mount_point
+
+        def initialize(opts = {})
+          @host = opts[:host]
+          @line = opts[:line]
+
+          @default = nil
+
+          @device = opts[:device] || nil
+          @opts = opts[:opts] || nil
+          @label = opts[:label] || nil
+          @uuid = opts[:uuid] || nil
+          @mount_point = opts[:mount_point] || nil
+          @type = opts[:type] || nil
+          @freq = opts[:freq] || nil
+          @passno = opts[:passno] || nil
+
+          @changed = false
+
+          @ucount = 0
+          @special = false
+          @invalid = false
+
+          if Kanrisuru::Util.present?(@line) && @line.instance_of?(String)
+            parse_line!
+          elsif (Kanrisuru::Util.present?(@opts) && @opts.instance_of?(String)) || @opts.instance_of?(Hash)
+            @opts = Kanrisuru::Remote::Fstab::Options.new(@type, @opts)
+          end
+        end
+
+        def inspect
+          str = '#<Kanrisuru::Remote::Fstab::Entry:0x%<object_id>s ' \
+                '@line=%<line>s @device=%<device>s @label=%<label>s' \
+                '@uuid=%<uuid>s @freq=%<freq>s @pasno=%<passno>s' \
+                '@opts=%<opts>s}>'
+
+          format(
+            str,
+            object_id: object_id,
+            line: @line,
+            device: @device,
+            label: @label,
+            uuid: @uuid,
+            freq: @freq,
+            passno: @passno,
+            opts: @opts.inspect
+          )
+        end
+
+        def valid?
+          !@invalid
+        end
+
+        def to_s(override = nil)
+          mode = override || @default
+
+          case mode
+          when 'uuid'
+            "UUID=#{@uuid} #{@mount_point} #{@type} #{@opts} #{@freq} #{@passno}"
+          when 'label'
+            "LABEL=#{@label} #{@mount_point} #{@type} #{@opts} #{@freq} #{@passno}"
+          else
+            "#{@device} #{@mount_point} #{@type} #{@opts} #{@freq} #{@passno}"
+          end
+        end
+
+        private
+
+        def parse_line!
+          fsline, mp, @type, opts, freq, passno = @line.split
+
+          @mount_point = mp
+          @freq = freq || '0'
+          @passno = passno || '0'
+
+          @opts = Fstab::Options.new(@type, opts)
+
+          case @line
+          when /^\s*LABEL=/
+            @default = 'label'
+            parse_label(fsline)
+          when /^\s*UUID=/
+            @default = 'uuid'
+            parse_uuid(fsline)
+          when %r{^\s*/dev}
+            @default = 'dev'
+            parse_dev(fsline)
+          else
+            # TODO: somewhat risky to assume that everything else
+            # can be considered a special device, but validating this
+            # is really tricky.
+            @special = true
+            @device = fsline
+          end
+
+          # Fstab entries not matching real devices have device unknown
+          @invalid = (@line.split.count != 6) # invalid entry if < 6 columns
+
+          if (@uuid.nil? && @label.nil? && !@special) ||
+             @device =~ /^unknown_/ ||
+             (!@host.inode?(@device) && !@special)
+            @invalid = true
+            @ucount += 1
+          end
+
+          @invalid = true unless @freq =~ /0|1|2/ && @passno =~ /0|1|2/
+        end
+
+        def parse_label(fsline)
+          @label = fsline.split('=').last.strip.chomp
+          path = @host.realpath("/dev/disk/by-label/#{@label}").path
+
+          @device = begin
+            "/dev/#{path.split('/').last}"
+          rescue StandardError
+            "unknown_#{@ucount}"
+          end
+
+          result = @host.blkid(device: @device)
+          @uuid = result.success? ? result[0].uuid : nil
+        end
+
+        def parse_uuid(fsline)
+          @uuid = fsline.split('=').last.strip.chomp
+          path = @host.realpath("/dev/disk/by-uuid/#{uuid}").path
+
+          @device = begin
+            "/dev/#{path.split('/').last}"
+          rescue StandardError
+            "unknown_#{@ucount}"
+          end
+
+          result = @host.blkid(device: @device)
+          @label = result.success? ? result[0].label : nil
+        end
+
+        def parse_dev(fsline)
+          @device = fsline
+          result = @host.blkid(device: @device)
+
+          @label = result.success? ? result[0].label : nil
+          @uuid = result.success? ? result[0].uuid : nil
+        end
+      end
+    end
+  end
+end