Open eliizaveta opened 7 months ago
Берем из 5го джаваюнита https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/BaseTest.java#L2-L3
вроде селенид это итак делает, почитай, могу ошибаться https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/BaseTest.java#L28
предлагаю все страницы наследовать от базовой страницы, где сделать метод чек, почитай про паттерн LoadableComponent https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/InterestsPage.java#L7
есть в базовом тесте https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginPage.java#L19
зачем $ если это итак уже селенид элемент, проверку на видимость и действия можно заинлайнить https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginPage.java#L23
можно вернуть страницу на которую попадаем https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginPage.java#L34
!!!!!!! тесты выносим в отдельный пакет от страниц
паттерн цепочка вызовов https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginTest.java#L25-L28
в базовый тест, зачем дублировать?? https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginTest.java#L23
убрать в один параметрайзд тест, ознакомься с другими источниками параметров https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginTest.java#L35-L87
не относятся к логину, вынести в другой класс https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginTest.java#L90-L108
тест кейсы расписать по правилам составления тест кейса, ожидаемые результаты в коде проверять после каждого шага
https://practicum.yandex.ru/blog/chto-takoe-test-keys-i-kak-ego-sostavit/
eliizaveta
commented
7 months ago eliizaveta
commented
7 months ago
Берем из 5го джаваюнита https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/BaseTest.java#L2-L3
вроде селенид это итак делает, почитай, могу ошибаться https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/BaseTest.java#L28
предлагаю все страницы наследовать от базовой страницы, где сделать метод чек, почитай про паттерн LoadableComponent https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/InterestsPage.java#L7
есть в базовом тесте https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginPage.java#L19
зачем $ если это итак уже селенид элемент, проверку на видимость и действия можно заинлайнить https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginPage.java#L23
можно вернуть страницу на которую попадаем https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginPage.java#L34
!!!!!!! тесты выносим в отдельный пакет от страниц
паттерн цепочка вызовов https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginTest.java#L25-L28
в базовый тест, зачем дублировать?? https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginTest.java#L23
убрать в один параметрайзд тест, ознакомься с другими источниками параметров https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginTest.java#L35-L87
не относятся к логину, вынести в другой класс https://github.com/trueuser3/web_automation/blob/9d05b1d157af5ee98832a6eca6959039739c3121/src/test/java/LoginTest.java#L90-L108
тест кейсы расписать по правилам составления тест кейса, ожидаемые результаты в коде проверять после каждого шага