avm-git 0.1.0 → 0.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b981cbcaacc3f23090895cd39d733ce3bd76067c7944331aaf5cd163bc6a39f7
-  data.tar.gz: ffffcbf994c1eb3a8e5464d5cd7b2ef5e8a82ad0ba29bc1263614dd7f213d404
+  metadata.gz: d460b71bec0badf0833da7e9d2f0fe66c943adf3962a5dc2b596bec89dc224c7
+  data.tar.gz: 69d2042b3b4dec08fc08695d78216121f0e5a4c5b85234d238e8dff549394418
 SHA512:
-  metadata.gz: 14f3e4c7c070e3479a23964439ceb5660772fe59106a85ea854918a487a24f13ce0530f103a86aa7ab9e8c72955e1c7408b31e9818938dbd010d781d0c9cd94f
-  data.tar.gz: 073e0cc2d85ad2c652467df568f64ddf1a1c5e3b14aff72bb4957767f2d2b33a72a0f4b0e3cb0586fb40afd916cf7015c4a9c235ab014b1c1cf2c31f1c4ba078
+  metadata.gz: f0d4e10c843fc8bd92befea6a35ba9d6160fcdc681e7a8488ac2f6004ed7de7d1229f55f692106211cc2ede750db1b2635bd7bcd1773a42a6c4c7f43f659def7
+  data.tar.gz: cc56195fd73c5e6457ee1186d146b23138e0ec7e0d52271f4fca11f188480d23ee08158364040727a4acb2f5a2895346dd7013acb7871effb15478761b66c94b

data/lib/avm/git/issue/complete/commits.rb

@@ -15,7 +15,7 @@ module Avm
             return [] unless branch_hash && follow_master?
 
             interval = remote_master_hash ? "#{remote_master_hash}..#{branch_hash}" : branch_hash
-            @git.execute!('rev-list', interval).each_line.map(&:strip)
+            launcher_git.execute!('rev-list', interval).each_line.map(&:strip)
           end
 
           def bifurcations_result
@@ -32,8 +32,8 @@ module Avm
           end
 
           def commit_parents(commit)
-            @git.execute!('log', '--pretty=%P', '-n', '1', commit).split(' ').map(&:strip)
-                .select(&:present?)
+            launcher_git.execute!('log', '--pretty=%P', '-n', '1', commit).split(' ').map(&:strip)
+                        .select(&:present?)
           end
         end
       end

data/lib/avm/git/issue/complete/git_subrepos.rb

@@ -12,7 +12,7 @@ module Avm
             return ::Avm::Result.error('Unclean workspace') unless clean_workspace?
 
             infom 'Checking Git subrepos...'
-            r = ::Avm::Git::SubrepoChecks.new(::EacGit::Local.new(@git)).add_all_subrepos
+            r = ::Avm::Git::SubrepoChecks.new(::EacGit::Local.new(launcher_git)).add_all_subrepos
             r.check_remote = true
             r.result
           end

data/lib/avm/git/issue/complete/local_branch.rb

@@ -7,16 +7,21 @@ module Avm
     module Issue
       class Complete
         module LocalBranch
+          NO_CURRENT_BRANCH_NAME = 'none'
+
+          # Retrieves the current local branch.
+          #
+          # @return [EacGit::Local::Branch, nil]
           def branch_uncached
-            @git.current_branch
+            eac_git.current_branch
           end
 
           def branch_hash_uncached
-            @git.rev_parse("refs/heads/#{branch}")
+            branch.if_present(&:current_commit_id)
           end
 
           def branch_name
-            branch.split('/')[-1]
+            branch.if_present(NO_CURRENT_BRANCH_NAME, &:name)
           end
 
           def branch_name_result
@@ -38,14 +43,16 @@ module Avm
           end
 
           def follow_master?
-            remote_master_hash ? @git.descendant?(branch_hash, remote_master_hash) : true
+            remote_master_hash ? launcher_git.descendant?(branch_hash, remote_master_hash) : true
           end
 
           def remove_local_branch
+            return unless branch
+
             info 'Removendo branch local...'
             bn = branch_name
-            git(['checkout', branch_hash])
-            git(['branch', '-D', bn])
+            git_execute(['checkout', branch_hash])
+            git_execute(['branch', '-D', bn])
           end
         end
       end

data/lib/avm/git/issue/complete/local_tag.rb

@@ -18,7 +18,7 @@ module Avm
 
           def delete_tag
             info 'Removendo tag...'
-            git(['tag', '-d', branch_name])
+            git_execute(['tag', '-d', branch_name])
           end
 
           def tag
@@ -26,11 +26,11 @@ module Avm
           end
 
           def tag_hash
