rubygems-update 3.3.7 → 3.4.8

data/test/rubygems/test_gem_source_git.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
-require_relative 'helper'
-require 'rubygems/source'
+require_relative "helper"
+require "rubygems/source"
 
 class TestGemSourceGit < Gem::TestCase
   def setup
@@ -10,36 +10,37 @@ class TestGemSourceGit < Gem::TestCase
 
     @hash = Digest::SHA1.hexdigest @repository
 
-    @source = Gem::Source::Git.new @name, @repository, 'master', false
+    @source = Gem::Source::Git.new @name, @repository, nil, false
   end
 
   def test_base_dir
-    assert_equal File.join(Gem.dir, 'bundler'), @source.base_dir
+    assert_equal File.join(Gem.dir, "bundler"), @source.base_dir
 
     @source.root_dir = "#{@gemhome}2"
 
-    assert_equal File.join("#{@gemhome}2", 'bundler'), @source.base_dir
+    assert_equal File.join("#{@gemhome}2", "bundler"), @source.base_dir
   end
 
   def test_checkout
     @source.checkout
 
-    assert_path_exist File.join @source.install_dir, 'a.gemspec'
+    assert_path_exist File.join @source.install_dir, "a.gemspec"
   end
 
-  def test_checkout_master
+  def test_checkout_default
     Dir.chdir @repository do
-      system @git, 'checkout', '-q', '-b', 'other'
-      system @git, 'mv',             'a.gemspec', 'b.gemspec'
-      system @git, 'commit',   '-q', '-a', '-m', 'rename gemspec'
-      system @git, 'checkout', '-q', 'master'
+      default_branch = Gem::Util.popen(@git, "branch", "--show-current").strip
+      system @git, "checkout", "-q", "-b", "other"
+      system @git, "mv",             "a.gemspec", "b.gemspec"
+      system @git, "commit",   "-q", "-a", "-m", "rename gemspec"
+      system @git, "checkout", "-q", default_branch
     end
 
-    @source = Gem::Source::Git.new @name, @repository, 'other', false
+    @source = Gem::Source::Git.new @name, @repository, "other", false
 
     @source.checkout
 
-    assert_path_exist File.join @source.install_dir, 'b.gemspec'
+    assert_path_exist File.join @source.install_dir, "b.gemspec"
   end
 
   def test_checkout_local
@@ -47,9 +48,9 @@ class TestGemSourceGit < Gem::TestCase
 
     @source.checkout
 
-    install_dir = File.join Gem.dir, 'bundler', 'gems', "a-#{@head[0..11]}"
+    install_dir = File.join Gem.dir, "bundler", "gems", "a-#{@head[0..11]}"
 
-    assert_path_not_exist File.join install_dir, 'a.gemspec'
+    assert_path_not_exist File.join install_dir, "a.gemspec"
   end
 
   def test_checkout_local_cached
@@ -59,25 +60,31 @@ class TestGemSourceGit < Gem::TestCase
 
     @source.checkout
 
-    assert_path_exist File.join @source.install_dir, 'a.gemspec'
+    assert_path_exist File.join @source.install_dir, "a.gemspec"
   end
 
   def test_checkout_submodules
-    source = Gem::Source::Git.new @name, @repository, 'master', true
+    # We need to allow to checkout submodules with file:// protocol
+    # CVE-2022-39253
+    # https://lore.kernel.org/lkml/xmqq4jw1uku5.fsf@gitster.g/
+    system(@git, *%W[config --global protocol.file.allow always])
 
-    git_gem 'b'
+    source = Gem::Source::Git.new @name, @repository, nil, true
 
-    Dir.chdir 'git/a' do
-      output, status = Open3.capture2e(@git, 'submodule', '--quiet', 'add', File.expand_path('../b'), 'b')
+    git_gem "b"
+
+    Dir.chdir "git/a" do
+      require "open3"
+      output, status = Open3.capture2e(@git, "submodule", "--quiet", "add", File.expand_path("../b"), "b")
       assert status.success?, output
 
