Display include and exclude filters separately

Author: vanstee

lib/ex_unit/lib/ex_unit/cli_formatter.ex

@@ -5,7 +5,7 @@ defmodule ExUnit.CLIFormatter do
 
   use GenServer.Behaviour
 
-  import ExUnit.Formatter, only: [format_time: 2, format_filters: 1, format_test_failure: 5, format_test_case_failure: 4]
+  import ExUnit.Formatter, only: [format_time: 2, format_filters: 2, format_test_failure: 5, format_test_case_failure: 4]
 
   defrecord Config, tests_counter: 0, invalids_counter: 0, failures_counter: 0,
                     skips_counter: 0, trace: false, color: true, previous: nil
@@ -155,9 +155,9 @@ defmodule ExUnit.CLIFormatter do
     end
   end
 
-  defp print_filters(filters) do
-    filters = Enum.reject filters, &Enum.empty?(elem(&1, 1))
-    unless Enum.empty?(filters), do: IO.puts format_filters(filters)
+  defp print_filters([include: include, exclude: exclude]) do
+    unless Enum.empty?(include), do: IO.puts format_filters(include, :include)
+    unless Enum.empty?(exclude), do: IO.puts format_filters(exclude, :exclude)
   end
 
   defp print_test_failure(ExUnit.Test[name: name, case: mod, state: { :failed, tuple }], config) do

lib/ex_unit/lib/ex_unit/formatter.ex

@@ -72,13 +72,16 @@ defmodule ExUnit.Formatter do
 
   ## Examples
 
-    iex> format_filters([run: true, slow: false])
-    "Using filters: [run: true, slow: false]"
+    iex> format_filters([run: true, slow: false], :include)
+    "Including tags: [run: true, slow: false]"
 
   """
-  @spec format_filters(Keyword.t) :: String.t
-  def format_filters(filters) do
-    "Using filters: #{inspect filters}"
+  @spec format_filters(Keyword.t, atom) :: String.t
+  def format_filters(filters, type) do
+    case type do
+      :include -> "Including tags: #{inspect filters}"
+      :exclude -> "Excluding tags: #{inspect filters}"
+    end
   end
 
   @doc %S"""

lib/ex_unit/test/ex_unit/formatter_test.exs

@@ -20,7 +20,8 @@ defmodule ExUnit.FormatterTest do
 
   test "formats test case filters" do
     filters = [run: true, slow: false]
-    assert format_filters(filters) =~ "Using filters: [run: true, slow: false]"
+    assert format_filters(filters, :include) =~ "Including tags: [run: true, slow: false]"
+    assert format_filters(filters, :exclude) =~ "Excluding tags: [run: true, slow: false]"
   end
 
   test "formats test errors" do