new: basedOnRequest strategy implemented

[eliseeviam]

A new request strategy was implemented according to an issue Добавить в моки возможность давать ответ в зависимости от body/query запроса.
From my point of view, the alternative way seems more explicit than extending the sequence strategy.

The point of this strategy is to provide the ability to send multiple parallel requests with the same path to an exact one service. Each request could be unique or similar.

For example, there is no way to mock these requests called in parallel:

/process?key=value1
/process?key=value2&opt_key

[lansfy]

Ну честно сказать не очень очевидный синтаксис. Я бы предпочел иметь что-то вроде

basedOnBodyFieldValue
basedOnQueryParamValue
basedOnHeaderValue

где возвращать результат в зависимости от значения поля в запросе, значения ключа или значения заголовка соответственно. Совсем странно использовать для этого requestConstraints, который создан совсем для другого.

[eliseeviam]

Хочу объяснить решаемую проблему и возможно найдется концептуально верное решение.

В процессе обработки запроса может происходить несколько запросов к одной и той же ручке, но с различными параметрами(телом, хидерами, etc) запроса. Также запросы могут быть отправлены параллельно.
Проблема в том, что ни одна из существующих стратегий не позволяет замокать такое поведение.
Из наиболее близких это uriVary, которой можно замокать только один запрос в нужным path, или sequence, которая позволяет замокать несколько запросов, но требовательна к порядку исполнения, который при параллельном вызове гарантировать нельзя.

Новая стратегия убирает ограничение уникальности с path и позволяет объявлять нескольно моков для одного и того же path.

requestConstraints я понимаю как способ проверить, что параметры запроса были сформированы ожидаемым образом. Что также соблюдается. Если ни один из запросов не удовлетворил requestConstraints, то тест будет провален.

[Cdayz]

Единственное что мне тут не нравится, это вот этот принт ошибки, кажется стоит его добавить в виде

if err != nil {
    requestDump = []byte(`failed to dump request`)
}

надеюсь ничего с типами не напутал.

[eliseeviam]

Спасибо! Поправил.
Заодно добавил возврат всех диффов верификации для запроса, для которого не нашлось мока.

[github-actions]

🚀 PR was released in v1.14.0 🚀