kpm 0.7.2 → 0.10.0

data/spec/kpm/remote/migrations_spec.rb

@@ -1,18 +1,19 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
-describe KPM::Migrations, :skip_me_if_nil => ENV['TOKEN'].nil? do
+require 'spec_helper'
 
+describe KPM::Migrations, skip_me_if_nil: ENV['TOKEN'].nil? do
   context 'plugins' do
     it 'should be able to find migrations for a java plugin' do
       migrations = KPM::Migrations.new('analytics-plugin-3.0.2', nil, 'killbill/killbill-analytics-plugin', ENV['TOKEN']).migrations
       # No migration yet
-      migrations.size.should == 0
+      expect(migrations.size).to eq 0
     end
 
     it 'should be able to find migrations for a ruby plugin' do
       migrations = KPM::Migrations.new('master', nil, 'killbill/killbill-cybersource-plugin', ENV['TOKEN']).migrations
       # No migration yet
-      migrations.size.should == 1
+      expect(migrations.size).to eq 1
     end
   end
 
@@ -20,19 +21,19 @@ describe KPM::Migrations, :skip_me_if_nil => ENV['TOKEN'].nil? do
     it 'should be able to find migrations between two versions' do
       migrations = KPM::Migrations.new('killbill-0.16.3', 'killbill-0.16.4', 'killbill/killbill', ENV['TOKEN']).migrations
 
-      migrations.size.should == 1
-      migrations.first[:name].should == 'V20160324060345__revisit_payment_methods_indexes_509.sql'
-      migrations.first[:sql].should == "drop index payment_methods_active_accnt on payment_methods;\n"
+      expect(migrations.size).to eq 1
+      expect(migrations.first[:name]).to eq 'V20160324060345__revisit_payment_methods_indexes_509.sql'
+      expect(migrations.first[:sql]).to eq "drop index payment_methods_active_accnt on payment_methods;\n"
 
-      KPM::Migrations.new('master', 'master', 'killbill/killbill', ENV['TOKEN']).migrations.size.should == 0
+      expect(KPM::Migrations.new('master', 'master', 'killbill/killbill', ENV['TOKEN']).migrations.size).to eq 0
     end
 
     it 'should be able to find migrations for a given version' do
       migrations = KPM::Migrations.new('killbill-0.16.4', nil, 'killbill/killbill', ENV['TOKEN']).migrations
 
-      migrations.size.should == 1
-      migrations.first[:name].should == 'V20160324060345__revisit_payment_methods_indexes_509.sql'
-      migrations.first[:sql].should == "drop index payment_methods_active_accnt on payment_methods;\n"
+      expect(migrations.size).to eq 1
+      expect(migrations.first[:name]).to eq 'V20160324060345__revisit_payment_methods_indexes_509.sql'
+      expect(migrations.first[:sql]).to eq "drop index payment_methods_active_accnt on payment_methods;\n"
     end
   end
 end

data/spec/kpm/remote/nexus_facade_spec.rb

@@ -1,60 +1,64 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 require 'rexml/document'
 
 describe KPM::NexusFacade do
-
-  let(:coordinates_map){ {:version  => '0.1.4',
-                      :group_id => 'org.kill-bill.billing',
-                      :artifact_id => 'killbill-platform-osgi-api',
-                      :packaging => 'jar',
-                      :classifier => nil} }
-  let(:coordinates_with_classifier_map){ {:version  => '0.1.1',
-                      :group_id => 'org.kill-bill.billing',
-                      :artifact_id => 'killbill-platform-osgi-bundles-jruby',
-                      :packaging => 'jar',
-                      :classifier => 'javadoc'} }
-  let(:coordinates) { KPM::Coordinates.build_coordinates(coordinates_map)}
-  let(:coordinates_with_classifier) { KPM::Coordinates.build_coordinates(coordinates_with_classifier_map)}
-  let(:nexus_remote) { described_class::RemoteFactory.create(nil, true)}
+  let(:coordinates_map) do
+    { version: '0.1.4',
+      group_id: 'org.kill-bill.billing',
+      artifact_id: 'killbill-platform-osgi-api',
+      packaging: 'jar',
+      classifier: nil }
+  end
+  let(:coordinates_with_classifier_map) do
+    { version: '0.1.1',
+      group_id: 'org.kill-bill.billing',
+      artifact_id: 'killbill-platform-osgi-bundles-jruby',
+      packaging: 'jar',
+      classifier: 'javadoc' }
+  end
+  let(:coordinates) { KPM::Coordinates.build_coordinates(coordinates_map) }
+  let(:coordinates_with_classifier) { KPM::Coordinates.build_coordinates(coordinates_with_classifier_map) }
+  let(:nexus_remote) { described_class::RemoteFactory.create(nil, true) }
 
   it 'when searching for artifacts' do
     response = nil
