-
Notifications
You must be signed in to change notification settings - Fork 131
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
refactor(Hint): transfer to functional component & refactor tests/stories #2715
base: master
Are you sure you want to change the base?
Conversation
Пытался написать тесты для ховеров, но ничего путного не получилось. Полагаю, что это происходит из-за того, что тело хинта рендерится в портале и при наведении на хинт В общем, нужно научиться тестировать порталы. Схожая проблема с тестированием порталов есть и в Toast'е. UPD: Я всё-таки научился тестировать порталы, но способ которым я это сделал не был нигде описан, так что я добавил новый комментарий со своим способом в ветку, которую я скидывал выше. |
Почему-то падает скриншотный тест с "наведением и кликом", хотя при ручном тесте всё отрабатывает нормально. |
…refactor/hint-to-functional
Смёржил актуальный мастер. |
…refactor/hint-to-functional
Перевёл контрол
Hint
на функциональные рельсы.Удалил тесты
'controlled opening only if manual prop passed'
и'manual opening'
, так как они тестировали детали реализации. Заменил их тестом'opens and closes manual hint on button click'
Также, уже не помню нужно ли прокидывать
rootNodeRef
в функциональные компоненты или они обходятся без него, поэтому удалил соответсвующий функционал.Этот контрол вскрыл ещё одну проблему перехода на функциональные компоненты: классовые компоненты не дружат с функциональными
ref
-ками, поэтому переезд нужно начинать с компонентов не содержащих вложенных компонентов и затем подниматься выше по иерархии.