Allow ReplaceBranch to use skip_if (#809)

* Allow ReplaceBranch to use skip_if

* Fix linting error

* Clearify specs

* Use bash instead of sh in spec

Author: oleander

lib/overcommit/hook/prepare_commit_msg/replace_branch.rb

@@ -85,7 +85,7 @@ def skipped_commit_types
     end
 
     def skip?
-      skipped_commit_types.include?(commit_message_source)
+      super || skipped_commit_types.include?(commit_message_source)
     end
   end
 end

spec/overcommit/hook/prepare_commit_msg/replace_branch_spec.rb

@@ -70,6 +70,26 @@ def remove_file(name)
           expect(File.read('COMMIT_EDITMSG')).to eq("[123] \n")
         end
       end
+
+      context 'when skip_if exits with a zero status' do
+        let(:config) { new_config('skip_if' => ['bash', '-c', 'exit 0']) }
+
+        it { is_expected.to pass }
+
+        it 'does not change the commit message' do
+          expect(File.read('COMMIT_EDITMSG')).to eq("\n")
+        end
+      end
+
+      context 'when skip_if exits with a non-zero status' do
+        let(:config) { new_config('skip_if' => ['bash', '-c', 'exit 1']) }
+
+        it { is_expected.to pass }
+
+        it 'does change the commit message' do
+          expect(File.read('COMMIT_EDITMSG')).to eq("[#123]\n")
+        end
+      end
     end
 
     context "when the checked out branch doesn't matches the pattern" do