rubygems-update 1.6.2 → 1.7.0

data/lib/rubygems/test_case.rb

@@ -310,7 +310,6 @@ class Gem::TestCase < MiniTest::Unit::TestCase
       s.author      = 'A User'
       s.email       = 'example@example.com'
       s.homepage    = 'http://example.com'
-      s.has_rdoc    = true
       s.summary     = "this is a summary"
       s.description = "This is a test description"
 
@@ -319,12 +318,12 @@ class Gem::TestCase < MiniTest::Unit::TestCase
 
     path = File.join "specifications", spec.spec_name
     written_path = write_file path do |io|
-      io.write(spec.to_ruby)
+      io.write spec.to_ruby_for_cache
     end
 
-    spec.loaded_from = written_path
+    spec.loaded_from = spec.loaded_from = written_path
 
-    Gem.source_index.add_spec spec
+    Gem.source_index.add_spec spec.for_cache
 
     return spec
   end
@@ -339,14 +338,14 @@ class Gem::TestCase < MiniTest::Unit::TestCase
       s.author      = 'A User'
       s.email       = 'example@example.com'
       s.homepage    = 'http://example.com'
-      s.has_rdoc    = true
       s.summary     = "this is a summary"
       s.description = "This is a test description"
 
       yield(s) if block_given?
     end
 
-    spec.loaded_from = @gemhome
+    path = File.join @gemhome, "specifications", spec.spec_name
+    spec.loaded_from = path
 
     Gem.source_index.add_spec spec
 
@@ -605,8 +604,8 @@ Also, a list:
   # Best used with +@all_gems+ from #util_setup_fake_fetcher.
 
   def util_setup_spec_fetcher(*specs)
-    specs = Hash[*specs.map { |spec| [spec.full_name, spec] }.flatten]
-    si = Gem::SourceIndex.new specs
+    si = Gem::SourceIndex.new
+    si.add_specs(*specs)
 
     spec_fetcher = Gem::SpecFetcher.fetcher
 

data/lib/rubygems/uninstaller.rb

@@ -57,11 +57,11 @@ class Gem::Uninstaller
     @user_install = options[:user_install] unless options[:install_dir]
 
     spec_dir = File.join @gem_home, 'specifications'
-    @source_index = Gem::SourceIndex.from_gems_in spec_dir
+    @source_index = Gem::SourceIndex.new [spec_dir]
 
     if @user_install then
       user_dir = File.join Gem.user_dir, 'specifications'
-      @user_index = Gem::SourceIndex.from_gems_in user_dir
+      @user_index = Gem::SourceIndex.new [user_dir]
     end
   end
 

data/lib/rubygems/user_interaction.rb

@@ -84,44 +84,40 @@ module Gem::UserInteraction
 
   include Gem::DefaultUserInteraction
 
-  ##
-  # :method: alert
+  def alert(*args)
+    ui.alert(*args)
+  end
 
-  ##
-  # :method: alert_error
+  def alert_error(*args)
+    ui.alert_error(*args)
+  end
 
-  ##
-  # :method: alert_warning
+  def alert_warning(*args)
+    ui.alert_warning(*args)
+  end
 
-  ##
-  # :method: ask
+  def ask(*args)
+    ui.ask(*args)
+  end
 
-  ##
-  # :method: ask_yes_no
+  def ask_for_password(*args)
+    ui.ask_for_password(*args)
+  end
 
-  ##
-  # :method: choose_from_list
+  def ask_yes_no(*args)
+    ui.ask_yes_no(*args)
+  end
 
-  ##
-  # :method: say
+  def choose_from_list(*args)
+    ui.choose_from_list(*args)
+  end
 
-  ##
-  # :method: terminate_interaction
-
-  [:alert,
-   :alert_error,
-   :alert_warning,
-   :ask,
-   :ask_for_password,
-   :ask_yes_no,
-   :choose_from_list,
-   :say,
-   :terminate_interaction ].each do |methname|
-    class_eval %{
-      def #{methname}(*args)
-        ui.#{methname}(*args)
-      end
-    }, __FILE__, __LINE__
+  def say(*args)
+    ui.say(*args)
+  end
+
+  def terminate_interaction(*args)
+    ui.terminate_interaction(*args)
   end
 end
 

