我正在重写现有的 Laravel 4 应用程序以确保进行充分的测试。长话短说，我已经使用 TDD 方法重写了我的 AccountController 类，但我有点头疼。

考虑以下呈现包含用户列表的页面的方法:

public function getIndex()
{
    // build the view
    //
    return \View::make('account.list-users')
        ->with('users', \Sentry::getUserProvider()->findAll());
}

我正在使用 Smarty 呈现我的 View 并使用 Sentry 进行身份验证。

现在，我想写一些这样的测试:

public function test_getIndex()
{
    // arrange
    //

    // set up some mocks here...


    // act
    //
    $response = $this->client->request("GET", "/list-users");


    // assert
    //

    // test for <table class="table">
    $this->assertFalse($response->filter("table.table")==null, "table not found");

    // test for some <a> tags for the "update" buttons
    $element = $response->filter("td a")->first()->extract(array("href", "class", "_text"));
    $this->assertTrue(strstr($element[0][0],"/my-update-url")!="");
    $this->assertTrue(strstr($element[0][1],"btn btn-xs btn-success")!="");
    $this->assertTrue(strstr($element[0][2],"Active")!="");

    // test for some other markup...

}

我一直在关注 Jeffrey Way 的书 Laravel Testing Decoded 并编写了类似上面的测试，它们运行良好。

令人头疼的是“在这里设置一些模拟...”部分。具体来说，我需要设置的模拟数量是荒谬的。这是因为，作为一个更大的 Web 应用程序的一部分，我正在使用一个 View composer，它将数据添加到 View 模型:当前用户模型、菜单结构、警报消息、新闻消息、应用程序版本号等。我我已经通过使用“基本”模板进行测试来减少其中的大部分内容，但它仍然有很多东西 - 以至于我要编写数百行代码来测试这个简单的单行方法。

有更好的方法吗？

在我看来，有两种方法可以做到这一点:

一个。我一直在做的方式

B.模拟 \View::make 调用，以便绕过我所有的模板渲染 - 就像这样

public function test_getIndex()
{
    // arrange
    //
    $userList = "this is a list of users";

    $userProvider = Mockery::mock("\Cartalyst\Sentry\Users\Eloquent\Provider");

    \Sentry::shouldReceive("getUserProvider")
        ->once()
        ->andReturn($userProvider);

    $userProvider->shouldReceive("findAll")
        ->once()
        ->andReturn($userList);

    $view = Mockery::mock("\Illuminate\View\View");

    \View::shouldReceive("make")
        ->with("account.list-users")
        ->once()
        ->andReturn($view);

    $view->shouldReceive("with")
        ->with("users", $userList)
        ->once()
        ->andReturn($view);

    $view->shouldReceive("render")
        ->once()
        ->andReturn("results");

    // act
    //
    $response = $this->call("GET", "/list-users");

    // assert
    //
    $this->assertResponseOk();
}

如果我采用这种方法，测试会简单得多，我只测试 Controller 方法中的代码，但我并没有真正测试调用该路由所涉及的所有内容(这可能是一件好事或者可能不会 - 我不确定)并且我担心我无法获得足够的保险。

那么，执行此操作的最佳方法是什么:(A)、(B) 或其他方法？

编辑

我对我的 Controller 方法的测试有相当多的困惑，@TheShiftExchange 在下面的回答和评论中变得更加清晰。我将尝试在这里解决这个问题，作为编辑，因为它给我更多的空间来讨论这个问题。

考虑下面答案中给出的第二个示例:

public function testMethod()
{
    $this->call('GET', '/list-users');

    $this->assertViewHas('users', \Sentry::getUserProvider()->findAll());
}

如果我运行这个测试，它会工作，但它会访问数据库，我试图通过模拟一些东西来避免这种情况。

所以，我可以稍微扩展一下这个测试:

public function testMethod()
{
    \Sentry::shouldReceive("getUserProvider")
        ->once()
        ->andReturn($userProvider);
    // plus a mock of the UserProvider class,...


    $this->call('GET', '/list-users');

    $this->assertViewHas('users', \Sentry::getUserProvider()->findAll());
}

此测试将不 起作用，因为除了 Controller 方法所需的模拟之外，我还需要对我的 View 编辑器中的代码进行模拟。此代码包括 $currentUser =\Sentry::getUser()(用户名显示在我的应用程序页面的右上角)。

所以代码实际上变成了:

public function testMethod()
{
    \Sentry::shouldReceive("getUserProvider")
        ->once()
        ->andReturn($userProvider);
    // plus a mock of the UserProvider class,...

    // plus a mock of ThisClass

    // and a mock of ThatClass

    // and a mock of SomeOtherClass

    // etc.

    // etc.

    $this->call('GET', '/list-users');

    $this->assertViewHas('users', \Sentry::getUserProvider()->findAll());
}

它很快就会失控。

这表明我做错了什么，但我不确定是什么。我怀疑问题源于我不确定我在这里测试的到底是什么。

所以，毕竟，问题变成了:

当我测试 Controller 的方法时，我到底想测试什么？

1. Controller 方法中的代码？或者，

2. 从请求到响应的全过程？

我要测试的是第一项——只是 Controller 方法中的代码。我的问题中的示例非常简单，但我确实有一些 Controller 方法可以根据用户输入执行表单验证或重定向等操作 - 我想测试该代码。

也许，我不需要通过 $this->call() 来测试代码，而是直接调用 Controller 方法？

最佳答案

作为 Laravel 框架的一部分，it includes some testing helpers .在这些助手中包括 View 测试助手:

断言 View 有一些数据

public function testMethod()
{
    $this->call('GET', '/');

    $this->assertViewHas('name');
    $this->assertViewHas('age', $value);
}

所以你可以这样做:

public function testMethod()
{
    \Sentry::shouldReceive("getUserProvider")
    ->once()
    ->andReturn('foo');

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

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

关于php - 在 Laravel 中测试 Controller 的正确方法是什么？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/23959302/