-    expect{  response = nexus_remote.search_for_artifacts(coordinates) }.not_to raise_exception
-    expect(REXML::Document.new(response).elements["//artifactId"].text).to eq(coordinates_map[:artifact_id])
+    expect { response = nexus_remote.search_for_artifacts(coordinates) }.not_to raise_exception
+    expect(REXML::Document.new(response).elements['//artifactId'].text).to eq(coordinates_map[:artifact_id])
   end
 
   it 'when searching for artifact with classifier' do
     response = nil
-    expect{  response = nexus_remote.search_for_artifacts(coordinates_with_classifier) }.not_to raise_exception
-    expect(REXML::Document.new(response).elements["//artifactId"].text).to eq(coordinates_with_classifier_map[:artifact_id])
+    expect { response = nexus_remote.search_for_artifacts(coordinates_with_classifier) }.not_to raise_exception
+    expect(REXML::Document.new(response).elements['//artifactId'].text).to eq(coordinates_with_classifier_map[:artifact_id])
   end
 
   it 'when getting artifact info' do
     response = nil
-    expect{  response = nexus_remote.get_artifact_info(coordinates) }.not_to raise_exception
-    expect(REXML::Document.new(response).elements["//version"].text).to eq(coordinates_map[:version])
+    expect { response = nexus_remote.get_artifact_info(coordinates) }.not_to raise_exception
+    expect(REXML::Document.new(response).elements['//version'].text).to eq(coordinates_map[:version])
   end
 
   it 'when getting artifact info with classifier' do
     response = nil
-    expect{  response = nexus_remote.get_artifact_info(coordinates_with_classifier) }.not_to raise_exception
-    expect(REXML::Document.new(response).elements["//version"].text).to eq(coordinates_with_classifier_map[:version])
+    expect { response = nexus_remote.get_artifact_info(coordinates_with_classifier) }.not_to raise_exception
+    expect(REXML::Document.new(response).elements['//version'].text).to eq(coordinates_with_classifier_map[:version])
   end
 
   it 'when pull artifact' do
     response = nil
     destination = Dir.mktmpdir('artifact')
-    expect{  response = nexus_remote.pull_artifact(coordinates,destination) }.not_to raise_exception
+    expect { response = nexus_remote.pull_artifact(coordinates, destination) }.not_to raise_exception
     destination = File.join(File.expand_path(destination), response[:file_name])
-    expect(File.exist?(destination)).to be_true
+    expect(File.exist?(destination)).to be_truthy
   end
 
   it 'when pull artifact with classifier' do
     response = nil
     destination = Dir.mktmpdir('artifact')
-    expect{  response = nexus_remote.pull_artifact(coordinates_with_classifier,destination) }.not_to raise_exception
+    expect { response = nexus_remote.pull_artifact(coordinates_with_classifier, destination) }.not_to raise_exception
     destination = File.join(File.expand_path(destination), response[:file_name])
-    expect(File.exist?(destination)).to be_true
+    expect(File.exist?(destination)).to be_truthy
   end
-
-end
+end

data/spec/kpm/remote/tenant_config_spec.rb

@@ -1,58 +1,59 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe KPM::TenantConfig do
   include_context 'connection_setup'
 
