feat: pass more params on webhook request

spec/coverage_reporter/api/webhook_spec.cr

@@ -1,9 +1,10 @@
 require "../../spec_helper"
 
 Spectator.describe CoverageReporter::Api::Webhook do
-  subject { described_class.new(config, "flag1,flag2") }
+  subject { described_class.new(config, "flag1,flag2", git_info) }
 
   let(config) { CoverageReporter::Config.new("token") }
+  let(git_info) { {:branch => "chore/add-tests", :head => {:message => "add tests"}} }
   let(endpoint) { "#{CoverageReporter::Config::DEFAULT_ENDPOINT}/webhook" }
 
   after_each { WebMock.reset }
@@ -25,6 +26,12 @@ Spectator.describe CoverageReporter::Api::Webhook do
         :build_num => nil,
         :status    => "done",
       },
+      :git => {
+        :branch => "chore/add-tests",
+        :head   => {
+          :message => "add tests",
+        },
+      },
     }.to_json
   end
 

spec/coverage_reporter/cli/cmd_spec.cr

@@ -176,6 +176,25 @@ Spectator.describe CoverageReporter::Cli do
       end
     end
 
+    context "without carryforward (new args)" do
+      let(options) do
+        %w(
+          done
+          -m
+          --build-number 3
+          --no-fail
+        )
+      end
+
+      it "doesn't fail" do
+        expect(subject).to eq 0
+        expect(reporter.carryforward).to eq nil
+        expect(reporter.overrides.try(&.to_h)).to eq({
+          :service_number => "3",
+        })
+      end
+    end
+
     context "with format option" do
       let(options) do
         %w(

src/coverage_reporter/api/webhook.cr

@@ -3,7 +3,11 @@ require "json"
 
 module CoverageReporter
   class Api::Webhook
-    def initialize(@config : Config, @carryforward : String?)
+    def initialize(
+      @config : Config,
+      @carryforward : String?,
+      @git : Hash(Symbol, Hash(Symbol, String) | String)
+    )
     end
 
     def send_request(dry_run : Bool = false)
@@ -18,6 +22,7 @@ module CoverageReporter
           :build_num => @config[:service_number]?,
           :status    => "done",
         },
+        :git => @git,
       }.compact)
 
       Log.debug "---\n⛑ Debug Output:\n#{data.to_pretty_json}"

src/coverage_reporter/cli/cmd.cr

@@ -23,7 +23,7 @@ module CoverageReporter::Cli
       overrides: opts.overrides,
       parallel: opts.parallel?,
       repo_token: opts.repo_token,
-      measure: opts.debug? || opts.measure?,
+      measure: opts.debug? || opts.measure?
     )
 
     if opts.parallel_done?

src/coverage_reporter/reporter.cr

@@ -63,7 +63,11 @@ module CoverageReporter
     # from a CI-specific ENV, or can be set explicitly via `COVERALLS_SERVICE_NUMBER`
     # environment variable.
     def parallel_done
-      api = Api::Webhook.new(config, settings.carryforward || config.carryforward)
+      api = Api::Webhook.new(
+        config: config,
+        carryforward: settings.carryforward || config.carryforward,
+        git: Git.info(config)
+      )
 
       measure("Webhook request") do
         api.send_request(settings.dry_run)