gitarro 0.1.85 → 0.1.86

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2559fce40754b64d75bb04d8f3bda7444e28b5f29333724d24411aaf815cda62
-  data.tar.gz: f52c91ed53608fef91d89e0c8939d0997f57919f09a1584bb6c565c42be45076
+  metadata.gz: d1060fb1ac1b396de8cf3d4262cab2b534f05cb1ca46aca564adb69bcc70991a
+  data.tar.gz: d855875451608261291992b8ed30dd5a55b38a0a0dc3f137d7d07ae439fd9977
 SHA512:
-  metadata.gz: ad01fcbbf254c3b8d3a0efacefdbb83e13dd8961f1e0310681dd96e73793f2d3261c7f55effc75fb30e38f06bb843c04456edba7a2805837ba6952273ce8d558
-  data.tar.gz: 1f4d107cf5dd7364ee1d964d3c71bc884d7472980269598179ac0a52bb025bc234360db56be8b0feef2c489515991d0e19025fe038f6a7b9fd8a891e7ced3461
+  metadata.gz: 1936f7fdc6ee3654cb93cab8c2c964b55ac6eb0943f60a85347333307c7db17a69787b26dd21675c3b3e2cd88866981a8dffb57aef138ee88fff03eb5a1c36d0
+  data.tar.gz: 6fa07c2bee9524c77f2a315e2129d215e3a3f17b32b828bc4254650a59fefa4305e0a007e22f5d686c10446eb270b29bc6e5c42c40e56077c2cffb0203970a61

data/bin/gitarro

@@ -8,23 +8,28 @@ require 'gitarro/git_op'
 require 'gitarro/backend'
 
 b = Backend.new
-exit 0 if b.triggered_by_pr_number?
+prs = b.required_prs
 
-prs = b.open_newer_prs
 exit 0 if prs.empty?
 
 prs.each do |pr|
-  puts '=' * 30 + "\n" + "TITLE_PR: #{pr.title}, NR: #{pr.number}\n" + '=' * 30
+  puts '=' * 30 + "\n" + "TITLE_PR: #{pr.title}, NR: #{pr.number}"
+  puts "DESCRIPTION:\n#{pr.body}\n" + '=' * 30
+
   # check if prs contains the branch given otherwise just break
   next unless b.pr_equal_specific_branch?(pr)
 
   # this check the last commit state, catch for review or not reviewd status.
   comm_st = b.client.status(b.repo, pr.head.sha)
 
+  # force test without any filter or check
+  break if b.force_run_test(pr)
+
   # retrigger if magic word found
   b.retrigger_check(pr)
   # 0) do test for unreviewed pr
   break if b.unreviewed_new_pr?(pr, comm_st)
+
   # we run the test in 2 conditions:
   # 1) the context  is not set, test didnt run
   # 2) the pending status is set on commit, repeat always when pending set

data/lib/gitarro/backend.rb

@@ -170,6 +170,13 @@ class Backend
     @gbexec = TestExecutor.new(@options)
   end
 
+  # public method retrieve pull request to process
+  def required_prs
+    return open_newer_prs if @options[:pr_number].nil?
+
+    [@client.pull_request(@repo, @options[:pr_number])]
+  end
+
   # public method for check if pr belong to user specified branch
   # if the pr belong to the branch continue tests
   # otherwise just skip tests without setting any status
@@ -192,9 +199,18 @@ class Backend
     prs
   end
 
+  # public forcing to run the test
+  def force_run_test(pr)
+    return false unless @force_test && defined?(@pr_number)
+
+    print_test_required
+    gbexec.export_pr_data(pr)
+    launch_test_and_setup_status(pr)
+  end
+
   # public for retrigger the test
   def retrigger_check(pr)
-    return unless retrigger_needed?(pr)
+    return false unless retrigger_needed?(pr)
 
     print_test_required
     gbexec.export_pr_data(pr)
@@ -216,7 +232,7 @@ class Backend
   def unreviewed_new_pr?(pr, comm_st)
     return unless commit_is_unreviewed?(comm_st)
 
-    return if empty_files_changed_by_pr?(pr)
+    return true if empty_files_changed_by_pr?(pr)
 
     # gb.check is true when there is a job running as scheduler
     # which doesn't execute the test but trigger another job
@@ -259,7 +275,7 @@ class Backend
   # this function will check if the PR contains a checkbox
   # for retrigger all the tests.
   def retriggered_by_checkbox?(pr, context)
-    return false unless pr.body.include? "[x] Re-run test \"#{context}\""
+    return false unless pr.body.match(/\[x\]\s+Re-run\s+test\s+#{context}/i)
 
     skipped = ''
     unless empty_files_changed_by_pr?(pr)

data/lib/gitarro/opt_parser.rb

@@ -37,6 +37,11 @@ module OptionalOptions
     opt.on('-C', '--check', desc) { |check| @options[:check] = check }
   end
 
+  def force_test_opt(opt)
+    desc = 'Force to run a test, even if is not marked to be re-triggered.'
+    opt.on('-F', '--force_test', desc) { |force_test| @options[:force_test] = force_test }
+  end
+
   def no_shallow(opt)
     desc = 'If enabled, gitarro will not use git shallow clone'
     opt.on('--noshallow', desc) { |noshallow| @options[:noshallow] = noshallow }
@@ -106,6 +111,7 @@ module OptionalOptions
     opt.separator "\n Optional options:"
     desc_opt(opt)
     check_opt(opt)
+    force_test_opt(opt)
     branch_opt(opt)
     no_shallow(opt)
     file_opt(opt)
@@ -171,6 +177,7 @@ class OptParserInternal
 
   # set some default values
   def defaults_false
+    @options[:force_test] = false if @options[:force_test].nil?
     @options[:check] = false if @options[:check].nil?
     @options[:target_url] = '' if @options[:target_url].nil?
     @options[:https] = false if @options[:https].nil?

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitarro
 version: !ruby/object:Gem::Version
-  version: 0.1.85
+  version: 0.1.86
 platform: ruby
 authors:
 - Dario Maiocchi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-03-25 00:00:00.000000000 Z
+date: 2019-06-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: english