Laravel胖模型/瘦控制器的测试策略探讨

针对「胖模型/瘦控制器」风格下的测试冗余问题，核心思路是分层聚焦测试目标：模型层做全量逻辑验证，控制器层只校验流程合规性，避免重复测试。

1. 模型层：做扎实的单元测试 #

针对User模型的updateProfilePhoto方法，编写全面的单元测试，覆盖所有业务场景：

• 合法图片上传后的存储路径、数据库字段更新逻辑
• 非法格式/大小超限的错误处理分支
• 替换旧头像时的文件清理逻辑
• 关联数据（如头像URL缓存）的同步更新等

这部分是核心业务逻辑的保障，要尽可能细致，确保模型方法的每一种执行分支都被验证通过。

2. 控制器层：只验证流程正确性，不重复逻辑测试 #

控制器的职责是「接收请求→参数校验→调用模型→返回响应」，测试只需聚焦这几个环节，不用重复模型已覆盖的逻辑：

• 验证参数校验规则：比如上传文件是否符合类型、大小要求，参数缺失时是否返回正确的验证错误
• 验证模型方法的调用：利用Laravel自带的Mocking工具（基于Mockery），模拟User模型，验证updateProfilePhoto是否被正确调用（包括调用次数、传入参数是否匹配）
• 验证响应结果：比如成功后的跳转、状态码，失败时的错误提示等

举个具体的测试示例：

public function test_update_profile_photo_triggers_model_method()
{
    // 创建测试用户
    $user = User::factory()->create();
    
    // 模拟User模型，指定updateProfilePhoto的调用预期
    $this->mock(User::class, function ($mock) {
        $mock->shouldReceive('updateProfilePhoto')
             ->once()
             ->with(\Illuminate\Http\UploadedFile::fake()->image('avatar.jpg'))
             ->andReturn(true);
    });

    // 发送带文件的请求
    $response = $this->actingAs($user)->post(route('users.update', $user), [
        'profile_photo' => \Illuminate\Http\UploadedFile::fake()->image('avatar.jpg'),
    ]);

    // 验证响应是否符合预期
    $response->assertRedirectToRoute('users.show', $user);
    $response->assertSessionHas('success');
}

这里通过Mock隔离了模型的实际逻辑，只验证控制器是否正确完成了「接收请求→调用模型方法→返回响应」的流程，完全避免了逻辑重复。

3. 可选：补充少量端到端测试 #

如果担心Mock会掩盖参数传递、依赖调用等实际流程中的问题，可以写1-2个极简的端到端测试：比如上传真实测试图片，最终验证数据库的头像字段、存储目录的文件是否存在。但不用覆盖所有场景，只是作为整体流程的兜底校验。

4. 额外优化：分层拆分测试执行 #

利用PHPUnit或Pest的测试分组功能，把模型单元测试和控制器HTTP测试分开执行。模型单元测试运行速度快，适合日常开发中频繁执行；控制器HTTP测试相对较慢，可以在CI阶段或版本发布前执行，兼顾效率和覆盖率。

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