Merge branch 'coverage_fix' into 'master'

Fix coverage colcalation on the commit page

See merge request !171

CHANGELOG

@@ -4,6 +4,7 @@ v7.13.0
   - Improved Lint stability
   - Disable colors in rake tasks automatically (if IO is not a TTY)
   - Implemented "rake env:info". Rake task to receive system information
+  - Fix coverage calculation on commit page
 
 v7.12.1
   - Runner without tag should pick builds without tag only

app/models/commit.rb

@@ -205,8 +205,11 @@ def finished_at
   end
 
   def coverage
-    if project.coverage_enabled? && builds.size > 0
-      builds.last.coverage
+    if project.coverage_enabled? && builds.count(:all) > 0
+      coverage_array = builds.map(&:coverage).compact
+      if coverage_array.size > 1
+        coverage_array.reduce(:+) / coverage_array.size
+      end
     end
   end
 

spec/models/commit_spec.rb

@@ -159,4 +159,27 @@
       commit.finished_at.should be_nil
     end
   end
+
+  describe "coverage" do
+    let(:project) { FactoryGirl.create :project, coverage_regex: "/.*/" }
+    let(:commit) { FactoryGirl.create :commit, project: project }
+
+    it "calculates average when there are two builds with coverage" do
+      FactoryGirl.create :build, coverage: 30, commit: commit
+      FactoryGirl.create :build, coverage: 40, commit: commit
+      commit.coverage.should == 35.0
+    end
+
+    it "calculates average when there are two builds with coverage and one with nil" do
+      FactoryGirl.create :build, coverage: 30, commit: commit
+      FactoryGirl.create :build, coverage: 40, commit: commit
+      FactoryGirl.create :build, commit: commit
+      commit.coverage.should == 35.0
+    end
+
+    it "calculates average when there is one build without coverage" do
+      FactoryGirl.create :build, commit: commit
+      commit.coverage.should be_nil
+    end
+  end
 end