ra10ke 4.5.1 → 4.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e89462c238286812deefe1c66b23bd3458f10f68545f19394cda6e5cae239fa3
-  data.tar.gz: be70614d1b793aaae4e3e9151186709b70d68506687b3a3590f6e79ed6fe4298
+  metadata.gz: a77b7170e6dfb723d530dc22553a16ca198a0b361ca3a2aac5169b05fb0e4c89
+  data.tar.gz: b8025d8e38a8051a8a3cf4baa8b214e74b96c8ea5230364a37b23ac22f210a61
 SHA512:
-  metadata.gz: 95bbe0ab0fea1b2598c75f4e0f7cfee5c1b77acc457574bd55627bd584d9a9ae2b0360f6bb70fcafb8b4df03559b05d63db2ee5abc1f8364276e126ddfaae047
-  data.tar.gz: 0c1d340000c757922d5929b27f61d814c4d7a1b8c1c7220d2a4bd50ba30f42c66bec026eb959bb03b74169ff567a6ec409a0c0c2ac357aa5744e86fbe2dce287
+  metadata.gz: 0047a09a069462ef4d25e9778b83558aa29e9c282106e1f32eafd1fcc87df65d25e31fb9becccfa4d4ae47e4b61d930dafc15fcd56cd56b4669801675c3c415f
+  data.tar.gz: 4d809b1fa050a40ab3e1a265999eb47c1eb337609586b06c87b7fa3537303e42721a6d8ad8c84bd3264d2b0a183e3f2a91334054520730b39bde47193fc346de

data/.github/labeler.yml

@@ -0,0 +1,5 @@
+---
+skip-changelog:
+  - head-branch:
+    - '^release-*'
+    - '^rel[0-9]+'

data/.github/workflows/labeler.yml

@@ -0,0 +1,17 @@
+---
+name: 🏷️ Pull Request Labeler
+
+on:
+  - pull_request_target
+
+permissions: {}
+
+jobs:
+  labeler:
+    name: Labeler
+    runs-on: ubuntu-latest
+    if: github.repository_owner == 'voxpupuli' || github.repository_owner == 'OpenVoxProject'
+    permissions:
+      pull-requests: write
+    steps:
+      - uses: actions/labeler@bf12e9b00b37c5c0ca2b87b79b2daf7891dbda13 # v7.0.0

data/.github/workflows/release.yml

@@ -15,7 +15,7 @@ jobs:
     name: Build the gem
     runs-on: ubuntu-24.04
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v7
       - name: Install Ruby
         uses: ruby/setup-ruby@v1
         with:
@@ -76,7 +76,7 @@ jobs:
         uses: actions/download-artifact@v8
         with:
           name: gem-artifact
-      - uses: rubygems/configure-rubygems-credentials@v2.0.0
+      - uses: rubygems/configure-rubygems-credentials@v2.1.0
       - name: Publish gem to rubygems.org
         shell: bash
         run: gem push *.gem

data/.github/workflows/test.yml

@@ -16,7 +16,7 @@ jobs:
     outputs:
       ruby: ${{ steps.ruby.outputs.versions }}
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v7
       - name: Install Ruby ${{ matrix.ruby }}
         uses: ruby/setup-ruby@v1
         with:
@@ -35,7 +35,7 @@ jobs:
         ruby: ${{ fromJSON(needs.rubocop_and_matrix.outputs.ruby) }}
     name: Ruby ${{ matrix.ruby }}
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v7
       - name: Install Ruby ${{ matrix.ruby }}
         uses: ruby/setup-ruby@v1
         with:

data/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 All notable changes to this project will be documented in this file.
 
