rubygems-update 2.6.0 → 3.4.22

data/test/rubygems/test_gem_uninstaller.rb

@@ -1,11 +1,13 @@
 # frozen_string_literal: true
-require 'rubygems/installer_test_case'
-require 'rubygems/uninstaller'
 
-class TestGemUninstaller < Gem::InstallerTestCase
+require_relative "installer_test_case"
+require "rubygems/uninstaller"
 
+class TestGemUninstaller < Gem::InstallerTestCase
   def setup
     super
+    @installer = setup_base_installer
+    @user_installer = setup_base_user_installer
     common_installer_setup
 
     build_rake_in do
@@ -22,13 +24,14 @@ class TestGemUninstaller < Gem::InstallerTestCase
   end
 
   def test_initialize_expand_path
-    uninstaller = Gem::Uninstaller.new nil, :install_dir => '/foo//bar'
+    FileUtils.mkdir_p "foo/bar"
+    uninstaller = Gem::Uninstaller.new nil, :install_dir => "foo//bar"
 
-    assert_match %r|/foo/bar$|, uninstaller.instance_variable_get(:@gem_home)
+    assert_match %r{foo/bar$}, uninstaller.instance_variable_get(:@gem_home)
   end
 
   def test_ask_if_ok
-    c = util_spec 'c'
+    c = util_spec "c"
 
     uninstaller = Gem::Uninstaller.new nil
 
@@ -52,14 +55,14 @@ class TestGemUninstaller < Gem::InstallerTestCase
       uninstaller.remove_all [@spec]
     end
 
-    refute_path_exists @spec.gem_dir
+    assert_path_not_exist @spec.gem_dir
   end
 
   def test_remove_executables_force_keep
     uninstaller = Gem::Uninstaller.new nil, :executables => false
 
-    executable = File.join Gem.bindir(@user_spec.base_dir), 'executable'
-    assert File.exist?(executable), 'executable not written'
+    executable = File.join Gem.bindir(@user_spec.base_dir), "executable"
+    assert File.exist?(executable), "executable not written"
 
     use_ui @ui do
       uninstaller.remove_executables @user_spec
@@ -73,8 +76,8 @@ class TestGemUninstaller < Gem::InstallerTestCase
   def test_remove_executables_force_remove
     uninstaller = Gem::Uninstaller.new nil, :executables => true
 
-    executable = File.join Gem.bindir(@user_spec.base_dir), 'executable'
-    assert File.exist?(executable), 'executable not written'
+    executable = File.join Gem.bindir(@user_spec.base_dir), "executable"
+    assert File.exist?(executable), "executable not written"
 
     use_ui @ui do
       uninstaller.remove_executables @user_spec
@@ -92,14 +95,14 @@ class TestGemUninstaller < Gem::InstallerTestCase
       uninstaller.remove_executables @user_spec
     end
 
-    exec_path = File.join Gem.user_dir, 'bin', 'executable'
-    refute File.exist?(exec_path), 'exec still exists in user bin dir'
+    exec_path = File.join Gem.user_dir, "bin", "executable"
+    refute File.exist?(exec_path), "exec still exists in user bin dir"
 
     assert_equal "Removing executable\n", @ui.output
   end
 
   def test_remove_executables_user_format
-    Gem::Installer.exec_format = 'foo-%s-bar'
+    Gem::Installer.exec_format = "foo-%s-bar"
 
     uninstaller = Gem::Uninstaller.new nil, :executables => true, :format_executable => true
 
@@ -107,8 +110,8 @@ class TestGemUninstaller < Gem::InstallerTestCase
       uninstaller.remove_executables @user_spec
     end
 
-    exec_path = File.join Gem.user_dir, 'bin', 'foo-executable-bar'
-    assert_equal false, File.exist?(exec_path), 'removed exec from bin dir'
+    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 foo-executable-bar\n", @ui.output
   ensure
