ra10ke 1.2.0 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c18588b9b0b4eb792bfcc75c573a7d8107840baf321c20ca4d0ba26aabc03912
-  data.tar.gz: 1503a4fa9911f99ac01e12aebd8cbc7f8e3402aa385c411f0861fe082bcaacfd
+  metadata.gz: 42a64a748cf4b63602804c2aac20987e23d8a7ad2a01ca3365d3ed998324f058
+  data.tar.gz: daa94b8601294ed8c480f431d396eda60c2a14d2c7aa608e4941189033c01b24
 SHA512:
-  metadata.gz: '078235ff75fc9d159333ef7e2eff5424e01e801e91480778fe035cd9aaba19a4073c0f0b48a2792f08afab4d1e9ac597a8ad4262b3afa220749862bdfb5e11ca'
-  data.tar.gz: 3820689a48c0df80436ab56c3b0e27093f5951753ac6997fbfbb27cbccb3c13d2f8151ac01d65f1b8732aebe0b24b002be8fcc6caf5677b089ebdeb94b83f2d5
+  metadata.gz: b1b0a572ecb7928f0e38254f6ed91f4e501f2755497a53ced15a164ada79dfa2b335afe7f4565c27ae63050a17d57c8df45dd39dd5fbac0271361f00c8063c5a
+  data.tar.gz: 251245599d77dbbd4b01c3946693083782fe8b1f0937236fc207ca3c69616b967743cafa0bb8e4151407199fd8d4e0be0479da8e7f83009bb22f0fe9192785e1

data/.github/workflows/test.yml

@@ -2,7 +2,6 @@ name: Test
 
 on:
   - pull_request
-  - push
 
 env:
   BUNDLE_WITHOUT: release
@@ -14,11 +13,9 @@ jobs:
       fail-fast: false
       matrix:
         include:
-          - ruby: "2.4"
-          - ruby: "2.5"
-          - ruby: "2.6"
           - ruby: "2.7"
           - ruby: "3.0"
+          - ruby: "3.1"
             coverage: "yes"
     env:
       COVERAGE: ${{ matrix.coverage }}

data/CHANGELOG.md

@@ -2,6 +2,31 @@
 
 All notable changes to this project will be documented in this file.
 
