Laravel依赖注入：你什么时候必须这样做？你什么时候可以嘲笑立面？这两种方法的优点是什么？选项#1：依赖注入选项#2：立面模拟

php unit-testing dependency-injection laravel-4 mockery

2022-08-31 00:29:25

我已经使用Laravel一段时间了，我一直在阅读很多关于依赖注入一个可测试的代码。在谈论立面和模拟对象时，我遇到了一个困惑的地步。我看到两种模式：

class Post extends Eloquent {

    protected $guarded = array();

    public static $rules = array();

}

这是我的后期模型。我可以跑去从我的博客中获取所有帖子。现在我想把它合并到我的控制器中。Post::all();

选项#1：依赖注入

我的第一个直觉是将模型作为依赖项注入：Post

class HomeController extends BaseController {

    public function __construct(Post $post)
    {
    $this->post = $post;
    }

    public function index()
    {
        $posts = $this->posts->all();
        return View::make( 'posts' , compact( $posts );
    }

}

我的单元测试将如下所示：

<?php 

use \Mockery;

class HomeControllerTest extends TestCase {

    public function tearDown()
    {
        Mockery::close();

        parent::tearDown();
    }
    public function testIndex()
    {
        $post_collection = new StdClass();

        $post = Mockery::mock('Eloquent', 'Post')
        ->shouldRecieve('all')
        ->once()
        ->andReturn($post_collection);

        $this->app->instance('Post',$post);

        $this->client->request('GET', 'posts');

        $this->assertViewHas('posts');
    }
}

选项#2：立面模拟

class HomeController extends BaseController {


    public function index()
    {
        $posts = Post::all();
        return View::make( 'posts' , compact( $posts );            
    }

}

我的单元测试将如下所示：

<?php 

use \Mockery;

class HomeControllerTest extends TestCase {


    public function testIndex()
    {
        $post_collection = new StdClass();

        Post::shouldRecieve('all')
        ->once()
        ->andReturn($post_collection);

        $this->client->request('GET', 'posts');

        $this->assertViewHas('posts');
    }
}

我理解这两种方法，但我不明白为什么我应该或何时应该使用一种方法而不是另一种方法。例如，我尝试将DI路由与类一起使用，但它不起作用，因此我必须使用Facade Mocks。如果能就此问题进行任何钙化，将不胜感激。Auth

答案 1

尽管您在选项 #1 上使用依赖注入，但您的控制器仍与 Eloquent ORM 耦合。（请注意，我在这里避免使用术语模型，因为在MVC中，模型不仅仅是一个类或对象，而是一个层。这是您的业务逻辑。

依赖注入允许依赖关系反转，但它们不是一回事。根据依赖关系反转原则，高级和低级代码都应该依赖于抽象。在你的例子中，高级代码是你的控制器，低级代码是从MySQL获取数据的雄辩ORM，但正如你所看到的，它们都不依赖于抽象。

因此，您无法在不影响控制器的情况下更改数据访问层。例如，您将如何从MySQL更改为MongoDB或文件系统？为此，您必须使用存储库（或任何您想要调用它的名称）。

因此，创建一个存储库接口，所有具体的存储库实现（MySQL，MongoDB，文件系统等）都应该实现。

interface PostRepositoriesInterface {

    public function getAll();
}

然后创建您的具体实现，例如MySQL

class DbPostRepository implements PostRepositoriesInterface {

    public function getAll()
    {

        return Post::all()->toArray();

        /* Why toArray()? This is the L (Liskov Substitution) in SOLID. 
           Any implementation of an abstraction (interface) should be substitutable
           in any place that the abstraction is accepted. But if you just return 
           Post:all() how would you handle the situation where another concrete 
           implementation would return another data type? Probably you would use an if
           statement in the controller to determine the data type but that's far from 
           ideal. In PHP you cannot force the return data type so this is something
           that you have to keep in mind.*/
    }
}

现在，控制器必须键入接口提示，而不是具体的实现。这就是“接口上的代码与非实现上的代码”的全部意义所在。这就是依赖关系反转。

class HomeController extends BaseController {

    public function __construct(PostRepositoriesInterface $repo)
    {
        $this->repo= $repo;
    }

    public function index()
    {
        $posts = $this->repo->getAll();

        return View::make( 'posts' , compact( $posts ) );
    }

}

这样，控制器将与数据层分离。它开放供扩展，但关闭以进行修改。您可以通过创建PostRepositoriesInterface（例如MongoPostRepository）的新具体实现来切换到MongoDB或文件系统，并仅更改绑定（请注意，我在这里不使用任何命名空间）：

App:bind('PostRepositoriesInterface','DbPostRepository');

自

App:bind('PostRepositoriesInterface','MongoPostRepository');

在理想情况下，控制器应仅包含应用程序，而不包含业务逻辑。如果你发现自己想从另一个控制器调用一个控制器，这表明你做错了什么。在这种情况下，控制器包含太多逻辑。

这也使测试更容易。现在，您可以在不实际命中数据库的情况下测试控制器。请注意，控制器测试必须仅在控制器正常运行时才进行测试，这意味着控制器调用正确的方法，获取结果并将其传递到视图。此时，您没有测试结果的有效性。这不是控制者的责任。

public function testIndexActionBindsPostsFromRepository()
{ 

    $repository = Mockery::mock('PostRepositoriesInterface');

    $repository->shouldReceive('all')->once()->andReturn(array('foo'));

    App::instance('PostRepositoriesInterface', $repository);

    $response = $this->action('GET', 'HomeController@index'); 

    $this->assertResponseOk(); 

    $this->assertViewHas('posts', array('foo')); 
}

编辑

如果您选择使用选项#1，则可以像这样测试它

class HomeControllerTest extends TestCase {

  public function __construct()
  {
      $this->mock = Mockery::mock('Eloquent', 'Post');
  }

  public function tearDown()
  {
      Mockery::close();
  }

  public function testIndex()
  {
      $this->mock
           ->shouldReceive('all')
           ->once()
           ->andReturn('foo');

      $this->app->instance('Post', $this->mock);

      $this->call('GET', 'posts');

      $this->assertViewHas('posts');
  }

}

答案 2