Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Оборачивать каждый тест в сабтесты #187

Closed
fetinin opened this issue Oct 7, 2022 · 3 comments · Fixed by #188
Closed

Оборачивать каждый тест в сабтесты #187

fetinin opened this issue Oct 7, 2022 · 3 comments · Fixed by #188
Assignees

Comments

@fetinin
Copy link
Contributor

fetinin commented Oct 7, 2022

Проблема следующая. При запуске тестов Gonkey сливает их все в одну кашу. Go test runner считает все тесты как один большой тест и все ошибки и логи по тестам сливаются.
Эта можно частично решить тем что каждый блок тестов или даже каждый тестовый файл заводится в коде отдельным тестом. Но это неудобно.
image

Предлагаю на каждый тест оборачивать в testing.T.Run() чтобы test runner понимал что каждый тест в .yaml файле это отдельный тестовый сценарий.

До:
Screen Recording 2022-10-07 at 19.00.35.webm

После:
Screen Recording 2022-10-07 at 18.57.53.webm

@fetinin
Copy link
Contributor Author

fetinin commented Oct 7, 2022

@leorush @keyclaim @luza @vitkarpenko напишите ваше мнение. Если никто не видит проблем, то оформлю PR.

@luza
Copy link
Contributor

luza commented Oct 7, 2022

Очень круто, если получится так сделать.

@keyclaim
Copy link
Contributor

Это очень-очень хорошо.
Особенно в сочетании с #151

@fetinin fetinin self-assigned this Oct 17, 2022
@fetinin fetinin linked a pull request Oct 26, 2022 that will close this issue
fetinin added a commit that referenced this issue Oct 27, 2022
fetinin added a commit that referenced this issue Oct 27, 2022
#187 Wrap test in subtest for go test runner
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants