Question

我正在处理客户端的应用程序，但我在测试存储库时遇到了问题。

要将存储库绑定到模型，我有以下代码：

<?php

namespace FD\Repo;

use App;
use Config;

/**
 * Service Provider for Repository
 */
class RepoServiceProvider extends \Illuminate\Support\ServiceProvider
{
    public function register()
    {
        $app = $this->app;

        $app->bind('FD\Repo\FactureSst\FactureSstInterface', function ($app) {
            return new FactureSst\EloquentFactureSst(App::make('FactureSst'), new \FD\Service\Cache\LaravelCache($app['cache'], 'factures_sst', 10));
        });
    }
}

然后，存储库扩展一个抽象类，其中包含来自雄辩的ORM（查找，所在，所有等）的函数。存储库的代码如下所示：

<?php

namespace FD\Repo\FactureSst;

use Illuminate\Database\Eloquent\Model;
use FD\Repo\AbstractBaseRepo;
use FD\Repo\BaseRepositoryInterface;
use FD\Service\Cache\CacheInterface;
use Illuminate\Support\Collection;

class EloquentFactureSst extends AbstractBaseRepo implements BaseRepositoryInterface, FactureSstInterface
{
    protected $model;
    protected $cache;

    public function __construct(Model $resource, CacheInterface $cache)
    {
        $this->model = $resource;
        $this->cache = $cache;
    }

    /**
     * Retrieve factures with the given SST and BDC IDs.
     *
     * @param int $sst_id
     * @param int $bdc_id
     * @return \Illuminate\Support\Collection
     */
    public function findWithSstAndBdc($sst_id, $bdc_id)
    {
        $return = new Collection;

        $factures = $this->model->where('id_sst', $sst_id)
            ->whereHas('facture_assoc', function ($query) use ($bdc_id) {
                $query->where('id_bdc', $bdc_id);
            })
            ->get();

        $factures->each(function ($facture) use (&$return) {
            $data = [
                'facture_id'   => $facture->id,
                'facture_name' => $facture->num_facture,
                'total_dsp'    => $facture->total_dsp(),
                'total_tradi'  => $facture->total_tradi()
            ];

            $return->push($data);
        });

        return $return;
    }
}

要测试对数据库的调用，我使用Mockery作为对数据库的调用将会太长。这是我的测试类：

<?php namespace App\Tests\Unit\Api\FactureSst;

use App;
use FactureSst;
use Illuminate\Database\Eloquent\Collection;
use Mockery as m;
use App\Tests\FdTestCase;

class FactureSstTest extends FdTestCase
{
    /**
     * The primary repository to test.
     */
    protected $repo;

    /**
     * Mocked version of the primary repo.
     */
    protected $mock;

    public function setUp()
    {
        parent::setUp();
        $this->repo = App::make('FD\Repo\FactureSst\FactureSstInterface');
        $this->mock = $this->mock('FD\Repo\FactureSst\FactureSstInterface');
    }

    public function tearDown()
    {
        parent::tearDown();
        m::close();
    }

    public function mock($class)
    {
        $mock = m::mock($class);
        $this->app->instance($class, $mock);
        return $mock;
    }

    public function testFindingBySstAndBdc()
    {
        $this->mock->shouldReceive('where')->with('id_sst', 10)->once()->andReturn($this->mock);
        $this->mock->shouldReceive('whereHas')->with('facture_assoc')->once()->andReturn($this->mock);
        $this->mock->shouldReceive('get');

        $result = $this->repo->findWithSstAndBdc(30207, 10);
        $this->assertEquals($result, new \Illuminate\Support\Collection);
        $this->assertEquals($result->count(), 0);
    }
}

正如您在测试中看到的那样，我只是尝试调用该函数并确保函数链接正确。但是我不断收到错误说：

应用\测试\单位\阿比\ FactureSst \ FactureSstTest :: testFindingBySstAndBdc Mockery \ Exception \ InvalidCountException：应调用Mockery_0_FD_Repo_FactureSst_FactureSstInterface中的（＆＃34; id_sst＆＃34;，10）的方法恰好1次，但被称为0次。

请有人帮我理解为什么这不起作用以及如何解决它。很抱歉，该代码为法语，该应用程序适用于法国客户。

提前致谢。

Answer 1

当你真的应该嘲笑该存储库的依赖关系时，看起来好像是在模拟存储库本身，即Illuminate\Database\Eloquent\Model，这是你将要访问数据库的地方。

更改setUp()，以便创建Illuminate\Database\Eloquent\Model的模拟对象，然后在实例化存储库时注入该模拟对象。

public function setUp()
{
    parent::setUp();
    $this->mock = m::mock('Illuminate\Database\Eloquent\Model');  // Or better if you mock 'FactureSst'
    $this->app->instance('FactureSst', $this->mock);
}

这有点令人困惑，因为您声明抽象类包含ORM方法，但是当您调用这些方法时，您在注入的Model依赖项上调用它们，而不是在抽象类上调用它们。这可能是混乱所在。

此外，如果您的模型扩展Illuminate\Database\Eloquent\Model，那么将模型注入存储库而不是Illuminate\Database\Eloquent\Model通常会更好。这样，您还可以利用您在存储库中的模型中设置的任何关系函数。

Answer 2

如果我没有弄错这一行

$result = $this->repo->findWithSstAndBdc(30207, 10);

实际应该是

$result = $this->mock->findWithSstAndBdc(30207, 10);

另请注意，您可以模拟整个调用链（对于这样的流畅查询很有用）：

$this->mock->shouldReceive('where->whereHas->get')
     ->once()->andReturn(/*MAKE A FAKE OBJECT HERE*/);

我也会使用PHP的内置foreach（），而不是使用$ factures-＆gt;每个，因为它可以减少测试。

Laravel 4 - 使用PHPUnit和Mockery进行存储库模式测试

2 个答案: