А
о, Дмитрий
Size: a a a
2020 August 30
А
причём тут интерфейс?
Д
NullLogger - это реализация, хоть и psr'ная. если заменить реализацию на интерфейс в этих тестах (и поправить место инициализцаии контейнера), то ничего не сломается. если не ломается, то и реализация значит не нужна, интерфейс выполняет своё дело.
А
Если задача теста проверить, что логгер берётся из контейнера по тому ID, который передан в конфиг
А
там отдельный тест на проверку отправки не LoggetrInterface
А
в случае простого контейнера нет разницы, что под каким ID хранится. Нет автовайринга же
Д
в случае простого контейнера нет разницы, что под каким ID хранится. Нет автовайринга же
пока нет :)
А
какой нафиг интерфейс, если в этом контейнере и референсов то нет
Д
кстати, а по умолчанию оно не может брать LoggerInterface?
Д
фабрика та
А
по умолчанию она ничего не берёт
Д
LoggerInterface => new NullLogger
А
если укажешь LoggerInterface то возьмёт
NO
А у тебя есть публичный репозиторий с JS кодом ?
Всегда с интересом читаю не свой код
Всегда с интересом читаю не свой код
если вопрос был ко мне
А
контейнер не ограничен именами классов. Для него есть тупо id и значение под этим id. Все остальные умные контейнеры - частные случаи, принятые реализации
A
если вопрос был ко мне
в том числе)
Д
т.е. кейсы с
NullLogger::class лишние? давай удалим тогда ихА
т.е. кейсы с
NullLogger::class лишние? давай удалим тогда ихпочему? как ты пришёл к такому выводу? :)
NO
GitHub - Nex-Otaku/nginx-host-manager: Tool to manage Nginx hosts
https://github.com/Nex-Otaku/nginx-host-manager
https://github.com/Nex-Otaku/nginx-host-manager
