вступление
Хорошо, я вижу, что есть одно решение для Mockery, поэтому, поскольку мне не нравится Mockery, я собираюсь дать вам альтернативу Prophecy, но я предлагаю вам сначала прочитать о разнице между Mockery и Prophecy.
Короче говоря : «Prophecy использует подход, называемый привязкой сообщения - это означает, что поведение метода не меняется со временем, а скорее изменяется другим методом».
Реальный проблемный код для покрытия
class Processor
{
/**
* @var MutatorResolver
*/
private $mutatorResolver;
/**
* @var ChunksStorage
*/
private $chunksStorage;
/**
* @param MutatorResolver $mutatorResolver
* @param ChunksStorage $chunksStorage
*/
public function __construct(MutatorResolver $mutatorResolver, ChunksStorage $chunksStorage)
{
$this->mutatorResolver = $mutatorResolver;
$this->chunksStorage = $chunksStorage;
}
/**
* @param Chunk $chunk
*
* @return bool
*/
public function process(Chunk $chunk): bool
{
$mutator = $this->mutatorResolver->resolve($chunk);
try {
$chunk->processingInProgress();
$this->chunksStorage->updateChunk($chunk);
$mutator->mutate($chunk);
$chunk->processingAccepted();
$this->chunksStorage->updateChunk($chunk);
}
catch (UnableToMutateChunkException $exception) {
$chunk->processingRejected();
$this->chunksStorage->updateChunk($chunk);
// Log the exception, maybe together with Chunk insert them into PostProcessing Queue
}
return false;
}
}
Решение PhpUnit Prophecy
class ProcessorTest extends ChunkTestCase
{
/**
* @var Processor
*/
private $processor;
/**
* @var MutatorResolver|ObjectProphecy
*/
private $mutatorResolverProphecy;
/**
* @var ChunksStorage|ObjectProphecy
*/
private $chunkStorage;
public function setUp()
{
$this->mutatorResolverProphecy = $this->prophesize(MutatorResolver::class);
$this->chunkStorage = $this->prophesize(ChunksStorage::class);
$this->processor = new Processor(
$this->mutatorResolverProphecy->reveal(),
$this->chunkStorage->reveal()
);
}
public function testProcessShouldPersistChunkInCorrectStatusBeforeAndAfterTheMutateOperation()
{
$self = $this;
// Chunk is always passed with ACK_BY_QUEUE status to process()
$chunk = $this->createChunk();
$chunk->ackByQueue();
$campaignMutatorMock = $self->prophesize(CampaignMutator::class);
$campaignMutatorMock
->mutate($chunk)
->shouldBeCalled();
$this->mutatorResolverProphecy
->resolve($chunk)
->shouldBeCalled()
->willReturn($campaignMutatorMock->reveal());
$this->chunkStorage
->updateChunk($chunk)
->shouldBeCalled()
->will(
function($args) use ($self) {
$chunk = $args[0];
$self->assertTrue($chunk->status() === Chunk::STATUS_PROCESSING_IN_PROGRESS);
$self->chunkStorage
->updateChunk($chunk)
->shouldBeCalled()
->will(
function($args) use ($self) {
$chunk = $args[0];
$self->assertTrue($chunk->status() === Chunk::STATUS_PROCESSING_UPLOAD_ACCEPTED);
return true;
}
);
return true;
}
);
$this->processor->process($chunk);
}
}
Резюме
Еще раз, Пророчество еще круче! Моя уловка состоит в том, чтобы использовать природу связывания сообщений в Prophecy, и хотя это, к сожалению, выглядит как типичный адский код обратного вызова javascript, начинающийся с $ self = $ this; поскольку вам очень редко приходится писать подобные модульные тесты, я думаю, что это хорошее решение, и за ним, безусловно, легко следить, отлаживать, поскольку оно фактически описывает выполнение программы.
BTW: есть вторая альтернатива, но она требует изменения кода, который мы тестируем. Мы могли бы обернуть смутьянов и вынести их в отдельный класс:
$chunk->processingInProgress();
$this->chunksStorage->updateChunk($chunk);
можно обернуть как:
$processorChunkStorage->persistChunkToInProgress($chunk);
и все, но поскольку я не хотел создавать для него еще один класс, я предпочитаю первый.