Для всех пейджей: нет метода check(), в котором будет проверка на базовые элементы страницы, по котором мы поймем, что страница прогрузилась. И перед тем, как кликать на элемент, брать текст и т.д. лучше проверять, что элемент присутствует. Метод check() может вызываться из конструктора класса.
katemewow
commented
1 year ago
Нужно сделать класс BaseTest, в котором будут все действия "до" и "после" тестов. Туда как раз можно вынести конфигурацию браузера, настройку окна нужной ширины и высоты, переход по урлу, окончание работы с браузером/драйвером. Каждый тест будет наследоваться от BaseTest. Класс PageBuilder как раз можно преобразовать, но работы с пейджами здесь быть не должно. Все действия с пейджами только в тестах. https://github.com/katemewow/SimpleTest/blob/57e02fe1eb6ee15e8d841f03f32ca87a89ea8312/FirsTest/src/test/java/ok/technopolis/helpers/PageBuilder.java#L15
Для всех пейджей: нет метода check(), в котором будет проверка на базовые элементы страницы, по котором мы поймем, что страница прогрузилась. И перед тем, как кликать на элемент, брать текст и т.д. лучше проверять, что элемент присутствует. Метод check() может вызываться из конструктора класса.
Когда сделали логин, нужно возвращать на void, а экземпляр класса, на который попали. Тогда можно было бы сделать chain of invocations в тестах. В классе ToolbarWrapper тоже самое. https://github.com/katemewow/SimpleTest/blob/57e02fe1eb6ee15e8d841f03f32ca87a89ea8312/FirsTest/src/test/java/ok/technopolis/pages/LoginPage.java#L25
Локаторы по правилам константы должны иметь вид USER_NAME. https://github.com/katemewow/SimpleTest/blob/57e02fe1eb6ee15e8d841f03f32ca87a89ea8312/FirsTest/src/test/java/ok/technopolis/pages/ToolbarWrapper.java#L20
Нужно убрать весь класс App, правильно сделать с помощью BaseTest, объявлять пейджи в самом тестовом методе. https://github.com/katemewow/SimpleTest/blob/57e02fe1eb6ee15e8d841f03f32ca87a89ea8312/FirsTest/src/test/java/ok/technopolis/tests/LoginTest.java#L19