From 00133c3d1e62bcfd6c47e8d1ac6839b0340c0479 Mon Sep 17 00:00:00 2001 From: Yury Yurchenko <99906419+Achitheus@users.noreply.github.com> Date: Mon, 12 Feb 2024 13:48:54 +0300 Subject: [PATCH] Update NOTES.md --- NOTES.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/NOTES.md b/NOTES.md index c1f0bfe..19991fa 100644 --- a/NOTES.md +++ b/NOTES.md @@ -5,17 +5,17 @@ - [Досадные обстоятельства](#досадные-обстоятельства) ### Не дает покоя (планы на будущее) --[x] Если [`metCondition()`](https://github.com/Achitheus/SelenideTestCase/blob/a49a8a2a2aa182f5570b3438b5d385e6bd77384c/src/main/java/helpers/SelenideCustom.java#L22C103-L22C103) +- [x] Если [`metCondition()`](https://github.com/Achitheus/SelenideTestCase/blob/a49a8a2a2aa182f5570b3438b5d385e6bd77384c/src/main/java/helpers/SelenideCustom.java#L22C103-L22C103) возвращает `false`, это значит что был брошен `Error`, а значит его перехватывает селенидовский листенер, а значит делается лишний скриншот и сохраняется лишняя страница. Исправить это можно, сделав метод через цикл со `Sleep`-ом и передавая в него `Condition`, который используется для `SelenideElement.is(Condition)`, который не задействует ожидания и ничего не выбрасывает (в обычной ситуации). --[x] Надо бы переделать метод +- [x] Надо бы переделать метод [`marketCheckAllPages()`](https://github.com/Achitheus/SelenideTestCase/blob/a49a8a2a2aa182f5570b3438b5d385e6bd77384c/src/test/java/ru/bellintegrator/ru/yandex/market/MarketTest.java#L82) во что-то реюзабельное, например создать класс вроде `PagesChecker` и с помощью него делать проверки с прогоном по всем страницам. Сейчас же он по сути является частью конкретного теста и захламляет класс тестов. Должен получиться отличный `helper`, который можно будет использовать вообще с любыми сайтами/страницами, где есть пагинация. --[x] Разобраться как следует в `maven` и заполнить раздел [Build](/README.md#build-раздел-в-разработке) +- [x] Разобраться как следует в `maven` и заполнить раздел [Build](/README.md#build-раздел-в-разработке) ### Понравилось - Помимо обычного `.gitignore`, есть еще его глобальная версия, которую обычно кладут