Хорошо ли делать поля (WebElements) public в реализации PageObject?

Question

[Александр Василенко]

Дело в том, что с некоторыми элементами операции проводятся действительно простые: кликнуть, ввести значение и т.д.
Насколько это bad/good practice делать поля PageObject публичными?

Answer 1

[1001001]

Это ваша личная кухня, как решите с коллегами так и делайте. )
Но держусь мнения, что в PageObject надо держать не поля а логику.
Вот есть у вас кнопка для закидывания товара в корзину, сделали поле в PO
By pay = By.id("pay")
(не важна By, или WebElement, или еще какая обертка). Теперь в нескольких тестах вы используете его

@Test
publuc void UserShoudBeAdblePay(){
  ...
  driver.findElement(pay).click();
}

А дальше приложение развивается и кнопка превращается в drop-down где два пукнка кинуть в корзину и купить сразу, теперь вы идете и лопатите весь код добавляя клик для открытия drop-down.
А если вы будете закладывать логику в PO, то править надо будет в 1 месте

public void putInBasket() {
        driver.findElement(By.id("pay")).click();
    }

public void putInBasket() {
        driver.findElement(By.id("drop-down")).click();
        driver.findElement(By.id("pay")).click();
    }

Другое дело, что возможно вам и не нуден PageObject, если у вас UI тесты, которые проверяют то, что нельзя проверить более низкоуровневыми тестами, есть шанс что PO в целом излишен )

Comments

[Александр Василенко]

Я немного не понял в части про поля и логику. Классический PO подразумевает, что мы при инициализации класса находим нужные нам элементы с помощью @FindBy аннотации, ну и прописываем методы работы с ними. Вы предлагаете вообще обойтись без этих полей? Просто методы? Просто PO как сервис, верно? Если не трудно, объясните, чем обусловлен именно такой выбор?

[1001001]

Ну вы говорили о публичных полях, значит скорее всего хотите производить манипуляции непосредственно в самом тесте с данными полями (ну я не вижу других причин). Я объяснил почему считаю это плохим подходом.
Делать или не делать поля дело вкуса, просто я часто вижу объявленные поля, которые используются 1 раз в объекте, по мне это излишество. Вообще пихать в переменные все подряд странные подход. ))
Вот по мне это намного удобней

public class LoginPage {
    public LoginPage setUsername(String username) {
        driver.findElement(By.id("username")).sendKeys(username);
        return this;    
    }
}

чем это

public class LoginPage {
    By usernameLocator = By.id("username");

    public LoginPage setUsername(String username) {
        driver.findElement(usernameLocator).sendKeys(username);
        return this;    
    }
}

или это

public class LoginPage {
    @FindBy(id = "username")
    WebElement usernameLocator

    public LoginPage setUsername(String username) {
        usernameLocator.sendKeys(username);
        return this;    
    }
}

Да если элемент используется много раз, логично его вытащить в переменную)

[Александр Василенко]

1001001, а, всё! Понял смысл! В целом, согласен!