Do not mark cases as loaded after failures, closes #4771

Author: José Valim

lib/ex_unit/lib/ex_unit/server.ex

@@ -55,11 +55,13 @@ defmodule ExUnit.Server do
     {:reply, diff, take_cases(%{state | loaded: :done})}
   end
 
-  def handle_cast({:add_async_case, name}, %{async_cases: cases} = state) do
+  def handle_cast({:add_async_case, name}, %{async_cases: cases, loaded: loaded} = state)
+      when is_integer(loaded) do
     {:noreply, take_cases(%{state | async_cases: [name | cases]})}
   end
 
-  def handle_cast({:add_sync_case, name}, %{sync_cases: cases} = state) do
+  def handle_cast({:add_sync_case, name}, %{sync_cases: cases, loaded: loaded} = state)
+      when is_integer(loaded) do
     {:noreply, %{state | sync_cases: [name | cases]}}
   end
 

lib/mix/lib/mix/compilers/test.ex

@@ -43,13 +43,8 @@ defmodule Mix.Compilers.Test do
       test_files ->
         try do
           task = Task.async(ExUnit, :run, [])
-
-          try do
-            Kernel.ParallelRequire.files(test_files, parallel_require_callbacks)
-          after
-            ExUnit.Server.cases_loaded()
-          end
-
+          Kernel.ParallelRequire.files(test_files, parallel_require_callbacks)
+          ExUnit.Server.cases_loaded()
           %{failures: failures} = results = Task.await(task, :infinity)
 
           if failures == 0 do