fix: ApiPlatform\Symfony\Bundle\Test\Client::getContainer return type

[VincentLanglet]

Q	A
Branch?	4.1 for bugfix
Tickets	Closes #...
License	MIT
Doc PR	api-platform/docs#...

Implementation was

    public function getContainer(): ?ContainerInterface
    {
        return $this->kernelBrowser->getContainer();
    }

but this->kernelBrowser is not nullable and KernelBrowser implementation is

    public function getContainer(): ContainerInterface
    {
        $container = $this->kernel->getContainer();

        return $container->has('test.service_container') ? $container->get('test.service_container') : $container;
    }

so it's never nullable.

[soyuka]

I like the change but its a bc break to change a return type https://symfony.com/doc/current/contributing/code/bc.html#changing-classes. Would you be able to target the main branch?

[VincentLanglet]

I like the change but its a bc break to change a return type

I don't think it is since the class is final, no ?

[soyuka]

thanks @VincentLanglet !