Bug #7618: TestGemDoctor failures

• Status: Closed
• Assignee: drbrain (Eric Hodel)
• Target version: 2.0.0
• ruby -v: ruby 1.8.7 (2012-02-08 patchlevel 358) [x86_64-linux]

[Eregon (Benoit Daloze)]

I guess it has already been noticed as shown on RubyCI but here is the failure:

[1/5] TestGemDoctor#test_doctor = 0.02 s

1. Failure:
   test_doctor(TestGemDoctor) [/home/eregon/prefix/src/ruby/test/rubygems/test_gem_doctor.rb:64]:
   --- expected
   +++ actual
   @@ -1,6 +1,6 @@
   "Checking /home/eregon/prefix/src/ruby/tmp/test/test_rubygems_22593/gemhome
   -Removed directory gems/b-2
   Removed directory gems/c-2
   +Removed directory gems/b-2
   Removed file specifications/c-2.gemspec

"

[5/5] TestGemDoctor#test_doctor_dry_run = 0.03 s
2) Failure:
test_doctor_dry_run(TestGemDoctor) [/home/eregon/prefix/src/ruby/test/rubygems/test_gem_doctor.rb:117]:
--- expected
+++ actual
@@ -1,6 +1,6 @@
"Checking /home/eregon/prefix/src/ruby/tmp/test/test_rubygems_22593/gemhome
-Extra directory gems/b-2
Extra directory gems/c-2
+Extra directory gems/b-2
Extra file specifications/c-2.gemspec

"

[luislavena (Luis Lavena)]

• Status changed from Open to Assigned

[drbrain (Eric Hodel)]

• Status changed from Assigned to Closed
• % Done changed from 0 to 100

This issue was solved with changeset r38691.
Benoit, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

• 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