@@ -116,7 +119,7 @@ class TestGemUninstaller < Gem::InstallerTestCase
   end
 
   def test_remove_executables_user_format_disabled
-    Gem::Installer.exec_format = 'foo-%s-bar'
+    Gem::Installer.exec_format = "foo-%s-bar"
 
     uninstaller = Gem::Uninstaller.new nil, :executables => true
 
@@ -124,8 +127,8 @@ class TestGemUninstaller < Gem::InstallerTestCase
       uninstaller.remove_executables @user_spec
     end
 
-    exec_path = File.join Gem.user_dir, 'bin', 'executable'
-    refute File.exist?(exec_path), 'removed exec from bin dir'
+    exec_path = File.join Gem.user_dir, "bin", "executable"
+    refute File.exist?(exec_path), "removed exec from bin dir"
 
     assert_equal "Removing executable\n", @ui.output
   ensure
@@ -133,9 +136,10 @@ class TestGemUninstaller < Gem::InstallerTestCase
   end
 
   def test_remove_not_in_home
+    Dir.mkdir "#{@gemhome}2"
     uninstaller = Gem::Uninstaller.new nil, :install_dir => "#{@gemhome}2"
 
-    e = assert_raises Gem::GemNotInHomeException do
+    e = assert_raise Gem::GemNotInHomeException do
       use_ui ui do
         uninstaller.remove @spec
       end
@@ -146,7 +150,78 @@ class TestGemUninstaller < Gem::InstallerTestCase
 
     assert_equal expected, e.message
 
-    assert_path_exists @spec.gem_dir
+    assert_path_exist @spec.gem_dir
+  end
+
+  def test_remove_symlinked_gem_home
+    pend "Symlinks not supported or not enabled" unless symlink_supported?
+
+    Dir.mktmpdir("gem_home") do |dir|
+      symlinked_gem_home = "#{dir}/#{File.basename(@gemhome)}"
+
+      FileUtils.ln_s(@gemhome, dir)
+
+      uninstaller = Gem::Uninstaller.new nil, :install_dir => symlinked_gem_home
+
+      use_ui ui do
+        uninstaller.remove @spec
+      end
+
+      assert_path_not_exist @spec.gem_dir
+    end
+  end
+
+  def test_remove_plugins
+    write_file File.join(@tempdir, "lib", "rubygems_plugin.rb") do |io|
+      io.write "# do nothing"
+    end
+
+    @spec.files += %w[lib/rubygems_plugin.rb]
+
+    Gem::Installer.at(Gem::Package.build(@spec), :force => true).install
+
+    plugin_path = File.join Gem.plugindir, "a_plugin.rb"
+    assert File.exist?(plugin_path), "plugin not written"
+
+    Gem::Uninstaller.new(nil).remove_plugins @spec
+
+    refute File.exist?(plugin_path), "plugin not removed"
+  end
+
+  def test_remove_plugins_with_install_dir
+    write_file File.join(@tempdir, "lib", "rubygems_plugin.rb") do |io|
+      io.write "# do nothing"
+    end
+
+    @spec.files += %w[lib/rubygems_plugin.rb]
+
+    Gem::Installer.at(Gem::Package.build(@spec), :force => true).install
+
+    plugin_path = File.join Gem.plugindir, "a_plugin.rb"
+    assert File.exist?(plugin_path), "plugin not written"
+
+    Dir.mkdir "#{@gemhome}2"
+    Gem::Uninstaller.new(nil, :install_dir => "#{@gemhome}2").remove_plugins @spec
+
+    assert File.exist?(plugin_path), "plugin unintentionally removed"
+  end
+
+  def test_regenerate_plugins_for
+    write_file File.join(@tempdir, "lib", "rubygems_plugin.rb") do |io|
+      io.write "# do nothing"
+    end
+
+    @spec.files += %w[lib/rubygems_plugin.rb]
+
+    Gem::Installer.at(Gem::Package.build(@spec), :force => true).install
+
+    plugin_path = File.join Gem.plugindir, "a_plugin.rb"
+    assert File.exist?(plugin_path), "plugin not written"
+
+    FileUtils.rm plugin_path
+    Gem::Uninstaller.new(nil).regenerate_plugins_for @spec, Gem.plugindir
+
+    assert File.exist?(plugin_path), "plugin not regenerated"
   end
 
   def test_path_ok_eh
