[FrameworkBundle] WebTestCase KernelBrowser::getContainer null return type

Simperfit · Simperfit · commit 5e143c145ff4 · 2019-04-29T09:44:14.000+02:00
diff --git a/src/Symfony/Component/HttpKernel/CHANGELOG.md b/src/Symfony/Component/HttpKernel/CHANGELOG.md
@@ -25,6 +25,7 @@ CHANGELOG
  * renamed `PostResponseEvent` to `TerminateEvent`
  * added `HttpClientKernel` for handling requests with an `HttpClientInterface` instance
  * added `trace_header` and `trace_level` configuration options to `HttpCache`
+ * Getting the container from a non-booted kernel Sis deprecated.
 
 4.2.0
 -----
diff --git a/src/Symfony/Component/HttpKernel/Kernel.php b/src/Symfony/Component/HttpKernel/Kernel.php
@@ -361,6 +361,10 @@ public function getProjectDir()
      */
     public function getContainer()
     {
+        if (null === $this->container) {
+            @trigger_error('Getting the container from a non-booted kernel is deprecated since Symfony 4.3 and will throw an exceptions in Symfony 5.0.', E_USER_DEPRECATED);
+        }
+
         return $this->container;
     }
 
diff --git a/src/Symfony/Component/HttpKernel/KernelInterface.php b/src/Symfony/Component/HttpKernel/KernelInterface.php
@@ -133,7 +133,7 @@ public function getRootDir();
     /**
      * Gets the current container.
      *
-     * @return ContainerInterface|null A ContainerInterface instance or null when the Kernel is shutdown
+     * @return ContainerInterface A ContainerInterface instance or null when the Kernel is shutdown
      */
     public function getContainer();
 
diff --git a/src/Symfony/Component/HttpKernel/Tests/KernelTest.php b/src/Symfony/Component/HttpKernel/Tests/KernelTest.php
@@ -36,6 +36,10 @@ public static function tearDownAfterClass()
         $fs->remove(__DIR__.'/Fixtures/var');
     }
 
+    /**
+     * @group legacy
+     * @expectedDeprecation Getting the container from a non-booted kernel is deprecated since Symfony 4.3 and will throw an exceptions in Symfony 5.0.
+     */
     public function testConstructor()
     {
         $env = 'test_env';
@@ -49,6 +53,10 @@ public function testConstructor()
         $this->assertNull($kernel->getContainer());
     }
 
+    /**
+     * @group legacy
+     * @expectedDeprecation Getting the container from a non-booted kernel is deprecated since Symfony 4.3 and will throw an exceptions in Symfony 5.0.
+     */
     public function testClone()
     {
         $env = 'test_env';
@@ -486,6 +494,18 @@ public function testTerminateReturnsSilentlyIfKernelIsNotBooted()
         $kernel->terminate(Request::create('/'), new Response());
     }
 
+    /**
+     * @group legacy
+     * @expectedDeprecation Getting the container from a non-booted kernel is deprecated since Symfony 4.3 and will throw an exceptions in Symfony 5.0.
+     */
+    public function testDeprecatedNullKernel()
+    {
+        $kernel =  $this->getKernel();
+        $kernel->shutdown();
+
+        $this->assertNull($kernel->getContainer());
+    }
+
     public function testTerminateDelegatesTerminationOnlyForTerminableInterface()
     {
         // does not implement TerminableInterface

Original file line number	Diff line number	Diff line change
`@@ -361,6 +361,10 @@ public function getProjectDir()`
`361`	`361`	`*/`
`362`	`362`	`public function getContainer()`
`363`	`363`	`{`
	`364`	`+ if (null === $this->container) {`
	`365`	`+ @trigger_error('Getting the container from a non-booted kernel is deprecated since Symfony 4.3 and will throw an exceptions in Symfony 5.0.', E_USER_DEPRECATED);`
	`366`	`+ }`
	`367`	`+`
`364`	`368`	`return $this->container;`
`365`	`369`	`}`
`366`	`370`
Original file line number	Diff line number	Diff line change
`@@ -133,7 +133,7 @@ public function getRootDir();`
`133`	`133`	`/**`
`134`	`134`	`* Gets the current container.`
`135`	`135`	`*`
`136`		`- * @return ContainerInterface\|null A ContainerInterface instance or null when the Kernel is shutdown`
	`136`	`+ * @return ContainerInterface A ContainerInterface instance or null when the Kernel is shutdown`
`137`	`137`	`*/`
`138`	`138`	`public function getContainer();`
`139`	`139`