Ревью

[eliizaveta]

LoginPageTest

не хватает проверки результата

MessagePageTest

if скрыть в методе поиски элементов тоже убрать в какой нибудь метод в пейдже, если это проверки на прогрузку чата с сообщениями, то можно сделать отдельный MessageLayer и вынести в метод isLoaded() getChats() можно сделать private и создать метод внутри MessagePage getFirstChat() {}

Общее

Вынести в константы локаторы private static final By BUTTON = By.xpath("");

Добавлять проверку перед действием с элементом, например $(BUTTON).shouldBe(visible.because("Не отображается кнопка")).click();

Переход к нужной странице лучше делать через методы с нажатием на элементы и возвращая ожидаемые страницы, а не по короткой ссылке

[kov-pavel]

LoginPageTest - теперь проверяет результат - после авторизации должна появиться лента новостей. MessagePageTest - удивительно, но у нас уже была такая прослойка: ChatsPageElement - однако мы не нашли ей подходящего применения. Замечания исправлены. Общее - в константы хпафы вынесены, проверки перед действиями добавлены. Добавлены более корректные переходы - логин возвращает NewsPage, в тестах чтобы получить MessagePage, кликаем на соответствующую кнопку, а не переходим по короткой ссылке.

[eliizaveta]

Про переход на другую страницу:

Лучше в пейдж, откуда переходим, добавить метод для перехода, пример добавить такой метод в NewsFeedPage, получается из ленты переходим в профиль через кнопку в левом меню

public ProfilePage goToProfileViaLeftMenu() { $(PROFILE_BUTTON).shouldBe(visible.because("News button isn't displayed")).click(); return new ProfilePage(); }

в коде будет понятно, куда и откуда перешли: ProfilePage profilePage = mainPage.goToProfileViaLeftMenu();

просто есть несколько страниц, откуда мы можем перейти на профиль, и не везде есть эта кнопка

[kov-pavel]

• Добавлены переходы в каждом пейдже через методы goTo</PageName/>
• Для проверки корректности переходов создан тест RingTest, где мы из каждого пейджа переходим в каждый в различных комбинациях
• Использованы паттерны: Page Object (MessagesPage etc.), Page Element (ChatPageElement etc.), Loadable Component (all), Chain of invocation (for buttons' clicks), Value Object (UserDetails)
• Аннотации JUnit: @Test, @ParameterizedTest, @ValueSource, @Nested, @DisplayName, @BeforeAll, @AfterAll, @BeforeEach, @AfterEach
• Использована библиотека матчинга google truth в соответствие с вариантом (см. MessagesPageTest -> Nested)