Remove suite warnings

Author: josevalim

lib/ex_unit/test/ex_unit/assertions_test.exs

@@ -765,8 +765,8 @@ defmodule ExUnit.AssertionsTest do
   @compile {:no_warn_undefined, Not.Defined}
 
   test "assert raise with some other error" do
-    "This should never be tested" =
-      assert_raise ArgumentError, fn -> Not.Defined.function(1, 2, 3) end
+    assert_raise ArgumentError, fn -> Not.Defined.function(1, 2, 3) end
+    flunk("This should never be tested")
   rescue
     error in [ExUnit.AssertionError] ->
       "Expected exception ArgumentError but got UndefinedFunctionError " <>
@@ -775,8 +775,8 @@ defmodule ExUnit.AssertionsTest do
   end
 
   test "assert raise with some other error includes stacktrace from original error" do
-    "This should never be tested" =
-      assert_raise ArgumentError, fn -> Not.Defined.function(1, 2, 3) end
+    assert_raise ArgumentError, fn -> Not.Defined.function(1, 2, 3) end
+    flunk("This should never be tested")
   rescue
     ExUnit.AssertionError ->
       [{Not.Defined, :function, [1, 2, 3], _} | _] = __STACKTRACE__
@@ -1057,7 +1057,7 @@ defmodule ExUnit.AssertionsTest do
       """ = Exception.message(error)
   end
 
-  defp ok(val), do: {:ok, val}
-  defp error(val), do: {:error, val}
+  defp ok(val), do: Process.get(:unused, {:ok, val})
+  defp error(val), do: Process.get(:unused, {:error, val})
   defp not_equal(left, right), do: left != right
 end

lib/ex_unit/test/ex_unit/formatter_test.exs

@@ -79,7 +79,7 @@ defmodule ExUnit.FormatterTest do
   test "formats test exits with function clause mfa" do
     {error, stack} =
       try do
-        Access.fetch(:foo, :bar)
+        Access.fetch(Process.get(:unused, :foo), :bar)
       catch
         :error, error -> {error, __STACKTRACE__}
       end
@@ -161,7 +161,7 @@ defmodule ExUnit.FormatterTest do
   test "formats test EXITs with function clause errors" do
     {error, stack} =
       try do
-        Access.fetch(:foo, :bar)
+        Access.fetch(Process.get(:unused, :foo), :bar)
       catch
         :error, error -> {error, __STACKTRACE__}
       end
@@ -411,7 +411,7 @@ defmodule ExUnit.FormatterTest do
   test "blames function clause error" do
     {error, stack} =
       try do
-        Access.fetch(:foo, :bar)
+        Access.fetch(Process.get(:unused, :foo), :bar)
       rescue
         exception -> {exception, __STACKTRACE__}
       end