search

vspochernin / autotests

Домашние задания по дисциплине "Автоматизированное тестирование" Технополис ВК в Политехе 2022-2023 уч. год.
0 stars 0 forks source link

Ревью 2 #2

Open eliizaveta opened 1 year ago

eliizaveta commented 1 year ago

я в тамтам https://tt.me/riinj

Можно сделать обертки, чтобы не писать такие длинные строчки каждый раз, и передавать в метод ViewAssertion параметр https://github.com/vspochernin/autotests/blob/d25a76af69dd021e6015bfad2a9027a99e30c2fa/app/src/androidTest/java/company/vk/polis/autotests/pages/MainPage.java#L32

Выносить в константы https://github.com/vspochernin/autotests/blob/d25a76af69dd021e6015bfad2a9027a99e30c2fa/app/src/androidTest/java/company/vk/polis/autotests/pages/MainActivityTest.java#L35

Кажется что выбор активити стандартен для всех тестов и можно было бы делать это скорее в базовом тестовом классе, а не базовой странице

https://github.com/vspochernin/autotests/blob/d25a76af69dd021e6015bfad2a9027a99e30c2fa/app/src/androidTest/java/company/vk/polis/autotests/pages/BasePage.java#L20-L22

А в базовую страницу можно добавить паттерн лоадабл компонента

eliizaveta commented 1 year ago

Немного страшно соединять андроид и веб проекты в едином репозитории Все таки лучше разделять на разные, а работающую версию приложения хранить в мастер ветке