-            @git.rev_parse(tag)
+            launcher_git.rev_parse(tag)
           end
 
           def create_tag
-            git(['tag', branch_name, branch_hash])
+            git_execute(['tag', branch_name, branch_hash])
           end
         end
       end

data/lib/avm/git/issue/complete/push.rb

@@ -12,13 +12,7 @@ module Avm
           def dry_push_result
             return ::Avm::Result.error('Nothing to push') if pushs.empty?
 
-            r = @git.execute(dry_push_args)
-            message = if r.fetch(:exit_code).zero?
-                        'ok'
-                      else
-                        r.fetch(:stderr) + "\n#{::Shellwords.join(dry_push_args)}"
-                      end
-            ::Avm::Result.success_or_error(r.fetch(:exit_code).zero?, message)
+            dry_push_execution_result
           end
 
           def push
@@ -26,7 +20,7 @@ module Avm
               info 'PUSH: Nada a enviar'
             else
               info "PUSH: enviando \"#{pushs}\"..."
-              git(%w[push origin] + pushs)
+              git_execute(%w[push origin] + pushs)
             end
           end
 
@@ -39,7 +33,7 @@ module Avm
           end
 
           def remove_branch_push
-            remote_branch_hash ? ":refs/heads/#{branch}" : nil
+            remote_branch_hash ? ":refs/heads/#{branch.name}" : nil
           end
 
           def tag_push
@@ -47,6 +41,18 @@ module Avm
 
             "#{branch_hash}:#{tag}"
           end
+
+          private
+
+          def dry_push_execution_result
+            r = launcher_git.execute(dry_push_args)
+            message = if r.fetch(:exit_code).zero?
+                        'ok'
+                      else
+                        r.fetch(:stderr) + "\n#{::Shellwords.join(dry_push_args)}"
+                      end
+            ::Avm::Result.success_or_error(r.fetch(:exit_code).zero?, message)
+          end
         end
       end
     end

data/lib/avm/git/issue/complete/remote.rb

@@ -10,7 +10,7 @@ module Avm
           end
 
           def remote_branch_hash
-            remote_hashs["refs/heads/#{branch}"]
+            remote_hashs["refs/heads/#{branch.name}"]
           end
 
           def remote_tag_hash
@@ -24,7 +24,7 @@ module Avm
           end
 
           def remote_hashs_uncached
-            @git.remote_hashs(remote_name)
+            launcher_git.remote_hashs(remote_name)
           end
         end
       end

data/lib/avm/git/issue/complete/test.rb

@@ -2,6 +2,8 @@
 
 require 'avm/sources/configuration'
 require 'avm/result'
+require 'avm/sources/base'
+require 'avm/sources/tests/builder'
 require 'eac_ruby_utils/fs/temp'
 
 module Avm
@@ -10,33 +12,26 @@ module Avm
       class Complete
         module Test
           def test_result
-            test_command = configuration.if_present(&:any_test_command)
-            return ::Avm::Result.success('unconfigured') if test_command.blank?
-
-            infom "Running test command \"#{test_command}\"..."
-            result = test_command.execute
-            test_result_log(result)
-            if result.fetch(:exit_code).zero?
-              ::Avm::Result.success('yes')
-            else
-              ::Avm::Result.error('no')
+            infom 'Running tests...'
+            test_performer.units.each do |single|
+              return ::Avm::Result.error(test_failed_result_message(single)) if single.failed?
             end
+            ::Avm::Result.success('all passed')
           end
 
           private
 
-          def test_result_log(result)
-            stdout_file = ::EacRubyUtils::Fs::Temp.file
-            stderr_file = ::EacRubyUtils::Fs::Temp.file
-            stdout_file.write(result.fetch(:stdout))
-            stderr_file.write(result.fetch(:stderr))
-            infov '  * Exit code', result.fetch(:exit_code)
-            infov '  * STDOUT', stdout_file.to_path
-            infov '  * STDERR', stderr_file.to_path
+          def test_failed_result_message(single)
+            { 'Source' => single, 'STDOUT' => single.logs[:stdout],
+              'STDERR' => single.logs[:stderr] }.map { |k, v| "#{k}: #{v}" }.join(', ')
           end
 
-          def configuration_uncached
-            ::Avm::Sources::Configuration.find_by_path(@git)
+          def test_performer
+            ::Avm::Sources::Tests::Builder
+              .new(::Avm::Registry.sources.detect(launcher_git))
+              .include_main(true)
+              .include_subs(true)
+              .performer
           end
         end
       end

data/lib/avm/git/issue/complete/tracker.rb

@@ -19,7 +19,7 @@ module Avm
           end
 
           def branch_short_hash
