[3/22/2017 1:35:43 PM] Victor Kolkovski .Net dev: Guten Tag!

Все сделал. Тесты готовы.
Пару моментов:
1. В связи с тем, что до текущего момента в develop не было функционала создать аккаунт, я искусственно вставил похожий код у себя. К вечеру поменяю и решу конфликты. Но все же прошу глянуть, плиз, как время будет, то что получилось. Глобально мало поменяется, после это замены.
2. По тестам:
  - Тест контроллера: замокать провайдер не получается, так как все, что могу замокать - это то что на выхлопе работы методов провайдера. Получается не айс.
Сделал интеграционный с проверкой внесения коллекции данных.
  - Тест провайдера: все работает, единственный момент, где нужно получить физический путь к файлу. Я решил добавить дополнительно файл client_secret.json в проект BL теста. Так как искать этот файл в проекте Profile.UI и создавать путь к этому проекту, как по мне не очень хорошая идея.
[3/22/2017 1:37:23 PM] Alexei Skiba: замокать провайдер не получается, так как все, что могу замокать - это то что на выхлопе работы методов провайдера. Получается не айс.так мокирование и есть "подставить то что на выхлопе работы методов"
[3/22/2017 1:37:33 PM] Alexei Skiba: а ты что пытался сделать?
[3/22/2017 1:38:32 PM] Alexei Skiba: Так как искать этот файл в проекте Profile.UI и создавать путь к этому проекту, как по мне не очень хорошая идея.ага, особенно из BL-теста
[3/22/2017 1:42:53 PM] Victor Kolkovski .Net dev: В провайдере можно замокать то, что придет от метода GetAccountsFromSpreadsheet. + то, что придет от метода CreateNewUser. При этом смысл теста исчезает.
А интеграционный получился вроде как норм.
[3/22/2017 1:47:34 PM | Edited 1:47:37 PM] Alexei Skiba: ничего не имею против интеграционного метода, но надо чтобы ты понял смысл тестирования
[3/22/2017 1:48:06 PM | Edited 1:50:51 PM] Alexei Skiba: если тестируешь контроллер и мокаешь для этого провайдер - всё, не нужно "в провайдере" ничего мокать
[3/22/2017 1:48:19 PM] Alexei Skiba: и что там могут вернуть другие методы - тоже становится не важно
[3/22/2017 1:48:41 PM] Alexei Skiba: с этим вроде проблем особых нет
[3/22/2017 1:49:07 PM] Alexei Skiba: но то, что ты не видишь смысла в тестировании контроллера - проблема
[3/22/2017 1:50:00 PM] Alexei Skiba: даже если бы там был вызов одного-единственного метода провайдера - юнит-тест имеет смысл. потому что ты должен быть уверен, что контроллер дергает то что нужно и возвращает то что положено
[3/22/2017 1:50:27 PM] Alexei Skiba: независимо от того, откуда это "положенное" взялось - из базы, гугла или было захардкожено в моках
[3/22/2017 5:06:26 PM] Victor Kolkovski .Net dev: Я не говорил, что нет смысла тестировать контроллер, я имел ввиду то, что у меня получилось в unit test контроллера выглядело не очень. И в этом случае более адекватно выглядит интеграционный тест.
Возможно стоило иначе модульный тест написать, подумаю. Вопрос тут у меня остался.
[3/22/2017 5:07:17 PM] Alexei Skiba: тебе не нравилось, что не проверяется никакой код, кроме кода контроллера, так ведь?
[3/22/2017 6:31:21 PM] Victor Kolkovski .Net dev: Мне не понравилось то, что мокаем выходные данные метода GetAccountsFromSpreadsheet, а потом выходные данные метода CreateNewUser, при этом связи между двумя выходными данными нет.
Понял, к чему ты, при тесте контроллера мы должны проверить только логику контроллера.
[3/22/2017 6:35:06 PM] Alexei Skiba: при этом связи между двумя выходными данными нет.почему нет?
[3/22/2017 6:35:15 PM] Alexei Skiba: насколько я понимаю, там должны мокаться один и те же данные
[3/22/2017 6:35:33 PM] Alexei Skiba: это конечно не особо принципиально, но намного нагляднее
[3/22/2017 6:40:03 PM] Victor Kolkovski .Net dev: позже восстановлю тот код и покажу, что мне не понравилось... я больше склоняюсь к тому, что мне следовало иначе написать тот тест
[3/22/2017 6:46:15 PM] Alexei Skiba: помнишь как девелоп сливать? сначала в стори, потом в таск
[3/22/2017 6:52:06 PM] Victor Kolkovski .Net dev: да, конечно, сча конфликты быстро решу, и залью изменения
[3/22/2017 6:54:18 PM] Victor Kolkovski .Net dev: тебе мои сообщения доходят? чет скайп тупит
[3/22/2017 6:55:13 PM] Alexei Skiba: я там еще пару комментов повесил
[3/22/2017 6:57:55 PM | Edited 6:57:58 PM] Alexei Skiba: тупит вообще по жести, но доходят
[3/22/2017 7:11:43 PM] Victor Kolkovski .Net dev: По ходу, не доходят некоторые сообщения... Что там не принципиально?
[3/22/2017 7:13:50 PM] Alexei Skiba: почему нет?
насколько я понимаю, там должны мокаться один и те же данные
это конечно не особо принципиально, но намного нагляднее
[3/22/2017 7:30:10 PM] Victor Kolkovski .Net dev: не доходят от тебя сообщения. в телефоне уведомления приходят, но в приложении их не видно
[3/22/2017 9:49:34 PM] Victor Kolkovski .Net dev: check...check
[3/22/2017 11:56:00 PM] Victor Kolkovski .Net dev: исправил