+## [v4.6.0](https://github.com/voxpupuli/ra10ke/tree/v4.6.0) (2026-10-13)
+
+[Full Changelog](https://github.com/voxpupuli/ra10ke/compare/v4.5.1...v4.6.0)
+
+**Implemented enhancements:**
+
+- feat: use git-gem in git\_repo class; drop support for git-gem version v1.x [\#152](https://github.com/voxpupuli/ra10ke/pull/152) ([rwaffen](https://github.com/rwaffen))
+
 ## [v4.5.1](https://github.com/voxpupuli/ra10ke/tree/v4.5.1) (2026-05-11)
 
 [Full Changelog](https://github.com/voxpupuli/ra10ke/compare/v4.5.0...v4.5.1)

data/lib/ra10ke/dependencies.rb

@@ -165,8 +165,7 @@ module Ra10ke::Dependencies
             "https://#{::Regexp.last_match(1)}/#{::Regexp.last_match(2)}"
           end
           repo = ::Ra10ke::GitRepo.new(source_url)
-          ref = repo.get_ref_like(mod.expected_version)
-          ref_name = ref ? ref[:name] : "bad url or tag #{mod.expected_version} is missing"
+          ref_name = repo.get_ref_like(mod.expected_version) || "bad url or tag #{mod.expected_version} is missing"
           <<~EOF
             mod '#{mod.name}',
               :git => '#{source_url}',

data/lib/ra10ke/git_repo.rb

@@ -1,16 +1,12 @@
 # frozen_string_literal: true
 
-require 'tempfile'
+require 'git'
+require 'tmpdir'
 
 module Ra10ke
   class GitRepo
     attr_reader :url
 
-    REMOTE_REFS_CMD = 'git ls-remote --symref'
-    CLONE_CMD = 'git clone --no-tags'
-    CURRENT_BRANCH_CMD = 'git symbolic-ref --short HEAD'
-    SHOW_CMD = 'git show'
-
     def initialize(url)
       @url = url
     end
@@ -19,18 +15,19 @@ module Ra10ke
     # @param path [String] - The path to the repository to check
     # @return [String] - The current active branch of the Git repo
     def self.current_branch(path)
-      Dir.chdir(path) do
-        data, success = run_command(CURRENT_BRANCH_CMD)
-        return success ? data.strip : nil
-      end
+      branch = Git.open(path).current_branch
+      branch unless branch == 'HEAD' || branch.to_s.empty?
+    rescue Git::Error, ArgumentError
+      nil
     end
 
-    # @return [Array] - the raw data from the git ls-remote command as lines array
-    # return empty array if url or command failed
+    # @return [Hash] - remote refs grouped by the git gem
+    # return empty hash if the command failed
     def remote_refs
       @remote_refs ||= begin
-        data, success = run_command("#{REMOTE_REFS_CMD} #{url}")
-        success ? data.lines : []
+        Git.ls_remote(url)
+      rescue Git::Error
+        {}
       end
     end
 
@@ -39,89 +36,55 @@ module Ra10ke
       !remote_refs.empty?
     end
 
-    # @return [Boolean] - return true if the commit sha is valid
-    # @param url [String] - the git string either https or ssh url
-    # @param ref [String] - the sha id
-    def valid_commit?(sha)
-      return false if sha.nil? || sha.empty?
-      return true if valid_ref?(sha)
+    def valid_commit?(ref)
+      return false if ref.nil? || ref.empty?
+      return true if valid_ref?(ref)
+      return false unless valid_url?
 
-      # cloning is a last resort if for some reason we cannot
-      # remotely get via ls-remote
       Dir.mktmpdir do |dir|
-        run_command("#{CLONE_CMD} #{url} #{dir}", silent: true)
-        Dir.chdir(dir) do
-          _, status = run_command("#{SHOW_CMD} #{sha}", silent: true)
-          status
-        end
+        repo = Git.clone(url, dir, bare: true)
+        repo.rev_parse("#{ref}^{commit}")
+        true
       end
+    rescue Git::Error
+      false
     end
 
-    # @return [Boolean] - return true if the ref is valid
-    # @param url [String] - the git string either https or ssh url
-    # @param ref [String] - the ref object, branch name, tag name, or commit sha, defaults to HEAD
+    # @param ref [String] branch name, tag name, HEAD, or a full/abbreviated SHA
+    # @return [Boolean]
     def valid_ref?(ref = 'HEAD')
-      return false if ref.nil?
+      return false if ref.nil? || ref.empty?
 
-      found = all_refs.find do |data|
-        # we don't need to bother with these types
-        next if %i[pull merge_request].include?(data[:type])
-
-        # is the name equal to the tag or branch?  Is the commit sha equal?
-        data[:name].eql?(ref) || data[:sha].slice(0, 8).eql?(ref.slice(0, 8))
-      end
-      !found.nil?
-    end
+      head = remote_refs.fetch('head', {})
+      return true if ref == 'HEAD' && !head.empty?
+      return true if sha_match?(head[:sha], ref)
 
-    # @return [Array] - an array of all the refs associated with the remote repository
-    # @param url [String] - the git string either https or ssh url
-    # @example
-    #   [{:sha=>"0ec707e431367bbe2752966be8ab915b6f0da754", :ref=>"refs/heads/74110ac", :type=>:branch, :subtype=>nil, :name=>"74110ac"},
-    #     :sha=>"07bb5d2d94db222dca5860eb29c184e8970f36f4", :ref=>"refs/pull/74/head", :type=>:pull, :subtype=>:head, :name=>"74"},
-    #     :sha=>"156ca9a8ea69e056e86355b27d944e59d1b3a1e1", :ref=>"refs/heads/master", :type=>:branch, :subtype=>nil, :name=>"master"},
-    #     :sha=>"fcc0532bbc5a5b65f3941738339e9cc7e3d767ce", :ref=>"refs/pull/249/head", :type=>:pull, :subtype=>:head, :name=>"249"},
-    #     :sha=>"8d54891fa5df75890ee15d53080c2a81b4960f92", :ref=>"refs/pull/267/head", :type=>:pull, :subtype=>:head, :name=>"267"}]
-    def all_refs
-      @all_refs ||= remote_refs.each_with_object([]) do |line, refs|
-        sha, ref = line.split("\t")
-        next refs if line.include?('redirecting')
-        next refs if sha.eql?('ref: refs/heads/master')
-
-        _, type, name, subtype = ref.chomp.split('/')
-        next refs unless name
-
-        type = :tag if type.eql?('tags')
-        type = type.to_sym
-        subtype = subtype.to_sym if subtype
-        type = :branch if type.eql?(:heads)
-        refs << { sha: sha, ref: ref.chomp, type: type, subtype: subtype, name: name }
+      %w[branches tags].any? do |type|
+        refs = remote_refs.fetch(type, {})
+        refs.key?(ref) || refs.values.any? { |data| sha_match?(data[:sha], ref) }
       end
     end
 
-    # @summary searches all the refs for a ref similar to the name provided
-    #          This is useful when looking for tags if a v or not a v
-    # @param ref_name [String]
-    # @return [String] the matching ref_name or nil
+    # @param ref_name [String] substring to find in a tag name
+    # @return [String, nil] matching tag name
     def get_ref_like(ref_name)
-      return nil unless valid_url?
+      return nil if ref_name.nil? || ref_name.empty?
 
-      all_refs.find do |ref|
-        ref[:name].include?(ref_name)
+      remote_refs.fetch('tags', {}).each_key.find do |name|
+        !name.end_with?('^{}') && name.include?(ref_name)
       end
     end
 
-    # useful for mocking easily
-    # @param cmd [String]
-    # @param silent [Boolean] set to true if you wish to send output to /dev/null, false by default
-    # @return [Array]
-    def self.run_command(cmd, silent: false)
-      out_args = silent ? '2>&1 > /dev/null' : '2>&1'
-      out = `#{cmd} #{out_args}`
-      [out, $CHILD_STATUS.success?]
-    end
+    private
+
+    def sha_match?(sha, ref)
+      return false unless sha
+
+      # \A and \z anchor the entire string; [0-9a-f] allows only hexadecimal digits.
+      # {8,64} accepts abbreviations of at least 8 digits up to a full SHA-256; i allows uppercase.
+      return false unless ref.match?(/\A[0-9a-f]{8,64}\z/i)
 
-    def run_command(cmd, silent: false)
-      self.class.run_command(cmd, silent: silent)
+      sha.start_with?(ref.downcase)
     end
   end
 end

data/lib/ra10ke/version.rb

@@ -1,3 +1,3 @@
 module Ra10ke
-  VERSION = '4.5.1'
+  VERSION = '4.6.0'
 end

data/ra10ke.gemspec

@@ -16,7 +16,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
   spec.required_ruby_version = '>= 3.1.0'
 
-  spec.add_dependency 'git', '>= 1.18', '< 5.0'
+  spec.add_dependency 'git', '>= 2.0', '< 5.0'
   spec.add_dependency 'puppet_forge', '~> 6.0'
   spec.add_dependency 'r10k', '~> 5.0'
   spec.add_dependency 'rake', '~> 13.0', '>= 13.0.6'

data/spec/ra10ke/dependencies_git_conversion_spec.rb

@@ -0,0 +1,53 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require 'ra10ke'
+
+RSpec.describe Ra10ke::Dependencies do
+  let(:url) { 'https://example.invalid/module.git' }
+  let(:metadata) do
+    PuppetForge::V3::Module.new(homepage_url: url, current_release: { metadata: { source: url } })
+  end
+  let(:forge_module) do
+    R10K::Module::Forge.new('example-module', fixtures_dir, version: '1.0.0')
+  end
+  let(:puppetfile) { instance_double(R10K::Puppetfile, load!: nil, forge: nil, modules: [forge_module]) }
+  let(:task_owner) { Ra10ke::RakeTask.new }
+
+  around do |example|
+    original_application = Rake.application
+    Rake.application = Rake::Application.new
+    example.run
+  ensure
+    Rake.application = original_application
+  end
+
+  before do
+    allow(forge_module).to receive_messages(v3_module: metadata, name: 'example-module')
+    allow(task_owner).to receive(:get_puppetfile).and_return(puppetfile)
+  end
+
+  it 'prints the matching tag name returned by GitRepo' do
+    allow(Git).to receive(:ls_remote).with(url).and_return(
+      'tags' => { 'v1.0.0' => { ref: 'refs', sha: 'abc12345' } },
+    )
+    expected = <<~PUPPETFILE
+      mod 'example-module',
+        :git => '#{url}',
+        :ref => 'v1.0.0'
+
+    PUPPETFILE
+    expect { Rake::Task['r10k:print_git_conversion'].execute }.to output(expected).to_stdout
+  end
+
+  it 'prints the existing error message when no tag matches' do
+    allow(Git).to receive(:ls_remote).with(url).and_return({})
+    expected = <<~PUPPETFILE
+      mod 'example-module',
+        :git => '#{url}',
+        :ref => 'bad url or tag 1.0.0 is missing'
+
+    PUPPETFILE
+    expect { Rake::Task['r10k:print_git_conversion'].execute }.to output(expected).to_stdout
+  end
+end

data/spec/ra10ke/git_repo_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 require 'spec_helper'
-require 'ra10ke/validate'
+require 'ra10ke/git_repo'
 
 RSpec.describe 'Ra10ke::GitRepo' do
   let(:url) { 'https://github.com/vshn/puppet-gitlab' }
@@ -10,11 +10,12 @@ RSpec.describe 'Ra10ke::GitRepo' do
     expect(Ra10ke::GitRepo.new(url)).to be_a Ra10ke::GitRepo
   end
 
-  it 'does not run_command more than once' do
+  it 'only fetches remote refs once' do
     i = Ra10ke::GitRepo.new(url)
-    expect(i).to receive(:run_command).with("git ls-remote --symref #{url}").once
+    expect(Git).to receive(:ls_remote).with(url).once.and_return({})
     i.valid_url?
-    i.all_refs
+    i.valid_ref?
+    i.get_ref_like('1.0.0')
   end
 
   describe 'good url' do
@@ -24,28 +25,56 @@ RSpec.describe 'Ra10ke::GitRepo' do
       Ra10ke::GitRepo.new(url)
     end
 
-    let(:reflist) { File.read(File.join(fixtures_dir, 'refs', 'gitlab.txt')) }
+    let(:reflist) do
+      File.readlines(File.join(fixtures_dir, 'refs', 'gitlab.txt')).each_with_object({}) do |line, refs|
+        sha, ref = line.strip.split("\t")
+        if ref == 'HEAD'
+          refs['head'] = { ref: ref, sha: sha }
+        else
+          prefix, type, name = ref.split('/', 3)
+          type = 'branches' if type == 'heads'
+          refs[type] ||= {}
+          refs[type][name] = { ref: prefix, sha: sha }
+        end
+      end
+    end
 
     before do
-      allow(instance).to receive(:run_command).with("git ls-remote --symref #{url}").and_return([reflist, true])
+      allow(Git).to receive(:ls_remote).with(url).and_return(reflist)
     end
 
-    it '#remote_refs is array' do
-      expect(instance.remote_refs).to be_a Array
+    it '#remote_refs returns the gem hash unchanged' do
+      expect(instance.remote_refs).to equal(reflist)
     end
 
     it '#remote_refs contains refs' do
-      expect(instance.remote_refs.first).to eq("1b3322d525e96bf7d0565b08703e2a44c90e7b4a\tHEAD\n")
+      expect(instance.remote_refs['head']).to eq(ref: 'HEAD', sha: '1b3322d525e96bf7d0565b08703e2a44c90e7b4a')
     end
 
     it '#get_ref_like' do
-      expect(instance.get_ref_like('8.0.0'))
+      expect(instance.get_ref_like('5.0.0')).to eq('v5.0.0')
     end
 
     it '#valid_ref?' do
       expect(instance.valid_ref?('master')).to be true
     end
 
+    it 'accepts HEAD explicitly and as the default ref' do
+      expect(instance.valid_ref?('HEAD')).to be true
+      expect(instance.valid_ref?).to be true
+    end
+
+    it 'excludes pull and merge request refs from validation' do
+      allow(Git).to receive(:ls_remote).with(url).and_return(
+        'pull' => { '74/head' => { ref: 'refs', sha: 'abc12345' } },
+        'merge_request' => { '12/merge' => { ref: 'refs', sha: 'def12345' } },
+      )
+      expect(instance.valid_ref?('74')).to be false
+      expect(instance.valid_ref?('abc12345')).to be false
+      expect(instance.valid_ref?('12')).to be false
+      expect(instance.valid_ref?('def12345')).to be false
+    end
+
     it '#valid_commit?' do
       expect(instance.valid_commit?('master')).to be true
     end
@@ -55,8 +84,9 @@ RSpec.describe 'Ra10ke::GitRepo' do
     end
 
     it '#valid_commit? but invalid sha' do
-      allow(instance).to receive(:run_command).with(/clone/, silent: true).and_return([nil, true])
-      allow(instance).to receive(:run_command).with(/git\sshow.*/, silent: true).and_return([nil, false])
+      repo = instance_double(Git::Base)
+      allow(Git).to receive(:clone).with(url, kind_of(String), bare: true).and_return(repo)
+      allow(repo).to receive(:rev_parse).with('invalid^{commit}').and_raise(Git::Error)
       expect(instance.valid_commit?('invalid')).to be false
     end
 
@@ -64,13 +94,28 @@ RSpec.describe 'Ra10ke::GitRepo' do
       expect(instance.valid_url?).to be true
     end
 
-    it '#all_refs is a Array of Hashes' do
-      refs = instance.all_refs
-      expect(refs).to be_a Array
-      expect(refs.last).to eq(
-        { name: 'v5.0.0^{}', ref: 'refs/tags/v5.0.0^{}', sha: '1febd15f90d32e6b3d6c242a70db386b2ef1942c', subtype: nil,
-          type: :tag, },
+    it 'accepts full and abbreviated SHAs but checks all supplied characters' do
+      sha = reflist['head'][:sha]
+      expect(instance.valid_ref?(sha)).to be true
+      expect(instance.valid_ref?(sha[0, 8])).to be true
+      expect(instance.valid_ref?(sha[0, 12])).to be true
+      expect(instance.valid_ref?("#{sha[0, 8]}#{'0' * 32}")).to be false
+      expect(instance.valid_ref?("#{sha}0")).to be false
+      expect(instance.valid_ref?(sha[0, 7])).to be false
+    end
+
+    it 'validates the peeled commit SHA of an annotated tag' do
+      expect(instance.valid_ref?('1febd15f90d32e6b3d6c242a70db386b2ef1942c')).to be true
+    end
+
+    it 'searches only tag names and skips peeled tag entries' do
+      allow(Git).to receive(:ls_remote).with(url).and_return(
+        'branches' => { 'feature/5.0.0' => { ref: 'refs', sha: 'abc12345' } },
+        'tags' => { 'v5.0.0^{}' => { ref: 'refs', sha: 'abc12345' }, 'v5.0.0' => { ref: 'refs', sha: 'def12345' } },
       )
+      expect(instance.get_ref_like('5.0.0')).to eq('v5.0.0')
+      expect(instance.get_ref_like('feature')).to be_nil
+      expect(instance.get_ref_like('^{}')).to be_nil
     end
   end
 
@@ -81,12 +126,143 @@ RSpec.describe 'Ra10ke::GitRepo' do
     end
 
     before do
-      allow(instance).to receive(:run_command).with("git ls-remote --symref #{url}").and_return(['', false])
+      allow(Git).to receive(:ls_remote).with(url).and_raise(Git::Error)
+    end
+
+    it 'returns empty remote refs and rejects HEAD' do
+      expect(instance.remote_refs).to eq({})
+      expect(instance.valid_ref?).to be false
+    end
+  end
+
+  describe 'failed remote lookup' do
+    it 'caches the failure and reports an invalid URL' do
+      repo = Ra10ke::GitRepo.new(url)
+      expect(Git).to receive(:ls_remote).with(url).once.and_raise(Git::Error)
+      expect(repo.valid_url?).to be false
+      expect(repo.remote_refs).to eq({})
+      expect(repo.get_ref_like('missing')).to be_nil
+      expect(Git).not_to receive(:clone)
+      expect(repo.valid_commit?('missing')).to be false
+    end
+  end
+
+  describe '#valid_commit?' do
+    let(:instance) { Ra10ke::GitRepo.new(url) }
+
+    before do
+      allow(Git).to receive(:ls_remote).with(url).and_return('head' => { ref: 'HEAD', sha: 'a' * 40 })
+    end
+
+    it 'does not clone for an empty SHA' do
+      expect(Git).not_to receive(:clone)
+      expect(instance.valid_commit?('')).to be false
+    end
+
+    it 'stops after a failed clone and removes the temporary directory' do
+      clone_dir = nil
+      expect(Git).to receive(:clone).with(url, kind_of(String), bare: true) do |_, dir|
+        clone_dir = dir
+        raise Git::Error
+      end
+      expect(instance.valid_commit?('missing')).to be false
+      expect(File.exist?(clone_dir)).to be false
+    end
+  end
+
+  describe '.current_branch' do
+    it 'handles nil returned by older git gem versions for detached HEAD' do
+      repo = instance_double(Git::Base, current_branch: nil)
+      allow(Git).to receive(:open).with('repo').and_return(repo)
+      expect(Ra10ke::GitRepo.current_branch('repo')).to be_nil
+    end
+
+    it 'returns nil for a directory without a repository' do
+      Dir.mktmpdir do |dir|
+        expect(Ra10ke::GitRepo.current_branch(dir)).to be_nil
+      end
+    end
+
+    it 'returns nil for a missing directory' do
+      Dir.mktmpdir do |dir|
+        expect(Ra10ke::GitRepo.current_branch(File.join(dir, 'missing'))).to be_nil
+      end
+    end
+  end
+
+  describe 'with a local repository' do
+    let(:tmpdir) { Dir.mktmpdir('ra10ke git repo ') }
+    let(:path) { File.join(tmpdir, 'source repo') }
+    let(:local_repo) { Git.init(path) }
+    let(:older_sha) { local_repo.revparse('HEAD~1') }
+
+    around do |example|
+      example.run
+    ensure
+      FileUtils.remove_entry(tmpdir)
+    end
+
+    before do
+      local_repo.config('user.name', 'Ra10ke Test')
+      local_repo.config('user.email', 'ra10ke@example.invalid')
+      local_repo.config('commit.gpgsign', 'false')
+      local_repo.config('tag.gpgsign', 'false')
+      File.write(File.join(path, 'file'), 'first')
+      local_repo.add('file')
+      local_repo.commit('first')
+      File.write(File.join(path, 'file'), 'second')
+      local_repo.add('file')
+      local_repo.commit('second')
+      local_repo.add_tag('v1.0.0')
+      local_repo.add_tag('v2.0.0', annotate: true, message: 'annotated tag')
+    end
+
+    it 'returns the branch without changing the working directory' do
+      cwd = Dir.pwd
+      expect(Ra10ke::GitRepo.current_branch(path)).to eq(local_repo.current_branch)
+      expect(Dir.pwd).to eq(cwd)
+    end
+
+    it 'returns nil for detached HEAD' do
+      local_repo.checkout(older_sha)
+      expect(Ra10ke::GitRepo.current_branch(path)).to be_nil
+    end
+
+    it 'finds branches, lightweight tags and annotated tags' do
+      repo = Ra10ke::GitRepo.new(path)
+      expect(repo.valid_url?).to be true
+      expect(repo.valid_ref?(local_repo.current_branch)).to be true
+      expect(repo.valid_ref?('v1.0.0')).to be true
+      expect(repo.valid_ref?('v2.0.0')).to be true
+      expect(repo.get_ref_like('2.0.0')).to eq('v2.0.0')
+    end
+
+    it 'preserves branch and tag names containing slashes' do
+      local_repo.branch('feature/example').create
+      local_repo.add_tag('release/v1.0.0')
+      repo = Ra10ke::GitRepo.new(path)
+      expect(repo.valid_ref?('feature/example')).to be true
+      expect(repo.valid_ref?('release/v1.0.0')).to be true
+      expect(repo.get_ref_like('release/')).to eq('release/v1.0.0')
+    end
+
+    it 'recognizes remote HEAD from the gem lookup' do
+      repo = Ra10ke::GitRepo.new(path)
+      expect(repo.valid_ref?).to be true
+      expect(repo.valid_ref?(local_repo.revparse('HEAD'))).to be true
+    end
+
+    it 'rejects a blob object during the commit fallback check' do
+      repo = Ra10ke::GitRepo.new(path)
+      blob_sha = local_repo.rev_parse('HEAD:file')
+      expect(repo.valid_commit?(blob_sha)).to be false
     end
 
-    it '#all_refs' do
-      expect(instance.all_refs).to be_a Array
-      expect(instance.all_refs).to eq []
+    it 'clones to validate a commit that is not advertised by a ref' do
+      repo = Ra10ke::GitRepo.new(path)
+      expect(repo.valid_ref?(older_sha)).to be false
+      expect(repo.valid_commit?(older_sha)).to be true
+      expect(repo.valid_commit?('invalid')).to be false
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ra10ke
 version: !ruby/object:Gem::Version
-  version: 4.5.1
+  version: 4.6.0
 platform: ruby
 authors:
 - Theo Chatzimichos
@@ -16,7 +16,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.18'
+        version: '2.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '5.0'
@@ -26,7 +26,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '1.18'
+        version: '2.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '5.0'
@@ -168,7 +168,9 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".github/dependabot.yml"
+- ".github/labeler.yml"
 - ".github/release.yml"
+- ".github/workflows/labeler.yml"
 - ".github/workflows/release.yml"
 - ".github/workflows/test.yml"
 - ".gitignore"
@@ -207,6 +209,7 @@ files:
 - spec/fixtures/refs/gitlab.txt
 - spec/fixtures/refs/r10k.txt
 - spec/fixtures/refs/reflist.txt
+- spec/ra10ke/dependencies_git_conversion_spec.rb
 - spec/ra10ke/dependencies_spec.rb
 - spec/ra10ke/deprecation_spec.rb
 - spec/ra10ke/diff_spec.rb
@@ -235,7 +238,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.6
+rubygems_version: 4.0.20
 specification_version: 4
 summary: Syntax check for the Puppetfile, check for outdated installed puppet modules
 test_files: []