kanrisuru 0.9.2 → 0.12.0

data/spec/functional/remote/env_spec.rb

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe Kanrisuru::Remote::Env do
+  let(:env) { described_class.new }
+
+  it 'adds a environment variable' do
+    env['VAR1'] = 'hello'
+    expect(env.count).to eq(1)
+    expect(env.to_h).to eq({ 'VAR1' => 'hello' })
+    expect(env.to_s).to eq('export VAR1=hello;')
+    expect(env['VAR1']).to eq('hello')
+  end
+
+  it 'adds multiple environment variables' do
+    env['var1'] = 'hello'
+    env['var2'] = 'world'
+
+    expect(env.count).to eq(2)
+    expect(env.to_h).to eq({ 'VAR1' => 'hello', 'VAR2' => 'world' })
+    expect(env.to_s).to eq('export VAR1=hello; export VAR2=world;')
+    expect(env['VAR1']).to eq('hello')
+    expect(env['VAR2']).to eq('world')
+  end
+
+  it 'deletes a variable' do
+    env[:var1] = 'foo'
+    expect(env.count).to eq(1)
+    expect(env[:var1]).to eq('foo')
+    env.delete(:var1)
+    expect(env.count).to eq(0)
+    expect(env.to_s).to eq('')
+  end
+
+  it 'clears the environment' do
+    env['VERSION'] = 1
+    env['SHELL'] = '/bin/zsh'
+    env['USER'] = 'ubuntu'
+    env['HOSTNAME'] = 'ubuntu'
+    expect(env.to_s).to eq('export VERSION=1; export SHELL=/bin/zsh; export USER=ubuntu; export HOSTNAME=ubuntu;')
+
+    expect(env.count).to eq(4)
+    env.clear
+
+    expect(env.count).to eq(0)
+  end
+end

data/spec/helper/stub_network.rb

@@ -17,7 +17,7 @@ class StubNetwork
 
       unless Kanrisuru::Remote::Os.instance_methods(false).include?(:initialize_alias)
         Kanrisuru::Remote::Os.class_eval do
-          alias_method :initialize_alias, :initialize 
+          alias_method :initialize_alias, :initialize
           define_method :initialize do |host|
             @host = host
 
@@ -32,30 +32,67 @@ class StubNetwork
         end
       end
 
-      unless Kanrisuru::Result.instance_methods(false).include?(:initialize_alias)
-        Kanrisuru::Result.class_eval do
-          alias_method :initialize_alias, :initialize 
-          def initialize(command)
-            @command = command
-            @data = nil
+      return if Kanrisuru::Result.instance_methods(false).include?(:initialize_alias)
 
-            @error = @command.to_a if @command.failure?
+      Kanrisuru::Result.class_eval do
+        alias_method :initialize_alias, :initialize
+        def initialize(command, parse_result = false, &block)
+          @command = command
+          @data = nil
+
+          @data = block.call(@command) if @command.success? && block_given? && parse_result
+          @error = @command.to_a if @command.failure?
+
+          ## Define getter methods on result that maps to
+          ## the same methods of a data struct.
+          return unless @command.success? && Kanrisuru::Util.present?(@data) && @data.class.ancestors.include?(Struct)
+
+          method_names = @data.members
+          self.class.class_eval do
+            method_names.each do |method_name|
+              define_method method_name do
+                @data[method_name]
+              end
+            end
+          end
+        end
+      end
+    end
+
+    def stub_command!(method, opts = {}, &block)
+      Kanrisuru::Remote::Host.class_eval do
+        alias_method "#{method}_alias", method
+
+        define_method(method) do |*args|
+          command = Kanrisuru::Command.new(method.to_s)
+
+          status = opts[:status] || 0
+          command.handle_status(status)
+
+          Kanrisuru::Result.new(command, true) do |_cmd|
+            block.call(args)
           end
         end
       end
     end
 
+    def unstub_command!(method)
+      Kanrisuru::Remote::Host.class_eval do
+        alias_method method, "#{method}_alias"
+      end
+    end
+
     def unstub!
       Kanrisuru::Remote::Host.class_eval do
         alias_method :execute_with_retries, :execute_with_retries_alias
       end
 
       Kanrisuru::Remote::Os.class_eval do
