giriş
Tamam, Mockery için sağlanan bir çözüm olduğunu görüyorum, bu yüzden Mockery'den hoşlanmadığım için, size bir Prophecy alternatifi vereceğim, ancak önce Mockery ile Prophecy arasındaki farkı okumanızı öneririm .
Uzun lafın kısası : "Kehanet, mesaj bağlama adı verilen yaklaşımı kullanır - bu, yöntemin davranışının zamanla değişmediği, aksine diğer yöntem tarafından değiştirildiği anlamına gelir."
Kapsanacak gerçek dünya sorunlu kodu
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 çözümü
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);
}
}
özet
Kehanet bir kez daha harika! Benim püf noktam Prophecy'nin ileti bağlama doğasından yararlanmak ve ne yazık ki tipik bir geri arama javascript cehennem kodu gibi görünse de, $ self = $ this;Çok nadiren böyle birim testleri yazmak zorunda kaldığınız için, bunun güzel bir çözüm olduğunu düşünüyorum ve programın yürütülmesini tanımladığı için takip etmesi, hata ayıklaması kesinlikle kolay.
BTW: İkinci bir alternatif var ancak test ettiğimiz kodu değiştirmeyi gerektiriyor. Sorun çıkaranları paketleyip ayrı bir sınıfa taşıyabiliriz:
$chunk->processingInProgress();
$this->chunksStorage->updateChunk($chunk);
şu şekilde sarılabilir:
$processorChunkStorage->persistChunkToInProgress($chunk);
ve bu kadar ama onun için başka bir sınıf oluşturmak istemediğim için ilkini tercih ediyorum.