adjustment: for latest Tesla / Hackney

Author: amineo

config/dev.exs

@@ -2,4 +2,5 @@ import Config
 
 config :ex_urlbox,
   api_key: {:system, "URLBOX_API_KEY"},
-  api_secret: {:system, "URLBOX_API_SECRET"}
+  api_secret: {:system, "URLBOX_API_SECRET"},
+  env: :dev

config/test.exs

@@ -4,6 +4,7 @@ config :tesla, ExUrlbox, adapter: Tesla.Mock
 
 config :ex_urlbox,
   api_key: "fakeAPIkey",
-  api_secret: "fakeAPIsecret"
+  api_secret: "fakeAPIsecret",
+  env: :test
 
 config :logger, level: :info

lib/config.ex

@@ -54,7 +54,18 @@ defmodule ExUrlbox.Config do
 
 
   # Determine the Tesla adapter to use
-  @doc false
-  def get_client_adapter, do:  Application.get_env(:tesla, ExUrlbox, [])[:adapter] || Tesla.Adapter.Hackney
+  @doc """
+  Returns the appropriate Tesla adapter based on the environment.
+  In test environment, returns Tesla.Mock.
+  In other environments, returns the configured adapter or defaults to Hackney.
+  """
+  def get_client_adapter(opts \\ []) do
+    env = Application.get_env(:ex_urlbox, :env, :prod)
+
+    case env do
+      :test -> Tesla.Mock
+      _ -> {Application.get_env(:tesla, ExUrlbox, [])[:adapter] || Tesla.Adapter.Hackney, opts}
+    end
+  end
 
 end

lib/ex_urlbox.ex

@@ -4,7 +4,7 @@ defmodule ExUrlbox do
 
   **Compatible with Urlbox API Version:** `v1`
 
-  A list of options that you can pass into `ExUrlBox.get/3`, `ExUrlBox.post/3`, `ExUrlBox.head/3`, and `ExUrlBox.delete/3` can be found here:
+  A list of options that you can pass into `ExUrlbox.get/3`, `ExUrlbox.post/3`, `ExUrlbox.head/3`, and `ExUrlbox.delete/3` can be found here:
   https://urlbox.io/docs/options
   """
 
@@ -20,7 +20,7 @@ defmodule ExUrlbox do
   Refer to the official documentation for all the available options: https://urlbox.io/docs/options
 
     **This action is `synchronous`. **
-    If you want to use an `async` flow that uses webhooks, use `ExUrlBox.post/3`.
+    If you want to use an `async` flow that uses webhooks, use `ExUrlbox.post/3`.
 
    **Function signature is: `url, [options], timeout`**
 
@@ -170,9 +170,16 @@ defmodule ExUrlbox do
     ]
     |> determine_endpoint(is_post?)
 
-    Tesla.client(middleware, Config.get_client_adapter)
+    # Pass timeout to Hackney adapter
+    adapter_opts = [recv_timeout: timeout]
+
+    # Use Config module to determine the appropriate adapter
+    adapter = ExUrlbox.Config.get_client_adapter(adapter_opts)
+
+    Tesla.client(middleware, adapter)
   end
 
+
   @doc false
   @spec determine_endpoint(list(), boolean()) :: list()
   defp determine_endpoint(middleware, is_post?) do

test/ex_urlbox_test.exs

@@ -1,6 +1,5 @@
 defmodule ExUrlboxTest do
   use ExUnit.Case, async: false
-  alias ExUrlbox.Config
   alias ExUrlbox.Utils
 
   doctest ExUrlbox
@@ -12,7 +11,7 @@ defmodule ExUrlboxTest do
     height: 768
   ]
 
-  @assembled_request_url "https://api.urlbox.io/v1/fakeAPIkey/1783c14642a8a56a36f6492f73f524bfe132ed91/png?url=https%3A%2F%google.com&width=1024&height=768"
+  @assembled_request_url "https://api.urlbox.io/v1/fakeAPIkey/3e2c5515f8a2de570445a8b4b5c041809fb9b97e/png?url=https%3A%2F%2Fgoogle.com&width=1024&height=768"
 
   # Mock requests
   setup_all do
@@ -40,6 +39,7 @@ defmodule ExUrlboxTest do
     options = [{:webhook_url, "https://example.com/webook"} | @test_options]
     {:ok, screenshot_request} = ExUrlbox.post(@test_url, options)
 
+
     # Headers
     assert Tesla.get_headers(screenshot_request, "authorization") === ["Bearer #{Application.get_env(:ex_urlbox, :api_secret)}"]
     assert Tesla.get_headers(screenshot_request, "content-type") === ["application/json"]