Как тестировать модуль, который получает данные от внешних сервисов?

Question

[Антон Иванов]

Всем привет.

Есть определенная функция, которая запрашивает "задание" с внешнего ресурса. И запрос и разбор ответа идет внутри функции.
Как можно/нужно написать тест для проверки работы в случае получения/не получения определенных данных?

Спасибо заранее.

Answer 1

[Константин Китманов]

Вынести разбор ответа в отдельную функцию/метод/модуль, чтобы можно было тестировать ее независимо.

Что же до запроса, то проще всего покрыть это e2e тестами. Ну или явно передавать в вашу функцию ту вещь, которая в вашем языке/платфоре делает запросы (например, в клиентском JS это XMLHTTPRequest или fetch), а в тестах ее подменять.

Comments

[Антон Иванов]

Ну вот, именно хотелось избежать вынесения разбора ответа в отдельную функцию :)
Спасибо.

[Константин Китманов]

Антон Иванов: а почему хотелось избежать? Наоборот же независимые кусочки и понимать легче и рефакторить и все такое...

[Антон Иванов]

Константин Китманов: Просто этот кусочек гарантированно нигде больше не будет использоваться и сам по себе относительно небольшой (10-15 строк). Не видел смысла выносить в отдельную функцию

[Константин Китманов]

Антон Иванов: и зря, 10-15 строк это как раз размер приличной функции. Разделять программу на кусочки надо не только из соображений повторного использования, а еще для разделения ответственности, повышения тестируемости, ну и просто для простоты восприятия.

[Антон Иванов]

Константин Китманов: ок, убедили :)

Answer 2

[Алексей]

Как обычно - придумать тестовые данные, после чего работать с ними. Зависимости, которые не имеют отношения к тестируемой логике напрямую можно замокать (в Java я использую Mockito, для вашего случая думаю тоже есть что то, упрощающее жизнь). В приведенном ниже примере возможно найдется что то полезное для вас.

import org.junit.Assert;

class Test {

    public static void main(String[] args) {

        String message = "Hello, World!\nCheck this message.";

        ExternalController external = job -> {
            System.out.println(job);
            return job.toUpperCase();
        };

        InternalController tested = new InternalController(external);
        String results = tested.processRequest(message);
        Assert.assertEquals("HELLO, WORLD!\nCHECK THIS MESSAGE.\n", results);
    }
}

class InternalController {
    private ExternalController external;

    InternalController(ExternalController external) {
        this.external = external;
    }

    String processRequest(String message) {
        String[] jobs = message.split("\n");
        StringBuilder results = new StringBuilder();
        for (String job : jobs) {
            String result = external.doWork(job);
            results.append(result).append("\n");
        }
        return results.toString();
    }
}

interface ExternalController {
    String doWork(String job);
}

Comments

[Антон Иванов]

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

[Алексей]

Антон Иванов: Все зависит от вашего флоу. Вы можете подтягивать данные из XML. Или явным образом создавать в коде соответствующий объект.
Честно говоря, у меня не совсем правильный мок получился. В идеале, он должен всегда возвращать статические данные в рамках одного кейса.
Кроме того, может вам с юнит тестов стоит переключиться на функциональные? Может тогда будет проще протестировать необходимый флоу?