Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public class Configuration {
- public static int timeout = 10;
- }
- /*
- правильнее тут выставить 4 секунды
- не всегда нужен такой большой таймаут - 10, 20 секунд
- пусть по умолчанию - будет более востребованное значение
- (не зря в Selenide таймаут по умолчанию - 4 секунды - часто его достаточно)
- а уже в самом тест-методе - переназначишь Configuration.timeout
- посмотри - как ты это делала в этом же задании - когда на Selenide это реализовывала
- */
- ****************************
- public abstract class ConsizeAPI {
- /*
- поискала разные написания concise
- consize - не вижу такого варианта
- кинь мне линку на почитать - если есть у тебя полезный для проверки написаний
- я проверила во всех своих словариках и погуглила
- не вижу варианта consize )
- допускаю, что чего-то не знаю )
- */
- ************************************
- actualTexts = new ArrayList<String>();
- actualTexts = getTexts(elements);
- /*
- в getTexts - есть и инициализация списка, и его заполнение
- в коде кондишена - достаточно actualTexts = getTexts(elements);
- */
- ***********************************************
- public static ExpectedCondition<List<WebElement>> text(final int indexOfVerifiedElement, final By elementLocator, final String expectedText) {
- /*
- про имя кондишена - писала в прошлый раз
- лучше - listNthElementHasText
- порядок и имена параметров
- final int indexOfVerifiedElement,
- будет достаточно index - имя кондишена будет подробным - будет достаточно
- параметр должен быть вторым
- все же надо бы идти от главного к второстепенному
- сначала - что проверяем - локатор / лейзи-прокси элемент/список
- потом - уточнения
- final By elementLocator,
- будет правильнее - elementsLocator
- мы же получаем список элементов
- и в нем ищем подходящий элемент
- это - локатор списка = elementsLocator
- параметр должен быть первым
- final String expectedText
- тут все ок - и название, и то, что это последний параметр
- */
- **************************************
- /*
- см прошлое ревью - строки 52-69
- это надо сделать
- */
- *****************************************
- @BeforeClass
- public static void driverOpen() {
- driver.get("http://gmail.com/");
- }
- /*
- открывать урл - единожды перед запуском всех тестов - перебор...
- лучше бы - пока у тебя один тест-метод
- это перенести в начало самого тест-метода
- а когда поймешь - что во всех тест-методах нужно открыть этот урл
- то тогда такой код разумно вынести в @Before-метод
- но - однозначно - не надо располагать код такой в предке тест-класса
- т к - остальное в BaseTest - универсально подходит в качестве предков для любіх тест-классов
- вот и стоит его оставить универсальным
- для driver.get(...) - в ConciseAPI организуй метод open(...) и его уже используй
- */
Advertisement
Add Comment
Please, Sign In to add comment