Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- public static Actions action(){
- return new Actions(getDriver());
- }
- /*
- Метод корректнее назвать actions()
- мы же создаем объект типа Actions
- */
- **********************
- public static WebElement $(ExpectedCondition<WebElement> conditionToWaitParentElement){
- /*
- я бы этот метод первым из методов $ привела
- и в $(By locator) - использовала уже его
- имя параметра conditionToWaitParentElement - упрости
- про аналогичное писала в прошлом ревью, строки 15-22
- */
- ******************************
- public String toString() {
- return String.format(index + " element in list should have text: " + expectedText + ". While actual text is " + elementText);
- }
- /*
- не забывай и локатор выводить
- да и я бы разделяла сообщение на несколько строк
- что мы проверяем
- для какого элемента/элементов мы проверяем
- что ожидаем
- что есть по факту
- тут было бы ок
- for list found by locator ...
- text of list element [index]
- should be :
- while actual text is:
- просмотри и другие сообщения
- */
- ***************************
- actualTexts.clear();
- actualTexts.addAll(Helpers.getTexts(actualElements));
- /*
- поскольку actualTexts - не final-переменная теперь
- то можно проще
- actualTexts = getTexts(actualElements);
- здесь и далее - используй import static
- для методов класса Helpers
- и так будет достаточно понятно и точно
- ну и усти по поводу работы со списками
- и другие места есть с аналогичным кодом
- */
- ******************************************
- public WebElement apply(WebDriver driver) {
- List<WebElement> webElementList = ConciseAPI.getDriver().findElements(elementsLocator);
- /*
- не нужно тебе в методе кондишена apply
- использовать вебдрайвер из ConciseAPI
- т к в apply - вебдрайвер передан как параметр
- */
- **********************************
- public class TodoMvcPage
- /*
- пейдж можно реализовать как пейдж-модуль
- это на твое усмотрение
- */
- ************************************
- $(byCss("#toggle-all"))
- //можно проще
- $("#toggle-all")
- //и для $(byCss("#clear-completed")) - аналогично
Advertisement
Add Comment
Please, Sign In to add comment