PHPUnit "zagnieżdżone" mocki Opcje

[Lion]

Sprawdzam sobie działanie mocków w PHPUnit 5.2.3. Mam pewien problem z "zagnieżdżonymi" mockami, kod wygląda tak:

[PHP] pobierz, plaintext 
 
interface EntityInterface
{
 
    public function setAncestor(EntityInterface $parent);
 
    public function getAncestor();
 
    public function addDescendant(EntityInterface $descendant);
 
    public function getName();
 
}
 
abstract class Entity implements EntityInterface
{
 
    protected $name;
 
    /** @var EntityInterface Ancestor */
    protected $ancestor = null;
 
    /** @var EntityInterface Descendants */
    protected $descendants = array();
 
    public function __construct($name)
    {
        $this->name = $name;
    }
 
    public function getAncestor()
    {
        return $this->ancestor;
    }
 
    public function setAncestor(EntityInterface $ancestor)
    {
        $this->ancestor = $ancestor;
    }
 
    public function addDescendant(EntityInterface $descendant)
    {
        $descendant->setAncestor($this);
        $this->descendants[$descendant->getName()] = $descendant;
    }
 
    public function getName()
    {
        return $this->name;
    }
 
}
 
[PHP] pobierz, plaintext 

Test wygląda tak:

[PHP] pobierz, plaintext 
 
class EntityTest extends PHPUnit_Framework_TestCase
{
 
    public function testEntityDescendantCanBeSet()
    {
        $entity = $this->getMockBuilder(Entity::class)
            ->setMethods(array('addDescendant'))
            ->disableOriginalConstructor()
            ->getMock();
        $descendant_entity = $this->getMockBuilder(Entity::class)
            ->setMethods(array('getName', 'setAncestor'))
            ->disableOriginalConstructor()
            ->getMock();
        $entity->expects($this->once())
            ->method('addDescendant');
        $descendant_entity->expects($this->once())
            ->method('setAncestor');
        $descendant_entity->expects($this->once())
            ->method('getName')
            ->will($this->returnValue('entity_name'));
 
        $entity->addDescendant($descendant_entity);
    }
 
}
 
[PHP] pobierz, plaintext 

Dostaję taki komunikat:

1) EntityTest::testEntityDescendantCanBeSet
Expectation failed for method name is equal to <string:setAncestor> when invoked 1 time(s).
Method was expected to be called 1 times, actually called 0 times.

Podobny komunikat jest dla getName jeśli test setAncestor jest zakomentowany. Co jest nie tak i jak to można przetestować?

[Crozin]

Utworzyłeś mocka, więc wywołanie metody addDescendant nie robi nic - nie wywołuje Twojego kodu. I tutaj należy zauważyć, że źle podchodzisz do tematu:

1. W miarę możliwości staraj się unikać tworzenia mocków.
2. Skoro testujesz tutaj klasę Entity, a jest on abstrakcyjna utwórz sobie nic nie robiącą jej implementację (class DummyEntity extends Entity { }) i korzystaj z jej obiektów.
3. Testuj kontrakt klasy, a nie jej implementację. Czasami faktycznie istotne jest to ile razy dana metoda została wywołana itp. itd., ale tutaj to nie ma znaczenia. To co w tym kodzie jest istotne to to czy:
3.1. Konstruktor faktycznie ustawia nazwę klasy zwracaną przez getName().

[PHP] pobierz, plaintext 
// arrange & act
$entity = new DummyEntity('name');
 
// assert
$this->assertEquals('name', $entity->getName());
[PHP] pobierz, plaintext 

3.2. addDescendant() dodaje obiekt jako dziecko i ustawia temu dziecku rodzica na siebie samego.

[PHP] pobierz, plaintext 
// arrange
$parent = new DummyEntity('parent');
$child = new DummyEntity('child');
 
// act
$parent->addDescedant($child);
 
// assert
$this->assertCount(1, $parent->getDescendants());
$this->assertArrayHasKey('child', $parent->getDescendants());
$this->assertSame($parent, $child->getParent());
[PHP] pobierz, plaintext 

Co Cię interesuje ile razy getName() było wywołane? Jakie to ma znaczenie tutaj?

[lukaskolista]

W php7 z pomocą przychodzą klasy anonimowe, np.

[PHP] pobierz, plaintext 
abstract class AbstractEntityAbc {}
 
class AbstractEntityAbcTest
{
    public function testFunctionX()
    {
        $abc = new class('arg1', 'arg2') extends AbstractEntityAbc {/* Tutaj możesz nadpisać metody lub zaimplementować metody abstrakcyjne */};
        $this->assertAbc($abc->functionX());
    }
}
[PHP] pobierz, plaintext 

Edit:
To samo dotyczy klas nieabstrakcyjnych, zamiast tworzyć mocki możesz tworzyć klasy anonimowe, które po nich dziedziczą.

Ten post edytował lukaskolista 19.02.2016, 08:23:40