* lib/rubygems/doctor.rb: Process directories in order in case the

filesystem doesn't.  [ruby-trunk - Bug #7618]

  Process specifications before other directories in case of bugs.
* test/rubygems/test_gem_doctor.rb:  Test for above.

* lib/rubygems.rb:  Updated version.

* test/rubygems/test_require.rb:  Fixed double require of benchmark.rb.
  RubyGems bug #420.

* test/rubygems/test_gem_commands_check_command.rb:  Fixed unused
  variable warnings.
* test/rubygems/test_gem_commands_query_command.rb:  ditto
* test/rubygems/test_gem_installer.rb:  ditto


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@38691 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

ChangeLog

@@ -1,3 +1,21 @@
+Fri Jan  4 16:26:45 2013  Eric Hodel  <drbrain@segment7.net>
+
+	* lib/rubygems/doctor.rb:  Process directories in order in case the
+	  filesystem doesn't.  [ruby-trunk - Bug #7618]
+
+	  Process specifications before other directories in case of bugs.
+	* test/rubygems/test_gem_doctor.rb:  Test for above.
+
+	* lib/rubygems.rb:  Updated version.
+
+	* test/rubygems/test_require.rb:  Fixed double require of
+	  benchmark.rb.  RubyGems bug #420.
+
+	* test/rubygems/test_gem_commands_check_command.rb:  Fixed unused
+	  variable warnings.
+	* test/rubygems/test_gem_commands_query_command.rb:  ditto
+	* test/rubygems/test_gem_installer.rb:  ditto
+
 Fri Jan  4 15:05:25 2013  Eric Hodel  <drbrain@segment7.net>
 
 	* lib/rdoc/cross_reference.rb:  Fixed matching of C#=== or #===.  RDoc

lib/rubygems.rb

@@ -98,7 +98,7 @@
 require 'rbconfig'
 
 module Gem
-  VERSION = '2.0.0.preview3'
+  VERSION = '2.0.0.preview3.1'
 end
 
 # Must be first since it unloads the prelude from 1.9.2

lib/rubygems/doctor.rb

@@ -19,16 +19,17 @@ class Gem::Doctor
   # Maps a gem subdirectory to the files that are expected to exist in the
   # subdirectory.
 
-  REPOSITORY_EXTENSION_MAP = { # :nodoc:
+  REPOSITORY_EXTENSION_MAP = [ # :nodoc:
-    'build_info' =>     '.info',
+    ['specifications', '.gemspec'],
-    'cache'      =>     '.gem',
+    ['build_info',     '.info'],
-    'doc'        =>     '',
+    ['cache',          '.gem'],
-    'gems'       =>     '',
+    ['doc',            ''],
-    'specifications' => '.gemspec'
+    ['gems',           ''],
-  }
+  ]
 
   raise 'Update REPOSITORY_EXTENSION_MAP' unless
-    Gem::REPOSITORY_SUBDIRECTORIES == REPOSITORY_EXTENSION_MAP.keys.sort
+    Gem::REPOSITORY_SUBDIRECTORIES.sort ==
+      REPOSITORY_EXTENSION_MAP.map { |(k,_)| k }.sort
 
   ##
   # Creates a new Gem::Doctor that will clean up +gem_repository+.  Only one
@@ -97,7 +98,7 @@ class Gem::Doctor
   def doctor_child sub_directory, extension # :nodoc:
     directory = @gem_repository + sub_directory
 
-    directory.each_child do |child|
+    directory.children.sort.each do |child|
       next unless child.exist?
 
       basename = child.basename(extension).to_s

test/rubygems/test_gem_commands_check_command.rb

@@ -45,7 +45,7 @@ class TestGemCommandsCheckCommand < Gem::TestCase
   end
 
   def test_doctor
-    a = gem 'a'
+    gem 'a'
     b = gem 'b'
 
     FileUtils.rm b.spec_file

test/rubygems/test_gem_commands_query_command.rb

@@ -405,8 +405,6 @@ pl \(1\)
       @cmd.execute
     end
 
-    str = @ui.output
-
     expected = <<-EOF
 
 *** LOCAL GEMS ***

test/rubygems/test_gem_doctor.rb

@@ -55,13 +55,13 @@ class TestGemDoctor < Gem::TestCase
 
     expected = <<-OUTPUT
 Checking #{@gemhome}
+Removed file specifications/c-2.gemspec
 Removed directory gems/b-2
 Removed directory gems/c-2
-Removed file specifications/c-2.gemspec
 
     OUTPUT
 
-    assert_equal expected.lines.sort, @ui.output.lines.sort
+    assert_equal expected, @ui.output
 
     assert_equal Gem.dir,  @userhome
     assert_equal Gem.path, [@gemhome, @userhome]
@@ -108,13 +108,13 @@ Removed file specifications/c-2.gemspec
 
     expected = <<-OUTPUT
 Checking #{@gemhome}
+Extra file specifications/c-2.gemspec
 Extra directory gems/b-2
 Extra directory gems/c-2
-Extra file specifications/c-2.gemspec
 
     OUTPUT
 
-    assert_equal expected.lines.sort, @ui.output.lines.sort
+    assert_equal expected, @ui.output
 
     assert_equal Gem.dir,  @userhome
     assert_equal Gem.path, [@gemhome, @userhome]

test/rubygems/test_gem_installer.rb

@@ -261,7 +261,7 @@ load Gem.bin_path('a', 'executable', version)
   end
 
   def test_ensure_loadable_spec_security_policy
-    a, a_gem = util_gem 'a', 2 do |s|
+    _, a_gem = util_gem 'a', 2 do |s|
       s.add_dependency 'garbage ~> 5'
     end
 
@@ -1015,8 +1015,6 @@ load Gem.bin_path('a', 'executable', version)
 
     installer = Gem::Installer.new gem, :install_dir => gemhome2
 
-    gem_home = Gem.dir
-
     build_rake_in do
       use_ui @ui do
         assert installer.pre_install_checks

test/rubygems/test_require.rb

@@ -57,9 +57,8 @@ class TestGemRequire < Gem::TestCase
   end
 
   def test_activate_via_require_respects_loaded_files
+    require 'benchmark' # stdlib
     save_loaded_features do
-      require 'benchmark' # stdlib
-
       a1 = new_spec "a", "1", {"b" => ">= 1"}, "lib/a.rb"
       b1 = new_spec "b", "1", nil, "lib/benchmark.rb"
       b2 = new_spec "b", "2", nil, "lib/benchmark.rb"