Ребята, всем привет, начал писать тесты под свой код, и

Question

Ребята, всем привет, начал писать тесты под свой код, и

не могу разобратся что можно сделать с этим классом https://pastebin.com/7zD6Rj8S.

Первое, что мене тормозит это приватный метод getPlayStrategy, которые возвращает внешний обьект. Замокать я ведь не могу, так как метод приватный. Нужно ли мне тогда его делать как протектед(хотя никаких дочерних нету, которым нужно было бы его вызывать)?

Второе, если есть дочерние классы, которые создаются и вызываются в конструкторе( к примеру MatchesPlanner). Они записываются в приватный проперти. Как мне это тоже глушить, я ведь не хочу тестировать и вообше запускать MatchesPlanner, так как он внешний

#programming #russian

0

20.02.2022

9 ответов

16 просмотров

Vitaliy Sheverov Автор вопроса

Pavel Chebotarev
Если в конструкторе создается объект то никак ты е...

фабрика достойный вариант

0

20.02.2022

Vitaliy Sheverov Автор вопроса

Pavel Chebotarev
Если в конструкторе создается объект то никак ты е...

Подскажите, пожалуйста, в этому коде LeaguePlayedEvent это ведь не внешння зависимось верно, где мне лучше всего обьявить этот класс? public function play(string $type = 'week'): array { $playStrategy = $this->playStrategyResolver->resolve($type); [ 'matches' => $played_matches, 'week' => $this->current_week ] = $playStrategy->play($this->matches_per_week, $this->current_week, $this->matches); LeaguePlayedEvent::dispatch($this); return $played_matches; }

0

20.02.2022

Pavel Chebotarev

Vitaliy Sheverov
Подскажите, пожалуйста, в этому коде LeaguePlayedE...

Какой класс? LeaguePlayedEvent ?

0

20.02.2022

Vitaliy Sheverov Автор вопроса

Pavel Chebotarev
Какой класс? LeaguePlayedEvent ?

да, или я недоконца понимаю что есть, а что не есть внешней зависимостью

0

20.02.2022

Pavel Chebotarev

Vitaliy Sheverov
да, или я недоконца понимаю что есть, а что не ест...

Не очень понимаю используемый термин, иногда зависимостями называются пакеты из композера ) Но вообще внешней зависимостью называются по факту все сервисы которые передаются в конструктор. Код внутри класс можно вынести во внешнюю зависимость - сделать класс с этим кодом. Тут надо создать LeaguePlayedEventDispatcher, положить его куда-нибудь в Common папку или поближе ко всяким конкретным имплементированным техническим штукам, передавать в конструктор как сервис, и уже вызывать $this->leaguePlayedEvent->dispatch($this);

0

20.02.2022

Vitaliy Sheverov Автор вопроса

Pavel Chebotarev
Не очень понимаю используемый термин, иногда завис...

понял, спасибо, заверну в диспатчер и передам через конструктор. Меня смущает, то, что внешний класс, который вызывает мой класс должен знать, обо всех его классах(которые он использует). получается, вот такое new League($teams, new PlayStrategyResolver, $this->matchesPlannerFactory, new LeaguePlayedEventDispatcher, $matches_per_week); Поскажите, это нормально?

0

20.02.2022

Pavel Chebotarev

Vitaliy Sheverov
понял, спасибо, заверну в диспатчер и передам чере...

Да норм, внешний класс который это делает должен называться LeagueFactory и быть фабрикой )

0

20.02.2022

Vitaliy Sheverov Автор вопроса

Pavel Chebotarev
Да норм, внешний класс который это делает должен н...

все, что создает запихать в фабрику. Делаем Сколько же моментом опущено, спасибо

0

20.02.2022

Pavel Chebotarev · Accepted Answer

Pavel Chebotarev

Если в конструкторе создается объект то никак ты его не замокаешь, придется с ним тестировать :) Либо опять же длинно отрефакторить и передавать в конструктор фабрику MatchesPlannerFactory

0

20.02.2022

169 похожих чатов

Ребята, всем привет, начал писать тесты под свой код, и

9 ответов

Похожие вопросы