Add Combined GitHub Status

README.md

@@ -138,6 +138,12 @@ Use `GithubStatusFormatter` to submit [commit status](https://github.com/blog/12
 $ PRONTO_GITHUB_ACCESS_TOKEN=token pronto run -f github_status -c origin/master
 ```
 
+If you want to show a one single status for all runners, instead of status per runner:
+
+```sh
+$ PRONTO_GITHUB_ACCESS_TOKEN=token pronto run -f github_combined_status -c origin/master
+```
+
 It's possible to combine multiple formatters.
 To get both pull request comments and commit status summary use:
 

lib/pronto.rb

@@ -42,6 +42,7 @@
 require 'pronto/formatter/pull_request_formatter'
 require 'pronto/formatter/github_formatter'
 require 'pronto/formatter/github_status_formatter'
+require 'pronto/formatter/github_combined_status_formatter'
 require 'pronto/formatter/github_pull_request_formatter'
 require 'pronto/formatter/github_pull_request_review_formatter'
 require 'pronto/formatter/gitlab_formatter'

lib/pronto/formatter/formatter.rb

@@ -13,6 +13,7 @@ def self.names
     FORMATTERS = {
       'github' => GithubFormatter,
       'github_status' => GithubStatusFormatter,
+      'github_combined_status' => GithubCombinedStatusFormatter,
       'github_pr' => GithubPullRequestFormatter,
       'github_pr_review' => GithubPullRequestReviewFormatter,
       'gitlab' => GitlabFormatter,

lib/pronto/formatter/github_combined_status_formatter.rb

@@ -0,0 +1,24 @@
+require_relative 'github_status_formatter/status_builder'
+
+module Pronto
+  module Formatter
+    class GithubCombinedStatusFormatter
+      def format(messages, repo, _)
+        client = Github.new(repo)
+        head = repo.head_commit_sha
+
+        create_status(client, head, messages.uniq || [])
+      end
+
+      private
+
+      def create_status(client, sha, messages)
+        builder = GithubStatusFormatter::StatusBuilder.new(nil, messages)
+        status = Status.new(sha, builder.state,
+                            'pronto', builder.description)
+
+        client.create_commit_status(status)
+      end
+    end
+  end
+end

spec/pronto/formatter/formatter_spec.rb

@@ -94,8 +94,8 @@ module Formatter
       subject { Formatter.names }
       it do
         should =~ %w[github github_pr github_pr_review github_status
-                     gitlab gitlab_mr bitbucket bitbucket_pr bitbucket_server_pr
-                     json checkstyle text null]
+                     github_combined_status gitlab gitlab_mr bitbucket bitbucket_pr
+                     bitbucket_server_pr json checkstyle text null]
       end
     end
   end

spec/pronto/formatter/github_combined_status_formatter_spec.rb

@@ -0,0 +1,102 @@
+module Pronto
+  module Formatter
+    RSpec.describe GithubCombinedStatusFormatter do
+      let(:formatter) { described_class.new }
+
+      describe '#format' do
+        subject { formatter.format(messages, repo, nil) }
+
+        let(:repo) { Git::Repository.new('spec/fixtures/test.git') }
+        let(:runner_class) do
+          Class.new do
+            def self.title
+              'fake_runner'
+            end
+          end
+        end
+        let(:message) do
+          Pronto::Message.new('app/path', nil, level, '', sha, runner_class)
+        end
+        let(:sha) { '64dadfdb7c7437476782e8eb024085862e6287d6' }
+        let(:status) { Status.new(sha, state, context, description) }
+        let(:context) { 'pronto' }
+
+        let(:messages) { [message, message] }
+
+        before do
+          Pronto::Github.any_instance
+            .should_receive(:create_commit_status)
+            .with(status)
+            .once
+            .and_return(nil)
+        end
+
+        context 'when has no messages' do
+          let(:messages) { [] }
+
+          let(:state) { :success }
+          let(:description) { 'Coast is clear!' }
+
+          it 'has no issues' do
+            subject
+          end
+        end
+
+        context 'when has one message' do
+          let(:messages) { [message, message] }
+
+          context 'when severity level is info' do
+            let(:level) { :info }
+
+            let(:state) { :success }
+            let(:description) { 'Found 1 info.' }
+
+            it 'has issue' do
+              subject
+            end
+          end
+
+          context 'when severity level is warning' do
+            let(:level) { :warning }
+
+            let(:state) { :failure }
+            let(:description) { 'Found 1 warning.' }
+
+            it 'has warning' do
+              subject
+            end
+          end
+        end
+
+        context 'when has multiple messages' do
+          let(:level) { :warning }
+          let(:level2) { :error }
+          let(:message2) do
+            Pronto::Message.new('app/path', nil, level2, '', sha, runner_class)
+          end
+
+          let(:state) { :failure }
+          let(:description) { 'Found 1 warning and 1 error.' }
+
+          context 'order of messages does not matter' do
+            context 'ordered' do
+              let(:messages) { [message, message2] }
+
+              it 'has issues' do
+                subject
+              end
+            end
+
+            context 'reversed' do
+              let(:messages) { [message2, message] }
+
+              it 'has issues' do
+                subject
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end