MS
через Mockito и mock для пагинатора
т.е. можно замокать поле, которое до этого самостоятельно проинициализировалось?
Size: a a a
2018 November 28
ES
т.е. можно замокать поле, которое до этого самостоятельно проинициализировалось?
точно, пагинатор не передается как зависимость.
MS
то есть тут нужен будет рефакторинг?
ES
скорее всего да, и нужно продумать как это лучше отрефакторить.
IG
если бы пришлось написать юнит тесты на презентеры, как, к примеру, убедиться, что при вызове метода
https://gitlab.com/terrakok/gitlab-client/blob/develop/app/src/main/java/ru/terrakok/gitlabclient/presentation/mergerequest/commits/MergeRequestCommitsPresenter.kt
MergeRequestCommitsPresenter#refreshCommits дергается paginator.refresh() ?https://gitlab.com/terrakok/gitlab-client/blob/develop/app/src/main/java/ru/terrakok/gitlabclient/presentation/mergerequest/commits/MergeRequestCommitsPresenter.kt
А зачем тестировать именно это? Тут же нужно убедиться что вью стейт получил ожидаемую нами команду.
IG
Ну, я о том, какую пользу несёт тест который проверяет именно вызов refresh у пагинатора. Тестировать нужно же чёрный ящик.
MS
А зачем тестировать именно это? Тут же нужно убедиться что вью стейт получил ожидаемую нами команду.
это уже задача тестов на класс Paginator, разве нет?
IG
это уже задача тестов на класс Paginator, разве нет?
Тестировать нужно поведение сущности. А каким образом это поведение реализовано нам не должно быть важно.
Ну, я это так понимаю.
Ну, я это так понимаю.
IG
Соответственно, вызвал он рефреш у пагинатора или сразу сказал вьюстейту показать прогресс должно быть пофиг
MS
Тестировать нужно поведение сущности. А каким образом это поведение реализовано нам не должно быть важно.
Ну, я это так понимаю.
Ну, я это так понимаю.
так или иначе нужно иметь возможность предварительно настроить пагинатор для того, чтобы протестировать все краевые случаи.
не всегда ведь метод
не всегда ведь метод
refreshCommits ведёт к вызову viewState.showCommits()IG
так или иначе нужно иметь возможность предварительно настроить пагинатор для того, чтобы протестировать все краевые случаи.
не всегда ведь метод
не всегда ведь метод
refreshCommits ведёт к вызову viewState.showCommits()Это не настройка пагинатора. Это настройка источника данных для него. Поведение пагинатора детерминировано.
IG
А источник данных можно мокнуть и вернуть необходимый для теста результат
MS
Это не настройка пагинатора. Это настройка источника данных для него. Поведение пагинатора детерминировано.
согласен. вопрос снят
IG
Другой вопрос в том, нужно ли вообще тестировать презентер)
KT
Всем привет! я знаю, что там есть МР на мне, я обязательно их разберу!
я был в разъездах и другой рутине, не думайте плохо 😉
сейчас вроде выбрался и скоро опять начну заниматься гитфоксом
я был в разъездах и другой рутине, не думайте плохо 😉
сейчас вроде выбрался и скоро опять начну заниматься гитфоксом
IG
А я постараюсь на этой неделе закончить задачу с MarkdownTextView
ES
а я сегодня хочу попробовать вкрутить code_highlight
IG
продумал. план такой: надо сделать кастомную вью MarkdownTextView, у которой будет свой презентер.
Это хорошо описано в мокси вики. делается просто. мы часто используем.
у этой вью будет метод setMarkdown(text: String, projectId: Long? = null)
она будет передавать все это дело своему презентеру, а тот будет грузить лейблы, отдвать на парсинг марквону и тд
важно: это то что можно засетить новый текст, до того как распарсился старый (в ресайклере). поэтому надо учесть отписку и запуск нового процесса и тд
это хорошее архитектурное задание 😊 готовы взять?
если да, то надо отдельным МР создать это в проекте (пока без лейблов, а как отдельную задачу)
в MarkdownPresenter будет инжектиться ProjectInteractor и MarkdownConverter
главное все правильно и аккуартно приготовить))
Это хорошо описано в мокси вики. делается просто. мы часто используем.
у этой вью будет метод setMarkdown(text: String, projectId: Long? = null)
она будет передавать все это дело своему презентеру, а тот будет грузить лейблы, отдвать на парсинг марквону и тд
важно: это то что можно засетить новый текст, до того как распарсился старый (в ресайклере). поэтому надо учесть отписку и запуск нового процесса и тд
это хорошее архитектурное задание 😊 готовы взять?
если да, то надо отдельным МР создать это в проекте (пока без лейблов, а как отдельную задачу)
в MarkdownPresenter будет инжектиться ProjectInteractor и MarkdownConverter
главное все правильно и аккуартно приготовить))
Небольшой вопрос. Не до конца понял, что ты имел в виду.
>если да, то надо отдельным МР создать это в проекте (пока без лейблов, а как отдельную задачу)
это значит от develop ветвиться или же от текущего состояния ветки extend-markdown-support?
>если да, то надо отдельным МР создать это в проекте (пока без лейблов, а как отдельную задачу)
это значит от develop ветвиться или же от текущего состояния ветки extend-markdown-support?
ES
Небольшой вопрос. Не до конца понял, что ты имел в виду.
>если да, то надо отдельным МР создать это в проекте (пока без лейблов, а как отдельную задачу)
это значит от develop ветвиться или же от текущего состояния ветки extend-markdown-support?
>если да, то надо отдельным МР создать это в проекте (пока без лейблов, а как отдельную задачу)
это значит от develop ветвиться или же от текущего состояния ветки extend-markdown-support?
а тебе для реализации этой логики надо то, что ты делал до этоого?
IG
а тебе для реализации этой логики надо то, что ты делал до этоого?
Ну, вообще нет, не нужно. Логично 🙂