Placed excluded count in parens to improve clarity

Author: kieraneglin

lib/ex_unit/lib/ex_unit/cli_formatter.ex

@@ -352,7 +352,7 @@ defmodule ExUnit.CLIFormatter do
       )
       |> if_true(
         config.excluded_counter > 0,
-        &(&1 <> ", #{config.excluded_counter} excluded")
+        &(&1 <> " (#{config.excluded_counter} excluded)")
       )
 
     cond do

lib/ex_unit/test/ex_unit_test.exs

@@ -331,19 +331,19 @@ defmodule ExUnitTest do
 
     {result, output} = run_with_filter([exclude: [even: true]], [ParityTest])
     assert result == %{failures: 0, skipped: 0, excluded: 1, total: 4}
-    assert output =~ "\n3 tests, 0 failures, 1 excluded\n"
+    assert output =~ "\n3 tests, 0 failures (1 excluded)\n"
 
     {result, output} = run_with_filter([exclude: :even], [ParityTest])
     assert result == %{failures: 0, skipped: 0, excluded: 3, total: 4}
-    assert output =~ "\n1 test, 0 failures, 3 excluded\n"
+    assert output =~ "\n1 test, 0 failures (3 excluded)\n"
 
     {result, output} = run_with_filter([exclude: :even, include: [even: true]], [ParityTest])
     assert result == %{failures: 1, skipped: 0, excluded: 2, total: 4}
-    assert output =~ "\n2 tests, 1 failure, 2 excluded\n"
+    assert output =~ "\n2 tests, 1 failure (2 excluded)\n"
 
     {result, output} = run_with_filter([exclude: :test, include: [even: true]], [ParityTest])
     assert result == %{failures: 1, skipped: 0, excluded: 3, total: 4}
-    assert output =~ "\n1 test, 1 failure, 3 excluded\n"
+    assert output =~ "\n1 test, 1 failure (3 excluded)\n"
   end
 
   test "log capturing" do
@@ -510,15 +510,15 @@ defmodule ExUnitTest do
     {result, output} = run_with_filter([exclude: :module], [])
 
     assert result == %{failures: 0, skipped: 0, excluded: 2, total: 2}
-    assert output =~ "\n0 tests, 0 failures, 2 excluded\n"
+    assert output =~ "\n0 tests, 0 failures (2 excluded)\n"
 
     {result, output} =
       [exclude: :test, include: [module: "ExUnitTest.SecondTestModule"]]
       |> run_with_filter([FirstTestModule, SecondTestModule])
 
     assert result == %{failures: 1, skipped: 0, excluded: 1, total: 2}
     assert output =~ "\n  1) test false (ExUnitTest.SecondTestModule)\n"
-    assert output =~ "\n1 test, 1 failure, 1 excluded\n"
+    assert output =~ "\n1 test, 1 failure (1 excluded)\n"
   end
 
   test "raises on reserved tag :file in module" do
@@ -681,7 +681,7 @@ defmodule ExUnitTest do
       end)
 
     refute output =~ max_failures_reached_msg()
-    assert output =~ "\n5 tests, 0 failures, 4 invalid, 1 skipped, 1 excluded\n"
+    assert output =~ "\n5 tests, 0 failures, 4 invalid, 1 skipped (1 excluded)\n"
   end
 
   test "parameterized tests" do
@@ -788,7 +788,7 @@ defmodule ExUnitTest do
         end)
 
       assert output =~ max_failures_reached_msg()
-      assert output =~ "\n5 tests, 2 failures, 1 skipped, 1 excluded\n"
+      assert output =~ "\n5 tests, 2 failures, 1 skipped (1 excluded)\n"
     end
 
     test ":max_failures is not reached" do
@@ -819,7 +819,7 @@ defmodule ExUnitTest do
         end)
 
       refute output =~ max_failures_reached_msg()
-      assert output =~ "\n6 tests, 2 failures, 1 skipped, 2 excluded\n"
+      assert output =~ "\n6 tests, 2 failures, 1 skipped (2 excluded)\n"
     end
 
     test ":max_failures has been reached" do
@@ -853,7 +853,7 @@ defmodule ExUnitTest do
         end)
 
       assert output =~ max_failures_reached_msg()
-      assert output =~ "\n5 tests, 2 failures, 2 skipped, 2 excluded\n"
+      assert output =~ "\n5 tests, 2 failures, 2 skipped (2 excluded)\n"
     end
 
     # Excluded and skipped tests are detected before setup_all
@@ -887,7 +887,7 @@ defmodule ExUnitTest do
         end)
 
       assert output =~ max_failures_reached_msg()
-      assert output =~ "\n3 tests, 0 failures, 2 invalid, 1 skipped, 1 excluded\n"
+      assert output =~ "\n3 tests, 0 failures, 2 invalid, 1 skipped (1 excluded)\n"
     end
 
     test ":max_failures flushes all async/sync cases" do
@@ -1060,7 +1060,7 @@ defmodule ExUnitTest do
       end)
 
     assert output =~ "All tests have been excluded.\n"
-    assert output =~ "0 tests, 0 failures, 2 excluded\n"
+    assert output =~ "0 tests, 0 failures (2 excluded)\n"
   end
 
   test "tests are run in compile order (FIFO)" do