define finch opts and update logger/send_test_event tests

Author: savhappy

config/config.exs

@@ -6,7 +6,7 @@ if config_env() == :test do
     tags: %{},
     enable_source_code_context: true,
     root_source_code_paths: [File.cwd!()],
-    hackney_opts: [recv_timeout: 50, pool: :sentry_pool],
+    finch_request_opts: [receive_timeout: 50],
     send_result: :sync,
     send_max_attempts: 1,
     dedup_events: false,
@@ -22,3 +22,6 @@ if config_env() == :test do
 end
 
 config :phoenix, :json_library, if(Code.ensure_loaded?(JSON), do: JSON, else: Jason)
+
+config :sentry,
+  client: Sentry.FinchClient

lib/mix/tasks/sentry.send_test_event.ex

@@ -67,7 +67,7 @@ defmodule Mix.Tasks.Sentry.SendTestEvent do
     end
 
     Mix.shell().info("current environment_name: #{inspect(to_string(Config.environment_name()))}")
-    Mix.shell().info("hackney_opts: #{inspect(Config.hackney_opts())}\n")
+    Mix.shell().info("finch_pool_opts: #{inspect(Config.finch_pool_opts())}\n")
   end
 
   defp send_event(opts) do

lib/sentry/config.ex

@@ -366,7 +366,7 @@ defmodule Sentry.Config do
       The maximum number of attempts to send an event to Sentry.
       """
     ],
-    finch_opts: [
+    finch_pool_opts: [
       type: :keyword_list,
       default: [size: 50, conn_max_idle_time: 5000],
       doc: """

lib/sentry/finch_client.ex

@@ -23,7 +23,7 @@ defmodule Sentry.FinchClient do
       Finch.child_spec(
         name: __MODULE__,
         pools: %{
-          :default => Sentry.Config.finch_opts()
+          :default => Sentry.Config.finch_pool_opts()
         }
       )
     else
@@ -39,7 +39,7 @@ defmodule Sentry.FinchClient do
   def post(url, headers, body) do
     request = Finch.build(:post, url, headers, body)
 
-    case Finch.request(request, __MODULE__) do
+    case Finch.request(request, __MODULE__, Sentry.Config.finch_request_opts()) do
       {:ok, %Finch.Response{status: status, headers: headers, body: body}} ->
         {:ok, status, headers, body}
 

test/mix/sentry.send_test_event_test.exs

@@ -5,7 +5,7 @@ defmodule Mix.Tasks.Sentry.SendTestEventTest do
   import Sentry.TestHelpers
 
   test "prints if :dsn is not set" do
-    put_test_config(dsn: nil, hackney_opts: [], environment_name: "some_env")
+    put_test_config(dsn: nil, finch_pool_opts: [], environment_name: "some_env")
 
     output =
       capture_io(fn ->
@@ -15,7 +15,7 @@ defmodule Mix.Tasks.Sentry.SendTestEventTest do
     assert output =~ """
            Client configuration:
            current environment_name: "some_env"
-           hackney_opts: []
+           finch_pool_opts: []
            """
 
     assert output =~ ~s(Event not sent because the :dsn option is not set)
@@ -35,7 +35,7 @@ defmodule Mix.Tasks.Sentry.SendTestEventTest do
     put_test_config(
       dsn: "http://public:secret@localhost:#{bypass.port}/1",
       environment_name: "test",
-      hackney_opts: []
+      finch_pool_opts: []
     )
 
     output =
@@ -49,7 +49,7 @@ defmodule Mix.Tasks.Sentry.SendTestEventTest do
            public_key: public
            secret_key: secret
            current environment_name: "test"
-           hackney_opts: []
+           finch_pool_opts: []
            """
 
     assert output =~ "Sending test event..."

test/sentry/logger_handler_test.exs

@@ -686,7 +686,7 @@ defmodule Sentry.LoggerHandlerTest do
     put_test_config(
       dsn: "http://public:secret@localhost:#{bypass.port}/1",
       dedup_events: false,
-      hackney_opts: [recv_timeout: 500, pool: :sentry_pool]
+      finch_request_opts: [receive_timeout: 500]
     )
 
     Bypass.expect(bypass, fn conn ->

test/sentry_test.exs

@@ -50,7 +50,7 @@ defmodule SentryTest do
   test "errors when taking too long to receive response", %{bypass: bypass} do
     Bypass.expect(bypass, fn _conn -> Process.sleep(:infinity) end)
 
-    put_test_config(hackney_opts: [recv_timeout: 50])
+    put_test_config(finch_request_opts: [receive_timeout: 50])
 
     assert {:error,
             %Sentry.ClientError{