-      system @git, 'commit', '--quiet', '-m', 'add submodule b'
+      system @git, "commit", "--quiet", "-m", "add submodule b"
     end
 
     source.checkout
 
-    assert_path_exist File.join source.install_dir, 'a.gemspec'
-    assert_path_exist File.join source.install_dir, 'b/b.gemspec'
+    assert_path_exist File.join source.install_dir, "a.gemspec"
+    assert_path_exist File.join source.install_dir, "b/b.gemspec"
   end
 
   def test_cache
@@ -86,7 +93,7 @@ class TestGemSourceGit < Gem::TestCase
     assert_path_exist @source.repo_cache_dir
 
     Dir.chdir @source.repo_cache_dir do
-      assert_equal @head, Gem::Util.popen(@git, 'rev-parse', 'master').strip
+      assert_equal @head, Gem::Util.popen(@git, "rev-parse", "HEAD").strip
     end
   end
 
@@ -114,17 +121,17 @@ class TestGemSourceGit < Gem::TestCase
     assert_equal @source, @source.dup
 
     source =
-      Gem::Source::Git.new @source.name, @source.repository, 'other', false
+      Gem::Source::Git.new @source.name, @source.repository, "other", false
 
     refute_equal @source, source
 
     source =
-      Gem::Source::Git.new @source.name, 'repo/other', @source.reference, false
+      Gem::Source::Git.new @source.name, "repo/other", @source.reference, false
 
     refute_equal @source, source
 
     source =
-      Gem::Source::Git.new 'b', @source.repository, @source.reference, false
+      Gem::Source::Git.new "b", @source.repository, @source.reference, false
 
     refute_equal @source, source
 
@@ -138,7 +145,7 @@ class TestGemSourceGit < Gem::TestCase
   def test_install_dir
     @source.cache
 
-    expected = File.join Gem.dir, 'bundler', 'gems', "a-#{@head[0..11]}"
+    expected = File.join Gem.dir, "bundler", "gems", "a-#{@head[0..11]}"
 
     assert_equal expected, @source.install_dir
   end
@@ -151,14 +158,14 @@ class TestGemSourceGit < Gem::TestCase
 
   def test_repo_cache_dir
     expected =
-      File.join Gem.dir, 'cache', 'bundler', 'git', "a-#{@hash}"
+      File.join Gem.dir, "cache", "bundler", "git", "a-#{@hash}"
 
     assert_equal expected, @source.repo_cache_dir
 
     @source.root_dir = "#{@gemhome}2"
 
     expected =
-      File.join "#{@gemhome}2", 'cache', 'bundler', 'git', "a-#{@hash}"
+      File.join "#{@gemhome}2", "cache", "bundler", "git", "a-#{@hash}"
 
     assert_equal expected, @source.repo_cache_dir
   end
@@ -169,20 +176,20 @@ class TestGemSourceGit < Gem::TestCase
     assert_equal @head, @source.rev_parse
 
     Dir.chdir @repository do
-      system @git, 'checkout', '--quiet', '-b', 'other'
+      system @git, "checkout", "--quiet", "-b", "other"
     end
 
-    master_head = @head
+    default_head = @head
 
-    git_gem 'a', 2
+    git_gem "a", 2
 
-    source = Gem::Source::Git.new @name, @repository, 'other', false
+    source = Gem::Source::Git.new @name, @repository, "other", false
 
     source.cache
 
-    refute_equal master_head, source.rev_parse
+    refute_equal default_head, source.rev_parse
 
-    source = Gem::Source::Git.new @name, @repository, 'nonexistent', false
+    source = Gem::Source::Git.new @name, @repository, "nonexistent", false
 
     source.cache
 
@@ -203,38 +210,38 @@ class TestGemSourceGit < Gem::TestCase
   end
 
   def test_spaceship
-    git       = Gem::Source::Git.new 'a', 'git/a', 'master', false
+    git       = Gem::Source::Git.new "a", "git/a", nil, false
     remote    = Gem::Source.new @gem_repo
     installed = Gem::Source::Installed.new
-    vendor    = Gem::Source::Vendor.new 'vendor/foo'
+    vendor    = Gem::Source::Vendor.new "vendor/foo"
 
