Update CHANGELOG

Author: José Valim

CHANGELOG.md

@@ -3,8 +3,16 @@
 ## v1.0.0-rc2
 
 * Enhancements
+  * [ExUnit] Print process mailbox on failing `assert_receive`
+  * [Mix] `mix deps.unlock` warns when given missing dependency
+  * [Kernel] Friendlier `unquote_splicing` error on inline quote
 
 * Bug fixes
+  * [Logger] Fix `Logger.log/2` not to raise on Logger exit
+  * [Logger] Recompute sync/async mode on Logger configuration
+
+* Backwards incompatible changes
+  * [Supervisor] Make `max_restarts` default to 3 (closer to upcoming Erlang defaults)
 
 ## v1.0.0-rc1 (2014-08-30)
 

lib/elixir/test/elixir/supervisor/spec_test.exs

@@ -64,7 +64,7 @@ defmodule Supervisor.SpecTest do
 
   test "supervise/2" do
     assert supervise([], strategy: :one_for_one) == {
-      :ok, {{:one_for_one, 5, 5}, []}
+      :ok, {{:one_for_one, 3, 5}, []}
    }
 
     children = [worker(GenEvent, [])]

lib/ex_unit/lib/ex_unit/assertions.ex

@@ -281,7 +281,7 @@ defmodule ExUnit.Assertions do
           unquote(expected) = received -> received
         after
           unquote(timeout) ->
-            flunk(unquote(message) <> mailbox(self))
+            flunk(unquote(message) <> ExUnit.Assertions.__mailbox__(self()))
         end
       end
 
@@ -291,19 +291,21 @@ defmodule ExUnit.Assertions do
   @max_mailbox_length 10
 
   @doc false
-  def mailbox(pid) do
+  def __mailbox__(pid) do
     {:messages, messages} = Process.info(pid, :messages)
     length = length(messages)
-    mailbox = Enum.take(messages, @max_mailbox_length)
-      |> Enum.map_join("\n", &inspect/1)
-    ". Process mailbox (#{length}):" <> mailbox_message(length, mailbox)
+    mailbox = Enum.take(messages, @max_mailbox_length) |> Enum.map_join("\n", &inspect/1)
+    mailbox_message(length, mailbox)
   end
 
-  defp mailbox_message(0, _mailbox), do: " []"
+  defp mailbox_message(0, _mailbox), do: ". The process mailbox is empty."
   defp mailbox_message(length, mailbox) when length > 10 do
-    "\n" <> mailbox <> "\nShowing only #{@max_mailbox_length} first messages."
+    ". Process mailbox:\n" <> mailbox
+      <> "\nShowing only #{@max_mailbox_length} of #{length} messages."
+  end
+  defp mailbox_message(_length, mailbox) do
+    ". Process mailbox:\n" <> mailbox
   end
-  defp mailbox_message(_length, mailbox), do: "\n" <> mailbox
 
   @doc """
   Asserts the `exception` is raised during `function` execution with

lib/ex_unit/test/ex_unit/assertions_test.exs

@@ -91,27 +91,30 @@ defmodule ExUnit.AssertionsTest do
       "This should never be tested" = assert_received :hello
     rescue
       error in [ExUnit.AssertionError] ->
-        "No message matching :hello. Process mailbox (0): []" = error.message
+        "No message matching :hello. The process mailbox is empty." = error.message
     end
   end
 
   test "assert received when different message" do
     send self, {:message, :not_expected, :at_all}
     try do
-      "This should never be tested" = assert_receive :hello, 5_000
+      "This should never be tested" = assert_received :hello
     rescue
       error in [ExUnit.AssertionError] ->
-        "No message matching :hello. Process mailbox (1):\n{:message, :not_expected, :at_all}" = error.message
+        "No message matching :hello. Process mailbox:\n{:message, :not_expected, :at_all}" = error.message
     end
   end
 
   test "assert received when different message having more than 10 on mailbox" do
     for i <- 1..11, do: send(self, {:message, i})
     try do
-      "This should never be tested" = assert_receive :hello, 5_000
+      "This should never be tested" = assert_received :hello
     rescue
       error in [ExUnit.AssertionError] ->
-        "No message matching :hello. Process mailbox (11):\n{:message, 1}\n{:message, 2}\n{:message, 3}\n{:message, 4}\n{:message, 5}\n{:message, 6}\n{:message, 7}\n{:message, 8}\n{:message, 9}\n{:message, 10}\nShowing only 10 first messages." = error.message
+        "No message matching :hello. Process mailbox:\n" <>
+        "{:message, 1}\n{:message, 2}\n{:message, 3}\n{:message, 4}\n" <>
+        "{:message, 5}\n{:message, 6}\n{:message, 7}\n{:message, 8}\n" <>
+        "{:message, 9}\n{:message, 10}\nShowing only 10 of 11 messages." = error.message
     end
   end