search

Coo3n / Quick-Cheque

2 stars 0 forks source link

Ревью #9

Open eliizaveta opened 1 year ago

eliizaveta commented 1 year ago

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

Можно обернуть эти методы, чтобы не писать такие длинные цепочки каждый раз https://github.com/Coo3n/Quick-Cheque/blob/88810349380c20ad402df4ed8a2f43f02a41b4d7/app/src/androidTest/java/com/example/quick_cheque/pages/RegisterPage.kt#L38-L53

Можно вынести в базовый тестовый класс https://github.com/Coo3n/Quick-Cheque/blob/88810349380c20ad402df4ed8a2f43f02a41b4d7/app/src/androidTest/java/com/example/quick_cheque/AuthTests.kt#L17-L18

Можно использовать цепочку вызовов https://github.com/Coo3n/Quick-Cheque/blob/88810349380c20ad402df4ed8a2f43f02a41b4d7/app/src/androidTest/java/com/example/quick_cheque/AuthTests.kt#L28-L32

Текст выносим в константы

Можно было бы добавить паттерн лоадабл компонент

А проверку? Это получается не тест.. https://github.com/Coo3n/Quick-Cheque/blob/88810349380c20ad402df4ed8a2f43f02a41b4d7/app/src/androidTest/java/com/example/quick_cheque/AuthTests.kt#L41