Extract Pronto::Github to be used by multiple formatters

lib/pronto.rb

@@ -1,4 +1,5 @@
 require 'rugged'
+require 'octokit'
 require 'forwardable'
 
 require 'pronto/git/repository'
@@ -10,6 +11,7 @@
 require 'pronto/plugin'
 require 'pronto/message'
 require 'pronto/runner'
+require 'pronto/github'
 
 require 'pronto/formatter/text_formatter'
 require 'pronto/formatter/json_formatter'

lib/pronto/formatter/github_formatter.rb

@@ -1,5 +1,3 @@
-require 'octokit'
-
 module Pronto
   module Formatter
     class GithubFormatter
@@ -33,12 +31,8 @@ def create_comment(repo, sha, position, path, body)
         end
       end
 
-      def access_token
-        ENV['GITHUB_ACCESS_TOKEN']
-      end
-
       def client
-        @client ||= Octokit::Client.new(access_token: access_token)
+        @client ||= Github.new
       end
     end
   end

lib/pronto/github.rb

@@ -0,0 +1,17 @@
+module Pronto
+  class Github
+    extend Forwardable
+
+    def_delegators :client, :commit_comments, :create_commit_comment
+
+    private
+
+    def client
+      @client ||= Octokit::Client.new(access_token: access_token)
+    end
+
+    def access_token
+      ENV['GITHUB_ACCESS_TOKEN']
+    end
+  end
+end

lib/pronto/rake_task/travis_pull_request.rb

@@ -1,5 +1,4 @@
 require 'pronto'
-require 'octokit'
 require 'rake'
 require 'rake/tasklib'
 

spec/pronto/formatter/github_formattter_spec.rb

@@ -8,7 +8,7 @@ module Formatter
       describe '#format' do
         subject { github_formatter.format(messages, repository) }
         let(:messages) { [message, message] }
-        let(:repository) { Pronto::Git::Repository.new('.') }
+        let(:repository) { Git::Repository.new('.') }
         let(:message) { Message.new('path/to', line, :warning, 'crucial') }
         let(:line) { double(new_lineno: 1, commit_sha: '123', position: nil) }
         before { line.stub(:commit_line).and_return(line) }