-    assert_equal(0, git.<=>(git),       'git <=> git')
+    assert_equal(0, git.<=>(git),       "git <=> git")
 
-    assert_equal(1, git.<=>(remote),    'git <=> remote')
-    assert_equal(-1, remote.<=>(git), 'remote <=> git')
+    assert_equal(1, git.<=>(remote),    "git <=> remote")
+    assert_equal(-1, remote.<=>(git), "remote <=> git")
 
-    assert_equal(1, git.<=>(installed), 'git <=> installed')
-    assert_equal(-1, installed.<=>(git), 'installed <=> git')
+    assert_equal(1, git.<=>(installed), "git <=> installed")
+    assert_equal(-1, installed.<=>(git), "installed <=> git")
 
-    assert_equal(-1, git.<=>(vendor), 'git <=> vendor')
-    assert_equal(1, vendor.<=>(git), 'vendor <=> git')
+    assert_equal(-1, git.<=>(vendor), "git <=> vendor")
+    assert_equal(1, vendor.<=>(git), "vendor <=> git")
   end
 
   def test_specs
-    source = Gem::Source::Git.new @name, @repository, 'master', true
+    source = Gem::Source::Git.new @name, @repository, nil, true
 
-    Dir.chdir 'git/a' do
-      FileUtils.mkdir 'b'
+    Dir.chdir "git/a" do
+      FileUtils.mkdir "b"
 
-      Dir.chdir 'b' do
-        b = Gem::Specification.new 'b', 1
+      Dir.chdir "b" do
+        b = Gem::Specification.new "b", 1
 
-        File.open 'b.gemspec', 'w' do |io|
+        File.open "b.gemspec", "w" do |io|
           io.write b.to_ruby
         end
 
-        system @git, 'add', 'b.gemspec'
-        system @git, 'commit', '--quiet', '-m', 'add b/b.gemspec'
+        system @git, "add", "b.gemspec"
+        system @git, "commit", "--quiet", "-m", "add b/b.gemspec"
       end
     end
 
@@ -251,11 +258,11 @@ class TestGemSourceGit < Gem::TestCase
     base_dir = File.dirname File.dirname source.install_dir
 
     assert_equal source.install_dir, a_spec.full_gem_path
-    assert_equal File.join(source.install_dir, 'a.gemspec'), a_spec.loaded_from
+    assert_equal File.join(source.install_dir, "a.gemspec"), a_spec.loaded_from
     assert_equal base_dir, a_spec.base_dir
 
     extension_dir =
-      File.join Gem.dir, 'bundler', 'extensions',
+      File.join Gem.dir, "bundler", "extensions",
         Gem::Platform.local.to_s, Gem.extension_api_version,
         "a-#{source.dir_shortref}"
 
@@ -263,8 +270,8 @@ class TestGemSourceGit < Gem::TestCase
 
     b_spec = specs.shift
 
-    assert_equal File.join(source.install_dir, 'b'), b_spec.full_gem_path
-    assert_equal File.join(source.install_dir, 'b', 'b.gemspec'),
+    assert_equal File.join(source.install_dir, "b"), b_spec.full_gem_path
+    assert_equal File.join(source.install_dir, "b", "b.gemspec"),
                  b_spec.loaded_from
     assert_equal base_dir, b_spec.base_dir
 
@@ -272,7 +279,7 @@ class TestGemSourceGit < Gem::TestCase
   end
 
   def test_specs_local
-    source = Gem::Source::Git.new @name, @repository, 'master', true
+    source = Gem::Source::Git.new @name, @repository, nil, true
     source.remote = false
 
     capture_output do
@@ -288,15 +295,15 @@ class TestGemSourceGit < Gem::TestCase
     assert_equal @hash, @source.uri_hash
 
     source =
-      Gem::Source::Git.new 'a', 'http://git@example/repo.git', 'master', false
+      Gem::Source::Git.new "a", "http://git@example/repo.git", nil, false
 