-        alias_method :initialize, :initialize_alias 
+        alias_method :initialize, :initialize_alias
       end
 
       Kanrisuru::Result.class_eval do
-        alias_method :initialize, :initialize_alias 
+        alias_method :initialize, :initialize_alias
       end
     end
 
@@ -90,13 +127,11 @@ class StubNetwork
           hardware_platform: 'x86_64',
           processor: 'x86_64',
           release: 'opensuse-leap',
-          version:15.2
+          version: 15.2
         }
       }
 
-      defaults[name].key?(property) ?
-        defaults[name][property] : nil
+      defaults[name][property] if defaults[name].key?(property)
     end
-
   end
 end

data/spec/helper/test_hosts.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'securerandom'
+
 class TestHosts
   class << self
     def each_os(opts = {}, &block)
@@ -7,7 +9,10 @@ class TestHosts
         next unless test?(os_name)
         next if opts[:only] && !only?(opts, os_name)
 
-        block.call(os_name)
+        host_json = TestHosts.host(os_name)
+        spec_dir = spec_dir(os_name, host_json)
+
+        block.call(os_name, host_json, spec_dir)
       end
     end
 
@@ -15,6 +20,11 @@ class TestHosts
       hosts(name)
     end
 
+    def spec_dir(os_name, host_json)
+      random_string = SecureRandom.hex
+      "#{host_json['home']}/.kanrisuru_spec_files_#{random_string[0..5]}"
+    end
+
     def only?(opts, name)
       ((opts[:only].instance_of?(Array) && opts[:only].include?(name)) ||
         (opts[:only].instance_of?(String) && opts[:only] == name))

data/spec/integration/core/apt_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::Apt do
-  TestHosts.each_os(only: %w[debian ubuntu]) do |os_name|
+TestHosts.each_os(only: %w[debian ubuntu]) do |os_name, host_json|
+  RSpec.describe Kanrisuru::Core::Apt do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],

data/spec/integration/core/archive_spec.rb

@@ -2,22 +2,20 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::Archive do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json, spec_dir|
+  RSpec.describe Kanrisuru::Core::Archive do
     context "with #{os_name}" do
       before(:all) do
-        host_json = TestHosts.host(os_name)
         host = Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
           username: host_json['username'],
           keys: [host_json['ssh_key']]
         )
 
-        host.mkdir("#{host_json['home']}/.kanrisuru_spec_files", silent: true)
+        host.mkdir(spec_dir, silent: true)
         host.disconnect
       end
 
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
@@ -26,22 +24,19 @@ RSpec.describe Kanrisuru::Core::Archive do
         )
       end
 
-      let(:spec_dir) { "#{host_json['home']}/.kanrisuru_spec_files" }
-
       after do
         host.disconnect
       end
 
       after(:all) do
-        host_json = TestHosts.host(os_name)
         host = Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
           username: host_json['username'],
           keys: [host_json['ssh_key']]
         )
 
-        host.rmdir("#{host_json['home']}/.kanrisuru_spec_files")
-        host.rmdir("#{host_json['home']}/extract-tar-files") if host.dir?("#{host_json['home']}/extract-tar-files")
+        host.rmdir(spec_dir)
+        host.rmdir("#{spec_dir}/extract-tar-files") if host.dir?("#{spec_dir}/extract-tar-files")
         host.disconnect
       end
 
@@ -126,10 +121,10 @@ RSpec.describe Kanrisuru::Core::Archive do
         paths = result.map(&:path)
         expect(paths.include?('test2.config')).to eq(false)
 
-        host.mkdir("#{host_json['home']}/extract-tar-files", silent: true)
-        host.tar('extract', 'archive.tar', directory: "#{host_json['home']}/extract-tar-files")
+        host.mkdir("#{spec_dir}/extract-tar-files", silent: true)
+        host.tar('extract', 'archive.tar', directory: "#{spec_dir}/extract-tar-files")
 
-        result = host.ls(path: "#{host_json['home']}/extract-tar-files")
+        result = host.ls(path: "#{spec_dir}/extract-tar-files")
         paths = result.map(&:path)
 
         expect(paths.include?('test1.config')).to eq(true)