@@ -160,7 +235,7 @@ class TestGemUninstaller < Gem::InstallerTestCase
 
     @spec.loaded_from = @spec.loaded_from.gsub @spec.full_name, '\&-legacy'
     @spec.internal_init # blow out cache. but why did ^^ depend on cache?
-    @spec.platform = 'legacy'
+    @spec.platform = "legacy"
 
     assert_equal true, uninstaller.path_ok?(@gemhome, @spec)
   end
@@ -174,14 +249,16 @@ class TestGemUninstaller < Gem::InstallerTestCase
   def test_uninstall
     uninstaller = Gem::Uninstaller.new @spec.name, :executables => true
 
-    gem_dir = File.join @gemhome, 'gems', @spec.full_name
+    gem_dir = File.join @gemhome, "gems", @spec.full_name
 
     Gem.pre_uninstall do
-      assert File.exist?(gem_dir), 'gem_dir should exist'
+      sleep(0.1) if win_platform?
+      assert File.exist?(gem_dir), "gem_dir should exist"
     end
 
     Gem.post_uninstall do
-      refute File.exist?(gem_dir), 'gem_dir should not exist'
+      sleep(0.1) if win_platform?
+      refute File.exist?(gem_dir), "gem_dir should not exist"
     end
 
     uninstaller.uninstall
@@ -193,40 +270,47 @@ class TestGemUninstaller < Gem::InstallerTestCase
   end
 
   def test_uninstall_default_gem
-    spec = new_default_spec 'default', '2'
+    spec = new_default_spec "default", "2"
 
     install_default_gems spec
 
     uninstaller = Gem::Uninstaller.new spec.name, :executables => true
 
-    e = assert_raises Gem::InstallError do
+    use_ui @ui do
       uninstaller.uninstall
     end
 
-    assert_equal 'gem "default" cannot be uninstalled ' +
-                 'because it is a default gem',
-                 e.message
+    lines = @ui.output.split("\n")
+
+    assert_equal "Gem default-2 cannot be uninstalled because it is a default gem", lines.shift
   end
 
   def test_uninstall_default_gem_with_same_version
-    default_spec = new_default_spec 'default', '2'
+    default_spec = new_default_spec "default", "2"
     install_default_gems default_spec
 
-    spec = new_spec 'default', '2'
+    spec = util_spec "default", "2"
     install_gem spec
 
     Gem::Specification.reset
 
     uninstaller = Gem::Uninstaller.new spec.name, :executables => true
 
-    uninstaller.uninstall
-
-    refute_path_exists spec.gem_dir
+    ui = Gem::MockGemUi.new "1\ny\n"
+    use_ui ui do
+      uninstaller.uninstall
+    end
+    expected = "Successfully uninstalled default-2\n" \
+      "There was both a regular copy and a default copy of default-2. The " \
+      "regular copy was successfully uninstalled, but the default copy " \
+      "was left around because default gems can't be removed.\n"
+    assert_equal expected, ui.output
+    assert_path_not_exist spec.gem_dir
   end
 
   def test_uninstall_extension
-    @spec.extensions << 'extconf.rb'
-    write_file File.join(@tempdir, 'extconf.rb') do |io|
+    @spec.extensions << "extconf.rb"
+    write_file File.join(@tempdir, "extconf.rb") do |io|
       io.write <<-RUBY
 require 'mkmf'
 create_makefile '#{@spec.name}'
