Если делаются тесты, то ставить проверки чтоб следить за программистом лишнее. Проверка ставятся на действия пользователя, или в местах где программа может повести себя по разному. А не для каждого метода, что он там возвращает.
Если программист следит за тем чтоб не изменять интерфейс классов, зная что one() возвращает либо модель либо null и ничего не может больше вернуть, опять же проверка лишняя. Беда в том что в Yii2, действительно, методы могут возвращать разные форматы данных, в случая с one() например, это может быть модель или массив. Так что нужно быть внимательно при программировании.
И напоследок мне больше нравится проверять так
тем самым мы даже подсказываем IDE что в переменной и помогаем с автодополнением.