-    assert_equal '291c4caac7feba8bb64c297987028acb3dde6cfe',
+    assert_equal "291c4caac7feba8bb64c297987028acb3dde6cfe",
                  source.uri_hash
 
     source =
-      Gem::Source::Git.new 'a', 'HTTP://git@EXAMPLE/repo.git', 'master', false
+      Gem::Source::Git.new "a", "HTTP://git@EXAMPLE/repo.git", nil, false
 
-    assert_equal '291c4caac7feba8bb64c297987028acb3dde6cfe',
+    assert_equal "291c4caac7feba8bb64c297987028acb3dde6cfe",
                  source.uri_hash
   end
 end

data/test/rubygems/test_gem_source_installed.rb

@@ -1,34 +1,34 @@
 # frozen_string_literal: true
-require_relative 'helper'
-require 'rubygems/source'
+require_relative "helper"
+require "rubygems/source"
 
 class TestGemSourceInstalled < Gem::TestCase
   def test_spaceship
-    a1 = quick_gem 'a', '1'
+    a1 = quick_gem "a", "1"
     util_build_gem a1
 
     remote    = Gem::Source.new @gem_repo
     specific  = Gem::Source::SpecificFile.new a1.cache_file
     installed = Gem::Source::Installed.new
     local     = Gem::Source::Local.new
-    git       = Gem::Source::Git.new 'a', 'a', 'master'
-    vendor    = Gem::Source::Vendor.new 'a'
+    git       = Gem::Source::Git.new "a", "a", nil
+    vendor    = Gem::Source::Vendor.new "a"
 
-    assert_equal(0, installed.<=>(installed), 'installed <=> installed')
+    assert_equal(0, installed.<=>(installed), "installed <=> installed")
 
-    assert_equal(-1, remote.<=>(installed), 'remote <=> installed')
-    assert_equal(1, installed.<=>(remote),    'installed <=> remote')
+    assert_equal(-1, remote.<=>(installed), "remote <=> installed")
+    assert_equal(1, installed.<=>(remote),    "installed <=> remote")
 
-    assert_equal(1, installed.<=>(local),     'installed <=> local')
-    assert_equal(-1, local.<=>(installed), 'local <=> installed')
+    assert_equal(1, installed.<=>(local),     "installed <=> local")
+    assert_equal(-1, local.<=>(installed), "local <=> installed")
 
-    assert_equal(-1, specific.<=>(installed), 'specific <=> installed')
-    assert_equal(1, installed.<=>(specific),  'installed <=> specific')
+    assert_equal(-1, specific.<=>(installed), "specific <=> installed")
+    assert_equal(1, installed.<=>(specific),  "installed <=> specific")
 
-    assert_equal(1, git. <=>(installed), 'git <=> installed')
-    assert_equal(-1, installed.<=>(git), 'installed <=> git')
+    assert_equal(1, git. <=>(installed), "git <=> installed")
+    assert_equal(-1, installed.<=>(git), "installed <=> git")
 
-    assert_equal(1, vendor.<=>(installed), 'vendor <=> installed')
-    assert_equal(-1, installed.<=>(vendor), 'installed <=> vendor')
+    assert_equal(1, vendor.<=>(installed), "vendor <=> installed")
+    assert_equal(-1, installed.<=>(vendor), "installed <=> vendor")
   end
 end

data/test/rubygems/test_gem_source_list.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
-require 'rubygems'
-require 'rubygems/source_list'
-require_relative 'helper'
+require "rubygems"
+require "rubygems/source_list"
+require_relative "helper"
 
 class TestGemSourceList < Gem::TestCase
   def setup
@@ -45,7 +45,7 @@ class TestGemSourceList < Gem::TestCase
   def test_clear
     sl = Gem::SourceList.new
 
-    sl << 'http://source.example'
+    sl << "http://source.example"
 
     sl.clear
 
@@ -76,7 +76,7 @@ class TestGemSourceList < Gem::TestCase
 
     assert_empty sl
 
-    sl << 'http://source.example'
+    sl << "http://source.example"
 
     refute_empty sl
   end

