git-fastclone 1.4.1 → 1.4.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7e915bc1276ac42ac35559122dc1c12f4a04778032ec4510fe96a64b37fc4a7a
-  data.tar.gz: 9fcb6e4a441106da6b09f2d4b39cf0e740942ae99de051712fba27656d962f04
+  metadata.gz: 4b755aef58d17ded702b989b16fb92cf93908413403bb99f92b542319c59fdbc
+  data.tar.gz: c7ad9ab44516bea864027aa562af274bd49999aea62eccbfc3df9421cc6674d6
 SHA512:
-  metadata.gz: fc52b41c18799ec2154000bc7762c8301ed50f28d2eb252e4adcdc82ff36cdc6e3fa772d85db4d52ea95782ea16e02ea0460f1488b5ce95a745ee8a451511a3d
-  data.tar.gz: f86877adef196e5416e69aac06c7129bfaf22a21a566a5afe05e4780da60cdc3f0e2379e81d1bc0b4e8bd129d18a893f925dbd7096a4057ef428f085a964b2f2
+  metadata.gz: 8f6cff1907474ac42d8a4936dba9e5e684440b72f9995ce80bdb0bc79bc169d79969386b32eb3c8e8cc64358fc1872b0cbb07011d2bebfd753b04423fc1198be
+  data.tar.gz: 889d56227ae2aff1aba3d9dfc33fdd7ab554b9f6ae77f21c036c66af5c2eb47756d49e62a2244ca922855442856caaeb772baace5bc1c9f093d1eae0a3043b02

data/lib/git-fastclone/version.rb

@@ -2,5 +2,5 @@
 
 # Version string for git-fastclone
 module GitFastCloneVersion
-  VERSION = '1.4.1'
+  VERSION = '1.4.3'
 end

data/lib/git-fastclone.rb

@@ -74,7 +74,7 @@ module GitFastClone
     DEFAULT_GIT_ALLOW_PROTOCOL = 'file:git:http:https:ssh'
 
     attr_accessor :reference_dir, :prefetch_submodules, :reference_updated, :reference_mutex,
-                  :options, :abs_clone_path, :using_local_repo, :verbose, :color,
+                  :options, :abs_clone_path, :using_local_repo, :verbose, :print_git_errors, :color,
                   :flock_timeout_secs
 
     def initialize
@@ -100,6 +100,8 @@ module GitFastClone
 
       self.verbose = false
 
+      self.print_git_errors = false
+
       self.color = false
 
       self.flock_timeout_secs = 0
@@ -133,9 +135,15 @@ module GitFastClone
         end
 
         opts.on('-v', '--verbose', 'Verbose mode') do
+          puts '--print_git_errors is redundant when using --verbose' if print_git_errors
           self.verbose = true
         end
 
+        opts.on('--print_git_errors', 'Print git output if a command fails') do
+          puts '--print_git_errors is redundant when using --verbose' if verbose
+          self.print_git_errors = true
+        end
+
         opts.on('-c', '--color', 'Display colored output') do
           self.color = true
         end
@@ -212,14 +220,14 @@ module GitFastClone
         clone_commands = ['git', 'clone', verbose ? '--verbose' : '--quiet']
         clone_commands << '--reference' << mirror.to_s << url.to_s << clone_dest
         clone_commands << '--config' << config.to_s unless config.nil?
-        fail_on_error(*clone_commands, quiet: !verbose)
+        fail_on_error(*clone_commands, quiet: !verbose, print_on_failure: print_git_errors)
       end
 
       # Only checkout if we're changing branches to a non-default branch
       if rev
-        Dir.chdir(File.join(abs_clone_path, src_dir)) do
-          fail_on_error('git', 'checkout', '--quiet', rev.to_s, quiet: !verbose)
-        end
+        fail_on_error('git', 'checkout', '--quiet', rev.to_s, quiet: !verbose,
+                                                              print_on_failure: print_git_errors,
+                                                              chdir: File.join(abs_clone_path, src_dir))
       end
 
       update_submodules(src_dir, url)
@@ -241,10 +249,9 @@ module GitFastClone
 
       threads = []
       submodule_url_list = []
-      output = ''
-      Dir.chdir(File.join(abs_clone_path, pwd).to_s) do
-        output = fail_on_error('git', 'submodule', 'init', quiet: !verbose)
-      end
+      output = fail_on_error('git', 'submodule', 'init', quiet: !verbose,
+                                                         print_on_failure: print_git_errors,
+                                                         chdir: File.join(abs_clone_path, pwd))
 
       output.split("\n").each do |line|
         submodule_path, submodule_url = parse_update_info(line)
@@ -260,11 +267,10 @@ module GitFastClone
     def thread_update_submodule(submodule_url, submodule_path, threads, pwd)
       threads << Thread.new do
         with_git_mirror(submodule_url) do |mirror, _|
