eliizaveta
commented
1 year ago тоже можно проверить на видимость https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/MusicPage.java#L67
он и без супер по идее вызовет https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/ProfilePage.java#L12-L14
этот лоад с любого места в одноклассниках будет работать? лучше не делать переходы прямо внутри страницы, а делать их с других страниц, где это точно есть https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/ProfilePage.java#L17-L21
можно сообщения добавлять в ассерты для более понятного вывода ошибок https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/test/java/AuthorizationTest.java#L23
такое лучше прятать в пейджах https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/test/java/MusicPageTest.java#L32-L36
StombieIT
я в тамтам: https://tt.me/riinj
а если ботов несколько это для каждого такой провайдер делать надо? https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/models/CredentialProviders.java#L3
вообще открывать ок логичнее в базовом тестовом классе https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/Page.java#L32
а на всех ли страницах он есть?
https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/Page.java#L37
в shouldBe есть встроенное ожидание(кстати какое?:)), зачем нам его каждый раз переопределять? https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/Page.java#L16
на всех страницах такое есть?
https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/BasePage.java#L15
это скорее тулбар а не базовая страница, тулбар есть не на всех страницах
https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/BasePage.java#L21-L29
а без этого работать не будет? https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/AuthorizationPage.java#L17 где проверки конкретно страницы авторизации?
а тут почему не используем обертки waitFor https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/AuthorizationPage.java#L22
может прикрутить какой то enum откуда доставать нужные значения в зависимости от принимаемых параметров а не дублировать код https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/GroupsPage.java#L34-L39
если не заходить в реализацию метода то неочевидно что был сделать какой то переход https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/GroupsPage.java#L53
кажется что новые и актуальные группы можно разнести на разные классы страниц групп но например сделать общего родителя с общими методами
вместо findAll по идее можно использовать $$ https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/GroupsPage.java#L62
это все полноценные шаги, не надо их прятать метод, лучше делать явно в тесте. после рефреша необходимо создавать новый экземпляр страницы потому что она могла просто не прогрузится заново https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/GroupsPage.java#L68-L72
можно еще проверять что атрибут у элемента есть перед тем как получать его shouldHave(attribute("href")); https://github.com/StombieIT/autotesting-course/blob/30578159262aa06fca5f90cc12d0c9bace51808f/src/main/java/org/ficko/pages/MusicPage.java#L62