-  let(:value) {"<?xml version=\"1.0\" encoding=\"UTF-8\" standalone=\"yes\"?>\n<catalog>\n    <effectiveDate>2017-04-25T15:57:43Z</effectiveDate>\n    <catalogName>DEFAULT</catalogName>\n    <recurringBillingMode>IN_ADVANCE</recurringBillingMode>\n    <currencies/>\n    <units/>\n    <products/>\n    <rules>\n        <changePolicy>\n            <changePolicyCase>\n                <policy>IMMEDIATE</policy>\n            </changePolicyCase>\n        </changePolicy>\n        <changeAlignment>\n            <changeAlignmentCase>\n                <alignment>START_OF_BUNDLE</alignment>\n            </changeAlignmentCase>\n        </changeAlignment>\n        <cancelPolicy>\n            <cancelPolicyCase>\n                <policy>IMMEDIATE</policy>\n            </cancelPolicyCase>\n        </cancelPolicy>\n        <createAlignment>\n            <createAlignmentCase>\n                <alignment>START_OF_BUNDLE</alignment>\n            </createAlignmentCase>\n        </createAlignment>\n        <billingAlignment>\n            <billingAlignmentCase>\n                <alignment>ACCOUNT</alignment>\n            </billingAlignmentCase>\n        </billingAlignment>\n        <priceList>\n            <priceListCase>\n                <toPriceList>DEFAULT</toPriceList>\n            </priceListCase>\n        </priceList>\n    </rules>\n    <plans/>\n    <priceLists>\n        <defaultPriceList name=\"DEFAULT\">\n            <plans/>\n        </defaultPriceList>\n    </priceLists>\n</catalog>\n"}
-  let(:key) {'CATALOG_RSPEC'}
-
-  let(:user) {'KPM Tenant Spec'}
-  let(:tenant_config_class) { described_class.new([killbill_api_key,killbill_api_secrets],
-                                             [killbill_user, killbill_password],url,logger)}
-  let(:options){{
-        :username => killbill_user,
-        :password => killbill_password,
-        :api_key => killbill_api_key,
-        :api_secret => killbill_api_secrets
-      }}
-                                             
+  let(:value) { "<?xml version=\"1.0\" encoding=\"UTF-8\" standalone=\"yes\"?>\n<catalog>\n    <effectiveDate>2017-04-25T15:57:43Z</effectiveDate>\n    <catalogName>DEFAULT</catalogName>\n    <recurringBillingMode>IN_ADVANCE</recurringBillingMode>\n    <currencies/>\n    <units/>\n    <products/>\n    <rules>\n        <changePolicy>\n            <changePolicyCase>\n                <policy>IMMEDIATE</policy>\n            </changePolicyCase>\n        </changePolicy>\n        <changeAlignment>\n            <changeAlignmentCase>\n                <alignment>START_OF_BUNDLE</alignment>\n            </changeAlignmentCase>\n        </changeAlignment>\n        <cancelPolicy>\n            <cancelPolicyCase>\n                <policy>IMMEDIATE</policy>\n            </cancelPolicyCase>\n        </cancelPolicy>\n        <createAlignment>\n            <createAlignmentCase>\n                <alignment>START_OF_BUNDLE</alignment>\n            </createAlignmentCase>\n        </createAlignment>\n        <billingAlignment>\n            <billingAlignmentCase>\n                <alignment>ACCOUNT</alignment>\n            </billingAlignmentCase>\n        </billingAlignment>\n        <priceList>\n            <priceListCase>\n                <toPriceList>DEFAULT</toPriceList>\n            </priceListCase>\n        </priceList>\n    </rules>\n    <plans/>\n    <priceLists>\n        <defaultPriceList name=\"DEFAULT\">\n            <plans/>\n        </defaultPriceList>\n    </priceLists>\n</catalog>\n" }
+  let(:key) { 'CATALOG_RSPEC' }
+
+  let(:user) { 'KPM Tenant Spec' }
+  let(:tenant_config_class) do
+    described_class.new([killbill_api_key, killbill_api_secret],
+                        [killbill_user, killbill_password], url, logger)
+  end
+  let(:options) do
+    {
+      username: killbill_user,
+      password: killbill_password,
+      api_key: killbill_api_key,
+      api_secret: killbill_api_secret
+    }
+  end
+
   describe '#initialize' do
     context 'when creating an instance of tenant config class' do
-
       it 'when initialized with defaults' do
         expect(described_class.new).to be_an_instance_of(KPM::TenantConfig)
       end
 
       it 'when initialized with options' do
-        tenant_config_class.should be_an_instance_of(KPM::TenantConfig)
+        expect(tenant_config_class).to be_an_instance_of(KPM::TenantConfig)
         expect(tenant_config_class.instance_variable_get(:@killbill_api_key)).to eq(killbill_api_key)
