Always round numbers down, never up

.gitignore

@@ -1,5 +1,6 @@
 .rvmrc
 .bundle
+/tmp
 
 ## MAC OS
 .DS_Store

.rubocop.yml

@@ -7,6 +7,11 @@ AllCops:
   DisplayCopNames: true
   NewCops: enable
   TargetRubyVersion: 2.5
+  Exclude:
+    - "test/fixtures/*"
+    - "tmp/**/*"
+    - "vendor/**/*"
+    - ".git/**/*"
 
 Bundler/DuplicatedGem:
   Enabled: false

Gemfile

@@ -19,6 +19,7 @@ group :test do
 end
 
 group :development do
+  gem "nokogiri"
   gem "rubocop"
   gem "rubocop-minitest"
   gem "rubocop-performance"

Gemfile.lock

@@ -24,7 +24,11 @@ GEM
     json (2.7.2-java)
     language_server-protocol (3.17.0.3)
     logger (1.6.1)
+    mini_portile2 (2.8.8)
     minitest (5.25.1)
+    nokogiri (1.18.1)
+      mini_portile2 (~> 2.8.2)
+      racc (~> 1.4)
     parallel (1.26.3)
     parser (3.3.5.0)
       ast (~> 2.4.1)
@@ -73,13 +77,13 @@ GEM
     unicode-display_width (2.5.0)
 
 PLATFORMS
-  arm64-darwin-22
   ruby
   universal-java-1.8
 
 DEPENDENCIES
   logger
   minitest
+  nokogiri
   rake (>= 11)
   rubocop
   rubocop-minitest

lib/simplecov-html.rb

@@ -55,8 +55,9 @@ def line_status?(source_file, line)
 
       def output_message(result)
         output = "Coverage report generated for #{result.command_name} to #{output_path}."
-        output += "\nLine Coverage: #{result.covered_percent.round(2)}% (#{result.covered_lines} / #{result.total_lines})"
-        output += "\nBranch Coverage: #{result.coverage_statistics[:branch].percent.round(2)}% (#{result.covered_branches} / #{result.total_branches})" if branchable_result?
+        output += "\nLine Coverage: #{result.covered_percent.floor(2)}% (#{result.covered_lines} / #{result.total_lines})"
+
+        output += "\nBranch Coverage: #{result.coverage_statistics[:branch].percent.floor(2)}% (#{result.covered_branches} / #{result.total_branches})" if branchable_result?
         output
       end
 

test/fixtures/branch_tester_script.rb

@@ -0,0 +1,50 @@
+# Adapted from https://github.com/simplecov-ruby/simplecov/pull/694#issuecomment-562097006
+# rubocop:disable all
+x = 1
+x.eql?(4) ? "4" : x
+
+puts x unless x.eql?(5)
+
+unless x == 5
+  puts "Ola.."
+end
+
+unless x != 5
+  puts "Ola.."
+end
+
+unless x != 5
+  puts "Ola.."
+else
+  puts "text"
+end
+
+puts x if x.eql?(5)
+if x != 5
+  puts "Ola.."
+end
+
+if x == 5
+  puts "Ola.."
+end
+
+if x != 5
+  puts "Ola.."
+else
+  puts "text"
+end
+x = 4
+if x == 1
+  puts "wow 1"
+  puts "such excite!"
+elsif x == 4
+  4.times { puts "4!!!!"}
+elsif x == 14
+  puts "magic"
+
+  puts "very"
+else
+  puts x
+end
+
+# rubocop:enable all

test/fixtures/branches.rb

@@ -0,0 +1,13 @@
+class Branches
+  def call(arg)
+    return if arg < 0
+
+    _val = (arg == 42 ? :yes : :no)
+
+    if arg.odd?
+      :yes
+    else
+      :no
+    end
+  end
+end

test/fixtures/case.rb

@@ -0,0 +1,14 @@
+module Case
+  def self.call(arg)
+    case arg
+    when 0...23
+      :foo
+    when 40..50
+      :bar
+    when Integer
+      :baz
+    else
+      :nope
+    end
+  end
+end

test/fixtures/case_without_else.rb

@@ -0,0 +1,12 @@
+module Case
+  def self.call(arg)
+    case arg
+    when 0...23
+      :foo
+    when 40..50
+      :bar
+    when Integer
+      :baz
+    end
+  end
+end

test/fixtures/elsif.rb

@@ -0,0 +1,13 @@
+module Elsif
+  def self.call(arg)
+    if arg.odd?
+      :odd
+    elsif arg == 30
+      :mop
+    elsif arg == 42
+      :yay
+    else
+      :nay
+    end
+  end
+end

test/fixtures/inline.rb

@@ -0,0 +1,13 @@
+class Inline
+  def call(arg)
+    String(arg == 42 ? :yes : :no)
+
+    String(
+      if arg.odd?
+        :yes
+      else
+        :no
+      end
+    )
+  end
+end

test/fixtures/nested_branches.rb

@@ -0,0 +1,15 @@
+# yes rubocop you are right but I want to test nesting!
+# rubocop:disable Metrics/BlockNesting
+module NestedBranches
+  def self.call(arg)
+    if arg.even?
+      if arg == 42
+        arg -= 1 while arg > 40
+        :ok
+      end
+    else
+      :nope
+    end
+  end
+end
+# rubocop:enable Metrics/BlockNesting

test/fixtures/never.rb

@@ -0,0 +1,2 @@
+# This class is purely some
+# comments

test/fixtures/nocov_complex.rb

@@ -0,0 +1,27 @@
+# So much skippping
+# rubocop:disable Metrics/MethodLength
+module NoCovComplex
+  def self.call(arg)
+    # :nocov:
+    if arg == 42
+      0
+    # :nocov:
+    else
+      puts "yolo"
+    end
+
+    arg += 1 if arg.odd?
+
+    # :nocov:
+    arg -= 1 while arg > 40
+
+    case arg
+    when 1..20
+      :nope
+    # :nocov:
+    when 30..40
+      :yas
+    end
+  end
+end
+# rubocop:enable Metrics/MethodLength

test/fixtures/sample.rb

@@ -0,0 +1,16 @@
+# Foo class
+class Foo
+  def initialize
+    @foo = "baz"
+  end
+
+  def bar
+    @foo
+  end
+
+  # :nocov:
+  def skipped
+    @foo * 2
+  end
+  # :nocov:
+end

test/fixtures/single_nocov.rb

@@ -0,0 +1,14 @@
+# :nocov:
+module SingleNocov
+  def self.call(arg)
+    if arg.odd?
+      :odd
+    elsif arg == 30
+      :mop
+    elsif arg == 42
+      :yay
+    else
+      :nay
+    end
+  end
+end

test/fixtures/uneven_nocovs.rb

@@ -0,0 +1,16 @@
+module UnevenNocov
+  def self.call(arg)
+    # :nocov:
+    if arg.odd?
+      :odd
+    elsif arg == 30
+      :mop
+    # :nocov:
+    elsif arg == 42
+      :yay
+    # :nocov:
+    else
+      :nay
+    end
+  end
+end

test/helper.rb

@@ -4,3 +4,6 @@
 require "simplecov"
 require "simplecov-html"
 require "minitest/autorun"
+
+require "pathname"
+require "nokogiri"