data/test/rubygems/test_gem_source_local.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
-require_relative 'helper'
-require 'rubygems/source'
+require_relative "helper"
+require "rubygems/source"
 
-require 'fileutils'
+require "fileutils"
 
 class TestGemSourceLocal < Gem::TestCase
   def setup
@@ -10,9 +10,9 @@ class TestGemSourceLocal < Gem::TestCase
 
     @sl = Gem::Source::Local.new
 
-    @a, @a_gem = util_gem "a", '1'
-    @ap, @ap_gem = util_gem "a", '2.a'
-    @b, @b_gem = util_gem "b", '1'
+    @a, @a_gem = util_gem "a", "1"
+    @ap, @ap_gem = util_gem "a", "2.a"
+    @b, @b_gem = util_gem "b", "1"
 
     FileUtils.mv @a_gem, @tempdir
     FileUtils.mv @ap_gem, @tempdir
@@ -84,7 +84,7 @@ class TestGemSourceLocal < Gem::TestCase
   end
 
   def test_spaceship
-    a1 = quick_gem 'a', '1'
+    a1 = quick_gem "a", "1"
     util_build_gem a1
 
     remote    = Gem::Source.new @gem_repo
@@ -92,15 +92,15 @@ class TestGemSourceLocal < Gem::TestCase
     installed = Gem::Source::Installed.new
     local     = Gem::Source::Local.new
 
-    assert_equal(0, local.<=>(local), 'local <=> local')
+    assert_equal(0, local.<=>(local), "local <=> local")
 
-    assert_equal(-1, remote.<=>(local), 'remote <=> local')
-    assert_equal(1, local.<=>(remote), 'local <=> remote')
+    assert_equal(-1, remote.<=>(local), "remote <=> local")
+    assert_equal(1, local.<=>(remote), "local <=> remote")
 
-    assert_equal(1, installed.<=>(local), 'installed <=> local')
-    assert_equal(-1, local.<=>(installed), 'local <=> installed')
+    assert_equal(1, installed.<=>(local), "installed <=> local")
+    assert_equal(-1, local.<=>(installed), "local <=> installed")
 
-    assert_equal(-1, specific.<=>(local), 'specific <=> local')
-    assert_equal(1, local.<=>(specific), 'local <=> specific')
+    assert_equal(-1, specific.<=>(local), "specific <=> local")
+    assert_equal(1, local.<=>(specific), "local <=> specific")
   end
 end

data/test/rubygems/test_gem_source_lock.rb

@@ -1,24 +1,24 @@
 # frozen_string_literal: true
-require_relative 'helper'
+require_relative "helper"
 
 class TestGemSourceLock < Gem::TestCase
   def test_fetch_spec
     spec_fetcher do |fetcher|
-      fetcher.spec 'a', 1
+      fetcher.spec "a", 1
     end
 
-    name_tuple = Gem::NameTuple.new 'a', v(1), 'ruby'
+    name_tuple = Gem::NameTuple.new "a", v(1), "ruby"
 
     remote = Gem::Source.new @gem_repo
     lock   = Gem::Source::Lock.new remote
 
     spec = lock.fetch_spec name_tuple
 
-    assert_equal 'a-1', spec.full_name
+    assert_equal "a-1", spec.full_name
   end
 
   def test_equals2
-    git    = Gem::Source::Git.new 'a', 'git/a', 'master', false
+    git    = Gem::Source::Git.new "a", "git/a", nil, false
     g_lock = Gem::Source::Lock.new git
 
     installed = Gem::Source::Installed.new
@@ -30,77 +30,77 @@ class TestGemSourceLock < Gem::TestCase
   end
 
   def test_spaceship
-    git    = Gem::Source::Git.new 'a', 'git/a', 'master', false
+    git    = Gem::Source::Git.new "a", "git/a", nil, false
     g_lock = Gem::Source::Lock.new git
 
     installed = Gem::Source::Installed.new
     i_lock    = Gem::Source::Lock.new installed
 
-    vendor = Gem::Source::Vendor.new 'vendor/a'
+    vendor = Gem::Source::Vendor.new "vendor/a"
     v_lock = Gem::Source::Lock.new vendor
 
-    assert_equal(0, g_lock.<=>(g_lock), 'g_lock <=> g_lock')
-    assert_equal(0, i_lock.<=>(i_lock), 'i_lock <=> i_lock')
-    assert_equal(0, v_lock.<=>(v_lock), 'v_lock <=> v_lock')
+    assert_equal(0, g_lock.<=>(g_lock), "g_lock <=> g_lock")
+    assert_equal(0, i_lock.<=>(i_lock), "i_lock <=> i_lock")
+    assert_equal(0, v_lock.<=>(v_lock), "v_lock <=> v_lock")
 
-    assert_equal(1, g_lock.<=>(i_lock), 'g_lock <=> i_lock')
-    assert_equal(-1, i_lock.<=>(g_lock), 'i_lock <=> g_lock')
+    assert_equal(1, g_lock.<=>(i_lock), "g_lock <=> i_lock")
+    assert_equal(-1, i_lock.<=>(g_lock), "i_lock <=> g_lock")
 
-    assert_equal(-1, g_lock.<=>(v_lock), 'g_lock <=> v_lock')
-    assert_equal(1, v_lock.<=>(g_lock), 'v_lock <=> g_lock')
+    assert_equal(-1, g_lock.<=>(v_lock), "g_lock <=> v_lock")
+    assert_equal(1, v_lock.<=>(g_lock), "v_lock <=> g_lock")
 
-    assert_equal(-1, i_lock.<=>(v_lock), 'i_lock <=> v_lock')
-    assert_equal(1, v_lock.<=>(i_lock), 'i_lock <=> v_lock')
+    assert_equal(-1, i_lock.<=>(v_lock), "i_lock <=> v_lock")
+    assert_equal(1, v_lock.<=>(i_lock), "i_lock <=> v_lock")
   end
 
   def test_spaceship_git
-    git  = Gem::Source::Git.new 'a', 'git/a', 'master', false
+    git  = Gem::Source::Git.new "a", "git/a", nil, false
     lock = Gem::Source::Lock.new git
 
-    assert_equal(1, lock.<=>(git),  'lock <=> git')
-    assert_equal(-1, git.<=>(lock), 'git <=> lock')
+    assert_equal(1, lock.<=>(git),  "lock <=> git")
+    assert_equal(-1, git.<=>(lock), "git <=> lock")
   end
 
   def test_spaceship_installed
     installed = Gem::Source::Installed.new
     lock      = Gem::Source::Lock.new installed
 
-    assert_equal(1, lock.<=>(installed), 'lock <=> installed')
-    assert_equal(-1, installed.<=>(lock), 'installed <=> lock')
+    assert_equal(1, lock.<=>(installed), "lock <=> installed")
+    assert_equal(-1, installed.<=>(lock), "installed <=> lock")
   end
 
   def test_spaceship_local
     local = Gem::Source::Local.new
     lock  = Gem::Source::Lock.new local # nonsense
 
-    assert_equal(1, lock.<=>(local), 'lock <=> local')
-    assert_equal(-1, local.<=>(lock), 'local <=> lock')
+    assert_equal(1, lock.<=>(local), "lock <=> local")
+    assert_equal(-1, local.<=>(lock), "local <=> lock")
   end
 
   def test_spaceship_remote
     remote = Gem::Source.new @gem_repo
     lock   = Gem::Source::Lock.new remote
 
-    assert_equal(1, lock.<=>(remote), 'lock <=> remote')
-    assert_equal(-1, remote.<=>(lock), 'remote <=> lock')
+    assert_equal(1, lock.<=>(remote), "lock <=> remote")
+    assert_equal(-1, remote.<=>(lock), "remote <=> lock")
   end
 
   def test_spaceship_specific_file
-    _, gem = util_gem 'a', 1
+    _, gem = util_gem "a", 1
 
     specific = Gem::Source::SpecificFile.new gem
     lock     = Gem::Source::Lock.new specific # nonsense
 
