Excel::assertQueuedWithChain throw an error of get_class(): Argument #1 ($object) must be of type object, null given

[yasuaki640]

I'm trying to test that an import job is queued and a notification is sent when the import is finished.

I wrote the test code but I can't test that the notification job was queued with the following error in the Excel::assertQueuedWithChain function.

  • Tests\Feature\UserControllerTest > queue import success
   TypeError 

  get_class(): Argument #1 ($object) must be of type object, null given

  at vendor/maatwebsite/excel/src/Fakes/ExcelFake.php:302
    298▕     }
    299▕ 
    300▕     public function assertQueuedWithChain($chain): void
    301▕     {
  ➜ 302▕         Queue::assertPushedWithChain(get_class($this->job), $chain);
    303▕     }
    304▕ 
    305▕     /**
    306▕      * @param string        $classname

      +2 vendor frames 
  3   tests/Feature/UserControllerTest.php:223
      Illuminate\Support\Facades\Facade::__callStatic("assertQueuedWithChain")

When I performed the manual test, I confirmed that the Excel file was imported and the notification was sent.

I will post the source code where the error occurred, so please point out any mistakes.

Environment

• Laravel Excel 3.1.33
• Laravel 8.55.0
• PHP 8.0.10

Codes

This is the code that is performing the import and queuing the notification job.

• Sorry for it has become a fat controller because of the sample code.

    /**
     * @param QueueImportPost $request
     * @return View|RedirectResponse
     */
    public function queueImport(QueueImportPost $request): View|RedirectResponse
    {
        try {
            Excel::queueImport(
                new UsersImport,
                $file = $request->file('users'),
                self::STORAGE_S3
            )->chain([
                new NotifyUserOfCompletedImport(
                    request()->user() ?? User::factory()->create(),
                    $request->file('users')->getClientOriginalName()
                )
            ]);

            $message = 'Successfully queued job of import an excel file';
            return \view('excel.import', compact('message'));

        } catch (Exception $e) {
            logger()->error($e);
            return redirect(route('users.excel.import.import-form'))
                ->withErrors($e->getMessage());
        }
    }

This is the test code.

    /**
     * @return void
     */
    public function test_queueImport_success()
    {
        Excel::fake();
        Storage::fake(UserController::STORAGE_S3);

        $user = User::factory()->create();

        $response = $this->actingAs($user)
            ->post(route('users.excel.import.queue'), [
                'users' => $file = UploadedFile::fake()->create(
                    'queue_import_success.xlsx',
                    100,
                    'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet',
                )
            ]);

        $response
            ->assertOk()
            ->assertViewHas('message');

        Excel::assertImported($file->getBasename(), UserController::STORAGE_S3);

        Excel::assertQueuedWithChain([
            new NotifyUserOfCompletedImport($user, $file->getClientOriginalName())
        ]);
    }

All source code is in this repository.

Thank you.

[yasuaki640]

I'm guessing that the cause of this error is a change in the get_class function specifications.

Note: Explicitly passing null as the object is no longer allowed as of PHP 7.2.0. The parameter is still optional and calling get_class() without a parameter from inside a class will work, but passing null now emits an E_WARNING notice.

Or maybe my code is wrong.

Thank you.

[patrickbrouwers]

@yasuaki640 assertQueuedWithChain currently only works for queued exports. Feel free to PR support for queuedImports.

[yasuaki640]

@patrickbrouwers
I've got it.
I will try to make a PR.