Испытайте улов Laravel Controller - PullRequest
Новая
       24

Испытайте улов Laravel Controller

0 голосов
/ 15 марта 2019

Я использую Laravel 5.7 и у меня есть этот контроллер 

public function upload(Request $request)
{
    $fileData   = $request->getContent();
    $fileName   = uniqid().'.xlsx';

    try {
        // save file locally
        Storage::disk('local')->put($fileName, $fileData);

        // dispatch to queue
        AnyJob::dispatch($fileName);

        // insert in import_statuses table for status checking
        AnyModel::create([
            'job_id'  => $fileName
        ]);

    } catch (\Exception $e) {
        error_log($e);

        return response()->json([
            'error' => 'Could not save file or queue not found.'
        ], 500);
    }

    return response()->json([
        'status' => 'OK',
        'jobId'  => $fileName,
    ]);
}

И я пытаюсь создать тест, который входит в улов, но я не могу заставить его работать.

Я уже пытался смоделировать Storage и ожидаю вызова методов disk и put, но он всегда жалуется на другие вызываемые методы и не ожидается.

Я также пытался насмехаться над AnyJob и AnyModel разными способами, но это, похоже, не влияет на запрос, сделанный $this->post.

Это мой метод испытаний прямо сейчас: 

public function testUploadException()
{
    $xlsxFile = UploadedFile::fake()->create('test.xlsx', 100);

    // ignore Storage and Job real work
    \Illuminate\Support\Facades\Storage::fake();
    $this->withoutJobs();

    $mock = Mockery::mock('\App\AnyModel');
    $mock
        ->shouldReceive('create')->times(3)
        ->andThrow(new \Exception('any error'));

    $this->app->instance('\App\AnyModel', $mock);

    $response = $this
        ->withHeaders(['content-type' => 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet'])
        ->post('/api/upload', [$xlsxFile]);

    $response
        ->assertStatus(500)
        ->assertJson([
            'error' => 'Could not save file or queue not found.'
        ]);
}

И что это возвращает 

root@3e5d84a4db42:/application/api# vendor/bin/phpunit 
PHPUnit 7.5.4 by Sebastian Bergmann and contributors.

....F                                                               5 / 5 (100%)

Time: 2.62 seconds, Memory: 20.00MB

There was 1 failure:

1) Tests\Feature\UploadTest::testUploadException
Expected status code 500 but received 200.
Failed asserting that false is true.

/application/api/vendor/laravel/framework/src/Illuminate/Foundation/Testing/TestResponse.php:133
/application/api/tests/Feature/UploadTest.php:50

FAILURES!
Tests: 5, Assertions: 8, Failures: 1.

Ответы [ 2 ]

1 голос
/ 16 марта 2019

Основная причина, по которой это не сработает, заключается в том, что вы не решаете модель из IoC, например. используя app() или внедрение зависимости. Это будет работать с фасадами, но не с моделями Eloquent.

Вы можете изменить определение метода контроллера на следующее, чтобы разрешить класс из контейнера: 

public function upload(Request $request, AnyModel $anyModel)

Тогда ваш звонок на create будет: 

$anyModel->create([
    'job_id'  => $fileName
]);

Есть также несколько проблем в вашем тесте:

  1. Полное имя класса 'App\AnyModel', а не '\App\AnyModel'. Я бы предложил использовать :: class вместо строки, т.е. AnyModel::class (убедитесь, что вы импортировали класс).
  2. Вы, кажется, звоните create() только один раз, поэтому ->times(3) выдаст ошибку.
0 голосов
/ 21 марта 2019

Помимо использования IoC, о котором говорил @RossWilson, я также обнаружил, что могу попасть в блок catch, используя 

Storage::shouldReceive('disk')->andThrow('error');
...