return :excluded instead of nil

Author: mitchellhenke

lib/sentry.ex

@@ -111,11 +111,13 @@ defmodule Sentry do
   @doc """
     Parses and submits an exception to Sentry if current environment is in included_environments.
   """
-  @spec capture_exception(Exception.t, Keyword.t) :: {:ok, String.t} | :error
+  @spec capture_exception(Exception.t, Keyword.t) :: {:ok, String.t} | :error | :excluded
   def capture_exception(exception, opts \\ []) do
     filter_module = Application.get_env(:sentry, :filter, Sentry.DefaultEventFilter)
     {source, opts} = Keyword.pop(opts, :event_source)
-    unless(filter_module.exclude_exception?(exception, source)) do
+    if(filter_module.exclude_exception?(exception, source)) do
+      :excluded
+    else
       Event.transform_exception(exception, opts)
       |> send_event()
     end

test/sentry_test.exs

@@ -18,7 +18,7 @@ defmodule SentryTest do
    Application.put_env(:sentry, :included_environments, [:test])
    Application.put_env(:sentry, :environment_name, :test)
 
-   Sentry.capture_exception(%RuntimeError{message: "error"}, [event_source: :plug])
-   Sentry.capture_exception(%ArithmeticError{message: "error"}, [event_source: :plug])
+   assert {:ok, _} = Sentry.capture_exception(%RuntimeError{message: "error"}, [event_source: :plug])
+   assert :excluded = Sentry.capture_exception(%ArithmeticError{message: "error"}, [event_source: :plug])
   end
 end