@@ -238,22 +322,22 @@ create_makefile '#{@spec.name}'
     use_ui @ui do
       path = Gem::Package.build @spec
 
-      installer = Gem::Installer.at path
+      installer = Gem::Installer.at path, :force => true
       installer.install
     end
 
-    assert_path_exists @spec.extension_dir, 'sanity check'
+    assert_path_exist @spec.extension_dir, "sanity check"
 
     uninstaller = Gem::Uninstaller.new @spec.name, :executables => true
     uninstaller.uninstall
 
-    refute_path_exists @spec.extension_dir
+    assert_path_not_exist @spec.extension_dir
   end
 
   def test_uninstall_nonexistent
-    uninstaller = Gem::Uninstaller.new 'bogus', :executables => true
+    uninstaller = Gem::Uninstaller.new "bogus", :executables => true
 
-    e = assert_raises Gem::InstallError do
+    e = assert_raise Gem::InstallError do
       uninstaller.uninstall
     end
 
@@ -261,61 +345,62 @@ create_makefile '#{@spec.name}'
   end
 
   def test_uninstall_not_ok
-    quick_gem 'z' do |s|
+    quick_gem "z" do |s|
       s.add_runtime_dependency @spec.name
     end
 
     uninstaller = Gem::Uninstaller.new @spec.name
 
-    gem_dir = File.join @gemhome, 'gems', @spec.full_name
-    executable = File.join @gemhome, 'bin', 'executable'
+    gem_dir = File.join @gemhome, "gems", @spec.full_name
+    executable = File.join @gemhome, "bin", "executable"
 
-    assert File.exist?(gem_dir),    'gem_dir must exist'
-    assert File.exist?(executable), 'executable must exist'
+    assert File.exist?(gem_dir),    "gem_dir must exist"
+    assert File.exist?(executable), "executable must exist"
 
     ui = Gem::MockGemUi.new "n\n"
 
-    assert_raises Gem::DependencyRemovalException do
+    assert_raise Gem::DependencyRemovalException do
       use_ui ui do
         uninstaller.uninstall
       end
     end
 
-    assert File.exist?(gem_dir),    'gem_dir must still exist'
-    assert File.exist?(executable), 'executable must still exist'
+    assert File.exist?(gem_dir),    "gem_dir must still exist"
+    assert File.exist?(executable), "executable must still exist"
   end
 
   def test_uninstall_user_install
-    @user_spec = Gem::Specification.find_by_name 'b'
+    @user_spec = Gem::Specification.find_by_name "b"
 
     uninstaller = Gem::Uninstaller.new(@user_spec.name,
-                                       :executables  => true,
+                                       :executables => true,
                                        :user_install => true)
 
     gem_dir = File.join @user_spec.gem_dir
 
     Gem.pre_uninstall do
-      assert_path_exists gem_dir
+      assert_path_exist gem_dir
     end
 
     Gem.post_uninstall do
-      refute_path_exists gem_dir
+      assert_path_not_exist gem_dir
     end
 
     uninstaller.uninstall
 
-    refute_path_exists gem_dir
+    assert_path_not_exist gem_dir
 
     assert_same uninstaller, @pre_uninstall_hook_arg
     assert_same uninstaller, @post_uninstall_hook_arg
   end
 
   def test_uninstall_wrong_repo
+    Dir.mkdir "#{@gemhome}2"
     Gem.use_paths "#{@gemhome}2", [@gemhome]
 
     uninstaller = Gem::Uninstaller.new @spec.name, :executables => true
 
-    e = assert_raises Gem::InstallError do
+    e = assert_raise Gem::InstallError do
       uninstaller.uninstall
     end
 
@@ -330,9 +415,9 @@ create_makefile '#{@spec.name}'
   def test_uninstall_selection
     util_make_gems
 
-    list = Gem::Specification.find_all_by_name 'a'
+    list = Gem::Specification.find_all_by_name "a"
 
