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

Translation Into Russian #221

Merged
merged 2 commits into from
Dec 14, 2022
Merged

Conversation

Saimon398
Copy link
Contributor

@Saimon398 Saimon398 commented Oct 4, 2022

Hello,
I decided to contribute to the project and have this manual translated into Russian ( #218 ).
Submitting it for review)


:white_check_mark: **Сделать:** Если существует необходимость в [snapshot testing](https://jestjs.io/docs/en/snapshot-testing), используйте короткие снимки (i.e. 3-7 lines), которые являются частью теста ([Inline Snapshot](https://jestjs.io/docs/en/snapshot-testing#inline-snapshots)), а не во внешних файлах. Соблюдение этого правила позволит вашим тестам оставаться понятными и надежными.

С другой стороны, руководства по "классическим снимкам" призывают хранить большие файлы (например, разметку рендеринга компонента, результат API JSON) на каком-то внешнем носителе и каждый раз при выполнении теста сравнивать полученный результат с сохраненной версией. Это может привести к тому, что, тест будет привязан к огромному количеству данных (например, 1000 строк), о которых разработчик никогда не задумывался. Почему так нельзя делать? аким образом, существует 1000 причин, по которым ваш тест может не пройти - достаточно изменения одной строки, чтобы снимок стал недействительным, а это, скорее всего, будет происходить часто. Как часто? Каждый пробел, комментарий или незначительное изменение HTML/CSS. Мало того, что название теста не дает представления о том, что пошло не так, поскольку он просто проверяет, что 1000 срок кода остались неизменны, так еще и вводит разработчика в заблуждение, заставляя принимать за желаемую истину документ, который он не сможет проверить. Все это является признаками теста, который стремиться захватить сразу многое.
Copy link

@Patr56 Patr56 Dec 6, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: "? аким образом"

@goldbergyoni
Copy link
Owner

@Saimon398 WOW, this is freakin awesome!

First, sorry for the very late reply, I was away from OSS work for a few months due to some circumstances

This PR is gorgeous, many-many thanks for this

Don't forget to also link to this from the international homepage, this is how Russian folks will find this

I'll try to promote this as much as possible - Twitter, news section in my repos, etc

Is there a Russian FB/Whatsapp group where we can publish this?

🎁 One last note - I'm now finalizing a 7-hours long comprehensive, advanced Node.js testing course - I'd like to gift to you as my way of saying that you for this awesome work. I'll bounce in a month+

@all-contributors please add @Saimon398 for content

@goldbergyoni goldbergyoni merged commit c50d73c into goldbergyoni:master Dec 14, 2022
@goldbergyoni
Copy link
Owner

https://github.com/all-contributors please add @Saimon398 for content

@goldbergyoni
Copy link
Owner

@all-contributors please add @Saimon398 for content

@allcontributors
Copy link
Contributor

@goldbergyoni

@Saimon398 already contributed before to content

@allcontributors
Copy link
Contributor

@goldbergyoni

I couldn't determine any contributions to add, did you specify any contributions?
Please make sure to use valid contribution names.

I couldn't determine any contributions to add, did you specify any contributions?
Please make sure to use valid contribution names.

I've put up a pull request to add @you! 🎉

I've put up a pull request to add @Saimon398! 🎉

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 this pull request may close these issues.

None yet

3 participants