Don't show total coverage delta if unavailable (#144)

* Don't show total coverage delta if unavailable

* Don't show total coverage delta if insignificant change

Co-authored-by: Devon Blandin <[email protected]>

Author: eutopian

lib/cc/presenters/pull_requests_presenter.rb

@@ -36,9 +36,11 @@ def skipped_message
       def coverage_message
         message = "#{formatted_percent(@covered_percent)}%"
 
-        if @covered_percent_delta > 0
+        return message if @covered_percent_delta.nil?
+
+        if @covered_percent_delta.round(2) > 0
           message += " (+#{formatted_percent(@covered_percent_delta)}%)"
-        elsif @covered_percent_delta < 0
+        elsif @covered_percent_delta.round(2) < 0
           message += " (#{formatted_percent(@covered_percent_delta)}%)"
         end
 

spec/cc/presenters/pull_requests_presenter_spec.rb

@@ -30,12 +30,20 @@
     expect("85%").to eq(build_presenter({}, "covered_percent" => 85, "covered_percent_delta" => 0).coverage_message)
   end
 
+  it "message coverage without delta" do
+    expect("85%").to eq(build_presenter({}, "covered_percent" => 85, "covered_percent_delta" => nil).coverage_message)
+  end
+
+  it "message coverage the same when rounded" do
+    expect("85%").to eq(build_presenter({}, "covered_percent" => 85, "covered_percent_delta" => 0.0005).coverage_message)
+  end
+
   it "message coverage up" do
     expect("85.5% (+2.46%)").to eq(build_presenter({}, "covered_percent" => 85.5, "covered_percent_delta" => 2.4567).coverage_message)
   end
 
   it "message coverage down" do
-    expect("85.35% (-3%)").to eq( build_presenter({}, "covered_percent" => 85.348, "covered_percent_delta" => -3.0).coverage_message)
+    expect("85.35% (-3%)").to eq(build_presenter({}, "covered_percent" => 85.348, "covered_percent_delta" => -3.0).coverage_message)
   end
 
   it "message approved" do