From 55b81eb9e659f641edd0c73aa504c7d35970ad8a Mon Sep 17 00:00:00 2001 From: drbrain Date: Wed, 27 Jul 2011 03:33:45 +0000 Subject: * lib/rubygems/uninstaller.rb: Add missing require and update messaging to avoid confusion with uninstall --format-executable. [Ruby 1.9 - Bug #4062] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@32697 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- test/rubygems/test_gem_commands_uninstall_command.rb | 13 ------------- test/rubygems/test_gem_uninstaller.rb | 12 +++++++++++- 2 files changed, 11 insertions(+), 14 deletions(-) (limited to 'test') diff --git a/test/rubygems/test_gem_commands_uninstall_command.rb b/test/rubygems/test_gem_commands_uninstall_command.rb index a2c11b7216..132c370f11 100644 --- a/test/rubygems/test_gem_commands_uninstall_command.rb +++ b/test/rubygems/test_gem_commands_uninstall_command.rb @@ -69,19 +69,6 @@ class TestGemCommandsUninstallCommand < Gem::InstallerTestCase Gem::Installer.exec_format = nil end - def test_execute_not_installed - @cmd.options[:args] = ["foo"] - e = assert_raises Gem::InstallError do - use_ui @ui do - @cmd.execute - end - end - - assert_match(/\Acannot uninstall, check `gem list -d foo`$/, e.message) - output = @ui.output.split "\n" - assert_empty output, "UI output should be empty after an uninstall error" - end - def test_execute_prerelease @spec = quick_spec "pre", "2.b" @gem = File.join @tempdir, @spec.file_name diff --git a/test/rubygems/test_gem_uninstaller.rb b/test/rubygems/test_gem_uninstaller.rb index a94fc43ff9..aecc4e3807 100644 --- a/test/rubygems/test_gem_uninstaller.rb +++ b/test/rubygems/test_gem_uninstaller.rb @@ -94,7 +94,7 @@ class TestGemUninstaller < Gem::InstallerTestCase exec_path = File.join Gem.user_dir, 'bin', 'foo-executable-bar' assert_equal false, File.exist?(exec_path), 'removed exec from bin dir' - assert_equal "Removing executable\n", @ui.output + assert_equal "Removing foo-executable-bar\n", @ui.output ensure Gem::Installer.exec_format = nil end @@ -158,6 +158,16 @@ class TestGemUninstaller < Gem::InstallerTestCase assert_same uninstaller, @post_uninstall_hook_arg end + def test_uninstall_nonexistent + uninstaller = Gem::Uninstaller.new 'bogus', :executables => true + + e = assert_raises Gem::InstallError do + uninstaller.uninstall + end + + assert_equal 'gem "bogus" is not installed', e.message + end + def test_uninstall_not_ok quick_gem 'z' do |s| s.add_runtime_dependency @spec.name -- cgit v1.2.3