主页 / coding / 问题 / 79311894
Accepted
Nishant
Asked: 2024-12-27 21:15:08 +0800 CST

如何在测试 Laravel 控制器方法时模拟控制器方法内部调用的方法

  • 772

定义的路线web.php是

Route::get('about-us', [PageController::class, 'renderAboutUsPage'])->name('pages.about-us');

在我的控制器中，我有方法

class PageController extends Controller
{
    protected $pageService;

    /**
     * Class contructor
     */
    public function __construct(PageService $pageService)
    {
        $this->pageService = $pageService;
    }

    /**
     * Method to render about us page
     */
    public function renderAboutUsPage(){
        return $this->renderStaticPage('about-us');
    }

    /**
     * Method to render static page
     * @param slug
     */
    private function renderStaticPage($slug) {
        Log::info("Rendering ".$slug." static page.");
        $page = $this->pageService->getActivePageBySlug($slug);
        return view('pages.static-page', ['data'=>$page]);
    }
}

我的理解是，当我测试该方法renderAboutUsPage()时，我应该pageService->getActivePageBySlug($slug)在测试中进行模拟，以避免真正调用该方法。这将有助于减少测试执行时间。

我对我的服务进行了单独的测试，并进行了getActivePageBySlug()独立测试。

我的测试用例是

    /**
     * @test
     * @testdox Whether the about us page returns a successful response, renders the correct view, contains view object {data} and log correct messages in log file.
     * @group website
     * @group static-pages
     */
    public function test_whether_about_us_page_renders_successfully()
    {
        Log::shouldReceive('info')->once()->with("Rendering about-us static page.");
        Log::shouldReceive('info')->once()->with("Getting page by active status and provided slug.");
        $response = $this->get('about-us');
        $response->assertStatus(200);
        $response->assertViewIs('pages.static-page'); 
        $response->assertViewHas('data');
    }

我不知道如何getActivePageBySlug($slug)在我的测试用例中模拟该方法以避免真正的调用。

方法的定义getActivePageBySlug($slug)是：

    /**
     * Method to get page by slug (Checks to page status is active)
     * @param string slug
     * @return Page page
     * @throws Throwable e 
     */
    public function getActivePageBySlug(string $slug)
    {
        Log::info("Getting page by active status and provided slug.");
        try
        {
            $page = Page::where('slug',$slug)->where('status', Status::active->value())->first();
            if(!$page) {
                throw new NotFoundHttpException("The page ". $slug ." not found.");
            } 
            return $page;
        }
        catch (Throwable $e)
        {
            Log::error("Error in getting page by slug.");
            throw $e;
        }
    }
laravel

1 个回答

  1. Best Answer

    实际上，您不想避免调用，否则它就不是真正的功能测试，而是“单元测试”。如果您正在项目中测试端点/URL，则它应该是功能测试，除非它是您真正想避免调用的第三方服务，否则您应该进行真正的调用。

    您的服务正在进行数据库调用，您应该调用它，这很好，特别是您在它之后没有进行任何繁重的工作。

    因此，如果您想进行真正的呼叫，您应该有如下代码（假设您有工厂并且一切都正确设置）：

    /**
 * @group website
 * @group static-pages
 */
public function test_about_us_page_should_render(): void
{
    Page::factory()->create([
        'slug' => $page = 'about-us',
        'status' => Status::active, // Be sure to cast 'status' => Status::class in your model
    ]);

    Log::shouldReceive('info')->once()->with("Rendering {$page} static page.");
    Log::shouldReceive('info')->once()->with("Getting page by active status and provided slug.");

    $this->get($page)
        ->assertOk()
        ->assertViewIs('pages.static-page')
        ->assertViewHas('data');
}

    这应该是有史以来最简单的测试，您可以在那里添加更多内容。请注意，我更改了测试名称（试图遵循标准）和其他小改动。

    现在，假设您也预计页面会失败（您的服务抛出错误），那么我们也来测试一下：

    /**
 * @group website
 * @group static-pages
 * @depends test_about_us_page_should_render
 * @dataProvider pageErrorDataProvider
 */
public function test_about_us_page_should_throw_an_error_when_model_not_found(callable $pageSeeder): void
{
    $pageSeeder();

    $page = 'about-us';

    Log::shouldReceive('error')->once()->with("Error in getting page by slug.");

    $this->get($page)
        ->assertServerError(); // This checks for status = 500

    // You should also assert that you got the exact error: "The page {$page} not found."
}

public function pageErrorDataProvider(): array
{
    return [
        'Page not active' => function () {
            return Page::factory()->create([
                'slug' => 'about-us',
                'status' => Status::disabled, // Or whatever the other state is
            ]);
        },
        'Different page' => function () {
            return Page::factory()->create([
                'slug' => 'about',
                'status' => Status::active,
            ]);
        },
        'Page does not exist' => function () {
            return null;
        },
    ];
}

    现在，你可以看到我们首先依赖第一个测试，如果成功，这个新案例也可以运行。我们还利用了数据提供者，所以我们用同一个案例测试不同的可能性，只是数据初始化不同。

    我现在不记得从端点检查错误的正确断言是什么，所以我们还要确保检查错误是否是预期的错误，但是请阅读官方文档并对其进行修改，你就会找到它。

    最后，如果你真的想避免进行任何呼叫，你的测试应该是这样的：

    /**
 * @group website
 * @group static-pages
 */
public function test_about_us_page_should_render(): void
{
    $page = 'about-us';

    $this->mock(PageService::class, function (MockInterface $mock) {
        $mock->shouldReceive('getActivePageBySlug')
            ->andReturn(
                Page::factory()->make([
                    'slug' => $page = 'about-us',
                    'status' => Status::active,
                ])
            );
    });

    Log::shouldReceive('info')->once()->with("Rendering {$page} static page.");
    Log::shouldReceive('info')->once()->with("Getting page by active status and provided slug.");

    $this->get($page)
        ->assertOk()
        ->assertViewIs('pages.static-page')
        ->assertViewHas('data');
}

    您想模拟该服务，因此您可以做任何事情。

    请记住，我使用的是 Laravel 11.x 文档，而且我不知道您的内容是什么Page，您还应该断言返回的内容是预期的内容，不仅data在您的视图中它有索引，而且它是什么数据。

    • 1

相关问题