Bug #7618

TestGemDoctor failures

Added by Benoit Daloze about 2 years ago. Updated about 2 years ago.

[ruby-core:51114]
Status:Closed
Priority:Normal
Assignee:Eric Hodel
ruby -v:ruby 1.8.7 (2012-02-08 patchlevel 358) [x86_64-linux] Backport:

Description

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

"

Associated revisions

Revision 38691
Added by Eric Hodel about 2 years ago

  • 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

Revision 38691
Added by Eric Hodel about 2 years ago

  • 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

History

#1 Updated by Luis Lavena about 2 years ago

  • Status changed from Open to Assigned

#2 Updated by Eric Hodel about 2 years ago

  • 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

Also available in: Atom PDF