kpm 0.8.2 → 0.9.0

data/lib/kpm/tasks.rb

@@ -483,7 +483,7 @@ module KPM
         desc 'account', 'export/import accounts'
         def account
           config_file = nil
-          raise Interrupt, '--killbill_url, required format -> http(s)://something' if options[:killbill_url] && %r{https?://[\S]+}.match(options[:killbill_url]).nil?
+          raise Interrupt, '--killbill_url, required format -> http(s)://something' if options[:killbill_url] && %r{https?://\S+}.match(options[:killbill_url]).nil?
 
           raise Interrupt, '--killbill_api_credentials, required format -> <api_key> <api_secrets>' if options[:killbill_api_credentials] && options[:killbill_api_credentials].size != 2
 
@@ -535,7 +535,7 @@ module KPM
                       desc: 'Killbill URL ex. http://127.0.0.1:8080'
         desc 'tenant_config', 'export all tenant-level configs.'
         def tenant_config
-          raise Interrupt, '--killbill_url, required format -> http(s)://something' if options[:killbill_url] && %r{https?://[\S]+}.match(options[:killbill_url]).nil?
+          raise Interrupt, '--killbill_url, required format -> http(s)://something' if options[:killbill_url] && %r{https?://\S+}.match(options[:killbill_url]).nil?
 
           raise Interrupt, '--killbill_api_credentials, required format -> <api_key> <api_secrets>' if options[:killbill_api_credentials] && options[:killbill_api_credentials].size != 2
 
@@ -608,7 +608,7 @@ module KPM
         def diagnostic
           raise Interrupt, '--account_export,  please provide a valid account id' if options[:account_export] && options[:account_export] == 'account_export'
 
-          raise Interrupt, '--killbill_url, required format -> http(s)://something' if options[:killbill_url] && %r{https?://[\S]+}.match(options[:killbill_url]).nil?
+          raise Interrupt, '--killbill_url, required format -> http(s)://something' if options[:killbill_url] && %r{https?://\S+}.match(options[:killbill_url]).nil?
 
           raise Interrupt, '--killbill_api_credentials, required format -> <api_key> <api_secrets>' if options[:killbill_api_credentials] && options[:killbill_api_credentials].size != 2
 

data/lib/kpm/tenant_config.rb

@@ -81,7 +81,7 @@ module KPM
       }
 
       begin
-        return KillBillClient::Model::Tenant.search_tenant_config(key_prefix, options)
+        KillBillClient::Model::Tenant.search_tenant_config(key_prefix, options)
       rescue KillBillClient::API::Unauthorized
         raise ArgumentError, "Unable to export tenant details, wrong credentials? username=#{@killbill_user}, password=#{mask(@killbill_password)}, api_key=#{@killbill_api_key}, api_secret=#{mask(@killbill_api_secret)}"
       end

data/lib/kpm/tomcat_manager.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require 'pathname'
 require 'net/http'
 require 'uri'
 

data/lib/kpm/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module KPM
-  VERSION = '0.8.2'
+  VERSION = '0.9.0'
 end

data/pom.xml

@@ -26,7 +26,7 @@
   <groupId>org.kill-bill.billing.installer</groupId>
   <artifactId>kpm</artifactId>
   <packaging>pom</packaging>
-  <version>0.8.2</version>
+  <version>0.9.0</version>
   <name>KPM</name>
   <url>http://github.com/killbill/killbill-cloud</url>
   <description>KPM: the Kill Bill Package Manager</description>

data/spec/kpm/remote/base_artifact_spec.rb

@@ -36,10 +36,12 @@ describe KPM::BaseArtifact do
       # Verify the download is skipped gracefully when Nexus isn't reachable
       test_download dir, 'foo-oss.pom.xml', true, false, sha1_file, nexus_down
       # Verify the download fails when Nexus isn't reachable and force_download is set
-      expect { test_download dir, 'foo-oss.pom.xml', nil, true, sha1_file, nexus_down }.to raise_error
+      expect { test_download dir, 'foo-oss.pom.xml', nil, true, sha1_file, nexus_down }.to raise_exception(SocketError, /(Failed to open TCP connection to does.not.exist:443|getaddrinfo)/)
       # Verify the download fails when Nexus isn't reachable and the Nexus cache is empty
-      KPM::Sha1Checker.from_file(sha1_file).cache_artifact_info('org.kill-bill.billing:killbill-oss-parent:pom:0.143.33', nil)
-      expect { test_download dir, 'foo-oss.pom.xml', nil, false, sha1_file, nexus_down }.to raise_error
+      expect(KPM::Sha1Checker.from_file(sha1_file).artifact_info('org.kill-bill.billing:killbill-oss-parent:pom:0.143.33')).to_not be_nil
+      KPM::Sha1Checker.from_file(sha1_file).remove_entry!('org.kill-bill.billing:killbill-oss-parent:pom:0.143.33')
+      expect(KPM::Sha1Checker.from_file(sha1_file).artifact_info('org.kill-bill.billing:killbill-oss-parent:pom:0.143.33')).to be_nil
+      expect { test_download dir, 'foo-oss.pom.xml', nil, false, sha1_file, nexus_down }.to raise_exception(SocketError, /(Failed to open TCP connection to does.not.exist:443|getaddrinfo)/)
     end
   end
 
@@ -53,14 +55,14 @@ describe KPM::BaseArtifact do
 
     Dir.mktmpdir do |dir|
       info = KPM::BaseArtifact.pull(@logger, group_id, artifact_id, packaging, classifier, version, dir)
-      info[:file_name].should be_nil
+      expect(info[:file_name]).to be_nil
 
       files_in_dir = Dir[info[:file_path] + '/*']
-      files_in_dir.size.should eq 20
+      expect(files_in_dir.size).to eq 20
 
-      File.file?(info[:file_path] + '/killbill-osgi-bundles-jruby-0.11.3.jar').should be_true
+      expect(File.file?(info[:file_path] + '/killbill-osgi-bundles-jruby-0.11.3.jar')).to be_truthy
 
-      info[:bundle_dir].should eq info[:file_path]
+      expect(info[:bundle_dir]).to eq info[:file_path]
     end
   end
 
@@ -75,11 +77,11 @@ describe KPM::BaseArtifact do
 
     Dir.mktmpdir do |dir|
       first_take = KPM::BaseArtifact.pull(@logger, group_id, artifact_id, packaging, classifier, version, dir)
-      File.file?(first_take[:file_path] + '/killbill-platform-osgi-bundles-jruby-0.36.2.jar').should be_true
+      expect(File.file?(first_take[:file_path] + '/killbill-platform-osgi-bundles-jruby-0.36.2.jar')).to be_truthy
 
       second_take = KPM::BaseArtifact.pull(@logger, group_id, artifact_id, packaging, classifier, second_bundle_version, dir)
-      File.file?(first_take[:file_path] + '/killbill-platform-osgi-bundles-jruby-0.36.2.jar').should be_false
-      File.file?(second_take[:file_path] + '/killbill-platform-osgi-bundles-jruby-0.36.10.jar').should be_true
+      expect(File.file?(first_take[:file_path] + '/killbill-platform-osgi-bundles-jruby-0.36.2.jar')).to be_falsey
+      expect(File.file?(second_take[:file_path] + '/killbill-platform-osgi-bundles-jruby-0.36.10.jar')).to be_truthy
     end
   end
 
@@ -87,8 +89,8 @@ describe KPM::BaseArtifact do
     path = filename.nil? ? dir : dir + '/' + filename
 
     info = KPM::BaseArtifact.pull(@logger, 'org.kill-bill.billing', 'killbill-oss-parent', 'pom', nil, '0.143.33', path, sha1_file, force_download, true, overrides, true)
-    info[:file_name].should eq(filename.nil? ? "killbill-oss-parent-#{info[:version]}.pom" : filename)
-    info[:skipped].should eq verify_is_skipped
-    info[:size].should eq File.size(info[:file_path]) unless info[:skipped]
+    expect(info[:file_name]).to eq(filename.nil? ? "killbill-oss-parent-#{info[:version]}.pom" : filename)
+    expect(info[:skipped]).to eq verify_is_skipped
+    expect(info[:size]).to eq File.size(info[:file_path]) unless info[:skipped]
   end
 end

data/spec/kpm/remote/base_installer_spec.rb

@@ -65,7 +65,7 @@ describe KPM::BaseInstaller do
       { file_path: '/Somewhere/xxx-foo/target/xxx-foo-abc-SNAPSHOT.jar', expected: 'xxx-foo' },
       { file_path: '/Somewhere/xxx-foo/target/xxx-foo-abc.jar', expected: 'xxx-foo' }
     ].each do |test|
-      KPM::Utils.get_plugin_name_from_file_path(test[:file_path]).should eq test[:expected]
+      expect(KPM::Utils.get_plugin_name_from_file_path(test[:file_path])).to eq test[:expected]
     end
   end
 
@@ -76,16 +76,16 @@ describe KPM::BaseInstaller do
 
     plugin_identifiers = read_plugin_identifiers(plugins_dir)
 
-    plugin_identifiers.size.should eq 1
+    expect(plugin_identifiers.size).to eq 1
 
-    plugin_identifiers['analytics']['plugin_name'].should eq 'analytics-plugin'
-    plugin_identifiers['analytics']['group_id'].should eq 'org.kill-bill.billing.plugin.java'
-    plugin_identifiers['analytics']['artifact_id'].should eq 'analytics-plugin'
-    plugin_identifiers['analytics']['packaging'].should eq 'jar'
-    plugin_identifiers['analytics']['version'].should eq '0.7.1'
-    plugin_identifiers['analytics']['language'].should eq 'java'
+    expect(plugin_identifiers['analytics']['plugin_name']).to eq 'analytics-plugin'
+    expect(plugin_identifiers['analytics']['group_id']).to eq 'org.kill-bill.billing.plugin.java'
+    expect(plugin_identifiers['analytics']['artifact_id']).to eq 'analytics-plugin'
+    expect(plugin_identifiers['analytics']['packaging']).to eq 'jar'
+    expect(plugin_identifiers['analytics']['version']).to eq '0.7.1'
+    expect(plugin_identifiers['analytics']['language']).to eq 'java'
 
-    File.file?(plugins_dir + '/plugins/java/analytics-plugin/0.7.1/tmp/disabled.txt').should be_false
+    expect(File.file?(plugins_dir + '/plugins/java/analytics-plugin/0.7.1/tmp/disabled.txt')).to be_falsey
   end
 
   def check_uninstallation(plugins_dir)
@@ -93,9 +93,9 @@ describe KPM::BaseInstaller do
 
     plugin_identifiers = read_plugin_identifiers(plugins_dir)
 
-    plugin_identifiers.size.should eq 0
+    expect(plugin_identifiers.size).to eq 0
 
-    File.file?(plugins_dir + '/plugins/java/analytics-plugin/0.7.1/tmp/disabled.txt').should be_true
+    expect(File.file?(plugins_dir + '/plugins/java/analytics-plugin/0.7.1/tmp/disabled.txt')).to be_truthy
   end
 
   def common_checks(plugins_dir)
@@ -107,14 +107,14 @@ describe KPM::BaseInstaller do
       plugins_dir + '/plugins/java/analytics-plugin/0.7.1',
       plugins_dir + '/plugins/java/analytics-plugin/0.7.1/tmp'
     ].each do |dir|
-      File.directory?(dir).should be_true
+      expect(File.directory?(dir)).to be_truthy
     end
 
     [
       plugins_dir + '/plugins/plugin_identifiers.json',
       plugins_dir + '/plugins/java/analytics-plugin/0.7.1/analytics-plugin-0.7.1.jar'
     ].each do |file|
-      File.file?(file).should be_true
+      expect(File.file?(file)).to be_truthy
     end
   end
 

data/spec/kpm/remote/github_api_calls_spec.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'rexml/document'
+
+describe KPM::NexusFacade, skip_me_if_nil: ENV['TOKEN'].nil? do
+  let(:logger) do
+    logger = ::Logger.new(STDOUT)
+    logger.level = Logger::INFO
+    logger
+  end
+  let(:coordinates_map) do
+    { version: '1.1.9',
+      group_id: 'org.kill-bill.billing.thirdparty',
+      artifact_id: 'qualpay-java-client',
+      packaging: 'pom',
+      classifier: nil }
+  end
+  let(:coordinates) { KPM::Coordinates.build_coordinates(coordinates_map) }
+  let(:nexus_remote) { described_class::GithubApiCalls.new({ :url => 'https://maven.pkg.github.com/killbill/qualpay-java-client', :token => ENV['TOKEN'] }, true, logger) }
+
+  it {
+    # Not implemented
+    expect { nexus_remote.search_for_artifacts(coordinates) }.to raise_exception(NoMethodError, 'GitHub Packages has no search support')
+  }
+
+  it {
+    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])
+  }
+
+  it {
+    response = nil
+    destination = Dir.mktmpdir('artifact')
+    expect { response = nexus_remote.pull_artifact(coordinates, destination) }.not_to raise_exception
+    destination = File.join(File.expand_path(destination), response[:file_name])
+    expect(File.read(destination)).to match(/qualpay-java-client/)
+  }
+end

data/spec/kpm/remote/installer_spec.rb

@@ -23,8 +23,8 @@ describe KPM::Installer do
       response = nil
       expect { response = installer.install }.to_not raise_exception
       response = JSON[response]
-      response['help'].should be_nil
-      response['killbill']['status'].should eq 'INSTALLED'
+      expect(response['help']).to be_nil
+      expect(response['killbill']['status']).to eq 'INSTALLED'
     end
   end
 
@@ -42,8 +42,8 @@ describe KPM::Installer do
       response = nil
       expect { response = installer.install }.to_not raise_exception
       response = JSON[response]
-      response['help'].should be_nil
-      response['kaui']['status'].should eq 'INSTALLED'
+      expect(response['help']).to be_nil
+      expect(response['kaui']['status']).to eq 'INSTALLED'
     end
   end
 
@@ -87,10 +87,10 @@ describe KPM::Installer do
 
       # Finally verify that for both (well behaved) ruby and java plugin, skipping the install will still correctly return the `:bundle_dir`
       info = installer.install_plugin('payment-test-plugin', nil, 'org.kill-bill.billing.plugin.ruby', 'payment-test-plugin', nil, nil, '1.8.7', plugins_dir)
-      info[:bundle_dir].should eq plugins_dir + '/plugins/ruby/killbill-payment-test/1.8.7'
+      expect(info[:bundle_dir]).to eq plugins_dir + '/plugins/ruby/killbill-payment-test/1.8.7'
 
       info = installer.install_plugin('analytics', nil, nil, nil, nil, nil, '0.7.1', plugins_dir)
-      info[:bundle_dir].should eq plugins_dir + '/plugins/java/analytics-plugin/0.7.1'
+      expect(info[:bundle_dir]).to eq plugins_dir + '/plugins/java/analytics-plugin/0.7.1'
     end
   end
 
@@ -110,7 +110,7 @@ describe KPM::Installer do
       plugins_dir + '/plugins/ruby/killbill-payment-test',
       plugins_dir + '/plugins/ruby/killbill-payment-test/1.8.7'
     ].each do |dir|
-      File.directory?(dir).should be_true
+      expect(File.directory?(dir)).to be_truthy
     end
 
     [
@@ -122,34 +122,34 @@ describe KPM::Installer do
       plugins_dir + '/plugins/java/stripe-plugin/7.0.0/stripe-plugin-7.0.0.jar',
       plugins_dir + '/plugins/ruby/killbill-payment-test/1.8.7/killbill.properties'
     ].each do |file|
-      File.file?(file).should be_true
+      expect(File.file?(file)).to be_truthy
     end
 
     plugin_identifiers = File.open(plugins_dir + '/plugins/plugin_identifiers.json', 'r') do |f|
       JSON.parse(f.read)
     end
 
-    plugin_identifiers.size.should eq 3
-
-    plugin_identifiers['analytics']['plugin_name'].should eq 'analytics-plugin'
-    plugin_identifiers['analytics']['group_id'].should eq 'org.kill-bill.billing.plugin.java'
-    plugin_identifiers['analytics']['artifact_id'].should eq 'analytics-plugin'
-    plugin_identifiers['analytics']['packaging'].should eq 'jar'
-    plugin_identifiers['analytics']['version'].should eq '0.7.1'
-    plugin_identifiers['analytics']['language'].should eq 'java'
-
-    plugin_identifiers['stripe']['plugin_name'].should eq 'stripe-plugin'
-    plugin_identifiers['stripe']['group_id'].should eq 'org.kill-bill.billing.plugin.java'
-    plugin_identifiers['stripe']['artifact_id'].should eq 'stripe-plugin'
-    plugin_identifiers['stripe']['packaging'].should eq 'jar'
-    plugin_identifiers['stripe']['version'].should eq '7.0.0'
-    plugin_identifiers['stripe']['language'].should eq 'java'
-
-    plugin_identifiers['payment-test-plugin']['plugin_name'].should eq 'killbill-payment-test'
-    plugin_identifiers['payment-test-plugin']['group_id'].should eq 'org.kill-bill.billing.plugin.ruby'
-    plugin_identifiers['payment-test-plugin']['artifact_id'].should eq 'payment-test-plugin'
-    plugin_identifiers['payment-test-plugin']['packaging'].should eq 'tar.gz'
-    plugin_identifiers['payment-test-plugin']['version'].should eq '1.8.7'
-    plugin_identifiers['payment-test-plugin']['language'].should eq 'ruby'
+    expect(plugin_identifiers.size).to eq 3
+
+    expect(plugin_identifiers['analytics']['plugin_name']).to eq 'analytics-plugin'
+    expect(plugin_identifiers['analytics']['group_id']).to eq 'org.kill-bill.billing.plugin.java'
+    expect(plugin_identifiers['analytics']['artifact_id']).to eq 'analytics-plugin'
+    expect(plugin_identifiers['analytics']['packaging']).to eq 'jar'
+    expect(plugin_identifiers['analytics']['version']).to eq '0.7.1'
+    expect(plugin_identifiers['analytics']['language']).to eq 'java'
+
+    expect(plugin_identifiers['stripe']['plugin_name']).to eq 'stripe-plugin'
+    expect(plugin_identifiers['stripe']['group_id']).to eq 'org.kill-bill.billing.plugin.java'
+    expect(plugin_identifiers['stripe']['artifact_id']).to eq 'stripe-plugin'
+    expect(plugin_identifiers['stripe']['packaging']).to eq 'jar'
+    expect(plugin_identifiers['stripe']['version']).to eq '7.0.0'
+    expect(plugin_identifiers['stripe']['language']).to eq 'java'
+
+    expect(plugin_identifiers['payment-test-plugin']['plugin_name']).to eq 'killbill-payment-test'
+    expect(plugin_identifiers['payment-test-plugin']['group_id']).to eq 'org.kill-bill.billing.plugin.ruby'
+    expect(plugin_identifiers['payment-test-plugin']['artifact_id']).to eq 'payment-test-plugin'
+    expect(plugin_identifiers['payment-test-plugin']['packaging']).to eq 'tar.gz'
+    expect(plugin_identifiers['payment-test-plugin']['version']).to eq '1.8.7'
+    expect(plugin_identifiers['payment-test-plugin']['language']).to eq 'ruby'
   end
 end

data/spec/kpm/remote/kaui_artifact_spec.rb

@@ -17,15 +17,15 @@ describe KPM::KillbillPluginArtifact do
                                     KPM::BaseArtifact::KAUI_CLASSIFIER,
                                     'LATEST',
                                     dir)
-      info[:file_name].should eq "kaui-standalone-#{info[:version]}.war"
-      info[:size].should eq File.size(info[:file_path])
+      expect(info[:file_name]).to eq "kaui-standalone-#{info[:version]}.war"
+      expect(info[:size]).to eq File.size(info[:file_path])
     end
   end
 
   it 'should be able to list versions' do
     versions = KPM::KauiArtifact.versions.to_a
     expect(versions.size).to be >= 2
-    versions[0].should eq '0.0.1'
-    versions[1].should eq '0.0.2'
+    expect(versions[0]).to eq '0.0.1'
+    expect(versions[1]).to eq '0.0.2'
   end
 end

data/spec/kpm/remote/killbill_plugin_artifact_spec.rb

@@ -20,15 +20,15 @@ describe KPM::KillbillPluginArtifact do
 
     Dir.mktmpdir do |dir|
       info = KPM::KillbillPluginArtifact.pull(@logger, group_id, artifact_id, packaging, classifier, version, plugin_name, dir)
-      info[:file_name].should be_nil
+      expect(info[:file_name]).to be_nil
 
       files_in_dir = Dir[info[:file_path] + '/*']
-      files_in_dir.size.should eq 1
-      files_in_dir[0].should eq info[:file_path] + '/killbill-payment-test'
+      expect(files_in_dir.size).to eq 1
+      expect(files_in_dir[0]).to eq info[:file_path] + '/killbill-payment-test'
 
-      File.read(info[:file_path] + '/killbill-payment-test/1.8.7/killbill.properties').should eq "mainClass=PaymentTest::PaymentPlugin\nrequire=payment_test\npluginType=PAYMENT\n"
+      expect(File.read(info[:file_path] + '/killbill-payment-test/1.8.7/killbill.properties')).to eq "mainClass=PaymentTest::PaymentPlugin\nrequire=payment_test\npluginType=PAYMENT\n"
 
-      info[:bundle_dir].should eq info[:file_path] + '/killbill-payment-test/1.8.7'
+      expect(info[:bundle_dir]).to eq info[:file_path] + '/killbill-payment-test/1.8.7'
     end
   end
 
@@ -44,8 +44,8 @@ describe KPM::KillbillPluginArtifact do
                                               'killbill-analytics',
                                               dir,
                                               sha1_file)
-      info[:file_name].should eq "analytics-plugin-#{info[:version]}.jar"
-      info[:size].should eq File.size(info[:file_path])
+      expect(info[:file_name]).to eq "analytics-plugin-#{info[:version]}.jar"
+      expect(info[:size]).to eq File.size(info[:file_path])
 
       check_yaml_for_resolved_latest_version(sha1_file, 'org.kill-bill.billing.plugin.java:analytics-plugin:jar', '3.0.0')
     end
@@ -63,7 +63,7 @@ describe KPM::KillbillPluginArtifact do
                                               sha1_file)
 
       # No file name - since we untar'ed it
-      info[:file_name].should be_nil
+      expect(info[:file_name]).to be_nil
 
       check_yaml_for_resolved_latest_version(sha1_file, 'org.kill-bill.billing.plugin.ruby:logging-plugin:tar.gz', '3.0.0')
     end
@@ -72,19 +72,19 @@ describe KPM::KillbillPluginArtifact do
   it 'should be able to list versions' do
     versions = KPM::KillbillPluginArtifact.versions
 
-    versions[:java].should_not be_nil
-    versions[:java]['analytics-plugin'].should_not be_nil
+    expect(versions[:java]).not_to be_nil
+    expect(versions[:java]['analytics-plugin']).not_to be_nil
     logging_plugin_versions = versions[:java]['analytics-plugin'].to_a
     expect(logging_plugin_versions.size).to be >= 3
-    logging_plugin_versions[0].should eq '0.6.0'
-    logging_plugin_versions[1].should eq '0.7.0'
-    logging_plugin_versions[2].should eq '0.7.1'
+    expect(logging_plugin_versions[0]).to eq '0.6.0'
+    expect(logging_plugin_versions[1]).to eq '0.7.0'
+    expect(logging_plugin_versions[2]).to eq '0.7.1'
 
-    versions[:ruby].should_not be_nil
-    versions[:ruby]['logging-plugin'].should_not be_nil
+    expect(versions[:ruby]).not_to be_nil
+    expect(versions[:ruby]['logging-plugin']).not_to be_nil
     logging_plugin_versions = versions[:ruby]['logging-plugin'].to_a
     expect(logging_plugin_versions.size).to be >= 1
-    logging_plugin_versions[0].should eq '1.7.0'
+    expect(logging_plugin_versions[0]).to eq '1.7.0'
   end
 
   private
@@ -96,11 +96,11 @@ describe KPM::KillbillPluginArtifact do
     sha1_checker = KPM::Sha1Checker.from_file(sha1_file)
 
     keys = sha1_checker.all_sha1.keys.select { |k| k.start_with? key_prefix }
-    keys.size.should eq 1
+    expect(keys.size).to eq 1
 
     parts = keys[0].split(':')
-    parts.size.should eq 4
-    parts[3].should_not eq 'LATEST'
+    expect(parts.size).to eq 4
+    expect(parts[3]).not_to eq 'LATEST'
     expect(parts[3]).to be >= minimum_version
   end
 end

data/spec/kpm/remote/killbill_server_artifact_spec.rb

@@ -18,16 +18,16 @@ describe KPM::KillbillServerArtifact do
                                               KPM::BaseArtifact::KILLBILL_CLASSIFIER,
                                               'LATEST',
                                               dir)
-      info[:file_name].should eq "killbill-profiles-killbill-#{info[:version]}.war"
-      info[:size].should eq File.size(info[:file_path])
+      expect(info[:file_name]).to eq "killbill-profiles-killbill-#{info[:version]}.war"
+      expect(info[:size]).to eq File.size(info[:file_path])
     end
   end
 
   it 'should be able to list versions' do
     versions = KPM::KillbillServerArtifact.versions(KPM::BaseArtifact::KILLBILL_ARTIFACT_ID).to_a
     expect(versions.size).to be >= 2
-    versions[0].should eq '0.11.10'
-    versions[1].should eq '0.11.11'
+    expect(versions[0]).to eq '0.11.10'
+    expect(versions[1]).to eq '0.11.11'
   end
 
   it 'should get dependencies information' do
@@ -36,23 +36,23 @@ describe KPM::KillbillServerArtifact do
     Dir.mktmpdir do |dir|
       sha1_file = "#{dir}/sha1.yml"
       info = KPM::KillbillServerArtifact.info('0.15.9', sha1_file)
-      info['killbill'].should eq '0.15.9'
-      info['killbill-oss-parent'].should eq '0.62'
-      info['killbill-api'].should eq '0.27'
-      info['killbill-plugin-api'].should eq '0.16'
-      info['killbill-commons'].should eq '0.10'
-      info['killbill-platform'].should eq '0.13'
-      KPM::Sha1Checker.from_file(sha1_file).killbill_info('0.15.9').should eq info
+      expect(info['killbill']).to eq '0.15.9'
+      expect(info['killbill-oss-parent']).to eq '0.62'
+      expect(info['killbill-api']).to eq '0.27'
+      expect(info['killbill-plugin-api']).to eq '0.16'
+      expect(info['killbill-commons']).to eq '0.10'
+      expect(info['killbill-platform']).to eq '0.13'
+      expect(KPM::Sha1Checker.from_file(sha1_file).killbill_info('0.15.9')).to eq info
 
       # Verify the download is skipped gracefully when Nexus isn't reachable
       KPM::KillbillServerArtifact.info('0.15.9', sha1_file, false, nil, nexus_down)
 
       # Verify the download fails when Nexus isn't reachable and force_download is set
-      expect { KPM::KillbillServerArtifact.info('0.15.9', sha1_file, true, nil, nexus_down) }.to raise_error
+      expect { KPM::KillbillServerArtifact.info('0.15.9', sha1_file, true, nil, nexus_down) }.to raise_exception(SocketError, /(Failed to open TCP connection to does.not.exist:443|getaddrinfo)/)
 
       # Verify the download fails when Nexus isn't reachable and the Nexus cache is empty
       KPM::Sha1Checker.from_file(sha1_file).cache_killbill_info('0.15.9', nil)
-      expect { KPM::KillbillServerArtifact.info('0.15.9', sha1_file, false, nil, nexus_down) }.to raise_error
+      expect { KPM::KillbillServerArtifact.info('0.15.9', sha1_file, false, nil, nexus_down) }.to raise_exception(SocketError, /(Failed to open TCP connection to does.not.exist:443|getaddrinfo)/)
     end
   end
 end