如何在Laravel中使用PHPUnit处理复杂场景的功能测试

问题根因 #

你当前测试失败的核心原因是使用了工厂的make()方法，该方法仅生成内存层面的模型实例，不会持久化到数据库，导致业务代码查询时找不到对应记录。

最优方案（无需修改业务代码） #

方案1：使用官方推荐的测试数据库方案（功能测试首选） #

这个方案是Laravel官方文档明确推荐的功能测试实现方式，完全不需要调整现有业务代码：

1. 在测试类中引入RefreshDatabase Trait，该工具会自动使用配置好的临时测试数据库，每次测试执行前执行迁移，执行完成后自动回滚数据，不会污染生产/开发环境数据。
2. 将测试代码中所有factory()->make()调用替换为factory()->create()，create()方法会将生成的模型实例持久化到测试数据库，业务代码查询时可以正常获取到对应记录。

注意：你当前测试逻辑存在错误：构造的用户ID为1，但连接的接收者ID为2，按照业务逻辑会返回403，需要将测试用户的ID改为2才能得到预期的200响应。
修正后的测试代码示例：

// tests/Feature/UserConnectionsTest.php
use Illuminate\Foundation\Testing\RefreshDatabase;

public function testSucceedIfConnectAuthorised()
{
    // 使用create方法持久化到测试库
    $connection = factory(Connection::class)->create([
        'sender_id'     => 1,
        'receiver_id'   => 2,
        'accepted'      => false,
        'connection_id' => 5,
    ]);

    $user = factory(User::class)->create([
        'id' => 2, // 和连接的接收者ID保持一致
    ]);

    $response = $this->actingAs($user)->post(
        '/app/connection-request/accept',
        [
            'accept'     => true,
            'request_id' => $connection->id,
        ]
    );

    $response->assertLocation('/')->assertStatus(200);
}

方案2：直接模拟依赖（单元测试首选） #

如果你不需要测试数据库读写逻辑，仅想测试Controller层的代码逻辑，Laravel自带的Mock功能完全不需要你额外定义接口调整业务代码，直接模拟Repository的方法返回即可：

public function testSucceedIfConnectAuthorised()
{
    $connection = factory(Connection::class)->make([
        'sender_id'     => 1,
        'receiver_id'   => 2,
        'accepted'      => false,
        'connection_id' => 5,
    ]);
    // 直接模拟UserRepository的方法返回
    $this->mock(\App\Http\Repositories\Frontend\UserRepository::class, function ($mock) use ($connection) {
        $mock->shouldReceive('GetConnectionById')
            ->with($connection->id)
            ->andReturn($connection);
    });

    $user = factory(User::class)->make([
        'id' => 2,
    ]);

    $response = $this->actingAs($user)->post(
        '/app/connection-request/accept',
        [
            'accept'     => true,
            'request_id' => $connection->id,
        ]
    );

    $response->assertLocation('/')->assertStatus(200);
}

疑问解答 #

• 你提到的接口注入方案确实会增加不必要的业务代码复杂度，该方案更适合多环境下需要切换完整数据层实现的场景，你的需求完全不需要用到。
• 该方案远不是唯一/最优解，Laravel官方没有提及是因为功能测试优先推荐用临时测试数据库做完整链路测试，Mock能力仅在单元测试隔离依赖时使用，且Laravel自带的Mock不需要依赖接口即可实现。

内容的提问来源于stack exchange，提问作者Picard