Intro
Ok, vedo che c'è una soluzione fornita per Mockery, quindi dato che non mi piace Mockery, ti offrirò un'alternativa a Prophecy ma ti suggerirei prima di leggere prima la differenza tra Mockery e Prophecy.
Per farla breve : "La profezia utilizza un approccio chiamato binding dei messaggi - significa che il comportamento del metodo non cambia nel tempo, ma piuttosto viene modificato dall'altro metodo".
Codice problematico del mondo reale da coprire
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;
}
}
Soluzione 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);
}
}
Sommario
Ancora una volta, Prophecy è più fantastico! Il mio trucco è sfruttare la natura vincolante dei messaggi di Prophecy e anche se purtroppo sembra un tipico codice javascript di callback, che inizia con $ self = $ this; dato che molto raramente devi scrivere unit test come questo, penso che sia una buona soluzione ed è sicuramente facile da seguire, eseguire il debug, poiché in realtà descrive l'esecuzione del programma.
BTW: C'è una seconda alternativa ma richiede la modifica del codice che stiamo testando. Potremmo avvolgere i piantagrane e spostarli in una classe separata:
$chunk->processingInProgress();
$this->chunksStorage->updateChunk($chunk);
potrebbe essere avvolto come:
$processorChunkStorage->persistChunkToInProgress($chunk);
e basta, ma poiché non volevo creare un'altra classe per questo, preferisco la prima.