Add minitest pre-push hook

Author: Dan Rabinowitz

config/default.yml

@@ -783,6 +783,12 @@ PrePush:
     description: 'Running RSpec test suite'
     required_executable: 'rspec'
 
+  Minitest:
+    enabled: false
+    description: 'Running Minitest test suite'
+    command: ['ruby', '-Ilib:test', 'test']
+    required_library: 'minitest'
+
 # Hooks that run during `git rebase`, before any commits are rebased.
 # If a hook fails, the rebase is aborted.
 PreRebase:

lib/overcommit/hook/pre_push/minitest.rb

@@ -0,0 +1,14 @@
+module Overcommit::Hook::PrePush
+  # Runs `minitest` test suite before push
+  #
+  # @see https://github.com/seattlerb/minitest
+  class Minitest < Base
+    def run
+      result = execute(command)
+      return :pass if result.success?
+
+      output = result.stdout + result.stderr
+      [:fail, output]
+    end
+  end
+end

spec/overcommit/hook/pre_push/minitest_spec.rb

@@ -0,0 +1,40 @@
+require 'spec_helper'
+
+describe Overcommit::Hook::PrePush::Minitest do
+  let(:config)  { Overcommit::ConfigurationLoader.default_configuration }
+  let(:context) { double('context') }
+  subject { described_class.new(config, context) }
+
+  context 'when minitest exits successfully' do
+    let(:result) { double('result') }
+
+    before do
+      result.stub(:success?).and_return(true)
+      subject.stub(:execute).and_return(result)
+    end
+
+    it { should pass }
+  end
+
+  context 'when minitest exits unsuccessfully' do
+    let(:result) { double('result') }
+
+    before do
+      result.stub(:success?).and_return(false)
+      subject.stub(:execute).and_return(result)
+    end
+
+    context 'with a runtime error' do
+      before do
+        result.stub(stdout: '', stderr: <<-EOS)
+          1) Error:
+          FooTest#test_: foo should bar. :
+          RuntimeError:
+          test/model/foo_test.rb:1:in `block (2 levels) in <class:FooTest>'
+        EOS
+      end
+
+      it { should fail_hook }
+    end
+  end
+end