myaumura / vk-java-sdet Goto Github PK

Привет!

1. ф-ии clickElement, checkConditions, checkCollection - это не сильно хороший подход. Во первых это очень не явно. Каждый элемент страницы может требовать своего подхода, также вы можете создавать свои проверки на основе имеющихся https://selenide.gitbooks.io/user-guide/content/ru/selenide-api/condition.html . На самом деле нет большого смысла для одного элемента проверять и присутствие в доме и видимость, часто нужно или то, или другое.
2. Для OKBasePage вижу checkPage(в котором ничего?), а для остальных ?
3. Конфигурирование браузера (setUp) обязательно нужно делать перед каждым тестом или можно перед всеми?
4. Хочется усложнить проект. Больше Пейджей, более сложные тест кейсы. И попробовать другие аннотации junit @nested, @ParameterizedTest или @testfactory (на выбор) + DisplayName, Tag
5. Тесты должны работать и быть зелеными, причем при запуске в любом порядке
6. Функции типа checkToolBarAccountItems которая возвращает кол-во ?? Немного странно. Лучше бы было если getCountToolBarItems()
7. $(LOCATOR).shouldBe(exist); проверки хотелось бы еще логировать
8. Литералы можно записывать в константы. Это удобнее, потому что по неймингу можно понять что значит та или иная константа и можно централизованно ее менять
9. OKHomePage homePage = new OKHomePage(); создавайте объекты pages непосредственно в местах их использования
10. Selenide.open(Configuration.baseUrl); лучше делать в base классе