Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- http://joxi.ru/DmBNWL6FNXWELm
- /*
- видишь повторения?
- нужна переменная)
- */
- ***************************
- private void followLink(int linkIndex)
- /*
- сравни
- */
- private void followLink(int index)
- /*
- в имени параметра - link - уже лишнее
- в имени метода - уже написали что и для какого объекта мы делаем
- в имени параметра - уже повторяться не надо
- */
- ***********************************
- public static ExpectedCondition<Boolean> sizeOf(final By elementsLocator, final int expectedSize) {
- ....
- public String toString() {
- return String.format("\nSize of list: %s\nExpected to be: %s\nActual size is: %s\n"
- , elements
- , expectedSize
- , listSize);
- }
- /*
- Сравни - как будет выглядеть сообщение об ошибке -
- если выводить туда elements или elementsLocator
- проверить легко
- вызови проверку этого кондишена - которая заведомо упадет
- */
- ***************************************
- public static ExpectedCondition<Boolean> textInElements(final By elementsLicator
- , final int linkIndex
- , final String expectedText)
- //сравни
- public static ExpectedCondition<Boolean> nthListElementHasText(final By elementsLocator
- , final int index
- , final String expectedText)
- /*
- название кондишена - чуть уточнила
- пару параметров - тоже предлагаю другое имя
- */
- public Boolean apply(WebDriver driver) {
- /*
- в целом реализация ОК
- молодец, замахнулся на что-то более серьезное
- и практически все верно реализовал)
- единственный момент - если ты вызовешь такую проверку
- для элемента по индексу за границей списка
- то - вызовется исключение IndexOfOutException
- добавь try-catch в этом методе
- */
- **********************************
- public String toString() {
- return String.format("\nCurrent text of element with index %s is: %s\nExpected text: %s"
- , linkIndex
- , currentText
- , expectedText);
- }
- /*
- Я бы еще локатор elementsLocator уточнила
- */
Advertisement
Add Comment
Please, Sign In to add comment