From 5d790264ea753d06de7a131387863d24199790fb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?David=20Rodr=C3=ADguez?= Date: Thu, 2 Mar 2023 14:30:33 +0100 Subject: [PATCH] [rubygems/rubygems] Remove hardcoded master branch references Let RubyGems use the default branch defined by each repo. https://github.com/rubygems/rubygems/commit/54bd3e9ebd --- test/rubygems/helper.rb | 3 +-- .../test_gem_request_set_lockfile_parser.rb | 4 +-- test/rubygems/test_gem_resolver_git_set.rb | 18 ++++++------- .../test_gem_resolver_git_specification.rb | 2 +- test/rubygems/test_gem_source.rb | 2 +- test/rubygems/test_gem_source_git.rb | 25 ++++++++++--------- test/rubygems/test_gem_source_installed.rb | 2 +- test/rubygems/test_gem_source_lock.rb | 6 ++--- test/rubygems/test_gem_source_vendor.rb | 2 +- 9 files changed, 32 insertions(+), 32 deletions(-) diff --git a/test/rubygems/helper.rb b/test/rubygems/helper.rb index c4c69be75e..54fe859174 100644 --- a/test/rubygems/helper.rb +++ b/test/rubygems/helper.rb @@ -563,14 +563,13 @@ class Gem::TestCase < Test::Unit::TestCase Dir.chdir directory do unless File.exist? ".git" system @git, "init", "--quiet" - system @git, "checkout", "-b", "master", "--quiet" system @git, "config", "user.name", "RubyGems Tests" system @git, "config", "user.email", "rubygems@example" end system @git, "add", gemspec system @git, "commit", "-a", "-m", "a non-empty commit message", "--quiet" - head = Gem::Util.popen(@git, "rev-parse", "master").strip + head = Gem::Util.popen(@git, "rev-parse", "HEAD").strip end return name, git_spec.version, directory, head diff --git a/test/rubygems/test_gem_request_set_lockfile_parser.rb b/test/rubygems/test_gem_request_set_lockfile_parser.rb index ecb145e697..69a3a328aa 100644 --- a/test/rubygems/test_gem_request_set_lockfile_parser.rb +++ b/test/rubygems/test_gem_request_set_lockfile_parser.rb @@ -257,7 +257,7 @@ DEPENDENCIES write_lockfile <<-LOCKFILE GIT remote: git://example/a.git - revision: master + revision: abranch specs: a (2) b (>= 3) @@ -289,7 +289,7 @@ DEPENDENCIES git_set.specs.values.first.dependencies expected = { - "a" => %w[git://example/a.git master], + "a" => %w[git://example/a.git abranch], } assert_equal expected, git_set.repositories diff --git a/test/rubygems/test_gem_resolver_git_set.rb b/test/rubygems/test_gem_resolver_git_set.rb index f7063b3d45..6005b1e40f 100644 --- a/test/rubygems/test_gem_resolver_git_set.rb +++ b/test/rubygems/test_gem_resolver_git_set.rb @@ -13,7 +13,7 @@ class TestGemResolverGitSet < Gem::TestCase def test_add_git_gem name, version, repository, = git_gem - @set.add_git_gem name, repository, "master", false + @set.add_git_gem name, repository, nil, false dependency = dep "a" @@ -27,7 +27,7 @@ class TestGemResolverGitSet < Gem::TestCase def test_add_git_gem_submodules name, _, repository, = git_gem - @set.add_git_gem name, repository, "master", true + @set.add_git_gem name, repository, nil, true dependency = dep "a" @@ -57,7 +57,7 @@ class TestGemResolverGitSet < Gem::TestCase def test_find_all name, _, repository, = git_gem - @set.add_git_gem name, repository, "master", false + @set.add_git_gem name, repository, nil, false dependency = dep "a", "~> 1.0" req = Gem::Resolver::DependencyRequest.new dependency, nil @@ -73,7 +73,7 @@ class TestGemResolverGitSet < Gem::TestCase def test_find_all_local name, _, repository, = git_gem - @set.add_git_gem name, repository, "master", false + @set.add_git_gem name, repository, nil, false @set.remote = false dependency = dep "a", "~> 1.0" @@ -88,7 +88,7 @@ class TestGemResolverGitSet < Gem::TestCase def test_find_all_prerelease name, _, repository, = git_gem "a", "1.a" - @set.add_git_gem name, repository, "master", false + @set.add_git_gem name, repository, nil, false dependency = dep "a", ">= 0" req = Gem::Resolver::DependencyRequest.new dependency, nil @@ -122,7 +122,7 @@ class TestGemResolverGitSet < Gem::TestCase def test_prefetch name, _, repository, = git_gem - @set.add_git_gem name, repository, "master", false + @set.add_git_gem name, repository, nil, false dependency = dep name req = Gem::Resolver::DependencyRequest.new dependency, nil @@ -136,7 +136,7 @@ class TestGemResolverGitSet < Gem::TestCase def test_prefetch_cache name, _, repository, = git_gem - @set.add_git_gem name, repository, "master", false + @set.add_git_gem name, repository, nil, false dependency = dep name req = Gem::Resolver::DependencyRequest.new dependency, nil @@ -154,7 +154,7 @@ class TestGemResolverGitSet < Gem::TestCase def test_prefetch_filter name, _, repository, = git_gem - @set.add_git_gem name, repository, "master", false + @set.add_git_gem name, repository, nil, false dependency = dep "b" req = Gem::Resolver::DependencyRequest.new dependency, nil @@ -168,7 +168,7 @@ class TestGemResolverGitSet < Gem::TestCase def test_prefetch_root_dir name, _, repository, = git_gem - @set.add_git_gem name, repository, "master", false + @set.add_git_gem name, repository, nil, false dependency = dep name req = Gem::Resolver::DependencyRequest.new dependency, nil diff --git a/test/rubygems/test_gem_resolver_git_specification.rb b/test/rubygems/test_gem_resolver_git_specification.rb index 454fd9c6e4..615949bc2a 100644 --- a/test/rubygems/test_gem_resolver_git_specification.rb +++ b/test/rubygems/test_gem_resolver_git_specification.rb @@ -84,7 +84,7 @@ class TestGemResolverGitSpecification < Gem::TestCase system @git, "commit", "--quiet", "-m", "Add extension files" end - source = Gem::Source::Git.new name, repository, "master", true + source = Gem::Source::Git.new name, repository, nil, true spec = source.specs.first diff --git a/test/rubygems/test_gem_source.rb b/test/rubygems/test_gem_source.rb index e164215b57..5a6c9ffc94 100644 --- a/test/rubygems/test_gem_source.rb +++ b/test/rubygems/test_gem_source.rb @@ -30,7 +30,7 @@ class TestGemSource < Gem::TestCase def test_initialize_git repository = "git@example:a.git" - source = Gem::Source::Git.new "a", repository, "master", false + source = Gem::Source::Git.new "a", repository, nil, false assert_equal repository, source.uri end diff --git a/test/rubygems/test_gem_source_git.rb b/test/rubygems/test_gem_source_git.rb index 147c6df1d2..c21dd88f8f 100644 --- a/test/rubygems/test_gem_source_git.rb +++ b/test/rubygems/test_gem_source_git.rb @@ -10,7 +10,7 @@ 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 @@ -27,12 +27,13 @@ class TestGemSourceGit < Gem::TestCase assert_path_exist File.join @source.install_dir, "a.gemspec" end - def test_checkout_master + def test_checkout_default Dir.chdir @repository do + 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", "master" + system @git, "checkout", "-q", default_branch end @source = Gem::Source::Git.new @name, @repository, "other", false @@ -68,7 +69,7 @@ class TestGemSourceGit < Gem::TestCase # https://lore.kernel.org/lkml/xmqq4jw1uku5.fsf@gitster.g/ system(@git, *%W[config --global protocol.file.allow always]) - source = Gem::Source::Git.new @name, @repository, "master", true + source = Gem::Source::Git.new @name, @repository, nil, true git_gem "b" @@ -92,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 @@ -178,7 +179,7 @@ class TestGemSourceGit < Gem::TestCase system @git, "checkout", "--quiet", "-b", "other" end - master_head = @head + default_head = @head git_gem "a", 2 @@ -186,7 +187,7 @@ class TestGemSourceGit < Gem::TestCase 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 @@ -209,7 +210,7 @@ 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" @@ -227,7 +228,7 @@ class TestGemSourceGit < Gem::TestCase 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" @@ -278,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 @@ -294,13 +295,13 @@ 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", 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", source.uri_hash diff --git a/test/rubygems/test_gem_source_installed.rb b/test/rubygems/test_gem_source_installed.rb index 7fb5017c59..5638e4b0b8 100644 --- a/test/rubygems/test_gem_source_installed.rb +++ b/test/rubygems/test_gem_source_installed.rb @@ -11,7 +11,7 @@ class TestGemSourceInstalled < Gem::TestCase 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" + git = Gem::Source::Git.new "a", "a", nil vendor = Gem::Source::Vendor.new "a" assert_equal(0, installed.<=>(installed), "installed <=> installed") diff --git a/test/rubygems/test_gem_source_lock.rb b/test/rubygems/test_gem_source_lock.rb index ff9465d0ad..6344feec59 100644 --- a/test/rubygems/test_gem_source_lock.rb +++ b/test/rubygems/test_gem_source_lock.rb @@ -18,7 +18,7 @@ class TestGemSourceLock < Gem::TestCase 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,7 +30,7 @@ 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 @@ -54,7 +54,7 @@ class TestGemSourceLock < Gem::TestCase 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") diff --git a/test/rubygems/test_gem_source_vendor.rb b/test/rubygems/test_gem_source_vendor.rb index 29846e5c13..50a446c90f 100644 --- a/test/rubygems/test_gem_source_vendor.rb +++ b/test/rubygems/test_gem_source_vendor.rb @@ -12,7 +12,7 @@ class TestGemSourceVendor < Gem::TestCase def test_spaceship vendor = Gem::Source::Vendor.new "vendor/foo" remote = Gem::Source.new @gem_repo - git = Gem::Source::Git.new "a", "a", "master" + git = Gem::Source::Git.new "a", "a", nil installed = Gem::Source::Installed.new assert_equal(0, vendor.<=>(vendor), "vendor <=> vendor")