Merge pull request #427 from letscolife/extend-meta-reporting

Handle elixir exceptions in MetadataReporter

lib/new_relic/error/metadata_reporter.ex

@@ -62,19 +62,21 @@ defmodule NewRelic.Error.MetadataReporter do
     end
   end
 
-  defp parse_reason({exception, stacktrace}) do
-    type = parse_type(exception)
+  defp parse_reason({%type{message: message} = exception, stacktrace}) do
     expected = parse_error_expected(exception)
-    message = Map.get(exception, :message)
+    type = inspect(type)
     reason = "(#{type}) #{message}"
 
     {type, reason, stacktrace, expected}
   end
 
-  defp parse_type(%type{}) do
-    type
-    |> Module.split()
-    |> Enum.join(".")
+  defp parse_reason({exception, stacktrace}) do
+    exception = Exception.normalize(:error, exception, stacktrace)
+    type = inspect(exception.__struct__)
+    message = Exception.message(exception)
+    reason = "(#{type}) #{message}"
+
+    {type, reason, stacktrace, false}
   end
 
   defp parse_process_name([], [{module, _f, _a, _} | _]), do: inspect(module)

test/error_test.exs

@@ -186,4 +186,20 @@ defmodule ErrorTest do
              intrinsic[:"error.class"] == "File.Error"
            end)
   end
+
+  test "Catch a function clause error inside a Task" do
+    TestHelper.restart_harvest_cycle(Collector.TransactionErrorEvent.HarvestCycle)
+
+    :proc_lib.spawn(fn ->
+      Task.async(fn -> ErrorDummy.handle_call(:foo, nil, nil) end) |> Task.await()
+    end)
+
+    :timer.sleep(100)
+
+    events = TestHelper.gather_harvest(Collector.TransactionErrorEvent.Harvester)
+
+    assert Enum.find(events, fn [intrinsic, _, _] ->
+             intrinsic[:"error.class"] == "FunctionClauseError"
+           end)
+  end
 end