+## [v2.0.0](https://github.com/voxpupuli/ra10ke/tree/v2.0.0) (2022-05-18)
+
+[Full Changelog](https://github.com/voxpupuli/ra10ke/compare/v1.2.0...v2.0.0)
+
+**Breaking changes:**
+
+- Bump minimal Ruby Version 2.4.0-\>2.7.0 [\#77](https://github.com/voxpupuli/ra10ke/pull/77) ([bastelfreak](https://github.com/bastelfreak))
+- Dependencies Table output  [\#72](https://github.com/voxpupuli/ra10ke/pull/72) ([logicminds](https://github.com/logicminds))
+
+**Implemented enhancements:**
+
+- Add Ruby 3.1 support [\#78](https://github.com/voxpupuli/ra10ke/pull/78) ([bastelfreak](https://github.com/bastelfreak))
+- Converts puppet forge module entries into puppetfile git entries [\#76](https://github.com/voxpupuli/ra10ke/pull/76) ([logicminds](https://github.com/logicminds))
+
+**Fixed bugs:**
+
+- incompatible with r10k gem \> 3.8 [\#57](https://github.com/voxpupuli/ra10ke/issues/57)
+- CI: Do not run twice [\#79](https://github.com/voxpupuli/ra10ke/pull/79) ([bastelfreak](https://github.com/bastelfreak))
+- Fix \#74 - server responded with 400 [\#75](https://github.com/voxpupuli/ra10ke/pull/75) ([logicminds](https://github.com/logicminds))
+- Fix \#70 - cache dir does not exist [\#71](https://github.com/voxpupuli/ra10ke/pull/71) ([logicminds](https://github.com/logicminds))
+
+**Closed issues:**
+
+- cache dir does not exist [\#70](https://github.com/voxpupuli/ra10ke/issues/70)
+
 ## [v1.2.0](https://github.com/voxpupuli/ra10ke/tree/v1.2.0) (2022-04-21)
 
 [Full Changelog](https://github.com/voxpupuli/ra10ke/compare/v1.1.0...v1.2.0)

data/README.md

@@ -68,6 +68,15 @@ Ignoring specific modules:
 Under specific conditions you may not wish to report on specific modules being out of date,
 to ignore a module create `.r10kignore` file in the same directory as your Puppetfile.
 
+### r10k::print_git_conversion
+This rake task will go through the puppetfile and convert forge based modules into git based modules using
+the modules't source repository and version tag.
+
+This feature is useful when you want to bring all the forge modules into git source control.  This assumes every module
+tags the release or provides a valid repo url.  We recommend to manually review
+the output provided by this task before replacing the forge based content in your puppetfile as not every module author 
+tagged a release or provided a working url.
+
 ### r10k:solve_dependencies
 
 Reads the Puppetfile in the current directory and uses the ruby 'solve' library to find

data/lib/ra10ke/dependencies.rb

@@ -1,37 +1,147 @@
+require 'semverse'
+require 'r10k/puppetfile'
+require 'puppet_forge'
+require 'table_print'
+require 'git'
+
 module Ra10ke::Dependencies
-  @@version_formats = {}
+  class Verification
 
-  # Registers a block that finds the latest version.
-  # The block will be called with a list of tags.
-  # If the block returns nil the next format will be tried.
-  def self.register_version_format(name, &block)
-    @@version_formats[name] = block
-  end
+    def self.version_formats
+      @version_formats ||= {}
+    end
+
+    # Registers a block that finds the latest version.
+    # The block will be called with a list of tags.
+    # If the block returns nil the next format will be tried.
+    def self.register_version_format(name, &block)
+      version_formats[name] = block
+    end
+
+    Ra10ke::Dependencies::Verification.register_version_format(:semver) do |tags|
+      latest_tag = tags.map do |tag|
+        begin
+          Semverse::Version.new tag[/\Av?(.*)\Z/, 1]
+        rescue Semverse::InvalidVersionFormat
+          # ignore tags that do not comply to semver
+          nil
+        end
+      end.select { |tag| !tag.nil? }.sort.last.to_s.downcase
+      latest_ref = tags.detect { |tag| tag[/\Av?(.*)\Z/, 1] == latest_tag }
+    end
+    attr_reader :puppetfile
+
+    def initialize(pfile)
+      @puppetfile = pfile
+       # semver is the default version format.
+  
+      puppetfile.load!
+    end
+
+    def get_latest_ref(remote_refs)
+      tags = remote_refs['tags'].keys
+      latest_ref = nil
+      self.class.version_formats.detect { |_, block| latest_ref = block.call(tags) }
+      latest_ref = 'undef (tags do not follow any known pattern)' if latest_ref.nil?
+      latest_ref
+    end
+
+    def ignored_modules
+      # ignore file allows for "don't tell me about this"
+      @ignored_modules ||= begin
+        File.readlines('.r10kignore').each {|l| l.chomp!} if File.exist?('.r10kignore')
+      end || []
+    end
+  
+    # @summary creates an array of module hashes with version info 
+    # @param {Object} supplied_puppetfile - the parsed puppetfile object 
+    # @returns {Array} array of version info for each module
+    # @note does not include ignored modules or modules up2date
+    def processed_modules(supplied_puppetfile = puppetfile)
+      threads = []
+      threads = supplied_puppetfile.modules.map do |puppet_module|
+      Thread.new do 
+        begin
+          next if ignored_modules.include? puppet_module.title
+          if puppet_module.class == ::R10K::Module::Forge
+            module_name = puppet_module.title.gsub('/', '-')
+            forge_version = ::PuppetForge::Module.find(module_name).current_release.version
+            installed_version = puppet_module.expected_version
+            {
+              name: puppet_module.title,
+              installed: installed_version,
+              latest: forge_version,
+              type: 'forge',
+              message: installed_version != forge_version ? :outdated : :current
+            }
+          
+          elsif puppet_module.class == R10K::Module::Git
+            # use helper; avoid `desired_ref`
+            # we do not want to deal with `:control_branch`
+            ref = puppet_module.version
+            next unless ref
+
+            remote = puppet_module.instance_variable_get(:@remote)
+            remote_refs = Git.ls_remote(remote)
+
+            # skip if ref is a branch
+            next if remote_refs['branches'].key?(ref)
+
+            if remote_refs['tags'].key?(ref)
+              # there are too many possible versioning conventions
+              # we have to be be opinionated here
+              # so semantic versioning (vX.Y.Z) it is for us
+              # as well as support for skipping the leading v letter
+              #
+              # register own version formats with
+              # Ra10ke::Dependencies.register_version_format(:name, &block)
+              latest_ref = get_latest_ref(remote_refs)
+            elsif ref.match(/^[a-z0-9]{40}$/)
+              ref = ref.slice(0,8)
+              # for sha just assume head should be tracked
+              latest_ref = remote_refs['head'][:sha].slice(0,8)
+            else
+              raise "Unable to determine ref type for #{puppet_module.title}"
+            end
+            {
+              name: puppet_module.title,
+              installed: ref,
+              latest: latest_ref,
+              type: 'git',
+              message: ref != latest_ref ? :outdated : :current
+            }
+
+          end
+        rescue R10K::Util::Subprocess::SubprocessError => e
+            {
+              name: puppet_module.title,
+              installed: nil,
+              latest: nil,
+              type: :error,
+              message: e.message
+            }
+        end
+        end
+      end
+      threads.map { |th| th.join.value }.compact
+    end
 
-  # semver is the default version format.
-  register_version_format(:semver) do |tags|
-    latest_tag = tags.map do |tag|
-      begin
-        Semverse::Version.new tag[/\Av?(.*)\Z/, 1]
-      rescue Semverse::InvalidVersionFormat
-        # ignore tags that do not comply to semver
-        nil
+    def outdated(supplied_puppetfile = puppetfile)
+      processed_modules.find_all do | mod |
+        mod[:message] == :outdated
       end
-    end.select { |tag| !tag.nil? }.sort.last.to_s.downcase
-    latest_ref = tags.detect { |tag| tag[/\Av?(.*)\Z/, 1] == latest_tag }
-  end
+    end
 
-  def get_latest_ref(remote_refs)
-    tags = remote_refs['tags'].keys
-    latest_ref = nil
-    @@version_formats.detect { |_, block| latest_ref = block.call(tags) }
-    latest_ref = 'undef (tags do not follow any known pattern)' if latest_ref.nil?
-    latest_ref
+    def print_table(mods)
+      puts
+      tp mods, { name: { width: 50 } }, :installed, :latest, :type, :message
+    end
   end
 
-  def define_task_dependencies(*_args)
-    desc "Print outdated forge modules"
-    task :dependencies do
+  def define_task_print_git_conversion(*_args)
+    desc "Convert and print forge modules to git format"
+    task :print_git_conversion do
+      require 'ra10ke/git_repo'
       require 'r10k/puppetfile'
       require 'puppet_forge'
 
@@ -45,49 +155,40 @@ module Ra10ke::Dependencies
       if File.exist?('.r10kignore')
         ignore_modules = File.readlines('.r10kignore').each {|l| l.chomp!}
       end
-
-      puppetfile.modules.each do |puppet_module|
-        next if ignore_modules.include? puppet_module.title
-        if puppet_module.class == R10K::Module::Forge
-          module_name = puppet_module.title.gsub('/', '-')
-          forge_version = PuppetForge::Module.find(module_name).current_release.version
-          installed_version = puppet_module.expected_version
-          if installed_version != forge_version
-            puts "#{puppet_module.title} is OUTDATED: #{installed_version} vs #{forge_version}"
-          end
-        end
-
-        if puppet_module.class == R10K::Module::Git
-          # use helper; avoid `desired_ref`
-          # we do not want to deal with `:control_branch`
-          ref = puppet_module.version
-          next unless ref
-
-          remote = puppet_module.instance_variable_get(:@remote)
-          remote_refs = Git.ls_remote(remote)
-
-          # skip if ref is a branch
-          next if remote_refs['branches'].key?(ref)
-
-          if remote_refs['tags'].key?(ref)
-            # there are too many possible versioning conventions
-            # we have to be be opinionated here
-            # so semantic versioning (vX.Y.Z) it is for us
-            # as well as support for skipping the leading v letter
-            #
-            # register own version formats with
-            # Ra10ke::Dependencies.register_version_format(:name, &block)
-            latest_ref = get_latest_ref(remote_refs)
-          elsif ref.match(/^[a-z0-9]{40}$/)
-            # for sha just assume head should be tracked
-            latest_ref = remote_refs['head'][:sha]
-          else
-            raise "Unable to determine ref type for #{puppet_module.title}"
+      forge_mods = puppetfile.modules.find_all { |mod| mod.instance_of?(R10K::Module::Forge) && mod.v3_module.homepage_url? }
+     
+      threads = forge_mods.map do |mod|
+       Thread.new do 
+          source_url = mod.v3_module.attributes.dig(:current_release, :metadata, :source) || mod.v3_module.homepage_url 
+          # git:// does not work with ls-remote command, convert to https
+          source_url = source_url.gsub('git://', 'https://')
+          source_url = source_url.gsub(/\Agit\@(.*)\:(.*)/) do
+            "https://#{$1}/#{$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"
+          <<~EOF
+          mod '#{mod.name}',
+            :git => '#{source_url}',
+            :ref => '#{ref_name}'
 
-          puts "#{puppet_module.title} is OUTDATED: #{ref} vs #{latest_ref}" if ref != latest_ref
+          EOF
         end
       end
+      output = threads.map { |th| th.join.value }
+      puts output
+    end
+  end
+
+  def define_task_dependencies(*_args)
+    desc "Print outdated forge modules"
+    task :dependencies do
+      PuppetForge.user_agent = "ra10ke/#{Ra10ke::VERSION}"
+      puppetfile = get_puppetfile
+      PuppetForge.host = puppetfile.forge if puppetfile.forge =~ /^http/
+      dependencies = Ra10ke::Dependencies::Verification.new(puppetfile)
+      dependencies.print_table(dependencies.outdated)
     end
   end
 end

data/lib/ra10ke/git_repo.rb

@@ -85,8 +85,8 @@ module Ra10ke
       @all_refs ||= begin
         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
 
@@ -99,6 +99,18 @@ module Ra10ke
       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
+    def get_ref_like(ref_name)
+      return nil unless valid_url?
+      ref = all_refs.find do |ref|
+        ref[:name].include?(ref_name)
+      end
+      ref
+    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

data/lib/ra10ke/puppetfile_parser.rb

@@ -27,11 +27,11 @@ module Ra10ke
       def modules(puppetfile)
         @modules ||= begin
           return [] unless File.exist?(puppetfile)
-
           all_lines = File.read(puppetfile).lines.map(&:strip_comment)
           # remove comments from all the lines
-          lines_without_comments = all_lines.reject { |line| line.match(/#.*\n/) }.join("\n")
+          lines_without_comments = all_lines.reject { |line| line.match(/#.*\n/) || line.empty? }.join("\n")
           lines_without_comments.split(/^mod/).map do |line|
+            
             next nil if line =~ /^forge/
             next nil if line.empty?
 

data/lib/ra10ke/solve.rb

@@ -6,6 +6,7 @@ require 'set'
 require 'solve'
 require 'yaml/store'
 require 'semverse/version'
+require 'fileutils'
 
 # How many versions to fetch from the Forge, at most
 FETCH_LIMIT = 3
@@ -35,6 +36,9 @@ module Ra10ke::Solve
       end
       # Actual new logic begins here:
       cache = (ENV['XDG_CACHE_DIR'] || File.expand_path('~/.cache'))
+
+      FileUtils.mkdir_p(cache)
+      
       # Metadata cache, since the Forge is slow:
       @metadata_cache = YAML::Store.new File.join(cache, 'ra10ke.metadata_cache')
       # The graph of available module versions

data/lib/ra10ke/version.rb

@@ -1,3 +1,3 @@
 module Ra10ke
-  VERSION = "1.2.0"
+  VERSION = "2.0.0"
 end

data/lib/ra10ke.rb

@@ -41,6 +41,7 @@ module Ra10ke
         define_task_duplicates(*args)
         define_task_install(*args)
         define_task_validate(*args)
+        define_task_print_git_conversion(*args)
       end
     end
 

data/ra10ke.gemspec

@@ -14,11 +14,11 @@ Gem::Specification.new do |spec|
 
   spec.files         = `git ls-files`.split($/)
   spec.require_paths = ["lib"]
-  spec.required_ruby_version = '>= 2.4.0'
+  spec.required_ruby_version = '>= 2.6.0'
 
   spec.add_dependency "rake"
   spec.add_dependency "puppet_forge"
-  spec.add_dependency "r10k"
+  spec.add_dependency "r10k", '>= 2.6.5'
   spec.add_dependency "git"
   spec.add_dependency "solve"
   spec.add_dependency 'semverse', '>= 2.0'

data/spec/fixtures/Puppetfile_deprecation_issue

@@ -0,0 +1,78 @@
+# used in profiles
+mod 'puppet/systemd', :latest
+mod 'puppet/lldpd', :latest
+mod 'puppet/ferm', :latest
+mod 'puppet/borg', :latest
+mod 'puppet/wireguard', :latest
+mod 'puppet/bird', :latest
+mod 'herculesteam/augeasproviders_pam', :latest
+mod 'herculesteam/augeasproviders_shellvar', :latest
+mod 'puppetlabs/vcsrepo', :latest
+mod 'saz/ssh', :latest
+mod 'puppet/r10k', :latest
+mod 'puppet/dbbackup', :latest
+mod 'puppet/mosquitto', :latest
+mod 'puppet/grafana', :latest
+mod 'puppet/nginx', :latest
+mod 'puppet/ssh_keygen', :latest
+mod 'puppet/unbound', :latest
+mod 'puppetlabs/inifile', :latest
+mod 'puppet/prometheus', :latest
+mod 'theforeman/foreman', :latest
+mod 'theforeman/puppet', :latest
+mod 'theforeman/foreman_proxy', :latest
+mod 'puppet/nftables', :latest
+mod 'herculesteam/augeasproviders_sysctl', :latest
+mod 'camptocamp/catalog_diff', :latest
+mod 'puppet/unattended_upgrades', :latest
+mod 'puppet/selinux', :latest
+mod 'choria/choria', :latest
+mod 'puppet/archive', :latest
+mod 'puppet/elasticsearch', :latest
+mod 'jsok/vault',
+  git: 'https://github.com/bastelfreak/puppet-vault',
+  ref: 'test'
+
+# dependencies
+mod 'herculesteam/augeasproviders_core', :latest
+mod 'puppetlabs/stdlib', :latest
+mod 'choria/mcollective', :latest
+mod 'puppetlabs/concat', :latest
+mod 'puppetlabs/apt', :latest
+mod 'puppetlabs/apache', :latest
+mod 'puppetlabs/postgresql', :latest
+mod 'puppet/extlib', :latest
+mod 'puppet/redis', :latest
+mod 'puppet/epel', :latest
+mod 'theforeman/puppetserver_foreman', :latest
+mod 'richardc/datacat', :latest
+mod 'theforeman/dns', :latest
+mod 'theforeman/dhcp', :latest
+mod 'theforeman/tftp', :latest
+mod 'puppetlabs/xinetd', :latest
+mod 'choria-mcollective_choria', :latest
+mod 'choria/mcollective_agent_puppet', :latest
+mod 'choria/mcollective_agent_package', :latest
+mod 'choria/mcollective_agent_service', :latest
+mod 'choria/mcollective_agent_filemgr', :latest
+mod 'choria/mcollective_agent_shell', :latest
+mod 'choria/mcollective_agent_nettest', :latest
+mod 'choria/mcollective_agent_puppetca', :latest
+mod 'choria/mcollective_agent_bolt_tasks', :latest
+mod 'choria/mcollective_agent_iptables', :latest
+mod 'choria/mcollective_agent_process', :latest
+mod 'choria/mcollective_util_actionpolicy', :latest
+mod 'optiz0r/mcollective_agent_puppet_env', :latest
+mod 'jay7x/mcollective_agent_query', :latest
+mod 'choria/mcollective_data_sysctl', :latest
+mod 'puppet/yum', :latest
+mod 'puppetlabs/java', :latest
+mod 'puppet/elastic_stack', :latest
+mod 'puppet/hashi_stack', :latest
+
+# core modules are not vendored on Gentoo
+mod 'puppetlabs-sshkeys_core', :latest
+mod 'puppetlabs-mount_core', :latest
+mod 'puppetlabs-augeas_core', :latest
+mod 'puppetlabs-selinux_core', :latest
+mod 'puppetlabs-yumrepo_core', :latest

data/spec/fixtures/Puppetfile_git_conversion

@@ -0,0 +1,28 @@
+mod 'choria',
+  :git => 'https://github.com/choria-io/puppet-choria',
+  :ref => '0.26.2'
+
+mod 'inifile',
+  :git => 'https://github.com/puppetlabs/puppetlabs-inifile',
+  :ref => '2.2.0'
+
+mod 'ruby',
+  :git => 'https://github.com/puppetlabs/puppetlabs-ruby.git',
+  :ref => 'v1.0.1'
+
+mod 'stdlib',
+  :git => 'https://github.com/puppetlabs/puppetlabs-stdlib',
+  :ref => '4.24.0'
+
+mod 'concat',
+  :git => 'https://github.com/puppetlabs/puppetlabs-concat',
+  :ref => '4.0.0'
+
+mod 'ntp',
+  :git => 'https://github.com/puppetlabs/puppetlabs-ntp',
+  :ref => '6.4.1'
+
+mod 'archive',
+  :git => 'https://github.com/voxpupuli/puppet-archive',
+  :ref => 'v3.1.1'
+

data/spec/ra10ke/dependencies_spec.rb

@@ -1,35 +1,59 @@
 # frozen_string_literal: true
-
+require 'r10k/puppetfile'
 require 'spec_helper'
 require 'ra10ke/dependencies'
+require 'ra10ke'
+
 RSpec::Mocks.configuration.allow_message_expectations_on_nil = true
 
 RSpec.describe 'Ra10ke::Dependencies::Verification' do
-  after(:each) do
-    # reset version formats
-    formats = Ra10ke::Dependencies.class_variable_get(:@@version_formats)
-    Ra10ke::Dependencies.class_variable_set(:@@version_formats, formats.select { |k, _v| k == :semver } )
+  let(:instance) do
+    pfile = R10K::Puppetfile.new(File.basename(puppetfile), nil,puppetfile, nil, false)
+    Ra10ke::Dependencies::Verification.new(pfile)
+  end
+
+  let(:puppetfile) do
+    File.join(fixtures_dir, 'Puppetfile')
   end
 
   context 'register_version_format' do
     it 'default contains semver' do
-      expect(Ra10ke::Dependencies.class_variable_get(:@@version_formats)).to have_key(:semver)
+      expect(Ra10ke::Dependencies::Verification.version_formats).to have_key(:semver)
     end
     it 'add new version format' do
-      Ra10ke::Dependencies.register_version_format(:test) do |tags|
+      Ra10ke::Dependencies::Verification.register_version_format(:test) do |tags|
         nil
       end
-      expect(Ra10ke::Dependencies.class_variable_get(:@@version_formats)).to have_key(:test)
+      expect(Ra10ke::Dependencies::Verification.version_formats).to have_key(:test)
     end
   end
 
-  context 'get_latest_ref' do
+  context 'show output in table format' do
     let(:instance) do
-      class DependencyDummy
-        include Ra10ke::Dependencies
-      end.new
+      pfile = R10K::Puppetfile.new(File.basename(puppetfile), nil,puppetfile, nil, false)
+      Ra10ke::Dependencies::Verification.new(pfile)
+    end
+  
+    let(:puppetfile) do
+      File.join(fixtures_dir, 'Puppetfile')
+    end
+
+    let(:processed_modules) do
+      instance.outdated
+    end
+
+    it 'have dependencies array' do
+      expect(processed_modules).to be_a Array
     end
 
+    it 'show dependencies as table' do
+      instance.print_table(processed_modules)
+    end
+
+  end
+
+  context 'get_latest_ref' do
+
     context 'find latest semver tag' do
       let(:latest_tag) do
           'v1.1.0'
@@ -60,7 +84,7 @@ RSpec.describe 'Ra10ke::Dependencies::Verification' do
       end
 
       it do
-        Ra10ke::Dependencies.register_version_format(:number) do |tags|
+        Ra10ke::Dependencies::Verification.register_version_format(:number) do |tags|
           tags.detect { |tag| tag == latest_tag }
         end
         expect(instance.get_latest_ref({
@@ -68,5 +92,17 @@ RSpec.describe 'Ra10ke::Dependencies::Verification' do
         })).to eq(latest_tag)
       end
     end
+
+    context 'convert to ref' do
+       
+      it 'run rake task' do
+        output_conversion = File.read(File.join(fixtures_dir, 'Puppetfile_git_conversion'))
+        require 'ra10ke'
+        Ra10ke::RakeTask.new do |t|
+          t.basedir = fixtures_dir
+        end
+        expect{Rake::Task['r10k:print_git_conversion'].invoke}.to output(output_conversion).to_stdout
+      end
+    end
   end
 end

data/spec/ra10ke/deprecation_spec.rb

@@ -34,4 +34,15 @@ RSpec.describe 'Ra10ke::Deprecation::Validation' do
 
     expect(instance.bad_mods?).to eq(false)
   end
+
+  describe 'handles large puppetfile' do
+    let(:puppetfile) do
+      File.join(fixtures_dir, 'Puppetfile_deprecation_issue')
+    end
+
+    it 'deprecated modules' do
+      expect(instance.bad_mods?).to eq(false)
+      expect(instance.deprecated_modules.count).to eq(0)
+    end
+  end
 end

data/spec/ra10ke/git_repo_spec.rb

@@ -38,6 +38,10 @@ RSpec.describe 'Ra10ke::GitRepo' do
       expect(instance.remote_refs.first).to eq("1b3322d525e96bf7d0565b08703e2a44c90e7b4a\tHEAD\n")
     end
 
+    it '#get_ref_like' do
+      expect(instance.get_ref_like('8.0.0'))
+    end
+
     it '#valid_ref?' do
       expect(instance.valid_ref?('master')).to be true
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ra10ke
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 2.0.0
 platform: ruby
 authors:
 - Theo Chatzimichos
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-04-21 00:00:00.000000000 Z
+date: 2022-05-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -45,14 +45,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.6.5
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.6.5
 - !ruby/object:Gem::Dependency
   name: git
   requirement: !ruby/object:Gem::Requirement
@@ -184,6 +184,8 @@ files:
 - lib/ra10ke/version.rb
 - ra10ke.gemspec
 - spec/fixtures/Puppetfile
+- spec/fixtures/Puppetfile_deprecation_issue
+- spec/fixtures/Puppetfile_git_conversion
 - spec/fixtures/Puppetfile_test
 - spec/fixtures/Puppetfile_with_bad_refs
 - spec/fixtures/Puppetfile_with_commit
@@ -213,7 +215,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.4.0
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="