-        expect(tenant_config_class.instance_variable_get(:@killbill_api_secrets)).to eq(killbill_api_secrets)
+        expect(tenant_config_class.instance_variable_get(:@killbill_api_secret)).to eq(killbill_api_secret)
         expect(tenant_config_class.instance_variable_get(:@killbill_user)).to eq(killbill_user)
         expect(tenant_config_class.instance_variable_get(:@killbill_password)).to eq(killbill_password)
         expect(tenant_config_class.instance_variable_get(:@killbill_url)).to eq(url)
-
       end
-
     end
+  end
 
-  end                                           
-  
   describe '#export' do
     it 'when retrieving tenant configuration' do
       KillBillClient.url = url
 
-      #Add a new tenant config
+      # Add a new tenant config
       tenant_config = KillBillClient::Model::Tenant.upload_tenant_user_key_value(key, value, user, nil, nil, options)
       expect(tenant_config.key).to eq(key)
-      
-      #get created tenant config
+
+      # get created tenant config
       export_file = tenant_config_class.export(key)
-      expect(File.exist?(export_file)).to be_true
-      expect(File.readlines(export_file).grep(/#{key}/)).to be_true
-      
-      #remove created tenant config
+      expect(File.exist?(export_file)).to be_truthy
+      expect(File.readlines(export_file).grep(/#{key}/)).to be_truthy
+
+      # remove created tenant config
       KillBillClient::Model::Tenant.delete_tenant_user_key_value(key, user, nil, nil, options)
-      
     end
   end
-end
+end

data/spec/kpm/remote/tomcat_manager_spec.rb

@@ -1,7 +1,8 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe KPM::TomcatManager do
-
   before(:all) do
     @logger = Logger.new(STDOUT)
     @logger.level = Logger::INFO
@@ -12,10 +13,10 @@ describe KPM::TomcatManager do
       manager = KPM::TomcatManager.new(dir, @logger)
 
       tomcat_path = manager.download
-      tomcat_path.should_not be_nil
+      expect(tomcat_path).not_to be_nil
 
       root_war_path = manager.setup
-      root_war_path.should_not be_nil
+      expect(root_war_path).not_to be_nil
     end
   end
 end

data/spec/kpm/unit/actions_spec.rb

@@ -0,0 +1,52 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe KPM::NexusFacade::Actions do
+  subject { described_class.new({}, nil, logger) }
+  let(:logger) { Logger.new(STDOUT) }
+  let(:nexus_mock) { double(KPM::NexusFacade::NexusApiCallsV2) }
+
+  before do
+    allow(KPM::NexusFacade::NexusApiCallsV2).to receive(:new).and_return(nexus_mock)
+  end
+
+  context 'when Nexus throws a non-retryable exception' do
+    it 'never retries' do
+      calls = 0
+      expect do
+        subject.send(:retry_exceptions, 'foo') do
+          calls += 1
+          raise StandardError, '404'
+        end
+      end.to raise_error(StandardError)
+      expect(calls).to eq(1)
+    end
+  end
+
+  context 'when Nexus throws a retryable exception' do
+    it 'retries until giving up' do
+      calls = 0
+      expect do
+        subject.send(:retry_exceptions, 'foo') do
+          calls += 1
+          raise KPM::NexusFacade::UnexpectedStatusCodeException, 503
+        end
+      end.to raise_error(StandardError)
+      expect(calls).to eq(3)
+    end
+  end
+
+  context 'when networking is flaky' do
+    it 'retries until call succeeds' do
+      calls = 0
+      expect(subject.send(:retry_exceptions, 'foo') do
+        calls += 1
+        raise OpenSSL::SSL::SSLErrorWaitReadable if calls < 2
+
+        true
+      end).to be_truthy
+      expect(calls).to eq(2)
+    end
+  end
+end

data/spec/kpm/unit/base_artifact_spec.rb

@@ -1,7 +1,8 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe KPM::BaseArtifact do
-
   before(:all) do
     @logger = Logger.new(STDOUT)
     @logger.level = Logger::INFO
@@ -13,16 +14,16 @@ describe KPM::BaseArtifact do
     Dir.mktmpdir do |dir|
       info = KPM::BaseArtifact.pull_from_fs(@logger, file_path, dir)
 
-      info[:skipped].should be_false
-      info[:is_tgz].should be_false
-      info[:repository_path].should == file_path
-      info[:dir_name].should == dir
-      info[:bundle_dir].should == dir
-      info[:file_name].should == 'sha1_test.yml'
+      expect(info[:skipped]).to be_falsey
+      expect(info[:is_tgz]).to be_falsey
+      expect(info[:repository_path]).to eq file_path
+      expect(info[:dir_name]).to eq dir
+      expect(info[:bundle_dir]).to eq dir
+      expect(info[:file_name]).to eq 'sha1_test.yml'
 
       files_in_dir = Dir[dir + '/*']
-      files_in_dir.size.should == 1
-      files_in_dir[0].should == info[:file_path]
+      expect(files_in_dir.size).to eq 1
+      expect(files_in_dir[0]).to eq info[:file_path]
     end
   end
 
@@ -75,18 +76,18 @@ describe KPM::BaseArtifact do
 
   def check_fs_info(specified_destination_path, repository_path, is_tgz, version, expected_dir_name, expected_file_name, expected_file_path)
     info = {
-        :repository_path => repository_path,
-        :is_tgz => is_tgz,
-        :version => version
+      repository_path: repository_path,
+      is_tgz: is_tgz,
+      version: version
     }
 
     KPM::BaseArtifact.send('populate_fs_info', info, specified_destination_path)
 
-    info[:repository_path].should == repository_path
-    info[:is_tgz].should == is_tgz
-    info[:version].should == version
-    info[:dir_name].should == expected_dir_name
-    info[:file_name].should == expected_file_name
-    info[:file_path].should == expected_file_path
+    expect(info[:repository_path]).to eq repository_path
+    expect(info[:is_tgz]).to eq is_tgz
+    expect(info[:version]).to eq version
+    expect(info[:dir_name]).to eq expected_dir_name
+    expect(info[:file_name]).to eq expected_file_name
+    expect(info[:file_path]).to eq expected_file_path
   end
 end

data/spec/kpm/unit/cpu_information_spec.rb

@@ -0,0 +1,67 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'kpm/system_helpers/system_proxy'
+require 'kpm/system_helpers/cpu_information'
+
+describe KPM::SystemProxy::CpuInformation do
+  subject { described_class.new }
+  let(:cpu_info) { subject.send(:build_hash, data) }
+
+  context 'when running on Linux' do
+    let(:data) { "processor: 0\nvendor_id: GenuineIntel\ncpu family: 6\nmodel: 78\nmodel name: Intel(R) Core(TM) i5-6287U CPU @ 3.10GHz\nstepping: 3\ncpu MHz: 3096.000\ncache size: 4096 KB\nphysical id: 0\nsiblings: 2\ncore id: 0\ncpu cores: 2\napicid: 0\ninitial apicid: 0\nfpu: yes\nfpu_exception: yes\ncpuid level: 22\nwp: yes\nflags: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx rdtscp lm constant_tsc rep_good nopl xtopology nonstop_tsc pni pclmulqdq ssse3 cx16 pcid sse4_1 sse4_2 movbe popcnt aes xsave avx rdrand hypervisor lahf_lm abm 3dnowprefetch fsgsbase avx2 invpcid rdseed clflushopt\nbugs:\nbogomips: 6192.00\nclflush size: 64\ncache_alignment: 64\naddress sizes: 39 bits physical, 48 bits virtual\npower management:\n\nprocessor: 1\nvendor_id: GenuineIntel\ncpu family: 6\nmodel: 78\nmodel name: Intel(R) Core(TM) i5-6287U CPU @ 3.10GHz\nstepping: 3\ncpu MHz: 3096.000\ncache size: 4096 KB\nphysical id: 0\nsiblings: 2\ncore id: 1\ncpu cores: 2\napicid: 1\ninitial apicid: 1\nfpu: yes\nfpu_exception: yes\ncpuid level: 22\nwp: yes\nflags: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx rdtscp lm constant_tsc rep_good nopl xtopology nonstop_tsc pni pclmulqdq ssse3 cx16 pcid sse4_1 sse4_2 movbe popcnt aes xsave avx rdrand hypervisor lahf_lm abm 3dnowprefetch fsgsbase avx2 invpcid rdseed clflushopt\nbugs:\nbogomips: 6192.00\nclflush size: 64\ncache_alignment: 64\naddress sizes: 39 bits physical, 48 bits virtual\npower management:\n\n" }
+
+    it {
+      expect(subject.labels).to eq([{ label: :cpu_detail },
+                                    { label: :value }])
+    }
+
+    it {
+      expect(cpu_info).to eq({ 'processor' => { cpu_detail: 'processor', value: '1' },
+                               'vendor_id' => { cpu_detail: 'vendor_id', value: 'GenuineIntel' },
+                               'cpu family' => { cpu_detail: 'cpu family', value: '6' },
+                               'model' => { cpu_detail: 'model', value: '78' },
+                               'model name' =>
+                                 { cpu_detail: 'model name',
+                                   value: 'Intel(R) Core(TM) i5-6287U CPU @ 3.10GHz' },
+                               'stepping' => { cpu_detail: 'stepping', value: '3' },
+                               'cpu MHz' => { cpu_detail: 'cpu MHz', value: '3096.000' },
+                               'cache size' => { cpu_detail: 'cache size', value: '4096 KB' },
+                               'physical id' => { cpu_detail: 'physical id', value: '0' },
+                               'siblings' => { cpu_detail: 'siblings', value: '2' },
+                               'core id' => { cpu_detail: 'core id', value: '1' },
+                               'cpu cores' => { cpu_detail: 'cpu cores', value: '2' },
+                               'apicid' => { cpu_detail: 'apicid', value: '1' },
+                               'initial apicid' => { cpu_detail: 'initial apicid', value: '1' },
+                               'fpu' => { cpu_detail: 'fpu', value: 'yes' },
+                               'fpu_exception' => { cpu_detail: 'fpu_exception', value: 'yes' },
+                               'cpuid level' => { cpu_detail: 'cpuid level', value: '22' },
+                               'wp' => { cpu_detail: 'wp', value: 'yes' },
+                               'bugs' => { cpu_detail: 'bugs', value: '' },
+                               'bogomips' => { cpu_detail: 'bogomips', value: '6192.00' },
+                               'clflush size' => { cpu_detail: 'clflush size', value: '64' },
+                               'cache_alignment' => { cpu_detail: 'cache_alignment', value: '64' },
+                               'address sizes' =>
+                                 { cpu_detail: 'address sizes', value: '39 bits physical, 48 bits virtual' },
+                               'power management' => { cpu_detail: 'power management', value: '' } })
+    }
+  end
+
+  context 'when running on MacOS' do
+    let(:data) { "      Processor Name: Intel Core i5\n      Processor Speed: 3.1 GHz\n      Number of Processors: 1\n      Total Number of Cores: 2\n      L2 Cache (per Core): 256 KB\n      L3 Cache: 4 MB\n" }
+
+    it {
+      expect(subject.labels).to eq([{ label: :cpu_detail },
+                                    { label: :value }])
+    }
+
+    it {
+      expect(cpu_info).to eq({ 'Processor Name' => { cpu_detail: 'Processor Name', value: 'Intel Core i5' },
+                               'Processor Speed' => { cpu_detail: 'Processor Speed', value: '3.1 GHz' },
+                               'Number of Processors' => { cpu_detail: 'Number of Processors', value: '1' },
+                               'Total Number of Cores' => { cpu_detail: 'Total Number of Cores', value: '2' },
+                               'L2 Cache (per Core)' => { cpu_detail: 'L2 Cache (per Core)', value: '256 KB' },
+                               'L3 Cache' => { cpu_detail: 'L3 Cache', value: '4 MB' } })
+    }
+  end
+end

data/spec/kpm/unit/disk_space_information_spec.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'kpm/system_helpers/system_proxy'
+require 'kpm/system_helpers/disk_space_information'
+
+describe KPM::SystemProxy::DiskSpaceInformation do
+  subject { described_class.new }
+  let(:data_keys) { [] }
+  let!(:disk_space_info) { subject.send(:build_hash, data, cols_count, true, data_keys) }
+
+  context 'when running on Linux' do
+    let(:cols_count) { 5 }
+    let(:data) { "Filesystem     1K-blocks     Used Available Use% Mounted on\nnone            58419028 24656532  30723884  45% /\ntmpfs              65536        0     65536   0% /dev\ntmpfs            5387012        0   5387012   0% /sys/fs/cgroup\n/dev/sda1       58419028 24656532  30723884  45% /etc/hosts\nshm                65536        0     65536   0% /dev/shm\ntmpfs            5387012        0   5387012   0% /sys/firmware\n" }
+
+    it {
+      expect(data_keys).to eq(['Filesystem', '1K-blocks', 'Used', 'Available', 'Use%', 'Mounted on'])
+    }
+
+    it {
+      expect(disk_space_info).to eq({ 'DiskInfo_2' => { :Filesystem => 'none', :"1K-blocks" => '58419028', :Used => '24656532', :Available => '30723884', :"Use%" => '45%', :Mounted_on => '/' },
+                                      'DiskInfo_3' => { :Filesystem => 'tmpfs', :"1K-blocks" => '65536', :Used => '0', :Available => '65536', :"Use%" => '0%', :Mounted_on => '/dev' },
+                                      'DiskInfo_4' => { :Filesystem => 'tmpfs', :"1K-blocks" => '5387012', :Used => '0', :Available => '5387012', :"Use%" => '0%', :Mounted_on => '/sys/fs/cgroup' },
+                                      'DiskInfo_5' => { :Filesystem => '/dev/sda1', :"1K-blocks" => '58419028', :Used => '24656532', :Available => '30723884', :"Use%" => '45%', :Mounted_on => '/etc/hosts' },
+                                      'DiskInfo_6' => { :Filesystem => 'shm', :"1K-blocks" => '65536', :Used => '0', :Available => '65536', :"Use%" => '0%', :Mounted_on => '/dev/shm' },
+                                      'DiskInfo_7' => { :Filesystem => 'tmpfs', :"1K-blocks" => '5387012', :Used => '0', :Available => '5387012', :"Use%" => '0%', :Mounted_on => '/sys/firmware' } })
+    }
+  end
+
+  context 'when running on MacOS' do
+    let(:cols_count) { 8 }
+    let(:data) { "Filesystem    512-blocks      Used Available Capacity iused               ifree %iused  Mounted on\n/dev/disk1s1   976490576 778131600 173031648    82% 2431747 9223372036852344060    0%   /\ndevfs                690       690         0   100%    1194                   0  100%   /dev\n/dev/disk1s4   976490576  23925200 173031648    13%       5 9223372036854775802    0%   /private/var/vm\nmap -hosts             0         0         0   100%       0                   0  100%   /net\nmap auto_home          0         0         0   100%       0                   0  100%   /home\n/dev/disk1s3   976490576    996584 173031648     1%      34 9223372036854775773    0%   /Volumes/Recovery\n" }
+
+    it {
+      expect(data_keys).to eq(['Filesystem', '512-blocks', 'Used', 'Available', 'Capacity', 'iused', 'ifree', '%iused', 'Mounted on'])
+    }
+
+    it {
+      expect(disk_space_info).to eq({ 'DiskInfo_2' => { :Filesystem => '/dev/disk1s1', :"512-blocks" => '976490576', :Used => '778131600', :Available => '173031648', :Capacity => '82%', :iused => '2431747', :ifree => '9223372036852344060', :"%iused" => '0%', :Mounted_on => '/' },
+                                      'DiskInfo_3' => { :Filesystem => 'devfs', :"512-blocks" => '690', :Used => '690', :Available => '0', :Capacity => '100%', :iused => '1194', :ifree => '0', :"%iused" => '100%', :Mounted_on => '/dev' },
+                                      'DiskInfo_4' => { :Filesystem => '/dev/disk1s4', :"512-blocks" => '976490576', :Used => '23925200', :Available => '173031648', :Capacity => '13%', :iused => '5', :ifree => '9223372036854775802', :"%iused" => '0%', :Mounted_on => '/private/var/vm' },
+                                      'DiskInfo_5' => { :Filesystem => 'map', :"512-blocks" => '-hosts', :Used => '0', :Available => '0', :Capacity => '0', :iused => '100%', :ifree => '0', :"%iused" => '0', :Mounted_on => '100% /net ' },
+                                      'DiskInfo_6' => { :Filesystem => 'map', :"512-blocks" => 'auto_home', :Used => '0', :Available => '0', :Capacity => '0', :iused => '100%', :ifree => '0', :"%iused" => '0', :Mounted_on => '100% /home ' },
+                                      'DiskInfo_7' => { :Filesystem => '/dev/disk1s3', :"512-blocks" => '976490576', :Used => '996584', :Available => '173031648', :Capacity => '1%', :iused => '34', :ifree => '9223372036854775773', :"%iused" => '0%', :Mounted_on => '/Volumes/Recovery' } })
+    }
+  end
+end