-          Dir.chdir(File.join(abs_clone_path, pwd).to_s) do
-            cmd = ['git', 'submodule', verbose ? nil : '--quiet', 'update', '--reference', mirror.to_s,
-                   submodule_path.to_s].compact
-            fail_on_error(*cmd, quiet: !verbose)
-          end
+          cmd = ['git', 'submodule',
+                 verbose ? nil : '--quiet', 'update', '--reference', mirror.to_s, submodule_path.to_s].compact
+          fail_on_error(*cmd, quiet: !verbose, print_on_failure: print_git_errors,
+                              chdir: File.join(abs_clone_path, pwd))
         end
 
         update_submodules(File.join(pwd, submodule_path), submodule_url)
@@ -337,13 +343,12 @@ module GitFastClone
     def store_updated_repo(url, mirror, repo_name, fail_hard)
       unless Dir.exist?(mirror)
         fail_on_error('git', 'clone', verbose ? '--verbose' : '--quiet', '--mirror', url.to_s, mirror.to_s,
-                      quiet: !verbose)
+                      quiet: !verbose, print_on_failure: print_git_errors)
       end
 
-      Dir.chdir(mirror) do
-        cmd = ['git', 'remote', verbose ? '--verbose' : nil, 'update', '--prune'].compact
-        fail_on_error(*cmd, quiet: !verbose)
-      end
+      cmd = ['git', 'remote', verbose ? '--verbose' : nil, 'update', '--prune'].compact
+      fail_on_error(*cmd, quiet: !verbose, print_on_failure: print_git_errors, chdir: mirror)
+
       reference_updated[repo_name] = true
     rescue RunnerExecutionRuntimeError => e
       # To avoid corruption of the cache, if we failed to update or check out we remove

data/lib/runner_execution.rb

@@ -22,7 +22,7 @@ module RunnerExecution
 
   # Runs a command that fails on error.
   # Uses popen2e wrapper. Handles bad statuses with potential for retries.
-  def fail_on_error(*cmd, stdin_data: nil, binmode: false, quiet: false, **opts)
+  def fail_on_error(*cmd, stdin_data: nil, binmode: false, quiet: false, print_on_failure: false, **opts)
     print_command('Running Shell Safe Command:', [cmd]) unless quiet
     shell_safe_cmd = shell_safe(cmd)
     retry_times = opts[:retry] || 0
@@ -39,7 +39,9 @@ module RunnerExecution
     end
 
     # Get out with the status, good or bad.
-    exit_on_status(output, [shell_safe_cmd], [status], quiet: quiet)
+    # When quiet, we don't need to print the output, as it is already streamed from popen2e_wrapper
+    needs_print_on_failure = quiet && print_on_failure
+    exit_on_status(output, [shell_safe_cmd], [status], quiet: quiet, print_on_failure: needs_print_on_failure)
   end
   module_function :fail_on_error
 
@@ -155,20 +157,21 @@ module RunnerExecution
   # return code of the first one.
   #
   # Otherwise returns first argument (output)
-  def exit_on_status(output, cmd_list, status_list, quiet: false)
+  def exit_on_status(output, cmd_list, status_list, quiet: false, print_on_failure: false)
     status_list.each_index do |index|
       status = status_list[index]
       cmd = cmd_list[index]
-      check_status(cmd, status, output: output, quiet: quiet)
+      check_status(cmd, status, output: output, quiet: quiet, print_on_failure: print_on_failure)
     end
 
     output
   end
   module_function :exit_on_status
 
-  def check_status(cmd, status, output: nil, quiet: false)
+  def check_status(cmd, status, output: nil, quiet: false, print_on_failure: false)
     return if status.exited? && status.exitstatus == 0
 
+    logger.info(output) if print_on_failure
     # If we exited nonzero or abnormally, print debugging info and explode.
     if status.exited?
       logger.debug("Process Exited normally. Exit status:#{status.exitstatus}") unless quiet

data/spec/git_fastclone_runner_spec.rb

@@ -91,7 +91,6 @@ describe GitFastClone::Runner do
     before(:each) do
       allow(runner_execution_double).to receive(:fail_on_error) {}
       allow(Dir).to receive(:pwd) { '/pwd' }
-      allow(Dir).to receive(:chdir).and_yield
       allow(subject).to receive(:with_git_mirror).and_yield('/cache', 0)
       expect(subject).to receive(:clear_clone_dest_if_needed).once {}
     end
@@ -99,11 +98,11 @@ describe GitFastClone::Runner do
     it 'should clone correctly' do
       expect(subject).to receive(:fail_on_error).with(
         'git', 'checkout', '--quiet', 'PH',
-        { quiet: true }
+        { chdir: '/pwd/.', print_on_failure: false, quiet: true }
       ) { runner_execution_double }
       expect(subject).to receive(:fail_on_error).with(
         'git', 'clone', '--quiet', '--reference', '/cache', 'PH', '/pwd/.',
-        { quiet: true }
+        { quiet: true, print_on_failure: false }
       ) { runner_execution_double }
 
       subject.clone(placeholder_arg, placeholder_arg, '.', nil)
