gitarro 0.1.56 → 0.1.70

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 66c7a56331a26d15804e98a354e00de75b7a9c08
-  data.tar.gz: 18914ad11c07fba2e904a50cd28e1f993f343208
+  metadata.gz: 2a2a34d84599e596da7d621b259fc5d28e0595d1
+  data.tar.gz: d625f03c948ff34129250332dffbc0fdb930f7bc
 SHA512:
-  metadata.gz: 44ef09f2898c29b82d5a7feecaa57965e3448d94c4a0e8954350f8ef4b8bef61d46f0bd30bfd5787f0d21c7e3eeb2477215e2aa85234ffd454d9b6db0c593f6b
-  data.tar.gz: ca747b9497342c94d91e9ef654ddf10ababbe2dcba7311ce5a8ad0b38dc762d696ca61c93598803e73fe5d45a9f3f68744d6ad035fd5a8fa5324920e40b876d5
+  metadata.gz: 2f6dedf76a39c784378cf4430f53f65d328269f3b8774e6d614f45cbfe03e4d0e92cee3566fef946a071d26515cc861ec167a3a02d2748c6916d1c335c9d2c3c
+  data.tar.gz: 660e5aecfd92a15f21407b494da427241d03b7c493883b9cbe3be203802e7280d9a598b743bd688fb2da9677db1e170923484ce4d5640065a3cf3440ebba424c

data/lib/gitarro/backend.rb

@@ -84,15 +84,11 @@ class TestExecutor
 
   # this will clone the repo and execute the tests
   def pr_test(pr)
-    git = GitOp.new(@git_dir, pr, @options)
-    # merge PR-branch to upstream branch
-    git.merge_pr_totarget(pr.base.ref, pr.head.ref)
+    clone_repo(@noshallow)
     # export variables
     export_pr_variables(pr)
     # do valid tests and store the result
     test_status = run_script
-    # del branch
-    git.del_pr_branch(pr.base.ref, pr.head.ref)
     test_status
   end
 
@@ -105,6 +101,23 @@ class TestExecutor
 
   private
 
+  def clone_repo(noshallow)
+    shallow = GitShallowClone.new(@git_dir, pr, @options)
+    # by default we use always shallow clone
+    unless noshallow
+      shallow.clone
+      return
+    end
+    # this is for using the merging to ref
+    full_clone
+  end
+
+  def full_clone
+    git = GitOp.new(@git_dir, pr, @options)
+    git.merge_pr_totarget(pr.base.ref, pr.head.ref)
+    git.del_pr_branch(pr.base.ref, pr.head.ref)
+  end
+
   def export_pr_variables(pr)
     ENV['GITARRO_PR_AUTHOR'] = pr.head.user.login.to_s
     ENV['GITARRO_PR_TITLE'] = pr.title.to_s

data/lib/gitarro/git_op.rb

@@ -3,6 +3,39 @@
 require 'English'
 require 'fileutils'
 require 'timeout'
+require 'tmpdir'
+
+# handle shallow clone
+class GitShallowClone
+  attr_reader :git_dir, :pr, :options, :repo_protocol
+  def initialize(git_dir, pr, options)
+    @git_dir = git_dir
+    # pr object for extract all relev. data.
+    @pr = pr
+    # All gitarro options
+    @options = options
+    gh = 'https://github.com/'
+    gg = 'git@github.com:'
+    @repo_protocol = @options[:https] ? gh : gg
+    @repo_url = @options[:https] ? pr.head.repo.html_url : pr.head.repo.ssh_url
+  end
+
+  # shallow clone
+  def clone
+    tmp_dir = create_tmp_dir!
+    git_local = "#{git_dir}/#{tmp_dir}"
+    puts `git clone --depth 1 #{@repo_url} -b #{pr.head.ref} #{git_local}`
+    exit 1 if $CHILD_STATUS.exitstatus.nonzero?
+    Dir.chdir git_local
+  end
+
+  private
+
+  def create_tmp_dir!
+    repo = options[:repo].split('/')[1]
+    "#{repo}#{Time.now.to_i}_#{rand(100)}"
+  end
+end
 
 # This class is used by lib/backend.rb
 # git operation for gitarro

data/lib/gitarro/opt_parser.rb

@@ -22,21 +22,11 @@ module MandatoryOptions
     end
   end
 
-  def git_opt(opt)
-    desc = 'Specify a location where gitarro will clone the GitHub project. '\
-           'If the dir does not exists, gitarro will create one. '\
-           'For example: /tmp/'
-    opt.on('-g', "--git_dir 'GIT_LOCAL_DIR'", desc) do |git_dir|
-      @options[:git_dir] = git_dir
-    end
-  end
-
   def mandatory_options(opt)
     opt.separator 'Mandatory options:'
     repo_opt(opt)
     context_opt(opt)
     test_opt(opt)
-    git_opt(opt)
   end
 end
 
@@ -47,12 +37,26 @@ module OptionalOptions
     opt.on('-C', '--check', desc) { |check| @options[:check] = check }
   end
 
+  def no_shallow(opt)
+    desc = 'If enabled, gitarro will not use git shallow clone'
+    opt.on('--noshallow', desc) { |noshallow| @options[:noshallow] = noshallow }
+  end
+
   def desc_opt(opt)
     opt.on('-d', "--description 'DESCRIPTION'", 'Test decription') do |d|
       @options[:description] = d
     end
   end
 
+  def git_opt(opt)
+    desc = 'Specify a location where gitarro will clone the GitHub project. '\
+           'If the dir does not exists, gitarro will create one. '\
+           'by default is the /tmp'
+    opt.on('-g', "--git_dir 'GIT_LOCAL_DIR'", desc) do |git_dir|
+      @options[:git_dir] = git_dir
+    end
+  end
+
   def url_opt(opt)
     desc = 'Specify the URL to append to add to the GitHub review. ' \
            'Usually you will use an URL to the Jenkins build log.'
@@ -95,11 +99,13 @@ module OptionalOptions
     opt.separator "\n Optional options:"
     desc_opt(opt)
     check_opt(opt)
+    no_shallow(opt)
     file_opt(opt)
     url_opt(opt)
     pr_number(opt)
     https_opt(opt)
     changed_since(opt)
+    git_opt(opt)
   end
 end
 
@@ -127,7 +133,7 @@ class OptParserInternal
 
   def parse(opt_parser)
     parse_options(opt_parser)
-    mandatory_options = %w[repo context test_file git_dir]
+    mandatory_options = %w[repo context test_file]
     mandatory_options.each { |opt| ck_mandatory_option(opt) }
     defaults_false
     defaults_to_text
@@ -159,6 +165,7 @@ class OptParserInternal
     @options[:check] = false if @options[:check].nil?
     @options[:target_url] = '' if @options[:target_url].nil?
     @options[:https] = false if @options[:https].nil?
+    @options[:noshallow] = false if @options[:noshallow].nil?
     @options[:changed_since] = -1 if @options[:changed_since].nil?
   end
 
@@ -166,6 +173,7 @@ class OptParserInternal
     desc = 'use option -d to set a custom test description.'
     @options[:description] = desc if @options[:description].nil?
     @options[:file_type] = 'notype' if @options[:file_type].nil?
+    @options[:git_dir] = '/tmp' if @options[:git_dir].nil?
   end
 end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitarro
 version: !ruby/object:Gem::Version
-  version: 0.1.56
+  version: 0.1.70
 platform: ruby
 authors:
 - Dario Maiocchi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-11-02 00:00:00.000000000 Z
+date: 2017-11-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: english
@@ -168,7 +168,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.2
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: gitarro gem