xgeek116

xgeek116

Problem mocking a function when doing unit testing

I’m testing a function that call another function to fetch external data using API calls, here is the main module code :

defmodule MyApp.MyModule.ExternalData do
  require Logger
  
  def fetch_external_data(params) do
    with {:ok, data} <-
      http_get_resource(params["url"], params["headers"]) do
        data
    else
      :error ->
        Logger.info("ERROR executing API call to : #{source} with path : #{url} ")
        %{}
    end
  end

  def http_get_resource(url, headers) do
    case HTTPoison.get(url, headers) do
      {:ok, %HTTPoison.Response{status_code: 200, body: body}} ->
        {:ok, Poison.decode!(body)}

      {:ok, %HTTPoison.Response{status_code: 404}} ->
        :error

      {:error, %HTTPoison.Error{reason: _reason}} ->
        :error
    end
  end
end

And here is my test file :

defmodule MyApp.MyModule.ExternalData.Test do
  use ExUnit.Case

  # Replace `http_get_resource/2` with a mock function to simulate the HTTP request
  def mock_http_get_resource(url, headers) do
    case url do
      # Simulate a successful request with a 200 status code
      "http://valid.com/resource" -> {:ok, %{status_code: 200, body: %{key1: "value1", key2: "value2"}}}
      # Simulate a failed request with a 404 status code
      "http://invalid.com/resource" -> {:ok, %{status_code: 404, body: "Resource not found"}}
      # Simulate a request that times out
      "http://timeout.com/resource" -> {:error, :timeout}
    end
  end

  def http_get_resource(url, headers) do
    mock_http_get_resource(url, headers)
  end

  test "fetch_one_external_resource/1" do
    # Replace the actual `http_get_resource/2` function with the mock function
    alias MyApp.MyModule.ExternalData, as: M
    M.http_get_resource = &http_get_resource/2

    # Test a successful request
    params = %{
      "url": "aaa", 
      "headers": "bbb"
    }
    assert M.fetch_one_external_resource(params) == %{new_key1: "value1", new_key2: "value2"}

    # Test a failed request
    params = %{
      "url": "aaa", 
      "headers": "bbb"
    }
    assert M.fetch_one_external_resource(params) == :error

    # Test a request that times out
    params = %{
      "url": "aaa", 
      "headers": "bbb"
    }
    assert M.fetch_one_external_resource(params) == :error
  end
end

I’m getting this error :

== Compilation error in file test/fetch_data_test.exs ==
** (CompileError) test/fetch_data_test.exs:26: cannot invoke remote function MyApp.MyModule.ExternalData.http_get_resource/0 inside a match
    test/fetch_data_test:23: (module)
    (elixir 1.14.2) lib/kernel/parallel_compiler.ex:455: Kernel.ParallelCompiler.require_file/2
    (elixir 1.14.2) lib/kernel/parallel_compiler.ex:348: anonymous fn/5 in Kernel.ParallelCompiler.spawn_workers/7

Any ideas or fix please, I think the problem is in the line M.http_get_resource = &http_get_resource/2

thanks

Most Liked

al2o3cr

al2o3cr

That’s not how Elixir modules work, you can’t “reassign” functions in them at runtime.

Consider using one of the mocking libraries like mox or meck that does this correctly.

Alternatively, consider refactoring fetch_external_data into separate functions that can be tested without a live HTTP call.

Where Next?

Popular in Questions Top

sergio
In Ruby, I can go: User.find_by(email: "foobar@email.com").update(email: "hello@email.com") How can I do something similar in Elixir? ...
New
lessless
I believe there are people here who are dealing with CSV files import on the daily basis, and since Excel is a really popular tool there ...
New
New
Harrisonl
We have an ECS cluster with 4 services, where each task joins a single cluster, via discovery ECS discovery service. Currently when I de...
New
nsuchy
Hi. I’ve noticed that Windows Powershell has it’s own IEX command and you cannot access Elixir’s IEX due to the conflict. This isn’t a cr...
New
makeitrein
Hey all, just started picking up Elixir last week and am writing a scraper as a learning project. Baby step #1 is extracting the number ...
New
johnnyicon
Hi all, I've just started learning Elixir and Phoenix Framework, so please pardon my n00bness at this stage. I'm trying to use Postg...
New
electic
Hi, I am new to Elixir. I am trying to use the DateTime component to insert a date into MySQL however the there seems to be no way to fo...
New
kostonstyle
Hi all I want to have a unix time, from the current time plus 1 hour. DateTime.now + 1 hour How to get it in elixir? Thanks
New
lanycrost
Hi everyone! I need implement if…else if…else condition from my elixir code, and anymore of this control flow structures not work proper...
New

Other popular topics Top

Brian
What is the proper way to load a module from a file in to IEX? In the python world, doing something like this pretty standard: from ....
New
peerreynders
Manning 2016 Halloween weekend sale via Deal of the Day Friday, October 28 - Half off all MEAPs - code WM102816LT Saturday, October 29 ...
326 29600 154
New
Tee
can someone please explain to me how Enum.reduce works with maps
New
joaquinalcerro
Hi there, I am working with Ecto-Postgresql and I need to call all of the records from a specific table but the table has 40,000 record...
New
Harrisonl
We have an ECS cluster with 4 services, where each task joins a single cluster, via discovery ECS discovery service. Currently when I de...
New
alice
Hey, Just curious what are the main benefits of Elixir compared to Clojure? When is Elixir more useful than Clojure and vice versa? Th...
New
fireproofsocks
Forgive me if this is obvious, but how does one delete a database record WITHOUT selecting it first? https://hexdocs.pm/ecto/Ecto.Repo.h...
New
msaraiva
Surface is an experimental library built on top of Phoenix LiveView and its new LiveComponent API that aims to provide a more declarative...
564 42633 214
New
electic
Hi, I am new to Elixir. I am trying to use the DateTime component to insert a date into MySQL however the there seems to be no way to fo...
New
jay1
Why is it that the mnesia database isn’t the most preferred database for use in Elixir/Phoenix?
New

We're in Beta

About us Mission Statement