color for case with 0 relevant lines

parroty · Oct 22, 2018 · 645ff9b · 645ff9b
1 parent 500b863
commit 645ff9b
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 8 deletions.
diff --git a/lib/excoveralls/html/view.ex b/lib/excoveralls/html/view.ex
@@ -35,7 +35,9 @@ defmodule ExCoveralls.Html.View do
     Safe.html_escape(data)
   end
 
-  def coverage_class(percent) do
+  def coverage_class(percent, sloc \\ nil)
+  def coverage_class(percent, 0), do: "none"
+  def coverage_class(percent, _) do
     cond do
       percent >= 75 -> "high"
       percent >= 50 -> "medium"

diff --git a/lib/templates/html/htmlcov/_style.html.eex b/lib/templates/html/htmlcov/_style.html.eex
diff --git a/lib/templates/html/htmlcov/coverage.html.eex b/lib/templates/html/htmlcov/coverage.html.eex
diff --git a/test/html/view_test.exs b/test/html/view_test.exs
@@ -22,14 +22,16 @@ defmodule ExCoveralls.Html.ViewTest do
   end
 
   test "#coverage_class" do
-    [h,m,l,t] = [View.coverage_class(75),
-                 View.coverage_class(50),
-                 View.coverage_class(25),
-                 View.coverage_class(0)]
+    [h,m,l,t,n] = [View.coverage_class(75),
+                   View.coverage_class(50),
+                   View.coverage_class(25),
+                   View.coverage_class(0),
+                   View.coverage_class(0, 0)]
 
     assert(h == "high")
     assert(m == "medium")
     assert(l == "low")
     assert(t == "terrible")
+    assert(n == "none")
   end
 end
diff --git a/test/html_test.exs b/test/html_test.exs
@@ -5,7 +5,7 @@ defmodule ExCoveralls.HtmlTest do
   alias ExCoveralls.Html
 
   @file_name "excoveralls.html"
-  @file_size 20184
+  @file_size 20212
   @test_output_dir "cover_test/"
   @test_template_path "lib/templates/html/htmlcov/"