modelsNetworkService.loadModels(page: 1) тесты которые лезут по настоящему в сеть не очень надежны, и не особо приветствуется. лучше замокать networkService, и проверить что при замоканном failure происходит вызов кложуры с верной ошибкой, а при замоканом успехе response не пустой, и в нем есть данные
и кажется urlRequest в ModelsAPI немного громоздкий получается, возможно стоит абстрагироваться до method и parameters, и уже в абстракции делать URLRequest
Information
Rating
Does not participate
Location
Санкт-Петербург, Санкт-Петербург и область, Россия
Мне кажется что большинство примеров личная преференция, и на ревью я бы пропустил оба варианта.
public static func += (event: Event, observer: Observer)
один и тот же observer может добавится много разне понимаю зачем использовать LinkedList, Set тут выигрывает по всем параметрам.
modelsNetworkService.loadModels(page: 1)
тесты которые лезут по настоящему в сеть не очень надежны, и не особо приветствуется. лучше замокать networkService, и проверить что при замоканном failure происходит вызов кложуры с верной ошибкой, а при замоканом успехе response не пустой, и в нем есть данныеи кажется urlRequest в ModelsAPI немного громоздкий получается, возможно стоит абстрагироваться до method и parameters, и уже в абстракции делать URLRequest