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

stjefim
Hello! Suppose you are building workflow (order / task / payment) processing system with the following requirements: Each workflow con...
New
Blokh
Hey guys, I’ve got a huge CSV ( around 10 GB ) that needs to be processed hourly Do you guys have any suggestions what is the best prac...
New
roeland
Kia ora, We have been using elixir-google-api to connect to Google Drive. However, with the updates to Tesla due to CVEs this is now bro...
New
kszambelanczyk
Hello! Could someone please give me a help/sample code, how to delete a file from s3 using waffle/waffle_ecto from Phoenix app. I creat...
New
Onor.io
I have what I’ve heard referred to as a “lookup table” in my database. This is a way of assigning codes to common values. One common lo...
New
jaybe78
Hello, I’m developing a online persistent chat system (what’s app) like using elixir/dynamodb/aws for a mobile app(flutter). The diffic...
New
Trolleger
What approach to take when sending live updates to “random” users Hi! I have a question, I have a little chat app, and when I create a DM...
New

Other Trending Topics Top

garrison
Hobbes is a low-level distributed database for the Elixir programming language. Hobbes provides a simple, safe, and scalable storage lay...
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
mcass19
ExRatatui lets you cook up rich terminal UIs in Elixir, powered by Rust’s ratatui via Rustler NIFs. Build interactive terminal applicatio...
New
netoum
Corex is an accessible, unstyled UI component library for Phoenix that integrates Zag.js state machines using Vanilla JavaScript and Live...
New
Damirados
Hello everyone. After busy few months I am happy to announce v0.1.0 of Emerge &amp; Solve. They are GUI (Emerge) and State management (S...
New
wintermeyer
There are three potential reasons for members of this forum to have a look at https://vutuv.de You are tired or annoyed of LinkedIn. Yo...
New

We're in Beta

About us Mission Statement

Options

Thread Display Mode




Thread Preview

Skip Thread Previews