xgeek116

xgeek116

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

Showing Posts 1 to 2

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.

xgeek116

xgeek116 OP

Hi @al2o3cr , thank you.

Please see my problem with Mox : Mox is ignoring my Mocked module and calling the real function

— All posts loaded —

Where Next? Top

Trending in Questions Top

RSP87
I’m working on a project that simulates the bumbl example in the programming phoenix book. It acts almost like an email client. We have a...
New
nseaSeb
Hello, I know there is an approach for handling lists that allows for optimized traversal, but I can’t recall the specific method (somet...
New
RemyXRenard
I’m seeing that a list inside a Kino.DataTable will be interpreted as a charlist, even if the Kino.configure() is set to charlists: :as_l...
New
velrest
So my question is quite simple and i have found no conclusive answer on forum, google or AI. Should we use :erlang.float for Integer to ...
New
brecabral
Documentation While reading the Scoped Routes section, I noticed that the documentation currently refers to a problem without explainin...
New
samoloth
Hi, I’ve just set up an application with ash_authentication. There is only magic link strategy for now, so there is no confirmation add o...
New
FlyingNoodle
If a change or preparation module uses Ash.Changeset.get_argument/2 or Ash.Query.get_argument/2 (or any of the other get_argument functio...
New

Other Trending Topics Top

mudasobwa
I am happy to introduce the very α version of the new programming language compiled to BEAM. Welcome Cure. It has literally three kille...
New
marciok
Hi there! We created Gust: A task orchestrator inspired by Airflow. For those who have never heard about Aiflow, it’s a Python-based wor...
New
jimsynz
Beam Bots (or just BB for short) is a framework for building fault-tolerant robotics applications in Elixir using familiar OTP patterns. ...
New
Dmk
Xamal is a deployment tool for Elixir apps that deploys native releases to bare metal servers over SSH. It’s a port of GitHub - basecamp/...
New
netoum
Corex is an accessible, unstyled UI component library for Phoenix that integrates Zag.js state machines using Vanilla JavaScript and Live...
New
webofbits
With AI doing more of the implementation work, I’ve been wondering how much coding I should deliberately keep doing myself. My main conc...
#ai
New

We're in Beta

About us Mission Statement

Options

Thread Display Mode




Thread Preview

Skip Thread Previews