Question

我是第一次与嘲讽/ Mockery搏斗，我不确定以下测试是否真的触及了我的代码，或者只是测试我做过的模拟？此外，我意识到这个代码并不适合存储库模式，尽管它的名称就是这样。我将继续这样做。

班级：

<?php namespace Acme\Cart\Repositories;

class EloquentCartRepository{
    protected $model_name = 'CartModel';
    protected $model;
    public function __construct($model = null)
    {
        $this->model = is_null($model) ? new $this->model_name : $model;
    }

    public function create_visitor_cart($session_id,$type = 'main'){
        return $this->create('visitor',$session_id,$type);
    }
    protected function create($user_type = null,$user_identifier = null,$type = 'main')
    {
        if(is_null($user_identifier)) throw new \Exception('Cannot create create cart, missing user identifier');
        if(is_null($user_type)) throw new \Exception('Cannot create create cart, missing user type');
        if($user_type == 'visitor')
        {
            $this->model->user_session_id = $user_identifier;
        }
        else
        {
            $this->model->user_id = $user_identifier;
        }
        $this->model->type = $type;
        $this->model->save();
        return $this->model;
    }
}

我的测试：

/** @test */
public function create_visitor_cart_calls_internal()
{
    $model = m::mock('Models\CartModel');
    $model->shouldReceive('user_session_id')->with('sess123');
    $model->shouldReceive('type')->with('main');
    $model->shouldReceive('save')->andReturn($model);

    $repository = new EloquentCartRepository($model);
    $created_model = $repository->create_visitor_cart('sess123','main');
    $this->assertEquals('sess123',$created_model->user_session_id);
    $this->assertEquals('main',$created_model->type);
}

这是测试我班级的正确方法吗？或者这是对Mockery / mocking的错误使用？

Answer 1

您应该测试它是否已保存，而不是测试返回的内容。这意味着，->save()运行。您在->save()上设置的期望值为$model->shouldReceive('save')->andReturn($model);。这没有意义，因为代码不使用->save()的返回值。

在编程中，您通常会处理两种类型的方法：命令和查询。查询可以获得一些值，做一些逻辑并返回一个值。命令可以获得一些值，与外部源（例如数据库）通信并且不返回任何内容。查询应该是存根的（这意味着，他们不应该对它的调用量做任何期望，但只能根据它返回的内容）和命令应该被模拟（这意味着，它们应该只包含对它的多少（以及如果）的期望被称为。）

->save()方法是一个命令：它与数据库通信。所以它应该被嘲笑。要模拟对象，请使用Mockery的->once()方法。它设定了一个应该被称为一次的期望：

/** @test */
public function create_visitor_cart_calls_internal()
{
    $model = m::mock('Models\CartModel');
    $model->shouldReceive('save')->once();

    $repository = new EloquentCartRepository($model);
    $created_model = $repository->create_visitor_cart('sess123','main');
    $this->assertEquals('sess123',$created_model->user_session_id);
    $this->assertEquals('main',$created_model->type);
}

尽管名称如此，Mockery默认是一个存根框架。除非您明确指定->once()
之类的期望，否则它不会验证是否调用了方法

有关详细信息，请参阅文档：https://github.com/padraic/mockery-docs/blob/master/reference/expectations.rst

不确定我是否正确使用嘲弄

1 个答案: