rubygems-update 3.2.17 → 3.2.18

data/rubygems-update.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
   s.name = "rubygems-update"
-  s.version = "3.2.17"
+  s.version = "3.2.18"
   s.authors = ["Jim Weirich", "Chad Fowler", "Eric Hodel", "Luis Lavena", "Aaron Patterson", "Samuel Giddins", "André Arko", "Evan Phoenix", "Hiroshi SHIBATA"]
   s.email = ["", "", "drbrain@segment7.net", "luislavena@gmail.com", "aaron@tenderlovemaking.com", "segiddins@segiddins.me", "andre@arko.net", "evan@phx.io", "hsbt@ruby-lang.org"]
 

data/test/rubygems/test_bundled_ca.rb

@@ -26,14 +26,14 @@ class TestBundledCA < Gem::TestCase
   end
 
   def assert_https(host)
-    self.assertions += 1
+    assert true
     http = Net::HTTP.new(host, 443)
     http.use_ssl = true
     http.verify_mode = OpenSSL::SSL::VERIFY_PEER
     http.cert_store = bundled_certificate_store
     http.get('/')
   rescue Errno::ENOENT, Errno::ETIMEDOUT, SocketError
-    skip "#{host} seems offline, I can't tell whether ssl would work."
+    pend "#{host} seems offline, I can't tell whether ssl would work."
   rescue OpenSSL::SSL::SSLError => e
     # Only fail for certificate verification errors
     if e.message =~ /certificate verify failed/

data/test/rubygems/test_deprecate.rb

@@ -96,7 +96,7 @@ class TestDeprecate < Gem::TestCase
   end
 
   def test_deprecated_method_calls_the_old_method
-    capture_io do
+    capture_output do
       thing = Thing.new
       thing.foo
       assert_equal "foo", thing.message
@@ -108,7 +108,7 @@ class TestDeprecate < Gem::TestCase
   end
 
   def test_deprecated_method_outputs_a_warning
-    out, err = capture_io do
+    out, err = capture_output do
       thing = Thing.new
       thing.foo
       thing.foo_arg("msg")
@@ -141,7 +141,7 @@ class TestDeprecate < Gem::TestCase
   end
 
   def test_deprecated_method_outputs_a_warning_old_way
-    out, err = capture_io do
+    out, err = capture_output do
       thing = OtherThing.new
       thing.foo
       thing.foo_arg("msg")

data/test/rubygems/test_gem.rb

@@ -106,7 +106,7 @@ class TestGem < Gem::TestCase
 
     assert_equal %w[a-1], installed.map {|spec| spec.full_name }
 
-    assert_path_exists File.join(gemhome2, 'gems', 'a-1')
+    assert_path_exist File.join(gemhome2, 'gems', 'a-1')
   end
 
   def test_self_install_in_rescue
@@ -212,7 +212,7 @@ class TestGem < Gem::TestCase
 
   def test_require_missing
     save_loaded_features do
-      assert_raises ::LoadError do
+      assert_raise ::LoadError do
         require "test_require_missing"
       end
     end
@@ -224,7 +224,7 @@ class TestGem < Gem::TestCase
 
       install_specs a1
 
-      assert_raises ::LoadError do
+      assert_raise ::LoadError do
         require "a*"
       end
 
@@ -261,7 +261,7 @@ class TestGem < Gem::TestCase
   end
 
   def test_self_activate_bin_path_no_exec_name
-    e = assert_raises ArgumentError do
+    e = assert_raise ArgumentError do
       Gem.activate_bin_path 'a'
     end
 
@@ -342,7 +342,7 @@ class TestGem < Gem::TestCase
 
     # c2 is missing, and b2 which has it as a dependency will be activated, so we should get an error about the orphaned dependency
 
-    e = assert_raises Gem::UnsatisfiableDependencyError do
+    e = assert_raise Gem::UnsatisfiableDependencyError do
       load Gem.activate_bin_path("a", "exec", ">= 0")
     end
 
@@ -390,7 +390,7 @@ class TestGem < Gem::TestCase
 
     File.open("Gemfile", "w") {|f| f.puts('source "https://rubygems.org"') }
 
-    e = assert_raises Gem::GemNotFoundException do
+    e = assert_raise Gem::GemNotFoundException do
       load Gem.activate_bin_path("bundler", "bundle", ">= 0.a")
     end
 
@@ -487,7 +487,7 @@ class TestGem < Gem::TestCase
 
     File.open("Gemfile", "w") {|f| f.puts('source "https://rubygems.org"') }
 
-    e = assert_raises Gem::GemNotFoundException do
+    e = assert_raise Gem::GemNotFoundException do
       load Gem.activate_bin_path("bundler", "bundle", "= 2.2.8")
     end
 
@@ -495,7 +495,7 @@ class TestGem < Gem::TestCase
   end
 
   def test_self_bin_path_no_exec_name
-    e = assert_raises ArgumentError do
+    e = assert_raise ArgumentError do
       Gem.bin_path 'a'
     end
 
@@ -516,20 +516,20 @@ class TestGem < Gem::TestCase
     util_spec 'a', '2' do |s|
       s.executables = ['exec']
     end
-    assert_raises(Gem::GemNotFoundException) do
+    assert_raise(Gem::GemNotFoundException) do
       Gem.bin_path('a', 'other', '2')
     end
   end
 
   def test_self_bin_path_no_bin_file
     util_spec 'a', '1'
-    assert_raises(ArgumentError) do
+    assert_raise(ArgumentError) do
       Gem.bin_path('a', nil, '1')
     end
   end
 
   def test_self_bin_path_not_found
-    assert_raises(Gem::GemNotFoundException) do
+    assert_raise(Gem::GemNotFoundException) do
       Gem.bin_path('non-existent', 'blah')
     end
   end
@@ -596,7 +596,7 @@ class TestGem < Gem::TestCase
   end
 
   def test_self_datadir_nonexistent_package
-    assert_raises(Gem::MissingSpecError) do
+    assert_raise(Gem::MissingSpecError) do
       Gem::Specification.find_by_name("xyzzy").datadir
     end
   end
@@ -692,12 +692,12 @@ class TestGem < Gem::TestCase
 
     Gem.ensure_gem_subdirectories @gemhome
 
-    assert_path_exists File.join @gemhome, 'build_info'
-    assert_path_exists File.join @gemhome, 'cache'
-    assert_path_exists File.join @gemhome, 'doc'
-    assert_path_exists File.join @gemhome, 'extensions'
-    assert_path_exists File.join @gemhome, 'gems'
-    assert_path_exists File.join @gemhome, 'specifications'
+    assert_path_exist File.join @gemhome, 'build_info'
+    assert_path_exist File.join @gemhome, 'cache'
+    assert_path_exist File.join @gemhome, 'doc'
+    assert_path_exist File.join @gemhome, 'extensions'
+    assert_path_exist File.join @gemhome, 'gems'
+    assert_path_exist File.join @gemhome, 'specifications'
   end
 
   def test_self_ensure_gem_directories_permissions
@@ -1058,7 +1058,7 @@ class TestGem < Gem::TestCase
 
     assert_equal ["\xCF", "\x80"], Gem.read_binary('test').chars.to_a
 
-    skip 'chmod not supported' if Gem.win_platform?
+    pend 'chmod not supported' if Gem.win_platform?
 
     begin
       File.chmod 0444, 'test'
@@ -1144,7 +1144,7 @@ class TestGem < Gem::TestCase
 
     assert_equal Gem::Requirement.create('>= 1.2.3'), Gem.env_requirement('foo')
     assert_equal Gem::Requirement.create('1.2.3'), Gem.env_requirement('bAr')
-    assert_raises(Gem::Requirement::BadRequirementError) { Gem.env_requirement('baz') }
+    assert_raise(Gem::Requirement::BadRequirementError) { Gem.env_requirement('baz') }
     assert_equal Gem::Requirement.default, Gem.env_requirement('qux')
   end
 
@@ -1349,7 +1349,7 @@ class TestGem < Gem::TestCase
       io.puts '# a_file.rb'
     end
 
-    e = assert_raises Gem::MissingSpecError do
+    e = assert_raise Gem::MissingSpecError do
       Gem.try_activate 'a_file'
     end
 
@@ -1370,7 +1370,7 @@ class TestGem < Gem::TestCase
       io.puts '# a_file.rb'
     end
 
-    e = assert_raises Gem::MissingSpecError do
+    e = assert_raise Gem::MissingSpecError do
       Gem.try_activate 'a_file'
     end
 
@@ -1389,7 +1389,7 @@ class TestGem < Gem::TestCase
       io.write spec.to_ruby_for_cache
     end
 
-    _, err = capture_io do
+    _, err = capture_output do
       refute Gem.try_activate 'nonexistent'
     end
 
@@ -1414,7 +1414,7 @@ class TestGem < Gem::TestCase
   end
 
   def test_setting_paths_does_not_warn_about_unknown_keys
-    stdout, stderr = capture_io do
+    stdout, stderr = capture_output do
       Gem.paths = { 'foo'      => [],
                     'bar'      => Object.new,
                     'GEM_HOME' => Gem.paths.home,
@@ -1432,7 +1432,7 @@ class TestGem < Gem::TestCase
   end
 
   def test_deprecated_paths=
-    stdout, stderr = capture_io do
+    stdout, stderr = capture_output do
       Gem.paths = { 'GEM_HOME' => Gem.paths.home,
                     'GEM_PATH' => [Gem.paths.home, 'foo'] }
     end
@@ -1727,7 +1727,7 @@ class TestGem < Gem::TestCase
   end
 
   def test_looks_for_gemdeps_files_automatically_on_start
-    skip "Requiring bundler messes things up" if Gem.java_platform?
+    pend "Requiring bundler messes things up" if Gem.java_platform?
 
     a = util_spec "a", "1", nil, "lib/a.rb"
     b = util_spec "b", "1", nil, "lib/b.rb"
@@ -1763,7 +1763,7 @@ class TestGem < Gem::TestCase
   end
 
   def test_looks_for_gemdeps_files_automatically_on_start_in_parent_dir
-    skip "Requiring bundler messes things up" if Gem.java_platform?
+    pend "Requiring bundler messes things up" if Gem.java_platform?
 
     a = util_spec "a", "1", nil, "lib/a.rb"
     b = util_spec "b", "1", nil, "lib/b.rb"
@@ -1881,7 +1881,7 @@ class TestGem < Gem::TestCase
   end
 
   def test_use_gemdeps_argument_missing
-    e = assert_raises ArgumentError do
+    e = assert_raise ArgumentError do
       Gem.use_gemdeps 'gem.deps.rb'
     end
 
@@ -1893,7 +1893,7 @@ class TestGem < Gem::TestCase
     rubygems_gemdeps, ENV['RUBYGEMS_GEMDEPS'] =
       ENV['RUBYGEMS_GEMDEPS'], 'gem.deps.rb'
 
-    e = assert_raises ArgumentError do
+    e = assert_raise ArgumentError do
       Gem.use_gemdeps 'gem.deps.rb'
     end
 
@@ -1966,9 +1966,11 @@ You may need to `gem install -g` to install missing gems
     EXPECTED
 
     Gem::Deprecate.skip_during do
-      assert_output nil, expected do
+      actual_stdout, actual_stderr = capture_output do
         Gem.use_gemdeps
       end
+      assert_empty actual_stdout
+      assert_equal(expected, actual_stderr)
     end
   ensure
     ENV['RUBYGEMS_GEMDEPS'] = rubygems_gemdeps
@@ -2073,7 +2075,7 @@ You may need to `gem install -g` to install missing gems
     refute_includes $LOAD_PATH, test_plugin_path
     $LOAD_PATH.unshift test_plugin_path
 
-    capture_io do
+    capture_output do
       yield
     end
   ensure

data/test/rubygems/test_gem_bundler_version_finder.rb

@@ -78,8 +78,8 @@ class TestGemBundlerVersionFinder < Gem::TestCase
   end
 
   def test_deleted_directory
-    skip "Cannot perform this test on windows" if win_platform?
-    skip "Cannot perform this test on Solaris" if /solaris/ =~ RUBY_PLATFORM
+    pend "Cannot perform this test on windows" if win_platform?
+    pend "Cannot perform this test on Solaris" if /solaris/ =~ RUBY_PLATFORM
     require "tmpdir"
 
     orig_dir = Dir.pwd

data/test/rubygems/test_gem_command.rb

@@ -118,7 +118,7 @@ class TestGemCommand < Gem::TestCase
     use_ui @ui do
       @cmd.when_invoked { true }
 
-      ex = assert_raises OptionParser::InvalidOption do
+      ex = assert_raise OptionParser::InvalidOption do
         @cmd.invoke('-zzz')
       end
 

data/test/rubygems/test_gem_command_manager.rb

@@ -22,7 +22,7 @@ class TestGemCommandManager < Gem::TestCase
   end
 
   def test_find_command_ambiguous
-    e = assert_raises Gem::CommandLineError do
+    e = assert_raise Gem::CommandLineError do
       @command_manager.find_command 'u'
     end
 
@@ -50,7 +50,7 @@ class TestGemCommandManager < Gem::TestCase
   end
 
   def test_find_command_unknown
-    e = assert_raises Gem::CommandLineError do
+    e = assert_raise Gem::CommandLineError do
       @command_manager.find_command 'xyz'
     end
 
@@ -65,7 +65,7 @@ class TestGemCommandManager < Gem::TestCase
     @command_manager.register_command :interrupt
 
     use_ui @ui do
-      assert_raises Gem::MockGemUi::TermError do
+      assert_raise Gem::MockGemUi::TermError do
         @command_manager.run %w[interrupt]
       end
       assert_equal '', ui.output
@@ -82,7 +82,7 @@ class TestGemCommandManager < Gem::TestCase
 
     @command_manager.register_command :crash
     use_ui @ui do
-      assert_raises Gem::MockGemUi::TermError do
+      assert_raise Gem::MockGemUi::TermError do
         @command_manager.run %w[crash]
       end
       assert_equal '', ui.output
@@ -96,7 +96,7 @@ class TestGemCommandManager < Gem::TestCase
 
   def test_process_args_bad_arg
     use_ui @ui do
-      assert_raises Gem::MockGemUi::TermError do
+      assert_raise Gem::MockGemUi::TermError do
         @command_manager.process_args %w[--bad-arg]
       end
     end

data/test/rubygems/test_gem_commands_build_command.rb

@@ -126,7 +126,7 @@ class TestGemCommandsBuildCommand < Gem::TestCase
 
       use_ui @ui do
         Dir.chdir @tempdir do
-          assert_raises Gem::InvalidSpecificationException do
+          assert_raise Gem::InvalidSpecificationException do
             @cmd.execute
           end
         end
@@ -180,7 +180,7 @@ class TestGemCommandsBuildCommand < Gem::TestCase
 
     use_ui @ui do
       Dir.chdir @tempdir do
-        assert_raises Gem::InvalidSpecificationException do
+        assert_raise Gem::InvalidSpecificationException do
           @cmd.execute
         end
       end
@@ -208,8 +208,8 @@ class TestGemCommandsBuildCommand < Gem::TestCase
     @cmd.options[:args] = [gemspec_file]
 
     out, err = use_ui @ui do
-      capture_io do
-        assert_raises Gem::MockGemUi::TermError do
+      capture_output do
+        assert_raise Gem::MockGemUi::TermError do
           @cmd.execute
         end
       end
@@ -225,7 +225,7 @@ class TestGemCommandsBuildCommand < Gem::TestCase
   def test_execute_missing_file
     @cmd.options[:args] = %w[some_gem]
     use_ui @ui do
-      assert_raises Gem::MockGemUi::TermError do
+      assert_raise Gem::MockGemUi::TermError do
         @cmd.execute
       end
     end
@@ -329,7 +329,7 @@ class TestGemCommandsBuildCommand < Gem::TestCase
     @cmd.options[:args] = ["*.gemspec"]
 
     use_ui @ui do
-      assert_raises Gem::MockGemUi::TermError do
+      assert_raise Gem::MockGemUi::TermError do
         @cmd.execute
       end
     end
@@ -527,7 +527,7 @@ class TestGemCommandsBuildCommand < Gem::TestCase
 
     use_ui @ui do
       Dir.chdir(gemspec_dir) do
-        assert_raises Gem::MockGemUi::TermError do
+        assert_raise Gem::MockGemUi::TermError do
           @cmd.execute
         end
       end
@@ -582,7 +582,7 @@ class TestGemCommandsBuildCommand < Gem::TestCase
   end
 
   def test_build_signed_gem
-    skip 'openssl is missing' unless Gem::HAVE_OPENSSL && !java_platform?
+    pend 'openssl is missing' unless Gem::HAVE_OPENSSL && !java_platform?
 
     trust_dir = Gem::Security.trust_dir
 
@@ -609,7 +609,7 @@ class TestGemCommandsBuildCommand < Gem::TestCase
   end
 
   def test_build_signed_gem_with_cert_expiration_length_days
-    skip 'openssl is missing' unless Gem::HAVE_OPENSSL && !java_platform?
+    pend 'openssl is missing' unless Gem::HAVE_OPENSSL && !java_platform?
 
     gem_path = File.join Gem.user_home, ".gem"
     Dir.mkdir gem_path
@@ -653,7 +653,7 @@ class TestGemCommandsBuildCommand < Gem::TestCase
   end
 
   def test_build_auto_resign_cert
-    skip 'openssl is missing' unless Gem::HAVE_OPENSSL && !java_platform?
+    pend 'openssl is missing' unless Gem::HAVE_OPENSSL && !java_platform?
 
     gem_path = File.join Gem.user_home, ".gem"
     Dir.mkdir gem_path

data/test/rubygems/test_gem_commands_cert_command.rb

@@ -54,7 +54,7 @@ class TestGemCommandsCertCommand < Gem::TestCase
     assert_equal PUBLIC_CERT.to_pem, match.first.to_pem
     assert_equal @trust_dir.cert_path(PUBLIC_CERT), match.last
 
-    assert_raises StopIteration do
+    assert_raise StopIteration do
       matches.next
     end
   end
@@ -69,7 +69,7 @@ class TestGemCommandsCertCommand < Gem::TestCase
     assert_equal ALTERNATE_CERT.to_pem, match.first.to_pem
     assert_equal @trust_dir.cert_path(ALTERNATE_CERT), match.last
 
-    assert_raises StopIteration do
+    assert_raise StopIteration do
       matches.next
     end
   end
@@ -83,7 +83,7 @@ class TestGemCommandsCertCommand < Gem::TestCase
 
     cert_path = @trust_dir.cert_path PUBLIC_CERT
 
-    assert_path_exists cert_path
+    assert_path_exist cert_path
 
     assert_equal "Added '/CN=nobody/DC=example'\n", @ui.output
     assert_empty @ui.error
@@ -138,8 +138,8 @@ Added '/CN=alternate/DC=example'
     assert_empty output
     assert_empty @build_ui.error
 
-    assert_path_exists File.join(@tempdir, 'gem-private_key.pem')
-    assert_path_exists File.join(@tempdir, 'gem-public_cert.pem')
+    assert_path_exist File.join(@tempdir, 'gem-private_key.pem')
+    assert_path_exist File.join(@tempdir, 'gem-public_cert.pem')
   end
 
   def test_execute_build_bad_email_address
@@ -152,15 +152,15 @@ Added '/CN=alternate/DC=example'
 
     use_ui @build_ui do
 
-      e = assert_raises Gem::CommandLineError do
+      e = assert_raise Gem::CommandLineError do
         @cmd.execute
       end
 
       assert_equal "Invalid email address #{email}",
         e.message
 
-      refute_path_exists File.join(@tempdir, 'gem-private_key.pem')
-      refute_path_exists File.join(@tempdir, 'gem-public_cert.pem')
+      assert_path_not_exist File.join(@tempdir, 'gem-private_key.pem')
+      assert_path_not_exist File.join(@tempdir, 'gem-public_cert.pem')
     end
   end
 
@@ -195,8 +195,8 @@ Added '/CN=alternate/DC=example'
     assert_empty output
     assert_empty @build_ui.error
 
-    assert_path_exists File.join(@tempdir, 'gem-private_key.pem')
-    assert_path_exists File.join(@tempdir, 'gem-public_cert.pem')
+    assert_path_exist File.join(@tempdir, 'gem-private_key.pem')
+    assert_path_exist File.join(@tempdir, 'gem-public_cert.pem')
 
     pem = File.read("#{@tempdir}/gem-public_cert.pem")
     cert = OpenSSL::X509::Certificate.new(pem)
@@ -214,7 +214,7 @@ Added '/CN=alternate/DC=example'
     @build_ui = Gem::MockGemUi.new "#{passphrase}\n#{passphrase_confirmation}"
 
     use_ui @build_ui do
-      e = assert_raises Gem::CommandLineError do
+      e = assert_raise Gem::CommandLineError do
         @cmd.execute
       end
 
@@ -232,8 +232,8 @@ Added '/CN=alternate/DC=example'
 
     end
 
-    refute_path_exists File.join(@tempdir, 'gem-private_key.pem')
-    refute_path_exists File.join(@tempdir, 'gem-public_cert.pem')
+    assert_path_not_exist File.join(@tempdir, 'gem-private_key.pem')
+    assert_path_not_exist File.join(@tempdir, 'gem-public_cert.pem')
   end
 
   def test_execute_build_key
@@ -254,8 +254,8 @@ Added '/CN=alternate/DC=example'
     assert_empty output
     assert_empty @ui.error
 
-    assert_path_exists File.join(@tempdir, 'gem-public_cert.pem')
-    refute_path_exists File.join(@tempdir, 'gem-private_key.pem')
+    assert_path_exist File.join(@tempdir, 'gem-public_cert.pem')
+    assert_path_not_exist File.join(@tempdir, 'gem-private_key.pem')
   end
 
   def test_execute_build_encrypted_key
@@ -276,7 +276,7 @@ Added '/CN=alternate/DC=example'
     assert_empty output
     assert_empty @ui.error
 
-    assert_path_exists File.join(@tempdir, 'gem-public_cert.pem')
+    assert_path_exist File.join(@tempdir, 'gem-public_cert.pem')
   end
 
   def test_execute_certificate
@@ -346,7 +346,7 @@ Added '/CN=alternate/DC=example'
 
     cert_path = @trust_dir.cert_path PUBLIC_CERT
 
-    assert_path_exists cert_path
+    assert_path_exist cert_path
 
     @cmd.handle_options %W[--remove nobody]
 
@@ -357,7 +357,7 @@ Added '/CN=alternate/DC=example'
     assert_equal "Removed '/CN=nobody/DC=example'\n", @ui.output
     assert_equal '', @ui.error
 
-    refute_path_exists cert_path
+    assert_path_not_exist cert_path
   end
 
   def test_execute_remove_multiple
@@ -367,8 +367,8 @@ Added '/CN=alternate/DC=example'
     public_path = @trust_dir.cert_path PUBLIC_CERT
     alternate_path = @trust_dir.cert_path ALTERNATE_CERT
 
-    assert_path_exists public_path
-    assert_path_exists alternate_path
+    assert_path_exist public_path
+    assert_path_exist alternate_path
 
     @cmd.handle_options %W[--remove example]
 
@@ -384,8 +384,8 @@ Removed '/CN=nobody/DC=example'
     assert_equal expected, @ui.output
     assert_equal '', @ui.error
 
-    refute_path_exists public_path
-    refute_path_exists alternate_path
+    assert_path_not_exist public_path
+    assert_path_not_exist alternate_path
   end
 
   def test_execute_remove_twice
@@ -395,8 +395,8 @@ Removed '/CN=nobody/DC=example'
     public_path = @trust_dir.cert_path PUBLIC_CERT
     alternate_path = @trust_dir.cert_path ALTERNATE_CERT
 
-    assert_path_exists public_path
-    assert_path_exists alternate_path
+    assert_path_exist public_path
+    assert_path_exist alternate_path
 
     @cmd.handle_options %W[--remove nobody --remove alternate]
 
@@ -412,8 +412,8 @@ Removed '/CN=alternate/DC=example'
     assert_equal expected, @ui.output
     assert_equal '', @ui.error
 
-    refute_path_exists public_path
-    refute_path_exists alternate_path
+    assert_path_not_exist public_path
+    assert_path_not_exist alternate_path
   end
 
   def test_execute_sign
@@ -552,7 +552,7 @@ Removed '/CN=alternate/DC=example'
     @cmd.handle_options %W[--sign #{path}]
 
     use_ui @ui do
-      assert_raises Gem::MockGemUi::TermError do
+      assert_raise Gem::MockGemUi::TermError do
         @cmd.execute
       end
     end
@@ -580,7 +580,7 @@ ERROR:  --certificate not specified and ~/.gem/gem-public_cert.pem does not exis
     @cmd.handle_options %W[--sign #{path}]
 
     use_ui @ui do
-      assert_raises Gem::MockGemUi::TermError do
+      assert_raise Gem::MockGemUi::TermError do
         @cmd.execute
       end
     end
@@ -686,7 +686,7 @@ ERROR:  --private-key not specified and ~/.gem/gem-private_key.pem does not exis
 
   def test_handle_options_add_bad
     nonexistent = File.join @tempdir, 'nonexistent'
-    e = assert_raises OptionParser::InvalidArgument do
+    e = assert_raise OptionParser::InvalidArgument do
       @cmd.handle_options %W[--add #{nonexistent}]
     end
 
@@ -696,7 +696,7 @@ ERROR:  --private-key not specified and ~/.gem/gem-private_key.pem does not exis
     bad = File.join @tempdir, 'bad'
     FileUtils.touch bad
 
-    e = assert_raises OptionParser::InvalidArgument do
+    e = assert_raise OptionParser::InvalidArgument do
       @cmd.handle_options %W[--add #{bad}]
     end
 
@@ -706,7 +706,7 @@ ERROR:  --private-key not specified and ~/.gem/gem-private_key.pem does not exis
 
   def test_handle_options_certificate
     nonexistent = File.join @tempdir, 'nonexistent'
-    e = assert_raises OptionParser::InvalidArgument do
+    e = assert_raise OptionParser::InvalidArgument do
       @cmd.handle_options %W[--certificate #{nonexistent}]
     end
 
@@ -716,7 +716,7 @@ ERROR:  --private-key not specified and ~/.gem/gem-private_key.pem does not exis
     bad = File.join @tempdir, 'bad'
     FileUtils.touch bad
 
-    e = assert_raises OptionParser::InvalidArgument do
+    e = assert_raise OptionParser::InvalidArgument do
       @cmd.handle_options %W[--certificate #{bad}]
     end
 
@@ -727,7 +727,7 @@ ERROR:  --private-key not specified and ~/.gem/gem-private_key.pem does not exis
 
   def test_handle_options_key_bad
     nonexistent = File.join @tempdir, 'nonexistent'
-    e = assert_raises OptionParser::InvalidArgument do
+    e = assert_raise OptionParser::InvalidArgument do
       @cmd.handle_options %W[--private-key #{nonexistent}]
     end
 
@@ -738,14 +738,14 @@ ERROR:  --private-key not specified and ~/.gem/gem-private_key.pem does not exis
     bad = File.join @tempdir, 'bad'
     FileUtils.touch bad
 
-    e = assert_raises OptionParser::InvalidArgument do
+    e = assert_raise OptionParser::InvalidArgument do
       @cmd.handle_options %W[--private-key #{bad}]
     end
 
     assert_equal "invalid argument: --private-key #{bad}: invalid RSA key",
                  e.message
 
-    e = assert_raises OptionParser::InvalidArgument do
+    e = assert_raise OptionParser::InvalidArgument do
       @cmd.handle_options %W[--private-key #{PUBLIC_KEY_FILE}]
     end
 
@@ -792,7 +792,7 @@ ERROR:  --private-key not specified and ~/.gem/gem-private_key.pem does not exis
 
   def test_handle_options_sign_nonexistent
     nonexistent = File.join @tempdir, 'nonexistent'
-    e = assert_raises OptionParser::InvalidArgument do
+    e = assert_raise OptionParser::InvalidArgument do
       @cmd.handle_options %W[
         --private-key #{ALTERNATE_KEY_FILE}