Мокайте только один метод на заглушках PHPSpec

Теперь это работает, тест зеленый. Но это кажется неправильным... Я что-то упустил или это ограничение PHPSpec?

Я думаю, это хорошо, что в этом случае это кажется неправильным, потому что так и должно быть. Как упоминалось выше @ l3l0, PHPSpec — это инструмент дизайна, и он дает вам четкое представление о вашем дизайне здесь.

С чем вы боретесь, так это с тем, что ваш Cart нарушает принцип единой ответственности — он делает больше, чем одну вещь — он управляет CartItems, а также знает, как генерировать из него RowId. Поскольку PHPSpec заставляет вас заглушить все поведение CartItem, это дает вам сообщение о необходимости реорганизации генерации RowId.

Теперь представьте, что вы извлекли RowIdGenerator в отдельный класс (с его собственными спецификациями, которые здесь не рассматриваются):

class RowIdGenerator
{
    public function fromCartItem(CartItem $item)
    {
        return md5($item->getId() . serialize($item->getOptions()));
    }
}

Затем вы вводите этот генератор через конструктор как зависимость от вашей корзины:

class Cart
{
    private $rowIdGenerator;

    public function __construct(RowIdGenerator $rowIdGenerator)
    {
        $this->rowIdGenerator = $rowIdGenerator;
    }
}

Тогда ваша окончательная спецификация может выглядеть так:

function let(RowIdGenerator $rowIdGenerator)
{
    $this->beConstructedWith($rowIdGenerator);
}

public function it_can_add_multiple_instances_of_a_cart_item(RowIdGenerator $rowIdGenerator, CartItem $cartItem1, CartItem $cartItem2)
{
    $rowIdGenerator->fromCartItem($cartItem1)->willReturn('abc');
    $rowIdGenerator->fromCartItem($cartItem1)->willReturn('def');

    $this->add($cartItem1);
    $this->add($cartItem2);

    $this->shouldHaveCount(2);
}

И поскольку вы издевались над поведением генератора идентификаторов (и вы знаете, что это общение должно произойти), теперь вы соответствуете SRP. Тебе сейчас лучше?

Итак, вы идете в ресторан, чтобы поужинать. Вы ожидаете, что вам будет предоставлен выбор еды, из которой вы выберете то, что вам действительно интересно съесть сегодня, и будете платить за это в конце ночи. Чего вы не ожидаете, так это того, что ресторан также возьмет с вас плату за прекрасную пару рядом с вами, заказавшую бутылку за бутылкой Chteau Margaux 95. Поэтому, когда вы обнаружите, что вы были strong> взимают плату за их еду, вы, вероятно, захотите немедленно позвонить в этот ресторан и в свой банк, потому что это совершенно не нормально, что это произошло без вашего ожидания!

Вопрос не в том, почему PhpSpec заставляет вас заглушать методы, которые вам сейчас не нужны. Вопрос в том, почему вы вызываете методы, которые вам сейчас не нужны. Если они не входят в ваши ожидания, PhpSpec просто позвонит в ваш банк, потому что это совершенно не нормально, что это произошло без вашего ожидания!

Да, вы можете назвать это «ограничением» phpspec. По сути, phpspec — это строгий TDD и инструмент проектирования объектной связи IMO.

Вы видите, что добавление $cartItem в коллекцию делает гораздо больше, чем вы ожидаете.

Во-первых, вам не нужно использовать заглушки (если вам не нужна связь между внутренними объектами), пример:

function it_adds_multiple_instances_of_a_cart_item()
{
    $this->add(new CartItem($id = 1, $options = ['size' => 1]));
    $this->add(new CartItem($id = 2, $options = ['size' => 2]));

    $this->shouldHaveCount(2);
}

function it_adds_two_same_items_with_different_sizes()
{
    $this->add(new CartItem($id = 1, $options = ['size' => 1]));
    $this->add(new CartItem($id = 1, $options = ['size' => 2]));

    $this->shouldHaveCount(2);   
}

function it_does_not_add_same_items()
{
    $this->add(new CartItem($id = 1, $options = []));
    $this->add(new CartItem($id = 1, $options = []));

    $this->shouldHaveCount(1);   
}

Вы можете сделать это и по-другому. С точки зрения связи, запрос много раз одного и того же экземпляра объекта не так эффективен. Многие общедоступные методы означают множество различных комбинаций. Вы можете спланировать общение и сделать что-то вроде этого:

function it_adds_multiple_instances_of_a_cart_item(CartItem $cartItem1, CartItem $cartItem2)
{
   $this->add($cartItem1);
   $cartItem1->isSameAs($cartItem2)->willReturn(false);
   $this->add($cartItem2);

   $this->shouldHaveCount(2);
}

function it_does_not_add_same_items((CartItem $cartItem1, CartItem $cartItem2)
{
    $this->add($cartItem1);
    $cartItem1->isSameAs($cartItem2)->willReturn(true);
    $this->add($cartItem2);

    $this->shouldHaveCount(1);   
}