eliizaveta
commented
1 year ago По исправлениям
Мало просто добавить логгер, хотелось бы использовать логи https://github.com/ted-bear/AutoTesting/blob/46800effa6f58c9304b76c535dccd623be0c65f9/src/test/java/homework1/GroupPage.java#L14
Тут все еще остался 4й джаваюнит https://github.com/ted-bear/AutoTesting/blob/46800effa6f58c9304b76c535dccd623be0c65f9/src/test/java/homework1/GroupPage.java#L11
Локаторы в константы https://github.com/ted-bear/AutoTesting/blob/46800effa6f58c9304b76c535dccd623be0c65f9/src/test/java/homework1/GroupPage.java#L18
Что в итоге с методами из Loadable Component, в чем разница и запускаются ли они? https://github.com/ted-bear/AutoTesting/blob/46800effa6f58c9304b76c535dccd623be0c65f9/src/test/java/homework1/OkGroupsPage.java#L32-L40
Я в тамтам https://tt.me/riinj
Предлагаю все методы превратить в дейтсвия и переименовать, чтобы не заходя в метод мы могли понять, что он такое делает. Сейчас больше похоже, что мы получаем какие-то пароли и логины, а не поля. Селенид-элементы мы и так можем хранить в константах, например, для чего нам методы их получения? Локаторы в константы выносим. https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/OkLoginPage.java#L8-L17
Логирование это круто, предлагаю использовать везде https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/ColumnMenuItemWrapper.java#L10
А ты доверяешь этому локатору? https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/ColumnMenuItemWrapper.java#L11
Еще хорошая практика добавлять сообщение visible.because("Не нашли картинку") https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/ColumnMenuItemWrapper.java#L18-L20
Мы тестируем гугл? https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/GoogleTest.java#L15
У нас все тестовые классы все тестовые методы начинаются с перехода по ссылке на портал, предлагаю сделать базовый класс, там поместить открытие страницы ОК, и наследовать от этого класса все тесты
Название результаты соответствует странице логина? Данные в константы https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/GoogleTest.java#L22-L25
Что будет если тест не пройдет на этапе логина? https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/GoogleTest.java#L30-L33
Все проверки делаем через Assertions, текст можно получать с помощью getText() метода у элемента https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/GoogleTest.java#L37
Локаторов в тесте быть не должно, собирать элементы в обертки следует внутри пейджа, на котором они находятся https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/GoogleTest.java#L63
Добавляем сообщения в ассерты на случай неудачи https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/GoogleTest.java#L66
Методы используются? В чем разница между load и isLoaded? Нужны ли нам оба метода? https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/GroupPage.java#L18-L26
Лучше использовать 5й джаваюнит https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/GroupPage.java#L9
До взаимодействия с элементами необходимо проверить, что они есть на странице https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/OkGroupsPage.java#L18
Из названий непонятна разница между OkGroupsPage и GroupPage
Константы большими буквами https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/OkProfilePage.java#L14-L17
Можно в одну строку, локаторы в константы, локаторы сократить, проверять на видимость элементы https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/OkProfilePage.java#L38-L43
В тестах используем проверки через ассерты, в пейджах лучше через shouldBe и др https://github.com/ted-bear/AutoTesting/blob/df94940b83aa37a6a2d36ab99f81ce09bcdd3dc4/src/test/java/homework1/OkProfilePage.java#L53