-    uninstaller = Gem::Uninstaller.new 'a'
+    uninstaller = Gem::Uninstaller.new "a"
 
     ui = Gem::MockGemUi.new "1\ny\n"
 
@@ -340,36 +425,39 @@ create_makefile '#{@spec.name}'
       uninstaller.uninstall
     end
 
-    updated_list = Gem::Specification.find_all_by_name('a')
+    updated_list = Gem::Specification.find_all_by_name("a")
     assert_equal list.length - 1, updated_list.length
 
-    assert_match ' 1. a-1',          ui.output
-    assert_match ' 2. a-2',          ui.output
-    assert_match ' 3. a-3.a',        ui.output
-    assert_match ' 4. All versions', ui.output
-    assert_match 'uninstalled a-1',  ui.output
+    assert_match " 1. a-1",          ui.output
+    assert_match " 2. a-2",          ui.output
+    assert_match " 3. a-3.a",        ui.output
+    assert_match " 4. All versions", ui.output
+    assert_match "uninstalled a-1",  ui.output
   end
 
   def test_uninstall_selection_greater_than_one
     util_make_gems
 
-    list = Gem::Specification.find_all_by_name('a')
+    list = Gem::Specification.find_all_by_name("a")
 
-    uninstaller = Gem::Uninstaller.new('a')
+    uninstaller = Gem::Uninstaller.new("a")
 
     use_ui Gem::MockGemUi.new("2\ny\n") do
       uninstaller.uninstall
     end
 
-    updated_list = Gem::Specification.find_all_by_name('a')
+    updated_list = Gem::Specification.find_all_by_name("a")
     assert_equal list.length - 1, updated_list.length
   end
 
   def test_uninstall_prompts_about_broken_deps
-    quick_gem 'r', '1' do |s| s.add_dependency 'q', '= 1' end
-    quick_gem 'q', '1'
+    quick_gem "r", "1" do |s|
+      s.add_dependency "q", "= 1"
+    end
+
+    quick_gem "q", "1"
 
-    un = Gem::Uninstaller.new('q')
+    un = Gem::Uninstaller.new("q")
     ui = Gem::MockGemUi.new("y\n")
 
     use_ui ui do
@@ -379,21 +467,27 @@ create_makefile '#{@spec.name}'
     lines = ui.output.split("\n")
     lines.shift
 
-    assert_match %r!You have requested to uninstall the gem:!, lines.shift
+    assert_match %r{You have requested to uninstall the gem:}, lines.shift
     lines.shift
     lines.shift
 
-    assert_match %r!r-1 depends on q \(= 1\)!, lines.shift
-    assert_match %r!Successfully uninstalled q-1!, lines.last
+    assert_match %r{r-1 depends on q \(= 1\)}, lines.shift
+    assert_match %r{Successfully uninstalled q-1}, lines.last
   end
 
   def test_uninstall_only_lists_unsatisfied_deps
-    quick_gem 'r', '1' do |s| s.add_dependency 'q', '~> 1.0' end
-    quick_gem 'x', '1' do |s| s.add_dependency 'q', '= 1.0'  end
-    quick_gem 'q', '1.0'
-    quick_gem 'q', '1.1'
+    quick_gem "r", "1" do |s|
+      s.add_dependency "q", "~> 1.0"
+    end
+
+    quick_gem "x", "1" do |s|
+      s.add_dependency "q", "= 1.0"
+    end
+
+    quick_gem "q", "1.0"
+    quick_gem "q", "1.1"
 
-    un = Gem::Uninstaller.new('q', :version => "1.0")
+    un = Gem::Uninstaller.new("q", :version => "1.0")
     ui = Gem::MockGemUi.new("y\n")
 
     use_ui ui do
@@ -403,20 +497,23 @@ create_makefile '#{@spec.name}'
     lines = ui.output.split("\n")
     lines.shift
 
