Комментарии 4
modelsNetworkService.loadModels(page: 1)
тесты которые лезут по настоящему в сеть не очень надежны, и не особо приветствуется. лучше замокать networkService, и проверить что при замоканном failure происходит вызов кложуры с верной ошибкой, а при замоканом успехе response не пустой, и в нем есть данные
и кажется urlRequest в ModelsAPI немного громоздкий получается, возможно стоит абстрагироваться до method и parameters, и уже в абстракции делать URLRequest
+1
А почему Вы в loadModels в completion-блоке не используете Result?
И Вы используете Combine? Не слежу просто за реактивными фреймворками.
Еще guard let strongSelf = self else { return } можно много где повыкидывать, т.к. все равно с nil работаете и выдаете
0
Зарегистрируйтесь на Хабре, чтобы оставить комментарий
Опыт внедрения многомодульности в (еще) не разросшееся приложение (Часть 1)