data/test/rubygems/test_gem.rb

@@ -406,22 +406,6 @@ class TestGem < Gem::TestCase
     assert_activate %w[d-1 e-1], e1, "d"
   end
 
-  def test_self_all_load_paths
-    util_make_gems
-
-    expected = [
-      File.join(@gemhome, *%W[gems #{@a1.full_name} lib]),
-      File.join(@gemhome, *%W[gems #{@a2.full_name} lib]),
-      File.join(@gemhome, *%W[gems #{@a3a.full_name} lib]),
-      File.join(@gemhome, *%W[gems #{@a_evil9.full_name} lib]),
-      File.join(@gemhome, *%W[gems #{@b2.full_name} lib]),
-      File.join(@gemhome, *%W[gems #{@c1_2.full_name} lib]),
-      File.join(@gemhome, *%W[gems #{@pl1.full_name} lib]),
-    ]
-
-    assert_equal expected, Gem.all_load_paths.sort
-  end
-
   def test_self_available?
     util_make_gems
     assert(Gem.available?("a"))
@@ -440,16 +424,6 @@ class TestGem < Gem::TestCase
     assert_equal @abin_path, Gem.bin_path('a', 'abin', '4')
   end
 
-  def test_self_bin_path_name
-    util_exec_gem
-    assert_equal @exec_path, Gem.bin_path('a')
-  end
-
-  def test_self_bin_path_name_version
-    util_exec_gem
-    assert_equal @exec_path, Gem.bin_path('a', nil, '4')
-  end
-
   def test_self_bin_path_nonexistent_binfile
     quick_spec 'a', '2' do |s|
       s.executables = ['exec']
@@ -461,14 +435,14 @@ class TestGem < Gem::TestCase
 
   def test_self_bin_path_no_bin_file
     quick_spec 'a', '1'
-    assert_raises(Gem::Exception) do
+    assert_raises(ArgumentError) do
       Gem.bin_path('a', nil, '1')
     end
   end
 
   def test_self_bin_path_not_found
     assert_raises(Gem::GemNotFoundException) do
-      Gem.bin_path('non-existent')
+      Gem.bin_path('non-existent', 'blah')
     end
   end
 
@@ -476,7 +450,6 @@ class TestGem < Gem::TestCase
     util_exec_gem
     quick_spec 'a', '10' do |s|
       s.executables = []
-      s.default_executable = nil
     end
     # Should not find a-10's non-abin (bug)
     assert_equal @abin_path, Gem.bin_path('a', 'abin')
@@ -692,20 +665,6 @@ class TestGem < Gem::TestCase
     assert_equal cwd, $LOAD_PATH.shift
   end
 
-  def test_self_latest_load_paths
-    util_make_gems
-
-    expected = [
-      File.join(@gemhome, *%W[gems #{@a3a.full_name} lib]),
-      File.join(@gemhome, *%W[gems #{@a_evil9.full_name} lib]),
-      File.join(@gemhome, *%W[gems #{@b2.full_name} lib]),
-      File.join(@gemhome, *%W[gems #{@c1_2.full_name} lib]),
-      File.join(@gemhome, *%W[gems #{@pl1.full_name} lib]),
-    ]
-
-    assert_equal expected, Gem.latest_load_paths.sort
-  end
-
   def test_self_loaded_specs
     foo = quick_spec 'foo'
     install_gem foo
@@ -1115,7 +1074,6 @@ class TestGem < Gem::TestCase
 
   def util_exec_gem
     spec, _ = quick_spec 'a', '4' do |s|
-      s.default_executable = 'exec'
       s.executables = ['exec', 'abin']
     end
 

data/test/rubygems/test_gem_commands_contents_command.rb

@@ -9,11 +9,18 @@ class TestGemCommandsContentsCommand < Gem::TestCase
     @cmd = Gem::Commands::ContentsCommand.new
   end
 
+  def gem name
+    spec = quick_gem name do |gem|
+      gem.files = %W[lib/#{name}.rb Rakefile]
+    end
+    write_file File.join(*%W[gems #{spec.full_name} lib #{name}.rb])
+    write_file File.join(*%W[gems #{spec.full_name} Rakefile])
+  end
+
   def test_execute
     @cmd.options[:args] = %w[foo]
-    quick_gem 'foo' do |gem|
-      gem.files = %w[lib/foo.rb Rakefile]
-    end
+
+    gem 'foo'
 
     use_ui @ui do
       @cmd.execute
@@ -27,13 +34,8 @@ class TestGemCommandsContentsCommand < Gem::TestCase
   def test_execute_all
     @cmd.options[:all] = true
 
-    quick_gem 'foo' do |gem|
-      gem.files = %w[lib/foo.rb Rakefile]
-    end
-
-    quick_gem 'bar' do |gem|
-      gem.files = %w[lib/bar.rb Rakefile]
-    end
+    gem 'foo'
+    gem 'bar'
 
     use_ui @ui do
       @cmd.execute
@@ -61,13 +63,8 @@ class TestGemCommandsContentsCommand < Gem::TestCase
 
   def test_execute_exact_match
     @cmd.options[:args] = %w[foo]
-    quick_gem 'foo' do |gem|
-      gem.files = %w[lib/foo.rb Rakefile]
-    end
-
-    quick_gem 'foo_bar' do |gem|
-      gem.files = %w[lib/foo_bar.rb Rakefile]
-    end
+    gem 'foo'
+    gem 'bar'
 
     use_ui @ui do
       @cmd.execute
@@ -82,9 +79,7 @@ class TestGemCommandsContentsCommand < Gem::TestCase
     @cmd.options[:args] = %w[foo]
     @cmd.options[:lib_only] = true
 
-    quick_gem 'foo' do |gem|
-      gem.files = %w[lib/foo.rb Rakefile]
-    end
+    gem 'foo'
 
     use_ui @ui do
       @cmd.execute
@@ -98,13 +93,9 @@ class TestGemCommandsContentsCommand < Gem::TestCase
 
   def test_execute_multiple
     @cmd.options[:args] = %w[foo bar]
-    quick_gem 'foo' do |gem|
-      gem.files = %w[lib/foo.rb Rakefile]
-    end
 
-    quick_gem 'bar' do |gem|
-      gem.files = %w[lib/bar.rb Rakefile]
-    end
+    gem 'foo'
+    gem 'bar'
 
     use_ui @ui do
       @cmd.execute
@@ -120,17 +111,15 @@ class TestGemCommandsContentsCommand < Gem::TestCase
     @cmd.options[:args] = %w[foo]
     @cmd.options[:prefix] = false
 
-    quick_gem 'foo' do |gem|
-      gem.files = %w[lib/foo.rb Rakefile]
-    end
+    gem 'foo'
 
     use_ui @ui do
       @cmd.execute
     end
 
     expected = <<-EOF
-lib/foo.rb
 Rakefile
+lib/foo.rb
     EOF
 
     assert_equal expected, @ui.output

data/test/rubygems/test_gem_commands_unpack_command.rb

@@ -136,6 +136,22 @@ class TestGemCommandsUnpackCommand < Gem::TestCase
     assert File.exist?(File.join(@tempdir, 'a-2')), 'a should be unpacked'
   end
 
+  def test_execute_spec
+    util_make_gems
+
+    @cmd.options[:args] = %w[a b]
+    @cmd.options[:spec] = true
+
+    use_ui @ui do
+      Dir.chdir @tempdir do
+        @cmd.execute
+      end
+    end
+
+    assert File.exist?(File.join(@tempdir, 'a-3.a.gemspec'))
+    assert File.exist?(File.join(@tempdir, 'b-2.gemspec'))
+  end
+
   def test_execute_sudo
     util_make_gems
 
@@ -197,5 +213,13 @@ class TestGemCommandsUnpackCommand < Gem::TestCase
     assert File.exist?(File.join(@tempdir, foo_spec.full_name))
   end
 
+  def test_handle_options_metadata
+    refute @cmd.options[:spec]
+
+    @cmd.send :handle_options, %w[--spec a]
+
+    assert @cmd.options[:spec]
+  end
+
 end
 

data/test/rubygems/test_gem_commands_update_command.rb

@@ -117,7 +117,7 @@ class TestGemCommandsUpdateCommand < Gem::TestCase
     @cmd.options[:generate_rdoc] = false
     @cmd.options[:generate_ri]   = false
 
-    assert_raises Gem::SystemExitException do
+    assert_raises Gem::MockGemUi::SystemExitException do
       use_ui @ui do
         @cmd.execute
       end
@@ -178,6 +178,31 @@ class TestGemCommandsUpdateCommand < Gem::TestCase
     assert_empty out
   end
 
+  def test_execute_system_specifically_to_latest_version
+    util_clear_gems
+    util_setup_rubygem9
+    util_setup_rubygem8
+    util_setup_spec_fetcher @rubygem8, @rubygem9
+    util_add_to_fetcher @rubygem8, @rubygem9
+
+    @cmd.options[:args]          = []
+    @cmd.options[:system]        = "9"
+    @cmd.options[:generate_rdoc] = false
+    @cmd.options[:generate_ri]   = false
+
+    use_ui @ui do
+      @cmd.execute
+    end
+
+    out = @ui.output.split "\n"
+    assert_equal "Updating rubygems-update", out.shift
+    assert_equal "Successfully installed rubygems-update-9", out.shift
+    assert_equal "Installing RubyGems 9", out.shift
+    assert_equal "RubyGems system software updated", out.shift
+
+    assert_empty out
+  end
+
   def test_execute_system_with_gems
     @cmd.options[:args]          = %w[gem]
     @cmd.options[:system]        = true

data/test/rubygems/test_gem_dependency_installer.rb

@@ -33,8 +33,10 @@ class TestGemDependencyInstaller < Gem::TestCase
       inst.install 'a'
     end
 
-    assert_equal Gem::SourceIndex.new(@a1.full_name => @a1),
-                 Gem::SourceIndex.from_installed_gems
+    si = Gem::SourceIndex.new
+    si.add_spec @a1
+
+    assert_equal Gem.source_index, si
 
     assert_equal [@a1], inst.installed_gems
   end
@@ -103,7 +105,7 @@ class TestGemDependencyInstaller < Gem::TestCase
       inst.install 'b'
     end
 
-    installed = Gem::SourceIndex.from_installed_gems.map { |n,s| s.full_name }
+    installed = Gem.source_index.map { |n,s| s.full_name }
 
     assert_equal %w[a-2 b-1], installed.sort
 
@@ -396,8 +398,10 @@ class TestGemDependencyInstaller < Gem::TestCase
       inst.install 'a'
     end
 
-    assert_equal Gem::SourceIndex.new(@a1.full_name => @a1),
-                 Gem::SourceIndex.from_installed_gems
+    si = Gem::SourceIndex.new
+    si.add_spec @a1
+
+    assert_equal Gem.source_index, si
 
     assert_equal %w[a-1], inst.installed_gems.map { |s| s.full_name }
   end

data/test/rubygems/test_gem_dependency_list.rb

@@ -22,12 +22,8 @@ class TestGemDependencyList < Gem::TestCase
   end
 
   def test_self_from_source_index
-    hash = {
-      'a-1' => @a1,
-      'b-2' => @b2,
-    }
-
-    si = Gem::SourceIndex.new hash
+    si = Gem::SourceIndex.new
+    si.add_specs @a1, @b2
     deps = Gem::DependencyList.from_source_index si
 
     assert_equal %w[b-2 a-1], deps.dependency_order.map { |s| s.full_name }