-    assert_match %r!You have requested to uninstall the gem:!, lines.shift
+    assert_match %r{You have requested to uninstall the gem:}, lines.shift
     lines.shift
     lines.shift
 
-    assert_match %r!x-1 depends on q \(= 1.0\)!, lines.shift
-    assert_match %r!Successfully uninstalled q-1.0!, lines.last
+    assert_match %r{x-1 depends on q \(= 1.0\)}, lines.shift
+    assert_match %r{Successfully uninstalled q-1.0}, lines.last
   end
 
   def test_uninstall_doesnt_prompt_when_other_gem_satisfies_requirement
-    quick_gem 'r', '1' do |s| s.add_dependency 'q', '~> 1.0' end
-    quick_gem 'q', '1.0'
-    quick_gem 'q', '1.1'
+    quick_gem "r", "1" do |s|
+      s.add_dependency "q", "~> 1.0"
+    end
+
+    quick_gem "q", "1.0"
+    quick_gem "q", "1.1"
 
-    un = Gem::Uninstaller.new('q', :version => "1.0")
+    un = Gem::Uninstaller.new("q", :version => "1.0")
     ui = Gem::MockGemUi.new("y\n")
 
     use_ui ui do
@@ -429,10 +526,13 @@ create_makefile '#{@spec.name}'
   end
 
   def test_uninstall_doesnt_prompt_when_removing_a_dev_dep
-    quick_gem 'r', '1' do |s| s.add_development_dependency 'q', '= 1.0' end
-    quick_gem 'q', '1.0'
+    quick_gem "r", "1" do |s|
+      s.add_development_dependency "q", "= 1.0"
+    end
+
+    quick_gem "q", "1.0"
 
-    un = Gem::Uninstaller.new('q', :version => "1.0")
+    un = Gem::Uninstaller.new("q", :version => "1.0")
     ui = Gem::MockGemUi.new("y\n")
 
     use_ui ui do
@@ -445,13 +545,16 @@ create_makefile '#{@spec.name}'
   end
 
   def test_uninstall_doesnt_prompt_and_raises_when_abort_on_dependent_set
-    quick_gem 'r', '1' do |s| s.add_dependency 'q', '= 1' end
-    quick_gem 'q', '1'
+    quick_gem "r", "1" do |s|
+      s.add_dependency "q", "= 1"
+    end
+
+    quick_gem "q", "1"
 
-    un = Gem::Uninstaller.new('q', :abort_on_dependent => true)
+    un = Gem::Uninstaller.new("q", :abort_on_dependent => true)
     ui = Gem::MockGemUi.new("y\n")
 
-    assert_raises Gem::DependencyRemovalException do
+    assert_raise Gem::DependencyRemovalException do
       use_ui ui do
         un.uninstall
       end
@@ -459,13 +562,42 @@ create_makefile '#{@spec.name}'
   end
 
   def test_uninstall_prompt_includes_dep_type
-    quick_gem 'r', '1' do |s|
-      s.add_development_dependency 'q', '= 1'
+    quick_gem "r", "1" do |s|
+      s.add_development_dependency "q", "= 1"
+    end
+
+    quick_gem "q", "1"
+
+    un = Gem::Uninstaller.new("q", :check_dev => true)
+    ui = Gem::MockGemUi.new("y\n")
+
+    use_ui ui do
+      un.uninstall
+    end
+
+    lines = ui.output.split("\n")
+    lines.shift
+
+    assert_match %r{You have requested to uninstall the gem:}, lines.shift
+    lines.shift
+    lines.shift
+
+    assert_match %r{r-1 depends on q \(= 1, development\)}, lines.shift
+    assert_match %r{Successfully uninstalled q-1}, lines.last
+  end
+
+  def test_uninstall_prompt_only_lists_the_dependents_that_prevented_uninstallation
+    quick_gem "r", "1" do |s|
+      s.add_development_dependency "q", "= 1"
+    end
+
+    quick_gem "s", "1" do |s|
+      s.add_dependency "q", "= 1"
     end
 
