0

我需要测试一个存储库,该存储库具有通过构造函数注入的 Eloquent 模型。

class EloquentOrderRepository implements OrderRepositoryInterface
{

    protected $model;

    public function __construct(Order $model)
    {
        $this->model = $model;
    }

    public function calculateValues(array $deliveryOption = null)
    {
        if (! is_null($deliveryOption)) {
            $this->model->value_delivery = (float) number_format($deliveryOption['price'], 2);
        }

        $this->model->value_products = (float) number_format($this->model->products->getTotal(), 2);
        $this->model->value_total    = (float) $this->model->value_products + $this->model->value_delivery;
    }
}

我的问题是当我调用$this->model->value_products(或任何属性)时。Eloquent 模型尝试调用模拟模型setAttribute中不存在的方法。如果我模拟此方法,我将无法正确设置属性,并且我的测试断言将失败。

这是我的测试:

<?php
class EloquentOrderRepositoryTest extends \PHPUnit_Framework_TestCase
{

    protected $model, $repository;

    public function setUp()
    {
        $this->model = Mockery::mock('Order');
    }

    public function test_calculate_values()
    {
        $repository = new EloquentOrderRepository($this->model);

        $this->model->products = m::mock('SomeCollection');
        $this->model->products->shouldReceive('getTotal')->once()->withNoArgs()->andReturn(25);

        $this->model->calculateValues(array('price' => 12));
        $this->assertEquals(12, $this->model->value_delivery);
        $this->assertEquals(25, $this->model->value_products);
        $this->assertEquals(37, $this->model->value_total);
    }
}

对此有什么想法吗?

4

1 回答 1

4

我认为您的主要问题是您没有正确使用存储库模式。您应该将构造函数中传递的模型视为原型。这不是一件可以使用的真实事物,而是您用于其他事物的事物的一个实例。在存储库中,您可能有一个 getUnpaidOrders 方法,它会执行类似return $this->model->wherePaid('0')->get();. 如您所见,我们并没有将实例作为实际的具体实例进行交互,而是更多地实现更广泛的范围。

在您的计算方法中,您实际上是在此原型模型上设置值。我不知道你打算用这些做什么,但据我所知,这不是存储库模式应该做的。存储库上的方法通常是类似静态的方法,您可以在其中调用它们(可能带有一些输入)并取回一些东西。它们不应该对任何内部状态产生影响,因为存储库不应该有任何内部状态。

希望这是有道理的。

于 2014-02-13T12:46:12.303 回答