data/spec/integration/core/disk_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::Disk do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json|
+  RSpec.describe Kanrisuru::Core::Disk do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],

data/spec/integration/core/dmi_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::Dmi do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json|
+  RSpec.describe Kanrisuru::Core::Dmi do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],

data/spec/integration/core/file_spec.rb

@@ -2,22 +2,20 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::File do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json, spec_dir|
+  RSpec.describe Kanrisuru::Core::File do
     context "with #{os_name}" do
       before(:all) do
-        host_json = TestHosts.host(os_name)
         host = Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
           username: host_json['username'],
           keys: [host_json['ssh_key']]
         )
 
-        host.mkdir("#{host_json['home']}/.kanrisuru_spec_files", silent: true)
+        host.mkdir(spec_dir, silent: true)
         host.disconnect
       end
 
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
@@ -26,25 +24,18 @@ RSpec.describe Kanrisuru::Core::File do
         )
       end
 
-      let(:spec_dir) { "#{host_json['home']}/.kanrisuru_spec_files" }
-
       after do
         host.disconnect
       end
 
       after(:all) do
-        host_json = TestHosts.host(os_name)
         host = Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
           username: host_json['username'],
           keys: [host_json['ssh_key']]
         )
 
-        host.rm("#{host_json['home']}/.kanrisuru_spec_files", force: true, recursive: true)
-        if host.dir?("#{host_json['home']}/extract-tar-files")
-          host.rm("#{host_json['home']}/extract-tar-files", force: true,
-                                                            recursive: true)
-        end
+        host.rm(spec_dir, force: true, recursive: true)
         host.disconnect
       end
 
@@ -356,7 +347,6 @@ RSpec.describe Kanrisuru::Core::File do
         data = host.cat('/etc/hosts').command.raw_result
         doc = data.map(&:lines).flatten
 
-        lines = doc.length
         words = doc.map(&:split).flatten
         chars = doc.map(&:length).flatten
 

data/spec/integration/core/find_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::Find do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json|
+  RSpec.describe Kanrisuru::Core::Find do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
@@ -19,6 +18,7 @@ RSpec.describe Kanrisuru::Core::Find do
       end
 
       it 'finds home directory' do
+        host.su('root')
         result = host.find(paths: '/home')
         home = result.find { |item| item.path == host_json['home'] }
         expect(home.path).to eq(host_json['home'])

data/spec/integration/core/group_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::Group do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json|
+  RSpec.describe Kanrisuru::Core::Group do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],

data/spec/integration/core/ip_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::IP do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json|
+  RSpec.describe Kanrisuru::Core::IP do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],

data/spec/integration/core/path_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::Path do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json|
+  RSpec.describe Kanrisuru::Core::Path do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],

data/spec/integration/core/socket_spec.rb

@@ -2,11 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::Socket do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json|
+  RSpec.describe Kanrisuru::Core::Socket do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
-
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],

data/spec/integration/core/stat_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::Stat do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json|
+  RSpec.describe Kanrisuru::Core::Stat do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],

data/spec/integration/core/stream_spec.rb

@@ -1,21 +1,21 @@
 # frozen_string_literal: true
 
-RSpec.describe Kanrisuru::Core::Stream do
-  TestHosts.each_os do |os_name|
+require 'spec_helper'
+
+TestHosts.each_os do |os_name, host_json, spec_dir|
+  RSpec.describe Kanrisuru::Core::Stream do
     context "with #{os_name}" do
       before(:all) do
-        host_json = TestHosts.host(os_name)
         host = Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
           username: host_json['username'],
           keys: [host_json['ssh_key']]
         )
 
-        host.mkdir("#{host_json['home']}/.kanrisuru_spec_files", silent: true)
+        host.mkdir(spec_dir, silent: true)
         host.disconnect
       end
 
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
@@ -24,21 +24,18 @@ RSpec.describe Kanrisuru::Core::Stream do
         )
       end
 
-      let(:spec_dir) { "#{host_json['home']}/.kanrisuru_spec_files" }
-
       after do
         host.disconnect
       end
 
       after(:all) do