-    quick_gem 'q', '1'
+    quick_gem "q", "1"
 
-    un = Gem::Uninstaller.new('q', :check_dev => true)
+    un = Gem::Uninstaller.new("q", :check_dev => false)
     ui = Gem::MockGemUi.new("y\n")
 
     use_ui ui do
@@ -475,11 +607,69 @@ create_makefile '#{@spec.name}'
     lines = ui.output.split("\n")
     lines.shift
 
-    assert_match %r!You have requested to uninstall the gem:!, lines.shift
+    assert_match %r{You have requested to uninstall the gem:}, lines.shift
     lines.shift
     lines.shift
 
-    assert_match %r!r-1 depends on q \(= 1, development\)!, lines.shift
-    assert_match %r!Successfully uninstalled q-1!, lines.last
+    assert_match %r{s-1 depends on q \(= 1\)}, lines.shift
+    assert_match %r{Successfully uninstalled q-1}, lines.last
+  end
+
+  def test_uninstall_no_permission
+    uninstaller = Gem::Uninstaller.new @spec.name, :executables => true
+
+    stub_rm_r = lambda do |*args|
+      _path = args.shift
+      options = args.shift || Hash.new
+      # Uninstaller calls a method in RDoc which also calls FileUtils.rm_rf which
+      # is an alias for FileUtils#rm_r, so skip if we're using the force option
+      raise Errno::EPERM unless options[:force]
+    end
+
+    FileUtils.stub :rm_r, stub_rm_r do
+      assert_raise Gem::UninstallError do
+        uninstaller.uninstall
+      end
+    end
+  end
+
+  def test_uninstall_keeps_plugins_up_to_date
+    write_file File.join(@tempdir, "lib", "rubygems_plugin.rb") do |io|
+      io.write "# do nothing"
+    end
+
+    plugin_path = File.join Gem.plugindir, "a_plugin.rb"
+
+    @spec.version = "1"
+    Gem::Installer.at(Gem::Package.build(@spec), :force => true).install
+
+    refute File.exist?(plugin_path), "version without plugin installed, but plugin written"
+
+    @spec.files += %w[lib/rubygems_plugin.rb]
+    @spec.version = "2"
+    Gem::Installer.at(Gem::Package.build(@spec), :force => true).install
+
+    assert File.exist?(plugin_path), "version with plugin installed, but plugin not written"
+    assert_match %r{\Arequire.*a-2/lib/rubygems_plugin\.rb}, File.read(plugin_path), "written plugin has incorrect content"
+
+    @spec.version = "3"
+    Gem::Installer.at(Gem::Package.build(@spec), :force => true).install
+
+    assert File.exist?(plugin_path), "version with plugin installed, but plugin removed"
+    assert_match %r{\Arequire.*a-3/lib/rubygems_plugin\.rb}, File.read(plugin_path), "old version installed, but plugin updated"
+
+    Gem::Uninstaller.new("a", :version => "1", :executables => true).uninstall
+
+    assert File.exist?(plugin_path), "plugin removed when old version uninstalled"
+    assert_match %r{\Arequire.*a-3/lib/rubygems_plugin\.rb}, File.read(plugin_path), "old version uninstalled, but plugin updated"
+
+    Gem::Uninstaller.new("a", version: "3", :executables => true).uninstall
+
+    assert File.exist?(plugin_path), "plugin removed when old version uninstalled and another version with plugin still present"
+    assert_match %r{\Arequire.*a-2/lib/rubygems_plugin\.rb}, File.read(plugin_path), "latest version uninstalled, but plugin not updated to previous version"
+
+    Gem::Uninstaller.new("a", version: "2", :executables => true).uninstall
+
+    refute File.exist?(plugin_path), "last version uninstalled, but plugin still present"
   end
 end