Переписал кучу e2e-тестов на варианты с динамическими параметрами в цикле.

Question

Переписал кучу e2e-тестов на варианты с динамическими параметрами в цикле.

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

Вопрос: плюсы и минусы использования циклов для формирования тестов? Ну, из очевидного - параметры - они не всегда универсальные.

#backend #javascript #node.js #programming #russian

0

09.05.2023

8 ответов

31 просмотр

][_ioN ‏ 😏 Автор вопроса

Grigorii K. Shartsev
Не очень понятно, что тут имеется в виду под цикло...

"Не очень понятно, что тут имеется в виду под циклом" Пример: const exchanges: = { coinbase: [ 'default', 'sandbox' ], exmo: [ 'default', 'margin' ], huobi: [ 'default', 'aws' ], nicehash: [ 'default' ], poloniex: [ 'default', 'futures' ], ... }; const routes = [ 'getAccountsData', 'getKChartData', 'getTickersData', ... ]; Object.keys( exchanges ).forEach( exchange => { describe( `${ exchange } should reply 500 if uid is not present in request`, () => { exchanges[ exchange ].forEach( type => { routes.forEach( route => { test( ... ); } ); } ); } ); } ); "или все тесты сводятся к сравнению ответа запроса при определённом запросе?" Ну, базовый набор тестов на 500, наличие отсутствие необходимых заголовков в ответе, 404 и прочие общие для всех роутов сценарии ответов.

0

09.05.2023

Grigorii K. Shartsev

Вообще вызывают вопросы тесты, которые проверяют, что бек кидает 500...

0

09.05.2023

][_ioN ‏ 😏 Автор вопроса

Grigorii K. Shartsev
Вообще вызывают вопросы тесты, которые проверяют, ...

Почему? Например кейс, что указанный параметр не поддерживается. В этом случае бек ничего трогать не должен и выкидывает ошибку.

0

09.05.2023

Grigorii K. Shartsev

][_ioN ‏ 😏
Почему? Например кейс, что указанный параметр не п...

Потому что 500 означает ошибку сервера

0

09.05.2023

][_ioN ‏ 😏 Автор вопроса

Grigorii K. Shartsev
Потому что 500 означает ошибку сервера

Это дефолт у Фастифая, выкидывать такие статусы в таких случаях 🤷‍♂️

0

09.05.2023

Тупая Инфузория

][_ioN ‏ 😏
Почему? Например кейс, что указанный параметр не п...

в тестах ты должен проверить работу своего кода, а не стороннего сервиса это уже другие тесты

0

09.05.2023

][_ioN ‏ 😏 Автор вопроса

это на ошибки валидации, знаю.

0

09.05.2023

Grigorii K. Shartsev · Accepted Answer

Grigorii K. Shartsev

Не очень понятно, что тут имеется в виду под циклом. > достаточно будет добавить роут к списку Но у разных роутов обычно разные тесты. или все тесты сводятся к сравнению ответа запроса при определённом запросе?

0

09.05.2023

170 похожих чатов

Переписал кучу e2e-тестов на варианты с динамическими параметрами в цикле.

8 ответов

Похожие вопросы