ProtectedBranches should be able to handle pushing tags gracefully

Author: Hamed Asghari

lib/overcommit/hook/pre_push/protected_branches.rb

@@ -21,6 +21,7 @@ def illegal_pushes
 
     def protected?(remote_ref)
       ref_name = remote_ref[%r{refs/heads/(.*)}, 1]
+      return false if ref_name.nil?
       protected_branch_patterns.any? do |pattern|
         File.fnmatch(pattern, ref_name)
       end

spec/overcommit/hook/pre_push/protected_branches_spec.rb

@@ -93,4 +93,14 @@
       include_examples 'protected branch'
     end
   end
+
+  context 'when pushing tags' do
+    let(:pushed_ref_name) { 'redundant' }
+
+    before do
+      pushed_ref.stub(:remote_ref).and_return("refs/tags/#{pushed_ref_name}")
+    end
+
+    it { should pass }
+  end
 end