Ну может типы где то не проставлены, или может где то final стоит использовать. Такие штуки косметические

Но лучше сначала тесты

👍

Тесты в любом случае помогут.

Но вообще arrays в плане релиза не прост.

Он мне не нравится в плане того, что там есть и как оно оформлено.

То есть там надо смотреть ещё на логичность набора фич, дубли, слишком сложные методы с флагами.

А стоит ли тогда с тестами сейчас морочаться, если большая вероятность переделки?

Конечно

Проще рефачить с тестами

понял

Подскажите, пожалуйста, как правильно сформировать файл aliases.php. У меня некоторые тесты db-migration не взлетают так-же как и в github actions. Пишет ошибку "yiisoft/composer-config-plugin-output/aliases.php): failed to open stream: No such file or directory"

Создаю этот файл, сыпется другая ошибка "Invalid path alias: @Yiisoft/Yii/Db/Migration/Tests/Build". Заполняю файл aliases массивом и всё взлетает:
[
   '@Yiisoft/Yii/Db/Migration/Tests/Build' => '/mnt/hdd-raid/PhpstormProjects/yii-dev-tool/dev/db-migration/tests/Build',
]

Как сгенерировать файл aliases.php правильно, а не так костыльно?

Вот эти ошибка в actions https://github.com/yiisoft/yii-db-migration/runs/1053181287?check_suite_focus=true#step:11:221

@Fantom409 https://github.com/yiisoft/yii-demo/pull/132 можно смотреть?

Интересно. Надо посмотреть что там не так...

Я гляну, постараюсь сегодня.

Хорошо, спасибо

Да

ArrayHelper::getValue($object, 'key', 'defaultValue')
Какое ожидаемое поведение в случае, если объект не возвращает свойство:
1) Exception
2) defaultValue

?

composer-plugin generates it automatically, a solution without using the composer-plugin would be to configure the aliases in TestCase manually.