-        host_json = TestHosts.host(os_name)
         host = Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
           username: host_json['username'],
           keys: [host_json['ssh_key']]
         )
 
-        host.rmdir("#{host_json['home']}/.kanrisuru_spec_files")
+        host.rmdir(spec_dir)
         host.disconnect
       end
 

data/spec/integration/core/system_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::System do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json|
+  RSpec.describe Kanrisuru::Core::System do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
@@ -107,7 +106,6 @@ RSpec.describe Kanrisuru::Core::System do
         host.su('root')
 
         result = host.last(failed_attempts: true)
-        puts result.data
         expect(result).to be_success
       end
 

data/spec/integration/core/transfer_spec.rb

@@ -2,22 +2,20 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::File do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json, spec_dir|
+  RSpec.describe Kanrisuru::Core::File do
     context "with #{os_name}" do
       before(:all) do
-        host_json = TestHosts.host(os_name)
         host = Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
           username: host_json['username'],
           keys: [host_json['ssh_key']]
         )
 
-        host.mkdir("#{host_json['home']}/.kanrisuru_spec_files", silent: true)
+        host.mkdir(spec_dir, silent: true)
         host.disconnect
       end
 
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
@@ -26,21 +24,18 @@ RSpec.describe Kanrisuru::Core::File do
         )
       end
 
-      let(:spec_dir) { "#{host_json['home']}/.kanrisuru_spec_files" }
-
       after do
         host.disconnect
       end
 
       after(:all) do
-        host_json = TestHosts.host(os_name)
         host = Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
           username: host_json['username'],
           keys: [host_json['ssh_key']]
         )
 
-        host.rmdir("#{host_json['home']}/.kanrisuru_spec_files")
+        host.rmdir(spec_dir)
         host.disconnect
       end
 

data/spec/integration/core/user_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::User do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json|
+  RSpec.describe Kanrisuru::Core::User do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
@@ -19,7 +18,6 @@ RSpec.describe Kanrisuru::Core::User do
       end
 
       after(:all) do
-        host_json = TestHosts.host(os_name)
         host = Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
           username: host_json['username'],

data/spec/integration/core/yum_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::Yum do
-  TestHosts.each_os(only: %w[fedora rhel centos]) do |os_name|
+TestHosts.each_os(only: %w[fedora rhel centos]) do |os_name, host_json|
+  RSpec.describe Kanrisuru::Core::Yum do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],

data/spec/integration/core/zypper_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Core::Zypper do
-  TestHosts.each_os(only: %w[sles opensuse]) do |os_name|
+TestHosts.each_os(only: %w[sles opensuse]) do |os_name, host_json|
+  RSpec.describe Kanrisuru::Core::Zypper do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],
@@ -30,19 +29,19 @@ RSpec.describe Kanrisuru::Core::Zypper do
 
       it 'installs a package' do
         host.su('root')
-        result = host.zypper('install', packages: 'ffmpeg')
+        result = host.zypper('install', packages: 'nginx')
         expect(result).to be_success
       end
 
       it 'installs multiple packages' do
         host.su('root')
-        result = host.zypper('install', packages: %w[curl ffmpeg])
+        result = host.zypper('install', packages: %w[curl nginx])
         expect(result).to be_success
       end
 
       it 'removes a package' do
         host.su('root')
-        result = host.zypper('remove', packages: ['ffmpeg'])
+        result = host.zypper('remove', packages: ['nginx'])
         expect(result).to be_success
       end
 

data/spec/integration/remote/cpu_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Remote::Cpu do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json|
+  RSpec.describe Kanrisuru::Remote::Cpu do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],

data/spec/integration/remote/env_spec.rb

@@ -2,10 +2,9 @@
 
 require 'spec_helper'
 
-RSpec.describe Kanrisuru::Remote::Env do
-  TestHosts.each_os do |os_name|
+TestHosts.each_os do |os_name, host_json|
+  RSpec.describe Kanrisuru::Remote::Env do
     context "with #{os_name}" do
-      let(:host_json) { TestHosts.host(os_name) }
       let(:host) do
         Kanrisuru::Remote::Host.new(
           host: host_json['hostname'],