-            git(['log', '--pretty=format:%h', '-1', '-q', branch_hash])
+            git_execute(['log', '--pretty=format:%h', '-1', '-q', branch_hash])
           end
         end
       end

data/lib/avm/git/issue/complete/validation.rb

@@ -11,6 +11,8 @@ module Avm
           enable_simple_cache
           common_constructor :parent, :key, :label
 
+          SKIPPED_RESULT_MESSAGE = 'skipped'
+
           def skip?
             parent.skip_validations.include?(key)
           end
@@ -18,11 +20,17 @@ module Avm
           private
 
           def result_uncached
-            if skip?
-              ::Avm::Result.neutral('skipped')
-            else
-              parent.send("#{key}_result")
-            end
+            skip? ? skipped_result : validation_result
+          end
+
+          def skipped_result
+            ::Avm::Result.neutral(SKIPPED_RESULT_MESSAGE)
+          end
+
+          def validation_result
+            parent.send("#{key}_result")
+          rescue ::RuntimeError => e
+            ::Avm::Result.error("error raised: #{e.message}")
           end
         end
       end

data/lib/avm/git/issue/complete/working_tree.rb

@@ -10,7 +10,7 @@ module Avm
           end
 
           def clean_workspace?
-            @git.dirty_files.none?
+            launcher_git.dirty_files.none?
           end
         end
       end

data/lib/avm/git/issue/complete.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require 'avm/tools/core_ext'
+require 'eac_git/local'
 
 module Avm
   module Git
@@ -10,14 +11,15 @@ module Avm
         enable_simple_cache
         enable_speaker
 
-        attr_reader :skip_validations
+        BRANCH_NAME_ISSUE_ID_PATTERN =
+
+          attr_reader :dir, :skip_validations
 
         def initialize(options)
           consumer = ::EacRubyUtils::OptionsConsumer.new(options)
-          dir, @skip_validations = consumer.consume_all(:dir, :skip_validations)
+          @dir, @skip_validations = consumer.consume_all(:dir, :skip_validations)
           validate_skip_validations
           consumer.validate
-          @git = ::Avm::Launcher::Git::Base.new(dir)
         end
 
         def start_banner
@@ -35,16 +37,29 @@ module Avm
         end
 
         def issue_id
-          m = branch_name.match(/\A#{Regexp.quote('issue_')}(\d+)\z/)
-          m ? m[1].to_i : nil
+          branch ? issue_id_parser.parse(branch_name) : nil
+        end
+
+        # @return [EacRubyUtils::RegexpParser]
+        def issue_id_parser
+          /\A#{Regexp.quote('issue_')}(\d+)\z/.to_parser { |m| m[1].to_i }
         end
 
         private
 
-        def git(args, exit_outputs = {})
-          r = @git.execute!(args, exit_outputs: exit_outputs)
+        # @return [EacGit::Local]
+        def eac_git_uncached
+          ::EacGit::Local.new(dir)
+        end
+
+        def git_execute(args, exit_outputs = {})
+          r = launcher_git.execute!(args, exit_outputs: exit_outputs)
           r.is_a?(String) ? r.strip : r
         end
+
+        def launcher_git_uncached
+          ::Avm::Launcher::Git::Base.new(dir)
+        end
       end
     end
   end

data/lib/avm/git/revision_test.rb

@@ -77,7 +77,7 @@ module Avm
       end
 
       def successful_uncached
-        if options.fetch(:no_cache) || !successful_cache.cached?
+        if options.fetch(:no_cache) || !successful_cache.stored?
           checkout_revision
           run_test
         end

data/lib/avm/git/scms/git.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require 'avm/git/scms/git_subrepo'
 require 'avm/scms/base'
 require 'eac_ruby_utils/core_ext'
 
@@ -56,6 +57,13 @@ module Avm
           commit_dirty(message)
         end
 
+        # @return [Enumerable<Avm::Git::Scms::GitSubrepo>]
+        def subs
+          git_repo.subrepos.map do |subrepo|
+            ::Avm::Git::Scms::GitSubrepo.new(subrepo.subpath.expand_path(path))
+          end
+        end
+
         def valid?
           path.join('.git').exist?
         end

data/lib/avm/git/version.rb

@@ -2,6 +2,6 @@
 
 module Avm
   module Git
-    VERSION = '0.1.0'
+    VERSION = '0.3.2'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: avm-git
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.3.2
 platform: ruby
 authors:
 - Put here the authors
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-10-13 00:00:00.000000000 Z
+date: 2022-01-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: avm-files
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.11'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.11'
 - !ruby/object:Gem::Dependency
   name: eac_ruby_utils
   requirement: !ruby/object:Gem::Requirement