-    assert_equal(1, lock.<=>(specific), 'lock <=> specific')
-    assert_equal(-1, specific.<=>(lock),      'specific <=> lock')
+    assert_equal(1, lock.<=>(specific), "lock <=> specific")
+    assert_equal(-1, specific.<=>(lock),      "specific <=> lock")
   end
 
   def test_spaceship_vendor
-    vendor = Gem::Source::Vendor.new 'vendor/a'
+    vendor = Gem::Source::Vendor.new "vendor/a"
     lock   = Gem::Source::Lock.new vendor
 
-    assert_equal(1, lock.<=>(vendor), 'lock <=> vendor')
-    assert_equal(-1, vendor.<=>(lock), 'vendor <=> lock')
+    assert_equal(1, lock.<=>(vendor), "lock <=> vendor")
+    assert_equal(-1, vendor.<=>(lock), "vendor <=> lock")
   end
 
   def test_uri

data/test/rubygems/test_gem_source_specific_file.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
-require_relative 'helper'
-require 'rubygems/source'
+require_relative "helper"
+require "rubygems/source"
 
 class TestGemSourceSpecificFile < Gem::TestCase
   def setup
     super
 
-    @a, @a_gem = util_gem "a", '1'
+    @a, @a_gem = util_gem "a", "1"
     @sf = Gem::Source::SpecificFile.new(@a_gem)
   end
 
@@ -37,7 +37,7 @@ class TestGemSourceSpecificFile < Gem::TestCase
   end
 
   def test_spaceship
-    a1 = quick_gem 'a', '1'
+    a1 = quick_gem "a", "1"
     util_build_gem a1
 
     remote    = Gem::Source.new @gem_repo
@@ -45,31 +45,31 @@ class TestGemSourceSpecificFile < Gem::TestCase
     installed = Gem::Source::Installed.new
     local     = Gem::Source::Local.new
 
-    assert_equal(0, specific.<=>(specific), 'specific <=> specific')
+    assert_equal(0, specific.<=>(specific), "specific <=> specific")
 
-    assert_equal(-1, remote.<=>(specific), 'remote <=> specific')
-    assert_equal(1, specific.<=>(remote), 'specific <=> remote')
+    assert_equal(-1, remote.<=>(specific), "remote <=> specific")
+    assert_equal(1, specific.<=>(remote), "specific <=> remote")
 
-    assert_equal(-1, specific.<=>(local),     'specific <=> local')
-    assert_equal(1, local.    <=>(specific),  'local <=> specific')
+    assert_equal(-1, specific.<=>(local),     "specific <=> local")
+    assert_equal(1, local.    <=>(specific),  "local <=> specific")
 
-    assert_equal(-1, specific. <=>(installed), 'specific <=> installed')
-    assert_equal(1, installed.<=>(specific), 'installed <=> specific')
+    assert_equal(-1, specific. <=>(installed), "specific <=> installed")
+    assert_equal(1, installed.<=>(specific), "installed <=> specific")
 
-    a2 = quick_gem 'a', '2'
+    a2 = quick_gem "a", "2"
     util_build_gem a2
 
-    b1 = quick_gem 'b', '1'
+    b1 = quick_gem "b", "1"
     util_build_gem b1
 
     a1_source = specific
     a2_source = Gem::Source::SpecificFile.new a2.cache_file
     b1_source = Gem::Source::SpecificFile.new b1.cache_file
 
-    assert_nil       a1_source.<=>(b1_source), 'a1_source <=> b1_source'
+    assert_nil       a1_source.<=>(b1_source), "a1_source <=> b1_source"
 
-    assert_equal(-1, a1_source.<=>(a2_source), 'a1_source <=> a2_source')
-    assert_equal(0, a1_source.<=>(a1_source), 'a1_source <=> a1_source')
-    assert_equal(1, a2_source.<=>(a1_source), 'a2_source <=> a1_source')
+    assert_equal(-1, a1_source.<=>(a2_source), "a1_source <=> a2_source")
+    assert_equal(0, a1_source.<=>(a1_source), "a1_source <=> a1_source")
+    assert_equal(1, a2_source.<=>(a1_source), "a2_source <=> a1_source")
   end
 end