@@ -113,11 +112,11 @@ describe GitFastClone::Runner do
       subject.verbose = true
       expect(subject).to receive(:fail_on_error).with(
         'git', 'checkout', '--quiet', 'PH',
-        { quiet: false }
+        { chdir: '/pwd/.', print_on_failure: false, quiet: false }
       ) { runner_execution_double }
       expect(subject).to receive(:fail_on_error).with(
         'git', 'clone', '--verbose', '--reference', '/cache', 'PH', '/pwd/.',
-        { quiet: false }
+        { quiet: false, print_on_failure: false }
       ) { runner_execution_double }
 
       subject.clone(placeholder_arg, placeholder_arg, '.', nil)
@@ -126,11 +125,26 @@ describe GitFastClone::Runner do
     it 'should clone correctly with custom configs' do
       expect(subject).to receive(:fail_on_error).with(
         'git', 'clone', '--quiet', '--reference', '/cache', 'PH', '/pwd/.', '--config', 'config',
-        { quiet: true }
+        { quiet: true, print_on_failure: false }
       ) { runner_execution_double }
 
       subject.clone(placeholder_arg, nil, '.', 'config')
     end
+
+    context 'with printing errors' do
+      before(:each) do
+        subject.print_git_errors = true
+      end
+
+      it 'prints failures' do
+        expect(subject).to receive(:fail_on_error).with(
+          'git', 'clone', '--quiet', '--reference', '/cache', 'PH', '/pwd/.', '--config', 'config',
+          { quiet: true, print_on_failure: true }
+        ) { runner_execution_double }
+
+        subject.clone(placeholder_arg, nil, '.', 'config')
+      end
+    end
   end
 
   describe '.clear_clone_dest_if_needed' do
@@ -323,7 +337,6 @@ describe GitFastClone::Runner do
 
     it 'should correctly update the hash' do
       allow(subject).to receive(:fail_on_error)
-      allow(Dir).to receive(:chdir) {}
 
       subject.reference_updated = placeholder_hash
       subject.store_updated_repo(placeholder_arg, placeholder_arg, placeholder_arg, false)
@@ -374,7 +387,6 @@ describe GitFastClone::Runner do
         expected_command = expected_commands.shift
         expect(command).to eq(expected_command)
       }
-      allow(Dir).to receive(:chdir).and_yield
 
       allow(subject).to receive(:print_formatted_error) {}
       allow(subject).to receive(:reference_repo_dir).and_return(test_reference_repo_dir)
@@ -389,7 +401,8 @@ describe GitFastClone::Runner do
       [
         ['git', 'clone', verbose ? '--verbose' : '--quiet', '--mirror', test_url_valid,
          test_reference_repo_dir],
-        ['git', 'remote', verbose ? '--verbose' : nil, 'update', '--prune'].compact
+        ['git', 'remote', verbose ? '--verbose' : nil, 'update',
+         '--prune'].compact
       ]
     end
 

data/spec/runner_execution_spec.rb

@@ -0,0 +1,58 @@
+# frozen_string_literal: true
+
+# Copyright 2023 Square Inc.
+
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+
+#     http://www.apache.org/licenses/LICENSE-2.0
+
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+require 'spec_helper'
+require 'git-fastclone'
+
+# Integration tests use real demo_tool.sh to inspect the E2E behavior
+describe RunnerExecution do
+  subject { described_class }
+  let(:external_tool) { "#{__dir__}/../script/spec_demo_tool.sh" }
+  let(:logger) { double('logger') }
+
+  before do
+    allow($stdout).to receive(:puts)
+    allow(logger).to receive(:info)
+    allow(logger).to receive(:debug)
+    allow(logger).to receive(:warn)
+    allow(RunnerExecution).to receive(:logger).and_return(logger)
+  end
+
+  describe '.fail_on_error' do
+    it 'should log failure info on command error' do
+      expect(logger).to receive(:info).with("My error output\n")
+
+      expect do
+        described_class.fail_on_error(external_tool, '1', 'My error output', quiet: true,
+                                                                             print_on_failure: true)
+      end.to raise_error(RunnerExecution::RunnerExecutionRuntimeError)
+    end
+
+    it 'should not log failure output on command success' do
+      expect($stdout).not_to receive(:info)
+
+      described_class.fail_on_error(external_tool, '0', 'My success output', quiet: true,
+                                                                             print_on_failure: true)
+    end
+
+    it 'should not log failure output when not in the quiet mode' do
+      expect($stdout).not_to receive(:info)
+
+      described_class.fail_on_error(external_tool, '0', 'My success output', quiet: false,
+                                                                             print_on_failure: true)
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: git-fastclone
 version: !ruby/object:Gem::Version
-  version: 1.4.1
+  version: 1.4.3
 platform: ruby
 authors:
 - Michael Tauraso
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-05-19 00:00:00.000000000 Z
+date: 2023-09-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colorize
@@ -44,6 +44,7 @@ files:
 - lib/runner_execution.rb
 - spec/git_fastclone_runner_spec.rb
 - spec/git_fastclone_url_helper_spec.rb
+- spec/runner_execution_spec.rb
 - spec/spec_helper.rb
 homepage: http://square.github.io/git-fastclone/
 licenses: