Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- core.Configuration.timeout или Configuration.timeout
- /*
- Configuration.timeout - достаточно подробно
- */
- **************************
- public class Configuration {
- public static int timeout = 15;
- }
- /*
- правильнее тут выставить 4 секунды
- не всегда нужен такой большой таймаут - 15 секунд
- пусть по умолчанию - будет более востребованное значение
- (не зря в Selenide таймаут по умолчанию - 4 секунды - часто его достаточно)
- а уже в самом тест-методе - переназначишь Configuration.timeout
- посмотри - как ты это делал в этом же задании - когда на Selenide это реализовывала
- */
- ***********************************
- return Boolean.valueOf(actualText.contains(expectedText));
- /*
- тут можно проще
- return actualText.contains(expectedText);
- */
- ********************************************
- public String toString() {
- return String.format("text (\'%s\') isn't present in element. Actual text is '%s'", expectedText, actualText);
- }
- ...
- public String toString() {
- return String.format("Text in element isn't contains actual text '%s'. " +
- "Expected text: '%s'", elementsTexts.toArray().toString(), expectedTexts.toString());
- }
- /*
- во всех кондишенах
- стоит строить фразу
- element of list ...
- т е - указать - про что речь - что проверяем (что тоже важно)
- should ...
- описываем expected
- while actual ...
- описываем actual
- не хватает описания проверяемой сущности
- ну и если совсем занудствовать - то корректнее - не утверждать - что проверка не выполнена
- а описать - проверяли тото, ждали тото, получили тото
- так корректнее - т к строка, возвращаемая toString() - описывает объект
- */
- ************************************
- /*
- тут этого не нужно
- но в общем - полезнее реализовать кондишены как
- ExpectedCondition<WebElement> listNthElementHasText(final List<WebElement> elements, final int index, final String expectedText)
- ExpectedCondition<List<WebElement>> texts(final List<WebElement> elements, final String... expectedTexts)
- советую попробовать)
- настаивать не буду
- */
Advertisement
Add Comment
Please, Sign In to add comment