Cover #177 with a test

Author: maxjacobson

lib/code_climate/test_reporter/git.rb

@@ -10,6 +10,10 @@ def info
           }
         end
 
+        def head_from_git_or_ci
+          head_from_git || head_from_ci
+        end
+
         def branch_from_git_or_ci
           clean_service_branch || clean_git_branch || "master"
         end
@@ -34,10 +38,6 @@ def clean_git_branch
 
         private
 
-        def head_from_git_or_ci
-          head_from_git || head_from_ci
-        end
-
         def head_from_git
           commit_hash = git("log -1 --pretty=format:'%H'")
           !commit_hash.empty? ? commit_hash : nil

spec/code_climate/test_reporter/git_spec.rb

@@ -79,6 +79,21 @@ def self.root
       end
     end
 
+    describe 'head_from_git_or_ci' do
+      it 'returns the head sha from git' do
+        expect(Git).to receive(:git).with("log -1 --pretty=format:'%H'").and_return("1234")
+
+        expect(Git.head_from_git_or_ci).to eq '1234'
+      end
+
+      it 'returns the head sha from ci if git is not available' do
+        expect(Git).to receive(:git).with("log -1 --pretty=format:'%H'").and_return("")
+        expect(Ci).to receive(:service_data).and_return({commit_sha: "4567"})
+
+        expect(Git.head_from_git_or_ci).to eq '4567'
+      end
+    end
+
     describe 'committed_at_from_git_or_ci' do
       it 'returns the committed_at from git' do
         expect(Git.committed_at_from_git_or_ci).to eq Git.send(:committed_at_from_git)