Эликсир - тестирование полного скрипта - PullRequest
0 голосов
/ 19 ноября 2018

Я пишу тест для проверки функции (автоматически вызываемой GenServer при входе нового файла в папку), которая вызывает другие функции в том же модуле с помощью каналов для чтения файла, обрабатывает его содержимое, чтобы вставить его, если нужно и возвращает список (: ошибки и: хорошо карты).

результаты выглядят так:

[
          error: "Data not found",
          ok: %MyModule{
            field1: field1data,
            field2: field2data
          },
          ok: %MyModule{
            field1: field1data,
            field2: field2data
          },
          error: "Data not found"

код:

  def processFile(file) do
    insertResultsMap =
      File.read!(file)
      |> getLines()
      |> extractMainData()
      |> Enum.map(fn(x) -> insertLines(x) end)
      |> Enum.group_by(fn x -> elem(x, 0) end)

    handleErrors(Map.get(insertResultsMap, :error))
    updateAnotherTableWithLines(Map.get(insertResultsMap, :ok))
  end

  defp getLines(docContent) do
    String.split(docContent, "\n")
  end

  defp extractMainData(docLines) do
    Enum.map(fn(x) -> String.split(x, ",") end)
  end

  defp insertLines([field1, field2, field3, field4]) do
    Attrs =  %{
      field1: String.trim(field1),
      field2: String.trim(field2),
      field3: String.trim(field3),
      field4: String.trim(field4)
    }

    mymodule.create_stuff(Attrs)
  end

  defp handleErrors(errors) do
    {:ok, file} = File.open(@errorsFile, [:append])
    saveErrors(file, errors)
    File.close(file)
  end

  defp saveErrors(_, []), do: :ok
  defp saveErrors(file, [{:error, changeset}|rest]) do
    changes = for {key, value} <- changeset.changes do
      "#{key} #{value}"
    end
    errors = for {key, {message, _}} <- changeset.errors do
      "#{key} #{message}"
    end

    errorData = "data: #{Enum.join(changes, ", ")} \nErrors: #{Enum.join(errors, ", ")}\n\n"

    IO.binwrite(file, errorData)
    saveErrors(file, rest)
  end

  defp updateAnotherTableWithLines(insertedLines) do
    Enum.map(insertedLines, fn {:ok, x} -> updateOtherTable(x) end)
  end

  defp updateOtherTable(dataForUpdate) do
    "CLOSE" -> otherModule.doStuff(dataForUpdate.field1, dataForUpdate.field2)
  end

У меня есть несколько вопросов, и некоторые из них будут довольно простыми, так как я все еще учусь:

  • Что вы думаете о коде? Любые советы? (примите во внимание, что я добровольно запутал имена).
  • Если я хочу проверить это, это правильный способ проверить только функцию processFile? Или я должен обнародовать больше из них и проверить их индивидуально?
  • Когда я проверяю функцию processFile, я проверяю, что я получаю список. Любой способ убедиться, что в этом списке есть только элементы, которые я жду, например error: "String" или ok: %{} "?

1 Ответ

0 голосов
/ 19 ноября 2018

Что вы думаете о коде?Любые советы?(примите во внимание, что я добровольно замаскировал имена).

Мнение основано.

Если я хочу проверить это, это правильный способ проверить только функцию processFile?

Да.

Или я должен обнародовать их больше и проверить их по отдельности?

Нет, это детали реализации итестирование это анти-паттерн.

Когда я тестирую функцию processFile, я проверяю, что я получаю список.Любой способ убедиться, что в этом списке есть только те элементы, которые я жду, поэтому ошибка: "String" или ok:% {} "?

Вы получаете Keyword. Для проверки явногозначение, можно использовать:

foo = processFile(file)
assert not is_nil(foo[:ok])

OTOH, я бы лучше вернуть карту оттуда и шаблон сопоставить его:

assert %{ok: _} = processFile(file)

Чтобы утверждать, что результат неесть что-нибудь, кроме :ok s и :error s, можно использовать вычитание списка:

assert Enum.uniq(Keyword.keys(result)) -- [:ok, :error] == []
...