PHPUnit exactly anything

[Vano]

Можна ли как-то добиться в PHPUnit такого - ожидаю вызов нескольких методов, с неважно каким параметром, но чтобы всегда был один и тот же?
Что-то типа:

$object1->expects($this->once())->with($this->anything())
//как-то сохранить что это было в параметре
$exactlyAsPrevious = ???
//и
$object2->expects($this->once())->with($exactlyAsPrevious);

 

[Вурдалак]

https://phpunit.de/manual/6.5/en/te...oubles.mock-objects.examples.SubjectTest3.php — можно через $this->callback().

 

[whirlwind]

тест попахивает

 

[Vano]

@whirlwind а почему?
Хочу добиться от метода,к примеру, чтобы он сохранял в файл и в кеш одну и ту же константу, его же класса.
Давайте попробуем написать:

class A
{
    private const LESHA = 'Lesha';
    private const PETYA = 'Ughuwewewe';

    public function save(bool $condition)
    {
        if ($condition) {
            $this->file->wirte(self::LESHA);
            $this->cache->save('name', self::LESHA);
        } else {
            $this->file->wirte(self::PETYA);
            $this->cache->save('name', self::PETYA);
        }
    }
}

// unit
public function testWithTrueParameter()
{
    $this->file->expects($this->once())
        ->method('write')
        ->with($this->anything());

    $this->cache->expects($this->once())
        ->method('save')
        ->with($this->anything());

    $this->a->save(true);
}

Что тут неправильно?

 

[AnrDaemon]

Всё неправильно.
Откуда у тебя взялись $this->file и $this->cache ?

 

[whirlwind]

@whirlwind а почему?

Что тут неправильно?

1) Если этот тест свалится в сете из 1000 тестов по названию теста можно понять в каком методе проблема, куда смотреть?
Я бы назвал так

testSave_IfCondition
testSave_IfNotCondition

2) Есть два типа тестирования: тестирование состояния и тестирование поведения. Здесь тестирование поведения - то есть тест взаимодействия со смежными интерфейсами. Что говорит этот тест? Тест говорит: запиши что нибудь. Если не смотреть в код класса, то из теста непонятно что конкретно юнит должен делать. Условие теста нечеткое - тест теряет характер спецификации и документации - тест не очень хороший.

3) Если возникают проблема при написании теста, то проблема не в тесте, а в дизайне. Используй декоратор поверх основного стораджа и сделай в нем кэш. И первоначальная проблема просто исчезнет.

Всегда - тяжело сформулировать тест/тест большой/много фикстуры/непонятный - проблема в дизайне.

PS. Думал, думал что мне здесь не нравится. Вообще здесь смешан сервисный слой и предметная область. Самое очевидное и правильное тут имхо разделить ответственности. Атрибуты инкапсулировать в VO - этим редуцируем сложность тестирования условия. А сервисный слой разбирает VO и сохраняет. Ну а кеш поверх стораджа прозрачно для потребителей - это классика.

 

[Vano]

@whirlwind спасибо. Особенно за розвернутость ответа.
@AnrDaemon file и cache, просто образно написал.